feat(permission0): add last update field

Author: saiintbrisson

pallets/permission0/src/ext/namespace_impl.rs

@@ -7,7 +7,7 @@ use pallet_permission0_api::Permission0NamespacesApi;
 use pallet_torus0_api::{NamespacePath, NamespacePathInner, Torus0Api};
 use polkadot_sdk::{
     frame_support::{dispatch::DispatchResult, ensure},
-    frame_system::ensure_signed,
+    frame_system::{self, ensure_signed},
     polkadot_sdk_frame::prelude::OriginFor,
     sp_core::Get,
     sp_runtime::{BoundedBTreeMap, BoundedBTreeSet, DispatchError},
@@ -309,6 +309,7 @@ pub(crate) fn update_namespace_permission<T: Config>(
     }
 
     permission.max_instances = max_instances;
+    permission.last_update = frame_system::Pallet::<T>::block_number();
     Permissions::<T>::set(permission_id, Some(permission));
 
     Ok(())

pallets/permission0/src/ext/stream_impl.rs

@@ -13,7 +13,7 @@ use pallet_permission0_api::{
 };
 use polkadot_sdk::{
     frame_support::{dispatch::DispatchResult, ensure, traits::ReservableCurrency},
-    frame_system::{ensure_signed, ensure_signed_or_root},
+    frame_system::{self, ensure_signed, ensure_signed_or_root},
     polkadot_sdk_frame::prelude::{BlockNumberFor, OriginFor},
     sp_core::{Get, TryCollect},
     sp_runtime::{
@@ -424,6 +424,7 @@ pub(crate) fn update_stream_permission<T: Config>(
     }
 
     permission.scope = PermissionScope::Stream(scope);
+    permission.last_update = frame_system::Pallet::<T>::block_number();
     Permissions::<T>::set(permission_id, Some(permission));
 
     Ok(())

pallets/permission0/src/migrations.rs

@@ -169,6 +169,7 @@ pub mod v6 {
                     duration: old_contract.duration,
                     revocation: old_contract.revocation,
                     enforcement: old_contract.enforcement,
+                    last_update: old_contract.created_at,
                     last_execution: old_contract.last_execution,
                     execution_count: old_contract.execution_count,
                     children: old_contract.children,

pallets/permission0/src/permission.rs

@@ -61,6 +61,8 @@ pub struct PermissionContract<T: Config> {
     pub revocation: RevocationTerms<T>,
     /// Enforcement authority that can toggle the permission
     pub enforcement: EnforcementAuthority<T>,
+    /// Last update block
+    pub last_update: BlockNumberFor<T>,
     /// Last execution block
     #[doc(hidden)]
     pub last_execution: Option<BlockNumberFor<T>>,
@@ -83,6 +85,7 @@ impl<T: Config> PermissionContract<T> {
         enforcement: EnforcementAuthority<T>,
         max_instances: u32,
     ) -> Self {
+        let now = frame_system::Pallet::<T>::block_number();
         Self {
             delegator,
             scope,
@@ -91,10 +94,11 @@ impl<T: Config> PermissionContract<T> {
             enforcement,
             max_instances,
 
+            last_update: now,
             last_execution: None,
             execution_count: 0,
             children: BoundedBTreeSet::new(),
-            created_at: frame_system::Pallet::<T>::block_number(),
+            created_at: now,
         }
     }
 }