Merge pull request #85 from agicommies/main

fix: false `InsufficientStake` error (#207)

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+spec version `121`
+
+Fixing `InsufficientStake` bug in proposal voting
+
 spec version `120`
 
 Storages:

pallets/governance/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "pallet-governance"
-version = "1.1.0"
+version = "1.1.1"
 description = "FRAME pallet for runtime logic of Subspace Blockchain."
 authors = ["Commune Community"]
 homepage = "https://communeai.org/"

pallets/governance/src/voting.rs

@@ -29,14 +29,15 @@ impl<T: Config> Pallet<T> {
             Error::<T>::AlreadyVoted
         );
 
-        let voter_stake = pallet_subspace::Pallet::<T>::get_delegated_stake(&key);
+        let voter_delegated_stake = pallet_subspace::Pallet::<T>::get_delegated_stake(&key);
+        let voter_owned_stake = pallet_subspace::Pallet::<T>::get_owned_stake(&key);
 
-        ensure!(voter_stake > 0, Error::<T>::InsufficientStake);
+        ensure!(
+            voter_delegated_stake > 0 || voter_owned_stake > 0,
+            Error::<T>::InsufficientStake
+        );
 
-        let stake_from_vector = pallet_subspace::Pallet::<T>::get_stake_from_vector(&key);
-        let has_stake_from =
-            || stake_from_vector.iter().any(|(k, stakes)| k == &key && *stakes > 0);
-        if !NotDelegatingVotingPower::<T>::get().contains(&key) && !has_stake_from() {
+        if !NotDelegatingVotingPower::<T>::get().contains(&key) && voter_delegated_stake == 0 {
             return Err(Error::<T>::VoterIsDelegatingVotingPower.into());
         }
 

runtime/src/lib.rs

@@ -125,7 +125,7 @@ pub const VERSION: RuntimeVersion = RuntimeVersion {
     //   `spec_version`, and `authoring_version` are the same between Wasm and native.
     // This value is set to 100 to notify Polkadot-JS App (https://polkadot.js.org/apps) to use
     //   the compatible custom types.
-    spec_version: 120,
+    spec_version: 121,
     impl_version: 1,
     apis: RUNTIME_API_VERSIONS,
     transaction_version: 1,