Property not being set

Question

Hi

In my MAUI app I am calling OnItemTapped method from a command.

I have implemented IsBusy logic in OnItemTapped to avoid it being executed again until it is done. At the end of OnItemTapped method I am setting IsBusy to false.

However IsBusy remains true for some reason. This means OnItemTapped will not execute except for the first time.

What am I doing wrong and how can I get IsBusy to become false again at the end of execution of OnItemTapped?

Thanks

Regards

bool _isBusy = false;
public bool IsBusy 
{
  get =>_isBusy;
  set =>SetProperty(ref _isBusy, value);
}

public HomeViewModel() 
{
  IsBusy = false;

  ItemTappedCommand = new Command < BookedJobs > (OnItemTapped);
}

private async void OnItemTapped(BookedJobs job) {
  if (IsBusy) 
  {
    return;
  }

  IsBusy = true;

  await Shell.Current.GoToAsync($ "{typeof(BookedJobsDetailsPage).FullName}", new Dictionary < string, object > { ["BookedJobs1"] = job
  });

  IsBusy = false;  // <== Busy set to false but it does not seem to work
}

protected bool SetProperty < T > (ref T backingStore, T value, [CallerMemberName] string propertyName = "", Action onChanged = null) {
  if (EqualityComparer < T > .Default.Equals(backingStore, value)) return false;

  backingStore = value;
  onChanged ? .Invoke();
  OnPropertyChanged(propertyName);
  return true;
}

protected void OnPropertyChanged([CallerMemberName] string propertyName = "") {
  var changed = PropertyChanged;
  if (changed == null) return;

  changed.Invoke(this, new PropertyChangedEventArgs(propertyName));
}

Answer 1

Hello,

However IsBusy remains true for some reason. This means OnItemTapped will not execute except for the first time.

I can reproduce this issue.

Because Navigation was executed multiple times before IsBusy = false; , I edited your OnItemTapped method. After executing navigation, please wait for this navigating finished, then set IsBusy = false;.

private async void OnItemTapped(BookedJobs job)
    {
        if (IsBusy)
        {
            return;
        }
       IsBusy = true;
        try
        {          
            await Shell.Current.GoToAsync($"{typeof(BookedJobsDetailsPage).FullName}", new Dictionary < string, object > { ["BookedJobs1"] = job
  });
        }
        finally
        {
            await Task.Delay(500);
            IsBusy = false;  // <== Busy set to false but it does not seem to work
        }
   }

Best Regards,

Leon Lu

---

If the answer is the right solution, please click "Accept Answer" and kindly upvote it. If you have extra questions about this answer, please click "Comment".

Note: Please follow the steps in our documentation to enable e-mail notifications if you want to receive the related email notification for this thread.