diff --git a/src/PostIt/PostIt/ViewModels/ActionStatusViewModelExtensions.cs b/src/PostIt/PostIt/ViewModels/ActionStatusViewModelExtensions.cs new file mode 100644 index 000000000..614d477c8 --- /dev/null +++ b/src/PostIt/PostIt/ViewModels/ActionStatusViewModelExtensions.cs @@ -0,0 +1,32 @@ +namespace PostIt.ViewModels; + +public interface IActionStatusViewModel +{ + string StatusMessage { get; set; } + StatusNotice ActionStatus { get; set; } +} + +public static class ActionStatusViewModelExtensions +{ + public static void SetInfoStatus(this IActionStatusViewModel viewModel, string message) + => viewModel.SetStatus(message, StatusSeverity.Info); + + public static void SetWarningStatus(this IActionStatusViewModel viewModel, string message) + => viewModel.SetStatus(message, StatusSeverity.Warning); + + public static void SetErrorStatus(this IActionStatusViewModel viewModel, string message) + => viewModel.SetStatus(message, StatusSeverity.Error); + + public static void SetStatus(this IActionStatusViewModel viewModel, string message, StatusSeverity severity) + { + var normalizedMessage = string.IsNullOrWhiteSpace(message) ? "Pret." : message.Trim(); + + viewModel.StatusMessage = normalizedMessage; + viewModel.ActionStatus = severity switch + { + StatusSeverity.Error => StatusNotice.Error(normalizedMessage), + StatusSeverity.Warning => StatusNotice.Warning(normalizedMessage), + _ => StatusNotice.Info(normalizedMessage), + }; + } +} \ No newline at end of file diff --git a/src/PostIt/PostIt/ViewModels/ActivitiesPageViewModel.cs b/src/PostIt/PostIt/ViewModels/ActivitiesPageViewModel.cs index 801cf21d4..bd13faf9b 100644 --- a/src/PostIt/PostIt/ViewModels/ActivitiesPageViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/ActivitiesPageViewModel.cs @@ -13,7 +13,7 @@ using Yavsc.Api.Client; namespace PostIt.ViewModels; -public partial class ActivitiesPageViewModel : ViewModelBase +public partial class ActivitiesPageViewModel : ViewModelBase, IActionStatusViewModel { private readonly ActivityApiClient _client; private readonly BillingApiClient _billingClient; @@ -46,11 +46,6 @@ public partial class ActivitiesPageViewModel : ViewModelBase [ObservableProperty] public partial StatusNotice ActionStatus { get; set; } = StatusNotice.Info("Choisissez une activité."); - partial void OnStatusMessageChanged(string value) - { - ActionStatus = StatusNotice.FromMessage(value); - } - public ActivityInfo? CurrentActivity => SelectedSpecialization ?? SelectedActivity; public string SelectedActivityLabel => SelectedActivity?.Name ?? "(aucune activité)"; public string CurrentActivityLabel => CurrentActivity?.Name ?? "(aucune)"; @@ -94,11 +89,11 @@ public partial class ActivitiesPageViewModel : ViewModelBase } catch (HttpRequestException ex) when (ex.StatusCode is HttpStatusCode.Unauthorized or HttpStatusCode.Forbidden) { - StatusMessage = "Accès refusé pour les activités (scope 'api'). Déconnectez puis reconnectez-vous."; + this.SetWarningStatus("Accès refusé pour les activités (scope 'api'). Déconnectez puis reconnectez-vous."); } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } } @@ -110,11 +105,11 @@ public partial class ActivitiesPageViewModel : ViewModelBase } catch (HttpRequestException ex) when (ex.StatusCode is HttpStatusCode.Unauthorized or HttpStatusCode.Forbidden) { - StatusMessage = "Accès refusé pour les activités (scope 'api'). Déconnectez puis reconnectez-vous."; + this.SetWarningStatus("Accès refusé pour les activités (scope 'api'). Déconnectez puis reconnectez-vous."); } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } } @@ -131,7 +126,7 @@ public partial class ActivitiesPageViewModel : ViewModelBase await ShowActivityAsync(first); if (first is null) { - StatusMessage = "Aucune activité disponible."; + this.SetInfoStatus("Aucune activité disponible."); } } catch (HttpRequestException ex) when (ex.StatusCode is HttpStatusCode.Unauthorized or HttpStatusCode.Forbidden) @@ -139,14 +134,14 @@ public partial class ActivitiesPageViewModel : ViewModelBase Activities = new ObservableCollection(); Specializations = new ObservableCollection(); Performers = new ObservableCollection(); - StatusMessage = "Accès refusé pour les activités (scope 'api'). Déconnectez puis reconnectez-vous."; + this.SetWarningStatus("Accès refusé pour les activités (scope 'api'). Déconnectez puis reconnectez-vous."); } catch (Exception ex) { Activities = new ObservableCollection(); Specializations = new ObservableCollection(); Performers = new ObservableCollection(); - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } finally { @@ -238,19 +233,19 @@ public partial class ActivitiesPageViewModel : ViewModelBase Performers = new ObservableCollection(items); SelectedPerformer = null; - StatusMessage = $"{activity.Name} · {Performers.Count} utilisateur(s)"; + this.SetInfoStatus($"{activity.Name} · {Performers.Count} utilisateur(s)"); } catch (HttpRequestException ex) when (ex.StatusCode is HttpStatusCode.Unauthorized or HttpStatusCode.Forbidden) { Performers = new ObservableCollection(); SelectedPerformer = null; - StatusMessage = "Accès refusé pour les activités (scope 'api'). Déconnectez puis reconnectez-vous."; + this.SetWarningStatus("Accès refusé pour les activités (scope 'api'). Déconnectez puis reconnectez-vous."); } catch (Exception ex) { Performers = new ObservableCollection(); SelectedPerformer = null; - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } finally { @@ -267,7 +262,7 @@ public partial class ActivitiesPageViewModel : ViewModelBase { if (SelectedPerformer is null || CurrentActivity is null) { - StatusMessage = "Sélectionnez un utilisateur et une activité avec formulaire."; + this.SetWarningStatus("Sélectionnez un utilisateur et une activité avec formulaire."); return; } diff --git a/src/PostIt/PostIt/ViewModels/AddCircleMemberDialogViewModel.cs b/src/PostIt/PostIt/ViewModels/AddCircleMemberDialogViewModel.cs index d3f9deb68..635073d1a 100644 --- a/src/PostIt/PostIt/ViewModels/AddCircleMemberDialogViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/AddCircleMemberDialogViewModel.cs @@ -29,7 +29,7 @@ namespace PostIt.ViewModels; /// CirclesPage then calls /// . /// -public partial class AddCircleMemberDialogViewModel : ViewModelBase +public partial class AddCircleMemberDialogViewModel : ViewModelBase, IActionStatusViewModel { private readonly IUserDirectory _directory; @@ -46,15 +46,10 @@ public partial class AddCircleMemberDialogViewModel : ViewModelBase public partial bool IsBusy { get; set; } [ObservableProperty] - public partial string StatusMessage { get; set; } = string.Empty; + public partial string StatusMessage { get; set; } = "Pret."; [ObservableProperty] - public partial StatusNotice ActionStatus { get; set; } = StatusNotice.FromMessage(string.Empty); - - partial void OnStatusMessageChanged(string value) - { - ActionStatus = StatusNotice.FromMessage(value); - } + public partial StatusNotice ActionStatus { get; set; } = StatusNotice.Info("Pret."); /// /// Raised when the user confirms a selection. The hosting @@ -87,7 +82,7 @@ public partial class AddCircleMemberDialogViewModel : ViewModelBase if (string.IsNullOrWhiteSpace(SearchQuery)) { Results.Clear(); - StatusMessage = "Tapez un nom ou un email"; + this.SetWarningStatus("Tapez un nom ou un email"); return; } @@ -96,11 +91,11 @@ public partial class AddCircleMemberDialogViewModel : ViewModelBase { var hits = await _directory.SearchAsync(SearchQuery, CancellationToken.None).ConfigureAwait(true); Results = new ObservableCollection(hits ?? Array.Empty()); - StatusMessage = $"{Results.Count} résultat(s)"; + this.SetInfoStatus($"{Results.Count} résultat(s)"); } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } finally { @@ -118,7 +113,7 @@ public partial class AddCircleMemberDialogViewModel : ViewModelBase { if (Selected is null) { - StatusMessage = "Sélectionnez un utilisateur"; + this.SetWarningStatus("Sélectionnez un utilisateur"); return; } Confirmed?.Invoke(this, Selected); diff --git a/src/PostIt/PostIt/ViewModels/BillingQueriesPageViewModel.cs b/src/PostIt/PostIt/ViewModels/BillingQueriesPageViewModel.cs index 098cdc6f4..d3c097f43 100644 --- a/src/PostIt/PostIt/ViewModels/BillingQueriesPageViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/BillingQueriesPageViewModel.cs @@ -14,7 +14,7 @@ using Yavsc.Abstract.Workflow; namespace PostIt.ViewModels; -public partial class BillingQueriesPageViewModel : ViewModelBase +public partial class BillingQueriesPageViewModel : ViewModelBase, IActionStatusViewModel { private readonly BillingApiClient _billingClient; @@ -37,12 +37,7 @@ public partial class BillingQueriesPageViewModel : ViewModelBase public partial string StatusMessage { get; set; } = "Chargement des commandes..."; [ObservableProperty] - public partial StatusNotice ActionStatus { get; set; } = StatusNotice.FromMessage("Chargement des commandes..."); - - partial void OnStatusMessageChanged(string value) - { - ActionStatus = StatusNotice.FromMessage(value); - } + public partial StatusNotice ActionStatus { get; set; } = StatusNotice.Info("Chargement des commandes..."); public string Title => IsReadOnly ? $"Demandes en cours ({Form.Title})" @@ -98,17 +93,17 @@ public partial class BillingQueriesPageViewModel : ViewModelBase .ToList(); Queries = new ObservableCollection(filtered); - StatusMessage = BuildLoadedStatusMessage(filtered.Count); + this.SetInfoStatus(BuildLoadedStatusMessage(filtered.Count)); } catch (HttpRequestException ex) when (ex.StatusCode is HttpStatusCode.Unauthorized or HttpStatusCode.Forbidden) { Queries = new ObservableCollection(); - StatusMessage = "Accès refusé au billing (scope 'api'). Déconnectez puis reconnectez-vous."; + this.SetWarningStatus("Accès refusé au billing (scope 'api'). Déconnectez puis reconnectez-vous."); } catch (Exception ex) { Queries = new ObservableCollection(); - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } finally { @@ -121,13 +116,13 @@ public partial class BillingQueriesPageViewModel : ViewModelBase { if (IsReadOnly) { - StatusMessage = "Mode lecture seule: l'ouverture en modification est désactivée."; + this.SetWarningStatus("Mode lecture seule: l'ouverture en modification est désactivée."); return; } if (SelectedQuery is null) { - StatusMessage = "Sélectionnez une commande."; + this.SetWarningStatus("Sélectionnez une commande."); return; } @@ -147,11 +142,11 @@ public partial class BillingQueriesPageViewModel : ViewModelBase } catch (HttpRequestException ex) when (ex.StatusCode is HttpStatusCode.Unauthorized or HttpStatusCode.Forbidden) { - StatusMessage = "Accès refusé au billing (scope 'api'). Déconnectez puis reconnectez-vous."; + this.SetWarningStatus("Accès refusé au billing (scope 'api'). Déconnectez puis reconnectez-vous."); } catch (Exception ex) { - StatusMessage = $"Erreur lors de l'ouverture: {ex.Message}"; + this.SetErrorStatus($"Erreur lors de l'ouverture: {ex.Message}"); } finally { diff --git a/src/PostIt/PostIt/ViewModels/CirclesPageViewModel.cs b/src/PostIt/PostIt/ViewModels/CirclesPageViewModel.cs index 73b87a181..cd068b4f4 100644 --- a/src/PostIt/PostIt/ViewModels/CirclesPageViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/CirclesPageViewModel.cs @@ -34,7 +34,7 @@ namespace PostIt.ViewModels; /// . The "remove" /// command is per-row and runs inline. /// -public partial class CirclesPageViewModel : ViewModelBase +public partial class CirclesPageViewModel : ViewModelBase, IActionStatusViewModel { private readonly CircleApiClient _client; @@ -63,17 +63,11 @@ public partial class CirclesPageViewModel : ViewModelBase public partial bool IsBusy { get; set; } [ObservableProperty] - public partial string StatusMessage { get; set; } = string.Empty; + public partial string StatusMessage { get; set; } = "Pret."; [ObservableProperty] public partial StatusNotice ActionStatus { get; set; } = StatusNotice.Info("Pret."); - partial void OnStatusMessageChanged(string value) - { - ActionStatus = StatusNotice.FromMessage(value); - } - - public CirclesPageViewModel(CircleApiClient client) { _client = client ?? throw new ArgumentNullException(nameof(client)); @@ -108,11 +102,11 @@ public partial class CirclesPageViewModel : ViewModelBase { var list = await _client.GetMyCirclesAsync(); Circles = new ObservableCollection(list ?? new()); - StatusMessage = $"{Circles.Count} cercle(s)"; + this.SetInfoStatus($"{Circles.Count} cercle(s)"); } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } finally { @@ -157,11 +151,11 @@ public partial class CirclesPageViewModel : ViewModelBase { var list = await _client.GetMembersAsync(circleId); Members = new ObservableCollection(list ?? new()); - StatusMessage = $"{Members.Count} membre(s)"; + this.SetInfoStatus($"{Members.Count} membre(s)"); } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); Members = new ObservableCollection(); } finally @@ -176,7 +170,7 @@ public partial class CirclesPageViewModel : ViewModelBase SelectedCircle = null; DraftName = string.Empty; DraftPublic = false; - StatusMessage = "Nouveau cercle"; + this.SetInfoStatus("Nouveau cercle"); } [RelayCommand] @@ -186,7 +180,7 @@ public partial class CirclesPageViewModel : ViewModelBase SelectedCircle = circle; DraftName = circle.Name; DraftPublic = circle.Public; - StatusMessage = $"Édition de « {circle.Name} »"; + this.SetInfoStatus($"Édition de « {circle.Name} »"); } [RelayCommand] @@ -194,7 +188,7 @@ public partial class CirclesPageViewModel : ViewModelBase { if (string.IsNullOrWhiteSpace(DraftName)) { - StatusMessage = "Le nom est obligatoire"; + this.SetWarningStatus("Le nom est obligatoire"); return; } @@ -208,22 +202,22 @@ public partial class CirclesPageViewModel : ViewModelBase Name = DraftName.Trim(), Public = DraftPublic, }); - StatusMessage = created is null - ? "Création échouée" - : $"Cercle « {created.Name} » créé"; + this.SetStatus( + created is null ? "Création échouée" : $"Cercle « {created.Name} » créé", + created is null ? StatusSeverity.Warning : StatusSeverity.Info); } else { SelectedCircle.Name = DraftName.Trim(); SelectedCircle.Public = DraftPublic; await _client.UpdateCircleAsync(SelectedCircle.Id, SelectedCircle); - StatusMessage = $"Cercle « {SelectedCircle.Name} » mis à jour"; + this.SetInfoStatus($"Cercle « {SelectedCircle.Name} » mis à jour"); } await RefreshAsync(); } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } finally { @@ -239,7 +233,7 @@ public partial class CirclesPageViewModel : ViewModelBase try { await _client.DeleteCircleAsync(circle.Id); - StatusMessage = $"Cercle « {circle.Name} » supprimé"; + this.SetInfoStatus($"Cercle « {circle.Name} » supprimé"); // If the deleted circle was the selected one, // clear the selection so the Members view goes // empty too (the partial setter on @@ -250,7 +244,7 @@ public partial class CirclesPageViewModel : ViewModelBase } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } finally { @@ -271,7 +265,7 @@ public partial class CirclesPageViewModel : ViewModelBase try { await _client.AddMemberAsync(SelectedCircle.Id, picked.Id); - StatusMessage = $"« {picked.DisplayName} » ajouté au cercle"; + this.SetInfoStatus($"« {picked.DisplayName} » ajouté au cercle"); await LoadMembersAsync(SelectedCircle.Id); } catch (Exception ex) @@ -286,7 +280,7 @@ public partial class CirclesPageViewModel : ViewModelBase var msg = ex.Message.Contains("409") || ex.Message.Contains("Conflict") ? "Déjà membre du cercle" : $"Erreur: {ex.Message}"; - StatusMessage = msg; + this.SetStatus(msg, msg == "Déjà membre du cercle" ? StatusSeverity.Warning : StatusSeverity.Error); } finally { @@ -307,11 +301,11 @@ public partial class CirclesPageViewModel : ViewModelBase { await _client.RemoveMemberAsync(SelectedCircle.Id, member.Id); Members.Remove(member); - StatusMessage = $"« {member.UserName} » retiré du cercle"; + this.SetInfoStatus($"« {member.UserName} » retiré du cercle"); } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } finally { diff --git a/src/PostIt/PostIt/ViewModels/CommandFormsPageViewModel.cs b/src/PostIt/PostIt/ViewModels/CommandFormsPageViewModel.cs index fd4c9ec8d..533fbc380 100644 --- a/src/PostIt/PostIt/ViewModels/CommandFormsPageViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/CommandFormsPageViewModel.cs @@ -11,7 +11,7 @@ using Yavsc.Api.Client; namespace PostIt.ViewModels; -public partial class CommandFormsPageViewModel : ViewModelBase +public partial class CommandFormsPageViewModel : ViewModelBase, IActionStatusViewModel { private readonly BillingApiClient _billingClient; @@ -25,15 +25,10 @@ public partial class CommandFormsPageViewModel : ViewModelBase public partial CommandFormSummary? SelectedForm { get; set; } [ObservableProperty] - public partial string StatusMessage { get; set; } + public partial string StatusMessage { get; set; } = "Pret."; [ObservableProperty] - public partial StatusNotice ActionStatus { get; set; } = StatusNotice.FromMessage(string.Empty); - - partial void OnStatusMessageChanged(string value) - { - ActionStatus = StatusNotice.FromMessage(value); - } + public partial StatusNotice ActionStatus { get; set; } = StatusNotice.Info("Pret."); public string Title => $"Formulaires pour {Performer.UserName}"; public string ContextLabel => $"{Activity.Name} · {Forms.Count} formulaire(s)"; @@ -63,9 +58,11 @@ public partial class CommandFormsPageViewModel : ViewModelBase .OrderBy(f => f.Title) .ThenBy(f => f.ActionName)); SelectedForm = Forms.FirstOrDefault(); - StatusMessage = Forms.Count == 0 - ? "Aucun formulaire n'est disponible pour cette activité." - : "Choisissez le formulaire à utiliser."; + this.SetStatus( + Forms.Count == 0 + ? "Aucun formulaire n'est disponible pour cette activité." + : "Choisissez le formulaire à utiliser.", + Forms.Count == 0 ? StatusSeverity.Warning : StatusSeverity.Info); } private bool CanOpenSelectedForm() => SelectedForm is not null; @@ -79,7 +76,7 @@ public partial class CommandFormsPageViewModel : ViewModelBase { if (SelectedForm is null) { - StatusMessage = "Sélectionnez un formulaire."; + this.SetWarningStatus("Sélectionnez un formulaire."); return; } @@ -100,7 +97,7 @@ public partial class CommandFormsPageViewModel : ViewModelBase { if (SelectedForm is null) { - StatusMessage = "Sélectionnez un formulaire."; + this.SetWarningStatus("Sélectionnez un formulaire."); return; } @@ -120,7 +117,7 @@ public partial class CommandFormsPageViewModel : ViewModelBase { if (SelectedForm is null) { - StatusMessage = "Sélectionnez un formulaire."; + this.SetWarningStatus("Sélectionnez un formulaire."); return; } diff --git a/src/PostIt/PostIt/ViewModels/Commands/BillingCommandPageViewModel.cs b/src/PostIt/PostIt/ViewModels/Commands/BillingCommandPageViewModel.cs index 5fd7d8461..5ad2b08f4 100644 --- a/src/PostIt/PostIt/ViewModels/Commands/BillingCommandPageViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/Commands/BillingCommandPageViewModel.cs @@ -9,7 +9,7 @@ using Yavsc.Models.Billing; namespace PostIt.ViewModels; -public abstract partial class BillingCommandPageViewModel : RemoteViewModelBase +public abstract partial class BillingCommandPageViewModel : RemoteViewModelBase, IActionStatusViewModel { protected readonly BillingApiClient _billingClient; @@ -21,7 +21,7 @@ public abstract partial class BillingCommandPageViewModel : RemoteViewModelBase public partial bool IsBusy { get; set; } [ObservableProperty] - public partial string StatusMessage { get; set; } + public partial string StatusMessage { get; set; } = "Pret."; [ObservableProperty] public partial StatusNotice ActionStatus { get; set; } = StatusNotice.Info("Pret."); @@ -78,7 +78,7 @@ public abstract partial class BillingCommandPageViewModel : RemoteViewModelBase Form = form ?? throw new ArgumentNullException(nameof(form)); _billingClient = billingClient ?? throw new ArgumentNullException(nameof(billingClient)); - StatusMessage = SupportMessage; + this.SetInfoStatus(SupportMessage); } partial void OnExistingQueryIdChanged(long? value) @@ -92,11 +92,6 @@ public abstract partial class BillingCommandPageViewModel : RemoteViewModelBase OnPropertyChanged(nameof(CanUseCurrentLocation)); } - partial void OnStatusMessageChanged(string value) - { - ActionStatus = StatusNotice.FromMessage(value); - } - public async Task InitializeAsync(BillingQueryDetailsDto? existingQuery = null) { await LoadAsync(); diff --git a/src/PostIt/PostIt/ViewModels/Commands/BrushViewModel.cs b/src/PostIt/PostIt/ViewModels/Commands/BrushViewModel.cs index d49fde803..72d9ce8ea 100644 --- a/src/PostIt/PostIt/ViewModels/Commands/BrushViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/Commands/BrushViewModel.cs @@ -58,18 +58,20 @@ public partial class BrushViewModel : RdvViewModel SelectedPrestation = AvailablePrestations.FirstOrDefault(); } - StatusMessage = AvailablePrestations.Count == 0 - ? "Aucune prestation coiffure disponible." - : SupportMessage; + this.SetStatus( + AvailablePrestations.Count == 0 + ? "Aucune prestation coiffure disponible." + : SupportMessage, + AvailablePrestations.Count == 0 ? StatusSeverity.Warning : StatusSeverity.Info); } catch (HttpRequestException ex) when (ex.StatusCode is HttpStatusCode.Unauthorized or HttpStatusCode.Forbidden) { - StatusMessage = "Accès refusé au catalogue de prestations (scope 'api'). Déconnectez puis reconnectez-vous."; + this.SetWarningStatus("Accès refusé au catalogue de prestations (scope 'api'). Déconnectez puis reconnectez-vous."); } catch (Exception ex) { - StatusMessage = $"Erreur lors du chargement des prestations: {ex.Message}"; + this.SetErrorStatus($"Erreur lors du chargement des prestations: {ex.Message}"); } finally { @@ -81,19 +83,19 @@ public partial class BrushViewModel : RdvViewModel { if (!Consent) { - StatusMessage = "Le consentement est requis pour poster la commande."; + this.SetWarningStatus("Le consentement est requis pour poster la commande."); return; } if (string.IsNullOrWhiteSpace(Address)) { - StatusMessage = "L'adresse du rendez-vous est requise."; + this.SetWarningStatus("L'adresse du rendez-vous est requise."); return; } if (SelectedPrestation is null) { - StatusMessage = "Sélectionnez une prestation coiffure."; + this.SetWarningStatus("Sélectionnez une prestation coiffure."); return; } @@ -143,18 +145,18 @@ public partial class BrushViewModel : RdvViewModel }).ConfigureAwait(true); } - StatusMessage = IsEditingExisting + this.SetInfoStatus(IsEditingExisting ? $"Commande #{ExistingQueryId} mise à jour sur {BillingRoute} pour {Performer.UserName}." - : $"Commande transmise sur {BillingRoute} pour {Performer.UserName}."; + : $"Commande transmise sur {BillingRoute} pour {Performer.UserName}."); } catch (HttpRequestException ex) when (ex.StatusCode is HttpStatusCode.Unauthorized or HttpStatusCode.Forbidden) { - StatusMessage = "Accès refusé au billing (scope 'api'). Déconnectez puis reconnectez-vous."; + this.SetWarningStatus("Accès refusé au billing (scope 'api'). Déconnectez puis reconnectez-vous."); } catch (Exception ex) { - StatusMessage = $"Erreur lors de l'envoi de la commande: {ex.Message}"; + this.SetErrorStatus($"Erreur lors de l'envoi de la commande: {ex.Message}"); } finally { diff --git a/src/PostIt/PostIt/ViewModels/Commands/MBrushViewModel.cs b/src/PostIt/PostIt/ViewModels/Commands/MBrushViewModel.cs index 793a77516..05481c2c3 100644 --- a/src/PostIt/PostIt/ViewModels/Commands/MBrushViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/Commands/MBrushViewModel.cs @@ -49,20 +49,20 @@ public partial class MBrushViewModel : BrushViewModel { if (!Consent) { - StatusMessage = "Le consentement est requis pour poster la commande."; + this.SetWarningStatus("Le consentement est requis pour poster la commande."); return; } if (string.IsNullOrWhiteSpace(Address)) { - StatusMessage = "L'adresse du rendez-vous est requise."; + this.SetWarningStatus("L'adresse du rendez-vous est requise."); return; } var selectedPrestations = MultiPrestations.Where(x => x.IsSelected).ToList(); if (selectedPrestations.Count == 0) { - StatusMessage = "Sélectionnez au moins une prestation coiffure."; + this.SetWarningStatus("Sélectionnez au moins une prestation coiffure."); return; } @@ -109,18 +109,18 @@ public partial class MBrushViewModel : BrushViewModel }).ConfigureAwait(true); } - StatusMessage = IsEditingExisting + this.SetInfoStatus(IsEditingExisting ? $"Commande #{ExistingQueryId} mise à jour sur {BillingRoute} pour {Performer.UserName}." - : $"Commande transmise sur {BillingRoute} pour {Performer.UserName}."; + : $"Commande transmise sur {BillingRoute} pour {Performer.UserName}."); } catch (HttpRequestException ex) when (ex.StatusCode is HttpStatusCode.Unauthorized or HttpStatusCode.Forbidden) { - StatusMessage = "Accès refusé au billing (scope 'api'). Déconnectez puis reconnectez-vous."; + this.SetWarningStatus("Accès refusé au billing (scope 'api'). Déconnectez puis reconnectez-vous."); } catch (Exception ex) { - StatusMessage = $"Erreur lors de l'envoi de la commande: {ex.Message}"; + this.SetErrorStatus($"Erreur lors de l'envoi de la commande: {ex.Message}"); } finally { diff --git a/src/PostIt/PostIt/ViewModels/Commands/RdvViewModel.cs b/src/PostIt/PostIt/ViewModels/Commands/RdvViewModel.cs index c2ff3294e..afd0c54bd 100644 --- a/src/PostIt/PostIt/ViewModels/Commands/RdvViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/Commands/RdvViewModel.cs @@ -54,7 +54,7 @@ public partial class RdvViewModel : BillingCommandPageViewModel Longitude = existingQuery.Location.Longitude; } - StatusMessage = $"Commande #{existingQuery.Id} chargée."; + this.SetInfoStatus($"Commande #{existingQuery.Id} chargée."); } [RelayCommand(CanExecute = nameof(CanUseCurrentLocation))] @@ -71,23 +71,23 @@ public partial class RdvViewModel : BillingCommandPageViewModel var result = await Platform.TryGetCurrentLocationAsync(default).ConfigureAwait(true); if (!result.IsSuccess || !result.Latitude.HasValue || !result.Longitude.HasValue) { - StatusMessage = result.Message; + this.SetWarningStatus(result.Message); return; } Latitude = result.Latitude.Value; Longitude = result.Longitude.Value; - StatusMessage = string.IsNullOrWhiteSpace(Address) + this.SetInfoStatus(string.IsNullOrWhiteSpace(Address) ? "Position récupérée. Complétez l'adresse puis envoyez la commande." - : result.Message; + : result.Message); } catch (OperationCanceledException) { - StatusMessage = "La récupération de la position a été annulée."; + this.SetWarningStatus("La récupération de la position a été annulée."); } catch (Exception ex) { - StatusMessage = $"Impossible de récupérer la position: {ex.Message}"; + this.SetErrorStatus($"Impossible de récupérer la position: {ex.Message}"); } finally { @@ -118,26 +118,26 @@ public partial class RdvViewModel : BillingCommandPageViewModel { if (!IsSupported) { - StatusMessage = SupportMessage; + this.SetWarningStatus(SupportMessage); return; } if (!Consent) { - StatusMessage = "Le consentement est requis pour poster la commande."; + this.SetWarningStatus("Le consentement est requis pour poster la commande."); return; } if (string.IsNullOrWhiteSpace(Address)) { - StatusMessage = "L'adresse du rendez-vous est requise."; + this.SetWarningStatus("L'adresse du rendez-vous est requise."); return; } if (string.IsNullOrWhiteSpace(Reason)) { - StatusMessage = "Le motif du rendez-vous est requis."; + this.SetWarningStatus("Le motif du rendez-vous est requis."); return; } @@ -186,17 +186,17 @@ public partial class RdvViewModel : BillingCommandPageViewModel }).ConfigureAwait(true); } - StatusMessage = IsEditingExisting + this.SetInfoStatus(IsEditingExisting ? $"Commande #{ExistingQueryId} mise à jour sur {BillingRoute} pour {Performer.UserName}." - : $"Commande transmise sur {BillingRoute} pour {Performer.UserName}."; + : $"Commande transmise sur {BillingRoute} pour {Performer.UserName}."); } catch (HttpRequestException ex) when (ex.StatusCode is HttpStatusCode.Unauthorized or HttpStatusCode.Forbidden) { - StatusMessage = "Accès refusé au billing (scope 'api'). Déconnectez puis reconnectez-vous."; + this.SetWarningStatus("Accès refusé au billing (scope 'api'). Déconnectez puis reconnectez-vous."); } catch (Exception ex) { - StatusMessage = $"Erreur lors de l'envoi: {ex.Message}"; + this.SetErrorStatus($"Erreur lors de l'envoi: {ex.Message}"); } finally { diff --git a/src/PostIt/PostIt/ViewModels/MainViewModel.cs b/src/PostIt/PostIt/ViewModels/MainViewModel.cs index 2ef6959bc..696470a3f 100644 --- a/src/PostIt/PostIt/ViewModels/MainViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/MainViewModel.cs @@ -12,7 +12,7 @@ using PostIt.Helpers; namespace PostIt.ViewModels; -public partial class MainViewModel : ViewModelBase +public partial class MainViewModel : ViewModelBase, IActionStatusViewModel { /// Window/tab title. Cosmetic — bound by /// MainPage.axaml if at all. Not the post title. @@ -55,16 +55,11 @@ public partial class MainViewModel : ViewModelBase public partial string StatusMessage { get; set; } [ObservableProperty] - public partial StatusNotice ActionStatus { get; set; } = StatusNotice.FromMessage(string.Empty); + public partial StatusNotice ActionStatus { get; set; } = StatusNotice.Info("Pret."); [ObservableProperty] public partial string SearchText { get; set; } - partial void OnStatusMessageChanged(string value) - { - ActionStatus = StatusNotice.FromMessage(value); - } - [ObservableProperty] public partial ObservableCollection Posts { get; set; } @@ -92,7 +87,7 @@ public partial class MainViewModel : ViewModelBase Posts.Add(post); } ApplyFilter(); - StatusMessage = $"Loaded {Posts.Count} posts."; + this.SetInfoStatus($"Loaded {Posts.Count} posts."); }); } @@ -112,7 +107,7 @@ public partial class MainViewModel : ViewModelBase // than to send a request the server will reject. if (string.IsNullOrWhiteSpace(DraftTitle)) { - StatusMessage = "Title is required."; + this.SetWarningStatus("Title is required."); return; } @@ -142,7 +137,7 @@ public partial class MainViewModel : ViewModelBase if (created is not null) { SelectedPost = created; - StatusMessage = $"Created post {created.Id}."; + this.SetInfoStatus($"Created post {created.Id}."); } } else @@ -158,7 +153,7 @@ public partial class MainViewModel : ViewModelBase DateModified = DateTime.UtcNow, }; await BlogClient!.UpdatePostAsync(SelectedPost.Id, update); - StatusMessage = $"Saved post {SelectedPost.Id}."; + this.SetInfoStatus($"Saved post {SelectedPost.Id}."); } await RefreshPostsAsync(); @@ -170,14 +165,14 @@ public partial class MainViewModel : ViewModelBase { if (SelectedPost is null || SelectedPost.Id == 0) { - StatusMessage = "Select an existing post before deleting."; + this.SetWarningStatus("Select an existing post before deleting."); return; } await ExecuteAsync(async () => { await BlogClient!.DeletePostAsync(SelectedPost.Id); - StatusMessage = $"Deleted post {SelectedPost.Id}."; + this.SetInfoStatus($"Deleted post {SelectedPost.Id}."); SelectedPost = null; await RefreshPostsAsync(); }); @@ -202,7 +197,7 @@ public partial class MainViewModel : ViewModelBase { if (SelectedPost is null || SelectedPost.Id == 0) { - StatusMessage = "Sélectionnez un billet existant pour changer sa publication."; + this.SetWarningStatus("Sélectionnez un billet existant pour changer sa publication."); return; } @@ -219,9 +214,9 @@ public partial class MainViewModel : ViewModelBase // locally flipped state until the round-trip // re-hydrates it. SelectedPost.IsPublished = publish; - StatusMessage = publish + this.SetInfoStatus(publish ? $"Billet {SelectedPost.Id} publié." - : $"Billet {SelectedPost.Id} remis en brouillon."; + : $"Billet {SelectedPost.Id} remis en brouillon."); }); } @@ -255,7 +250,7 @@ public partial class MainViewModel : ViewModelBase { if (SelectedPost is null) { - StatusMessage = "Select an existing post before managing ACL."; + this.SetWarningStatus("Select an existing post before managing ACL."); return; } @@ -354,7 +349,7 @@ public partial class MainViewModel : ViewModelBase FilteredPosts = new ObservableCollection(); SelectedPost = null; IsBusy = false; - StatusMessage = "Ready"; + this.SetInfoStatus("Ready"); Settings = settings ?? new Settings(); SearchText = Settings.SearchText; WindowTitle = "PostIt"; @@ -484,12 +479,12 @@ public partial class MainViewModel : ViewModelBase try { IsBusy = true; - StatusMessage = "Working..."; + this.SetInfoStatus("Working..."); await action(); } catch (Exception ex) { - StatusMessage = $"Error: {ex.Message}"; + this.SetErrorStatus($"Error: {ex.Message}"); } finally { diff --git a/src/PostIt/PostIt/ViewModels/PostAclDialogViewModel.cs b/src/PostIt/PostIt/ViewModels/PostAclDialogViewModel.cs index 994a4bcbd..be1b220fa 100644 --- a/src/PostIt/PostIt/ViewModels/PostAclDialogViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/PostAclDialogViewModel.cs @@ -37,7 +37,7 @@ public sealed class PostAclEntry /// any 403 / 404 will surface as an exception caught by the /// command and routed to . /// -public partial class PostAclDialogViewModel : ViewModelBase +public partial class PostAclDialogViewModel : ViewModelBase, IActionStatusViewModel { private readonly BlogAclApiClient _aclClient; private readonly CircleApiClient _circleClient; @@ -61,15 +61,10 @@ public partial class PostAclDialogViewModel : ViewModelBase public partial bool IsBusy { get; set; } [ObservableProperty] - public partial string StatusMessage { get; set; } = string.Empty; + public partial string StatusMessage { get; set; } = "Pret."; [ObservableProperty] - public partial StatusNotice ActionStatus { get; set; } = StatusNotice.FromMessage(string.Empty); - - partial void OnStatusMessageChanged(string value) - { - ActionStatus = StatusNotice.FromMessage(value); - } + public partial StatusNotice ActionStatus { get; set; } = StatusNotice.Info("Pret."); /// /// Idempotency gate for : the dialog @@ -123,12 +118,12 @@ public partial class PostAclDialogViewModel : ViewModelBase AclEntries = new ObservableCollection(AclEntries.Select(a => ToAclEntry(a.CircleId))); - StatusMessage = $"{AclEntries.Count} autorisation(s)"; + this.SetInfoStatus($"{AclEntries.Count} autorisation(s)"); _loaded = true; } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } finally { @@ -141,7 +136,7 @@ public partial class PostAclDialogViewModel : ViewModelBase { if (SelectedCircleToAdd is null) { - StatusMessage = "Sélectionnez un cercle à ajouter"; + this.SetWarningStatus("Sélectionnez un cercle à ajouter"); return; } @@ -150,7 +145,7 @@ public partial class PostAclDialogViewModel : ViewModelBase { if (AclEntries.Any(a => a.CircleId == SelectedCircleToAdd.Id)) { - StatusMessage = $"Cercle « {SelectedCircleToAdd.Name} » déjà autorisé"; + this.SetWarningStatus($"Cercle « {SelectedCircleToAdd.Name} » déjà autorisé"); return; } @@ -162,11 +157,11 @@ public partial class PostAclDialogViewModel : ViewModelBase if (created is not null) { AclEntries.Add(ToAclEntry(created.CircleId)); - StatusMessage = $"Cercle « {SelectedCircleToAdd.Name} » autorisé"; + this.SetInfoStatus($"Cercle « {SelectedCircleToAdd.Name} » autorisé"); } else { - StatusMessage = "Autorisation refusée par le serveur"; + this.SetWarningStatus("Autorisation refusée par le serveur"); } } catch (HttpRequestException ex) when (ex.StatusCode == HttpStatusCode.Conflict) @@ -174,11 +169,11 @@ public partial class PostAclDialogViewModel : ViewModelBase // Conflict means the link already exists in backend. Resync // from the dedicated ACL API so the UI reflects server truth. await ReloadAclEntriesFromServerAsync(); - StatusMessage = $"Cercle « {SelectedCircleToAdd.Name} » déjà autorisé"; + this.SetWarningStatus($"Cercle « {SelectedCircleToAdd.Name} » déjà autorisé"); } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } finally { @@ -197,11 +192,11 @@ public partial class PostAclDialogViewModel : ViewModelBase var existing = AclEntries.FirstOrDefault(e => e.CircleId == acl.CircleId); if (existing is not null) AclEntries.Remove(existing); - StatusMessage = "Autorisation révoquée"; + this.SetInfoStatus("Autorisation révoquée"); } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } finally { diff --git a/src/PostIt/PostIt/ViewModels/SignaturePageViewModel.cs b/src/PostIt/PostIt/ViewModels/SignaturePageViewModel.cs index 47667bddf..23ba70cf9 100644 --- a/src/PostIt/PostIt/ViewModels/SignaturePageViewModel.cs +++ b/src/PostIt/PostIt/ViewModels/SignaturePageViewModel.cs @@ -30,7 +30,7 @@ namespace PostIt.ViewModels; /// until the Yavsc.Org endpoint exists; the contract there will /// be POST /api/signature/{devisId} with this same payload. /// -public partial class SignaturePageViewModel : ViewModelBase +public partial class SignaturePageViewModel : ViewModelBase, IActionStatusViewModel { /// /// Default capture surface, in DIPs. 3:1 ratio matches a @@ -43,12 +43,7 @@ public partial class SignaturePageViewModel : ViewModelBase public partial string StatusMessage { get; set; } = "Prêt."; [ObservableProperty] - public partial StatusNotice ActionStatus { get; set; } = StatusNotice.FromMessage("Prêt."); - - partial void OnStatusMessageChanged(string value) - { - ActionStatus = StatusNotice.FromMessage(value); - } + public partial StatusNotice ActionStatus { get; set; } = StatusNotice.Info("Prêt."); [ObservableProperty] public partial int StrokeCount { get; set; } @@ -115,7 +110,7 @@ public partial class SignaturePageViewModel : ViewModelBase private void OnStrokeCompleted(object? sender, SignaturePadData data) { - StatusMessage = $"Trait terminé. {data.StrokeCount} trait(s)."; + this.SetInfoStatus($"Trait terminé. {data.StrokeCount} trait(s)."); RefreshCounts(); } @@ -133,7 +128,7 @@ public partial class SignaturePageViewModel : ViewModelBase public void Clear() { _control?.Clear(); - StatusMessage = "Effacé."; + this.SetInfoStatus("Effacé."); RefreshCounts(); } @@ -142,14 +137,14 @@ public partial class SignaturePageViewModel : ViewModelBase { if (_control is null) { - StatusMessage = "Contrôle non attaché."; + this.SetWarningStatus("Contrôle non attaché."); return; } var data = _control.Snapshot(); if (data.IsEmpty) { - StatusMessage = "Rien à capturer."; + this.SetWarningStatus("Rien à capturer."); return; } @@ -157,11 +152,11 @@ public partial class SignaturePageViewModel : ViewModelBase { var path = WriteCapture(data); LastCapturedPath = path; - StatusMessage = $"Capture enregistrée: {path}"; + this.SetInfoStatus($"Capture enregistrée: {path}"); } catch (Exception ex) { - StatusMessage = $"Erreur: {ex.Message}"; + this.SetErrorStatus($"Erreur: {ex.Message}"); } await Task.CompletedTask; } diff --git a/src/PostIt/PostIt/ViewModels/StatusNotice.cs b/src/PostIt/PostIt/ViewModels/StatusNotice.cs index 2d14b28b3..972c64350 100644 --- a/src/PostIt/PostIt/ViewModels/StatusNotice.cs +++ b/src/PostIt/PostIt/ViewModels/StatusNotice.cs @@ -32,27 +32,4 @@ public sealed class StatusNotice public static StatusNotice Info(string message) => new(message, StatusSeverity.Info); public static StatusNotice Warning(string message) => new(message, StatusSeverity.Warning); public static StatusNotice Error(string message) => new(message, StatusSeverity.Error); - - public static StatusNotice FromMessage(string? message) - { - if (string.IsNullOrWhiteSpace(message)) - { - return Info("Pret."); - } - - var text = message.Trim(); - var lower = text.ToLowerInvariant(); - - if (lower.StartsWith("erreur") || lower.StartsWith("echec") || lower.StartsWith("impossible")) - { - return Error(text); - } - - if (lower.Contains("refuse") || lower.Contains("annule") || lower.Contains("obligatoire") || lower.Contains("deja")) - { - return Warning(text); - } - - return Info(text); - } }