Automatically create the asset of a community if it doesnt exist yet

pallets/communities/src/functions.rs

@@ -1,4 +1,4 @@
-use super::{origin::DecisionMethod, *};
+use super::*;
 use fc_traits_memberships::{GenericRank, Inspect, Rank};
 use frame_support::{
 	dispatch::PostDispatchInfo,
@@ -95,7 +95,7 @@ impl<T: Config> Pallet<T> {
 			};
 
 			let say = *match (vote, decision_method) {
-				(Vote::AssetBalance(say, asset, ..), DecisionMethod::CommunityAsset(a)) if asset == a => say,
+				(Vote::AssetBalance(say, asset, ..), DecisionMethod::CommunityAsset(a, _)) if asset == a => say,
 				(Vote::NativeBalance(say, ..), DecisionMethod::NativeToken)
 				| (Vote::Standard(say), DecisionMethod::Membership | DecisionMethod::Rank) => say,
 				_ => fail!(Error::<T>::InvalidVoteType),

pallets/communities/src/lib.rs

@@ -146,6 +146,7 @@ pub mod pallet {
 		Blake2_128Concat, Parameter,
 	};
 	use frame_system::pallet_prelude::{ensure_signed, BlockNumberFor, OriginFor};
+	use sp_runtime::traits::AccountIdConversion;
 	use sp_runtime::{
 		traits::{Dispatchable, StaticLookup},
 		TokenError,
@@ -193,7 +194,7 @@ pub mod pallet {
 		>;
 
 		/// Type represents interactions between fungibles (i.e. assets)
-		type Assets: fungibles::Inspect<Self::AccountId>
+		type Assets: fungibles::Inspect<Self::AccountId, Balance = NativeBalanceOf<Self>>
 			+ fungibles::Mutate<Self::AccountId>
 			+ fungibles::Create<Self::AccountId>
 			+ fungibles::hold::Inspect<Self::AccountId, Reason = Self::RuntimeHoldReason>
@@ -477,6 +478,15 @@ pub mod pallet {
 			decision_method: DecisionMethodFor<T>,
 		) -> DispatchResult {
 			T::AdminOrigin::ensure_origin(origin)?;
+			if let DecisionMethod::CommunityAsset(ref asset, min_vote) = decision_method {
+				// best effort attemt to create the asset if it doesn't exist
+				let _ = <T::Assets as fungibles::Create<T::AccountId>>::create(
+					asset.clone(),
+					T::PalletId::get().into_account_truncating(),
+					false,
+					min_vote,
+				);
+			}
 			CommunityDecisionMethod::<T>::set(community_id, decision_method);
 			Self::deposit_event(Event::DecisionMethodSet { id: community_id });
 			Ok(())

pallets/communities/src/mock.rs

@@ -24,8 +24,9 @@ pub use virto_common::{CommunityId, MembershipId};
 
 use crate::{
 	self as pallet_communities,
-	origin::{DecisionMethod, EnsureCommunity, EnsureSignedPays},
+	origin::{EnsureCommunity, EnsureSignedPays},
 	types::{Tally, VoteWeight},
+	DecisionMethod,
 };
 
 // Weights constants
@@ -455,7 +456,7 @@ pub(crate) struct TestEnvBuilder {
 	assets_config: AssetsConfig,
 	balances: Vec<(AccountId, Balance)>,
 	communities: Vec<CommunityId>,
-	decision_methods: sp_std::collections::btree_map::BTreeMap<CommunityId, DecisionMethod<AssetId>>,
+	decision_methods: sp_std::collections::btree_map::BTreeMap<CommunityId, DecisionMethod<AssetId, Balance>>,
 	members: Vec<(CommunityId, AccountId)>,
 	memberships: Vec<(CommunityId, MembershipId)>,
 	tracks: Vec<(TrackIdOf<Test, ()>, TrackInfoOf<Test>)>,
@@ -498,7 +499,7 @@ impl TestEnvBuilder {
 	pub(crate) fn add_community(
 		mut self,
 		community_id: CommunityId,
-		decision_method: DecisionMethod<AssetId>,
+		decision_method: DecisionMethod<AssetId, Balance>,
 		members: &[AccountId],
 		memberships: &[MembershipId],
 		maybe_track: Option<TrackInfoOf<Test>>,

pallets/communities/src/origin.rs

@@ -126,16 +126,6 @@ pub enum Subset<T: Config> {
 	AtLeastRank(GenericRank),
 }
 
-/// The mechanism used by the community or one of its subsets to make decisions
-#[derive(Clone, Debug, Decode, Default, Encode, Eq, MaxEncodedLen, PartialEq, TypeInfo)]
-pub enum DecisionMethod<AssetId> {
-	#[default]
-	Membership,
-	NativeToken,
-	CommunityAsset(AssetId),
-	Rank,
-}
-
 #[cfg(feature = "xcm")]
 impl<T> TryConvert<RuntimeOriginFor<T>, xcm::v3::MultiLocation> for RawOrigin<T>
 where

pallets/communities/src/tests/governance.rs

@@ -6,9 +6,8 @@ use parity_scale_codec::Encode;
 use sp_runtime::{str_array as s, BoundedVec, TokenError};
 
 use crate::{
-	origin::DecisionMethod,
 	types::{Tally, Vote},
-	Call,
+	Call, DecisionMethod,
 };
 use frame_support::assert_noop;
 use pallet_referenda::TrackInfo;
@@ -108,7 +107,7 @@ fn new_test_ext() -> sp_io::TestExternalities {
 		// Community-asset based
 		.add_community(
 			COMMUNITY_B,
-			DecisionMethod::CommunityAsset(COMMUNITY_B_ASSET_ID),
+			DecisionMethod::CommunityAsset(COMMUNITY_B_ASSET_ID, 0),
 			&[BOB, CHARLIE],
 			memberships_of(COMMUNITY_B),
 			Some(CommunityTrack::get()),

pallets/communities/src/types.rs

@@ -1,4 +1,3 @@
-use crate::origin::DecisionMethod;
 use crate::{CommunityDecisionMethod, Config};
 use fc_traits_memberships::{Inspect, Rank};
 use frame_support::pallet_prelude::*;
@@ -16,7 +15,7 @@ pub type NativeBalanceOf<T> = <<T as Config>::Balances as fungible::Inspect<Acco
 pub type AccountIdOf<T> = <T as frame_system::Config>::AccountId;
 pub type CommunityIdOf<T> = <T as Config>::CommunityId;
 pub type VoteOf<T> = Vote<AssetIdOf<T>, AssetBalanceOf<T>, NativeBalanceOf<T>>;
-pub type DecisionMethodFor<T> = DecisionMethod<AssetIdOf<T>>;
+pub type DecisionMethodFor<T> = DecisionMethod<AssetIdOf<T>, AssetBalanceOf<T>>;
 pub type PollIndexOf<T> = <<T as Config>::Polls as Polling<Tally<T>>>::Index;
 pub type AccountIdLookupOf<T> = <<T as frame_system::Config>::Lookup as StaticLookup>::Source;
 pub type PalletsOriginOf<T> =
@@ -54,6 +53,16 @@ pub enum CommunityState {
 	Blocked,
 }
 
+/// The mechanism used by the community or one of its subsets to make decisions
+#[derive(Clone, Debug, Decode, Default, Encode, Eq, MaxEncodedLen, PartialEq, TypeInfo)]
+pub enum DecisionMethod<AssetId, MinVote> {
+	#[default]
+	Membership,
+	NativeToken,
+	CommunityAsset(AssetId, MinVote),
+	Rank,
+}
+
 // Governance
 pub type VoteWeight = u32;
 
@@ -126,7 +135,7 @@ impl<T: Config> Tally<T> {
 			DecisionMethod::Membership => T::MemberMgmt::members_total(&community_id),
 			DecisionMethod::Rank => T::MemberMgmt::ranks_total(&community_id),
 			DecisionMethod::NativeToken => T::Balances::total_issuance().saturated_into::<VoteWeight>(),
-			DecisionMethod::CommunityAsset(asset_id) => {
+			DecisionMethod::CommunityAsset(asset_id, _) => {
 				T::Assets::total_issuance(asset_id).saturated_into::<VoteWeight>()
 			}
 		}