fix: removed unnecessary validation from splitter

andromedaprotocol · Dec 27, 2024 · 0a35200 · 0a35200
1 parent 219c9ba
commit 0a35200
Showing 1 changed file with 2 additions and 21 deletions.
diff --git a/contracts/finance/andromeda-conditional-splitter/src/contract.rs b/contracts/finance/andromeda-conditional-splitter/src/contract.rs
@@ -19,7 +19,6 @@ use cosmwasm_std::{
     attr, ensure, entry_point, BankMsg, Binary, Coin, CosmosMsg, Deps, DepsMut, Env, MessageInfo,
     Reply, Response, StdError, SubMsg, Uint128,
 };
-use cw_utils::nonpayable;
 
 // version info for migration info
 const CONTRACT_NAME: &str = "crates.io:andromeda-conditional-splitter";
@@ -213,16 +212,7 @@ fn execute_update_thresholds(
     ctx: ExecuteContext,
     thresholds: Vec<Threshold>,
 ) -> Result<Response, ContractError> {
-    let ExecuteContext {
-        deps, info, env, ..
-    } = ctx;
-
-    nonpayable(&info)?;
-
-    ensure!(
-        ADOContract::default().is_owner_or_operator(deps.storage, info.sender.as_str())?,
-        ContractError::Unauthorized {}
-    );
+    let ExecuteContext { deps, env, .. } = ctx;
 
     let conditional_splitter = CONDITIONAL_SPLITTER.load(deps.storage)?;
 
@@ -245,16 +235,7 @@ fn execute_update_thresholds(
 }
 
 fn execute_update_lock(ctx: ExecuteContext, lock_time: Expiry) -> Result<Response, ContractError> {
-    let ExecuteContext {
-        deps, info, env, ..
-    } = ctx;
-
-    nonpayable(&info)?;
-
-    ensure!(
-        ADOContract::default().is_owner_or_operator(deps.storage, info.sender.as_str())?,
-        ContractError::Unauthorized {}
-    );
+    let ExecuteContext { deps, env, .. } = ctx;
 
     let mut conditional_splitter = CONDITIONAL_SPLITTER.load(deps.storage)?;