authority lower call size

Author: ferrell-code

authority/src/lib.rs

@@ -267,7 +267,7 @@ pub mod module {
 		#[pallet::weight(T::WeightInfo::fast_track_scheduled_dispatch())]
 		pub fn fast_track_scheduled_dispatch(
 			origin: OriginFor<T>,
-			initial_origin: T::PalletsOrigin,
+			initial_origin: Box<T::PalletsOrigin>,
 			task_id: ScheduleTaskIndex,
 			when: DispatchTime<T::BlockNumber>,
 		) -> DispatchResult {
@@ -285,15 +285,15 @@ pub mod module {
 			T::Scheduler::reschedule_named((&initial_origin, task_id).encode(), when)
 				.map_err(|_| Error::<T>::FailedToFastTrack)?;
 
-			Self::deposit_event(Event::FastTracked(initial_origin, task_id, dispatch_at));
+			Self::deposit_event(Event::FastTracked(*initial_origin, task_id, dispatch_at));
 			Ok(())
 		}
 
 		/// Delay a scheduled dispatchable.
 		#[pallet::weight(T::WeightInfo::delay_scheduled_dispatch())]
 		pub fn delay_scheduled_dispatch(
 			origin: OriginFor<T>,
-			initial_origin: T::PalletsOrigin,
+			initial_origin: Box<T::PalletsOrigin>,
 			task_id: ScheduleTaskIndex,
 			additional_delay: T::BlockNumber,
 		) -> DispatchResult {
@@ -308,21 +308,21 @@ pub mod module {
 			let now = frame_system::Pallet::<T>::block_number();
 			let dispatch_at = now.saturating_add(additional_delay);
 
-			Self::deposit_event(Event::Delayed(initial_origin, task_id, dispatch_at));
+			Self::deposit_event(Event::Delayed(*initial_origin, task_id, dispatch_at));
 			Ok(())
 		}
 
 		/// Cancel a scheduled dispatchable.
 		#[pallet::weight(T::WeightInfo::cancel_scheduled_dispatch())]
 		pub fn cancel_scheduled_dispatch(
 			origin: OriginFor<T>,
-			initial_origin: T::PalletsOrigin,
+			initial_origin: Box<T::PalletsOrigin>,
 			task_id: ScheduleTaskIndex,
 		) -> DispatchResult {
 			T::AuthorityConfig::check_cancel_schedule(origin, &initial_origin)?;
 			T::Scheduler::cancel_named((&initial_origin, task_id).encode()).map_err(|_| Error::<T>::FailedToCancel)?;
 
-			Self::deposit_event(Event::Cancelled(initial_origin, task_id));
+			Self::deposit_event(Event::Cancelled(*initial_origin, task_id));
 			Ok(())
 		}
 	}

authority/src/tests.rs

@@ -207,7 +207,7 @@ fn fast_track_scheduled_dispatch_work() {
 		let pallets_origin = schedule_origin.caller().clone();
 		assert_ok!(Authority::fast_track_scheduled_dispatch(
 			Origin::root(),
-			pallets_origin,
+			Box::new(pallets_origin),
 			0,
 			DispatchTime::At(4),
 		));
@@ -234,7 +234,7 @@ fn fast_track_scheduled_dispatch_work() {
 
 		assert_ok!(Authority::fast_track_scheduled_dispatch(
 			Origin::root(),
-			frame_system::RawOrigin::Root.into(),
+			Box::new(frame_system::RawOrigin::Root.into()),
 			1,
 			DispatchTime::At(4),
 		));
@@ -284,7 +284,7 @@ fn delay_scheduled_dispatch_work() {
 		let pallets_origin = schedule_origin.caller().clone();
 		assert_ok!(Authority::delay_scheduled_dispatch(
 			Origin::root(),
-			pallets_origin,
+			Box::new(pallets_origin),
 			0,
 			4,
 		));
@@ -311,7 +311,7 @@ fn delay_scheduled_dispatch_work() {
 
 		assert_ok!(Authority::delay_scheduled_dispatch(
 			Origin::root(),
-			frame_system::RawOrigin::Root.into(),
+			Box::new(frame_system::RawOrigin::Root.into()),
 			1,
 			4,
 		));
@@ -358,7 +358,11 @@ fn cancel_scheduled_dispatch_work() {
 		};
 
 		let pallets_origin = schedule_origin.caller().clone();
-		assert_ok!(Authority::cancel_scheduled_dispatch(Origin::root(), pallets_origin, 0));
+		assert_ok!(Authority::cancel_scheduled_dispatch(
+			Origin::root(),
+			Box::new(pallets_origin),
+			0
+		));
 		System::assert_last_event(mock::Event::Authority(Event::Cancelled(
 			OriginCaller::Authority(DelayedOrigin {
 				delay: 1,
@@ -381,7 +385,7 @@ fn cancel_scheduled_dispatch_work() {
 
 		assert_ok!(Authority::cancel_scheduled_dispatch(
 			Origin::root(),
-			frame_system::RawOrigin::Root.into(),
+			Box::new(frame_system::RawOrigin::Root.into()),
 			1
 		));
 		System::assert_last_event(mock::Event::Authority(Event::Cancelled(