feat(runtime): create UDs manually with new call universalDividend.claim_uds

Closes #11 (closed) #3 (closed)

pallets/universal-dividend/src/lib.rs

@@ -274,38 +274,35 @@ pub mod pallet {
             total_weight
         }
         fn do_claim_uds(who: &T::AccountId) -> DispatchResultWithPostInfo {
-            let (uds_count, uds_total) =
-                T::MembersStorage::try_mutate_exists(who, |maybe_first_eligible_ud| {
-                    if let Some(FirstEligibleUd(Some(ref mut first_ud_index))) =
-                        maybe_first_eligible_ud
-                    {
-                        let current_ud_index = CurrentUdIndex::<T>::get();
-                        if first_ud_index.get() >= current_ud_index {
-                            Ok((0, Zero::zero()))
-                        } else {
-                            let (uds_count, uds_total) = compute_claim_uds::compute_claim_uds(
-                                current_ud_index,
-                                first_ud_index.get(),
-                                PastReevals::<T>::get().into_iter(),
-                            );
-                            let _ = core::mem::replace(
-                                first_ud_index,
-                                core::num::NonZeroU16::new(current_ud_index)
-                                    .expect("unrechable because current_ud_index is never zero."),
-                            );
-                            Ok((uds_count, uds_total))
-                        }
+            T::MembersStorage::try_mutate_exists(who, |maybe_first_eligible_ud| {
+                if let Some(FirstEligibleUd(Some(ref mut first_ud_index))) = maybe_first_eligible_ud
+                {
+                    let current_ud_index = CurrentUdIndex::<T>::get();
+                    if first_ud_index.get() >= current_ud_index {
+                        DispatchResultWithPostInfo::Ok(().into())
                     } else {
-                        Err::<_, DispatchError>(Error::<T>::AccountNotAllowedToClaimUds.into())
+                        let (uds_count, uds_total) = compute_claim_uds::compute_claim_uds(
+                            current_ud_index,
+                            first_ud_index.get(),
+                            PastReevals::<T>::get().into_iter(),
+                        );
+                        let _ = core::mem::replace(
+                            first_ud_index,
+                            core::num::NonZeroU16::new(current_ud_index)
+                                .expect("unrechable because current_ud_index is never zero."),
+                        );
+                        T::Currency::deposit_creating(who, uds_total);
+                        Self::deposit_event(Event::UdsClaimed {
+                            count: uds_count,
+                            total: uds_total,
+                            who: who.clone(),
+                        });
+                        Ok(().into())
                     }
-                })?;
-            T::Currency::deposit_creating(who, uds_total);
-            Self::deposit_event(Event::UdsClaimed {
-                count: uds_count,
-                total: uds_total,
-                who: who.clone(),
-            });
-            Ok(().into())
+                } else {
+                    Err(Error::<T>::AccountNotAllowedToClaimUds.into())
+                }
+            })
         }
         fn do_transfer_ud(
             origin: OriginFor<T>,