fix(editgroups): trim post entries + better messages

Author: mrflos

actions/EditGroupsAction.php

@@ -25,7 +25,7 @@ public function run()
         $selectedGroupName = '';
         $action = '';
         $error_message = '';
-        error_log(implode("\n", $_POST));
+        error_log('$_POST content: ' . print_r($_POST ?? null, true));
 
         if (empty($_POST)) {
             error_log('$_POST empty');
@@ -47,7 +47,7 @@ public function run()
                         $type = 'success';
                         $message = str_replace('{group}', $selectedGroupName, _t('GROUP_CREATED'));
                     } elseif (!empty($_POST['action-update'])) {
-                        $members = $_POST['members'];
+                        $members = array_map('trim', $_POST['members']);
                         $groupController->update($selectedGroupName, $members);
                         $message = str_replace('{group}', $selectedGroupName, _t('GROUP_SAVED'));
                         $type = 'success';
@@ -59,7 +59,7 @@ public function run()
                     }
                 } catch (Throwable $th) {
                     $type = 'danger';
-                    $message = _t('ERROR_WHILE_EDITING_GROUP') . ':<br/>' . $th->getMessage();
+                    $message = _t('ERROR_WHILE_EDITING_GROUP') . '"' . $selectedGroupName . '" :<br/>' . $th->getMessage();
                 }
             }
         }

includes/controllers/GroupController.php

@@ -252,9 +252,9 @@ private function addOrUpdate(string $groupName, array $members, bool $add): void
                 case 0:
                     break;
                 case 1:
-                    throw new UserNameDoesNotExistException(_t('USER_NAME_DOES_NOT_EXIST') . ' : ' . $member . ' ');
+                    throw new UserNameDoesNotExistException(_t('USER_NAME_DOES_NOT_EXIST') . ' : ' . $member);
                 case 2:
-                    throw new GroupNameDoesNotExistException('included ' . _t('GROUP_NAME_DOES_NOT_EXIST'));
+                    throw new GroupNameDoesNotExistException('included "' . $member . '" ' . _t('GROUP_NAME_DOES_NOT_EXIST'));
                 case 3:
                     throw new InvalidInputException(_t('ERROR_RECURSIVE_GROUP'));
             }

lang/yeswiki_fr.php

@@ -576,13 +576,13 @@
     'REACTION_BAD_IMAGE_FORMAT' => 'Mauvais format d\'image : doit être un fichier, un icône utf8 ou une classe Fontawesome',
     'REACTION_NO_IMAGE' => 'Image manquante',
     'GROUP_NAME_ALREADY_USED' => 'le groupe existe déjà',
-    'USER_NAME_DOES_NOT_EXIST' => 'l\'utlisateur n\'existe pas',
+    'USER_NAME_DOES_NOT_EXIST' => 'l\'utilisateur n\'existe pas',
     'GROUP_NAME_DOES_NOT_EXIST' => 'le groupe n\'existe pas',
     'INVALID_GROUP_NAME' => 'le nom du groupe n\'est pas valide',
     'GROUP_SAVED' => 'groupe {group} mis à jour avec succès',
     'GROUP_CREATED' => 'groupe {group} créé avec succès',
     'GROUP_DELETED' => 'groupe {group} supprimé avec succès',
-    'ERROR_WHILE_EDITING_GROUP' => 'error durant l\'action sur le groupe.',
+    'ERROR_WHILE_EDITING_GROUP' => 'Erreur durant l\'action sur le groupe ',
     'TO_ANOTHER_YESWIKI' => 'dans un autre YesWiki',
     'WIKI_URL_RECENT' => 'Nécessite un YesWiki en version >= 4.5.0',
     'WIKI_URL' => 'Url du YesWiki',