fix issue in edit invitation not showing user details and remove legacy add user modals links

Author: ipula

controllers/grid/settings/user/UserGridHandler.php

@@ -92,23 +92,6 @@ public function initialize($request, $args = null)
         // Basic grid configuration.
         $this->setTitle('grid.user.currentUsers');
 
-        // Grid actions.
-        $router = $request->getRouter();
-
-        $this->addAction(
-            new LinkAction(
-                'addUser',
-                new AjaxModal(
-                    $router->url($request, null, null, 'addUser', null, null),
-                    __('grid.user.add'),
-                    null,
-                    true
-                ),
-                __('grid.user.add'),
-                'add_user'
-            )
-        );
-
         //
         // Grid columns.
         //
@@ -171,7 +154,7 @@ public function getTemplateVarsFromRow($row): array
                               ->withActive();
                     })
                     ->get();
-                
+
                 $roles = $userGroups->map(fn (UserGroup $userGroup) => $userGroup->getLocalizedData('name'))->join(__('common.commaListSeparator'));
                 return ['label' => $roles];
                 }
@@ -550,18 +533,18 @@ public function removeUser($args, $request)
         if (!$request->checkCSRF()) {
             return new JSONMessage(false);
         }
-    
+
         $context = $request->getContext();
         $user = $request->getUser();
-    
+
         // Identify the user Id.
         $userId = $request->getUserVar('rowId');
-    
+
         if ($userId !== null && Validation::getAdministrationLevel($userId, $user->getId(), $context->getId()) === Validation::ADMINISTRATION_PROHIBITED) {
             // We don't have administrative rights over this user.
             return new JSONMessage(false, __('grid.user.cannotAdminister'));
         }
-    
+
         // Check if this user has any active user group assignments for this context.
         $activeUserGroupCount = UserGroup::query()
             ->withContextIds($context->getId())
@@ -570,7 +553,7 @@ public function removeUser($args, $request)
                     ->withActive();
             })
             ->count();
-        
+
         if (!$activeUserGroupCount) {
             return new JSONMessage(false, __('grid.user.userNoRoles'));
         } else {
@@ -582,7 +565,7 @@ public function removeUser($args, $request)
                     $query->withContextIds($context->getId());
                 })
                 ->update(['date_end' => now()]);
-        
+
             return \PKP\db\DAO::getDataChangedEvent($userId);
         }
     }

controllers/grid/users/author/AuthorGridRow.php

@@ -101,24 +101,6 @@ public function initialize($request, $template = null)
                         'delete'
                     )
                 );
-
-                $author = Repo::author()->get((int) $rowId, $this->getPublication()->getId());
-
-                if ($author && !Repo::user()->getByEmail($author->getEmail(), true)) {
-                    $this->addAction(
-                        new LinkAction(
-                            'addUser',
-                            new AjaxModal(
-                                $router->url($request, null, null, 'addUser', null, $actionArgs),
-                                __('grid.user.add'),
-                                null,
-                                true
-                            ),
-                            __('grid.user.add'),
-                            'add_user'
-                        )
-                    );
-                }
             }
         }
     }

locale/en/user.po

@@ -855,7 +855,7 @@ msgid "orcid.field.unverified.shouldRequest"
 msgstr "This ORCID has not been verified. Please remove this unverified ORCID and request verification from the user/author directly."
 
 msgid "user.removeRole.message"
-msgstr "Are you sure want remove this role permanently?"
+msgstr "Are you sure you want to permanently remove this role? This action will revoke the user's access to all information and permissions associated with this role"
 
 msgid "user.futureRole.notification.message"
 msgstr "Your role is scheduled to begin on {$roleStartDate}"

pages/invitation/InvitationHandler.php

@@ -22,13 +22,10 @@
 use APP\handler\Handler;
 use APP\template\TemplateManager;
 use PKP\context\Context;
-use PKP\core\PKPApplication;
 use PKP\facades\Locale;
-use PKP\i18n\LocaleMetadata;
 use PKP\invitation\core\enums\InvitationAction;
 use PKP\invitation\core\Invitation;
 use PKP\invitation\stepTypes\SendInvitationStep;
-use PKP\user\User;
 use PKP\userGroup\relationships\UserUserGroup;
 
 class InvitationHandler extends Handler
@@ -155,8 +152,8 @@ public function invite(array $args, Request $request): void
             $invitationModel = $invitation->invitationModel->toArray();
 
             $invitationMode = 'edit';
-            $payload['email']=$invitationModel['email'];
-            $invitationData = $this->generateInvitationPayload($invitationModel['userId'],$payload,$request->getContext())['invitationPayload'];
+            $payload['email'] = $invitationModel['email'];
+            $invitationData = $this->generateInvitationPayload($invitationModel['userId'], $payload, $request->getContext());
             $user = $invitationData['user'];
             $invitationPayload = $invitationData['invitationPayload'];
         }
@@ -206,7 +203,8 @@ public function invite(array $args, Request $request): void
             'pageTitleDescription' => $invitation ?
                 __(
                     'invitation.wizard.viewPageTitleDescription',
-                    ['name' => $invitationPayload['givenName'][Locale::getLocale()]]
+                    ['name' => $invitationPayload['givenName'][Locale::getLocale()] ?
+                        $invitationPayload['givenName'][Locale::getLocale()] : $invitationPayload['inviteeEmail']]
                 )
                 : __('invitation.wizard.pageTitleDescription'),
         ]);
@@ -228,9 +226,9 @@ public function invite(array $args, Request $request): void
     public function editUser($args, $request): void
     {
         $invitation = null;
-        if(!empty($args)) {
+        if (!empty($args)) {
             $invitationMode = 'editUser';
-            $invitationData = $this->generateInvitationPayload($args[0],[],$request->getContext());
+            $invitationData = $this->generateInvitationPayload($args[0], [], $request->getContext());
             $user = $invitationData['user'];
             $invitationPayload = $invitationData['invitationPayload'];
             $templateMgr = TemplateManager::getManager($request);
@@ -257,7 +255,7 @@ public function editUser($args, $request): void
             ];
             $steps = new SendInvitationStep();
             $templateMgr->setState([
-                'steps' => $steps->getSteps($invitation, $context,$user),
+                'steps' => $steps->getSteps($invitation, $context, $user),
                 'emailTemplatesApiUrl' => $request
                     ->getDispatcher()
                     ->url(
@@ -295,7 +293,7 @@ public function editUser($args, $request): void
      * @param Context $context
      * @param int $id
      */
-    private function getUserUserGroups(int $id , Context $context): array
+    private function getUserUserGroups(int $id, Context $context): array
     {
         $userGroups = [];
         $userUserGroups = UserUserGroup::query()
@@ -325,11 +323,11 @@ private function getUserUserGroups(int $id , Context $context): array
     private function generateInvitationPayload($userId, array $payload, Context $context): array
     {
         $user = null;
-        if($userId){
-            $user = Repo::user()->get($userId,true);
+        if ($userId) {
+            $user = Repo::user()->get($userId, true);
         }
 
-        $invitationPayload =[];
+        $invitationPayload = [];
         $invitationPayload['userId'] = $user ? $user->getId() : $userId;
         $invitationPayload['inviteeEmail'] = $user ? $user->getEmail() : $payload['email'];
         $invitationPayload['orcid'] = $user ? $user->getData('orcid') : $payload['orcid'];
@@ -341,12 +339,12 @@ private function generateInvitationPayload($userId, array $payload, Context $con
         $invitationPayload['phone'] = $user?->getPhone();
         $invitationPayload['mailingAddress'] = $user?->getMailingAddress();
         $invitationPayload['signature'] = $user?->getSignature(null);
-        $invitationPayload['locales'] = $user? $this->getWorkingLanguages($context,$user->getLocales()) : null;
+        $invitationPayload['locales'] = $user ? $this->getWorkingLanguages($context, $user->getLocales()) : null;
         $invitationPayload['reviewInterests'] = $user?->getInterestString();
         $invitationPayload['homePageUrl'] = $user?->getUrl();
         $invitationPayload['disabled'] = $user?->getData('disabled');
         $invitationPayload['userGroupsToAdd'] = !$payload['userGroupsToAdd'] ? [] : $payload['userGroupsToAdd'];
-        $invitationPayload['currentUserGroups'] = !$userId ? [] : $this->getUserUserGroups($userId,$context);
+        $invitationPayload['currentUserGroups'] = !$userId ? [] : $this->getUserUserGroups($userId, $context);
         $invitationPayload['userGroupsToRemove'] = [];
         $invitationPayload['emailComposer'] = [
             'emailBody' => '',
@@ -364,7 +362,7 @@ private function generateInvitationPayload($userId, array $payload, Context $con
      * @param $userLocales
      * @return string
      */
-    private function getWorkingLanguages(Context $context,$userLocales): string
+    private function getWorkingLanguages(Context $context, $userLocales): string
     {
         $locales = $context->getSupportedLocaleNames();
         return join(__('common.commaListSeparator'), array_map(fn($key) => $locales[$key], $userLocales));