Skip to content

fix: remove dead storage helpers in campaign/src/storage.rs (#80)#86

Open
Xhristin3 wants to merge 1 commit into
OrbitChainLabs:mainfrom
Xhristin3:fix/issue-80-remove-dead-storage-helpers
Open

fix: remove dead storage helpers in campaign/src/storage.rs (#80)#86
Xhristin3 wants to merge 1 commit into
OrbitChainLabs:mainfrom
Xhristin3:fix/issue-80-remove-dead-storage-helpers

Conversation

@Xhristin3

Copy link
Copy Markdown
Contributor

Summary

Three pub helpers in campaign/src/storage.rs were defined but never called from any contract entrypoint or referenced outside their own definitions: bump_all_persistent, get_contract_status, and set_contract_status. The DataKey::ContractStatus variant existed solely to back the latter two. Removed all dead code to reduce maintenance surface and eliminate unreachable public API.

Changes

  • Removed get_contract_status and set_contract_status from campaign/src/storage.rs
  • Removed bump_all_persistent from campaign/src/storage.rs
  • Removed DataKey::ContractStatus variant from campaign/src/types.rs

Testing

Existing campaign contract tests pass without modification — the change is pure dead code removal with no behavioural impact.

Closes #80

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[LOW] storage::bump_all_persistent, get_contract_status, set_contract_status are dead code

1 participant