Adjust error message for non-existing subscription details #758

Author: hupf

src/app/import/utils/subscription-details/validation.ts

@@ -106,6 +106,14 @@ export const assertPersonExists: SubscriptionDetailValidationFn = (entry) => {
   return valid;
 };
 
+/**
+ * The /Subscriptions/{subscriptionId}/SubscriptionDetails endpoint does not
+ * return details that don't have VssInternet="E". So there is no way to
+ * differentiate between the case where no subscription exists for a given
+ * person/event and the case where a subscription exists, but can't be edited
+ * through the Internet (VssInternet!="E"). We solve this by using a generic
+ * error message that covers both cases.
+ */
 export const assertSubscriptionDetailExists: SubscriptionDetailValidationFn = (
   entry,
 ) => {
@@ -117,6 +125,11 @@ export const assertSubscriptionDetailExists: SubscriptionDetailValidationFn = (
   return valid;
 };
 
+/**
+ * Actually, the /Subscriptions/{subscriptionId}/SubscriptionDetails endpoint
+ * does not return details that don't have VssInternet="E". But we leave this
+ * assertion in place for now.
+ */
 export const assertSubscriptionDetailEditable: SubscriptionDetailValidationFn =
   (entry) => {
     const detail = entry.data.subscriptionDetail;

src/app/shared/services/subscriptions-rest.service.ts

@@ -97,6 +97,10 @@ export class SubscriptionsRestService extends RestService<typeof Subscription> {
       .pipe(switchMap(decodeArray(Subscription)));
   }
 
+  /**
+   * This endpoint only returns subscription details that have VssInternet="E"
+   * (i.e. editable via Internet).
+   */
   getSubscriptionDetailsById(
     id: number | string,
   ): Observable<ReadonlyArray<SubscriptionDetail>> {

src/assets/locales/de-CH.json

@@ -394,7 +394,7 @@
         "MissingValueError": "Der Wert fehlt",
         "EventNotFoundError": "Der Anlass ist nicht vorhanden",
         "PersonNotFoundError": "Die Person ist nicht vorhanden",
-        "SubscriptionDetailNotFoundError": "Die Person ist nicht auf den Anlass angemeldet",
+        "SubscriptionDetailNotFoundError": "Das Anmeldedetail ist nicht für die Interneteingabe freigegeben oder die Person ist nicht auf den Anlass angemeldet",
         "SubscriptionDetailNotEditableError": "Das Anmeldedetail ist nicht für die Interneteingabe freigegeben",
         "InvalidValueTypeError": "Ungültiger Wert für dieses Anmeldedetail",
         "InvalidDropdownValueError": "Ungültiger Wert für dieses Anmeldedetail"

src/assets/locales/fr-CH.json

@@ -394,7 +394,7 @@
         "MissingValueError": "La valeur est manquante",
         "EventNotFoundError": "L'événement n'existe pas",
         "PersonNotFoundError": "La personne n'existe pas",
-        "SubscriptionDetailNotFoundError": "La personne n'est pas inscrite à l'événement",
+        "SubscriptionDetailNotFoundError": "Le détail d'inscription n'est pas validé pour la saisie sur Internet ou la personne n'est pas inscrite à l'événement",
         "SubscriptionDetailNotEditableError": "Le détail d'inscription n'est pas validé pour la saisie sur Internet",
         "InvalidValueTypeError": "Valeur non valide pour ce détail d'inscription",
         "InvalidDropdownValueError": "Valeur non valide pour ce détail d'inscription"