Add set_partial_params dispatchable function

substrate/frame/core-fellowship/src/lib.rs

@@ -84,6 +84,15 @@ pub mod weights;
 pub use pallet::*;
 pub use weights::WeightInfo;
 
+macro_rules! override_params {
+	( $x:expr, $y: expr ) => {
+		for (base_element, new_element) in $x.iter_mut().zip($y) {
+			if new_element.is_some() {
+				*base_element = new_element.unwrap();
+			}
+		}
+	};
+}
 /// The desired outcome for which evidence is presented.
 #[derive(Encode, Decode, Eq, PartialEq, Copy, Clone, TypeInfo, MaxEncodedLen, RuntimeDebug)]
 pub enum Wish {
@@ -196,6 +205,8 @@ pub mod pallet {
 	}
 
 	pub type ParamsOf<T, I> = ParamsType<<T as Config<I>>::Balance, BlockNumberFor<T>, RANK_COUNT>;
+	pub type PartialParamsOf<T, I> =
+		ParamsType<Option<<T as Config<I>>::Balance>, Option<BlockNumberFor<T>>, RANK_COUNT>;
 	pub type MemberStatusOf<T> = MemberStatus<BlockNumberFor<T>>;
 	pub type RankOf<T, I> = <<T as Config<I>>::Members as RankedMembers>::Rank;
 
@@ -530,6 +541,31 @@ pub mod pallet {
 
 			Ok(Pays::No.into())
 		}
+
+		/// Set the parameters.
+		///
+		/// - `origin`: An origin complying with `ParamsOrigin` or root.
+		/// - `params`: The new parameters for the pallet.
+		#[pallet::weight(T::WeightInfo::set_params())]
+		#[pallet::call_index(9)]
+		pub fn set_partial_params(
+			origin: OriginFor<T>,
+			partial_params: Box<PartialParamsOf<T, I>>,
+		) -> DispatchResult {
+			T::ParamsOrigin::ensure_origin_or_root(origin)?;
+			Params::<T, I>::mutate(|p| {
+				override_params!(p.active_salary, partial_params.active_salary);
+				override_params!(p.passive_salary, partial_params.passive_salary);
+				override_params!(p.demotion_period, partial_params.demotion_period);
+				override_params!(p.min_promotion_period, partial_params.min_promotion_period);
+				if partial_params.offboard_timeout.is_some() {
+					p.offboard_timeout = partial_params.offboard_timeout.unwrap();
+				}
+			});
+			let params = Params::<T, I>::get();
+			Self::deposit_event(Event::<T, I>::ParamsChanged { params });
+			Ok(())
+		}
 	}
 
 	impl<T: Config<I>, I: 'static> Pallet<T, I> {

substrate/frame/core-fellowship/src/tests/unit.rs

@@ -184,6 +184,24 @@ fn set_params_works() {
 	});
 }
 
+#[test]
+fn set_partial_params_works() {
+	new_test_ext().execute_with(|| {
+		let params = ParamsType {
+			active_salary: [None; 9],
+			passive_salary: [None; 9],
+			demotion_period: [None, Some(10), None, None, None, None, None, None, None],
+			min_promotion_period: [None; 9],
+			offboard_timeout: Some(1),
+		};
+		assert_noop!(
+			CoreFellowship::set_partial_params(signed(2), Box::new(params.clone())),
+			DispatchError::BadOrigin
+		);
+		assert_ok!(CoreFellowship::set_partial_params(signed(1), Box::new(params)));
+	});
+}
+
 #[test]
 fn induct_works() {
 	new_test_ext().execute_with(|| {