[user_accounts] Prevent crash when one user has 0 site/project affili…

…ations (aces#8139)

Although this use case should never happen as it is prevented by rules in the edit_user logic, a couple bugs have let it happen in the past and the entire module becomes inaccessible. This makes the module load and displays a message in the user row informing the editor that the user does not currently have any sites and projects.

The only way to test this would be to manually delete all site associations directly in the DB user_psc_rel OR all project associations user_project_rel

Note that the module crashes even if the user not having any sites or projects is not the editor nor the editee... any user can cause this issue.

modules/user_accounts/jsx/userAccountsIndex.js

@@ -77,6 +77,11 @@ class UserAccountsIndex extends Component {
             .join(', ')}
           </td>
         );
+        if (cell.length === 0) {
+          result = (
+            <td>This user has no site affiliations</td>
+          );
+        }
         break;
       case 'Project':
         // If user has multiple projects, join array of sites into string
@@ -86,6 +91,11 @@ class UserAccountsIndex extends Component {
             ).join(', ')}
           </td>
         );
+        if (cell.length === 0) {
+          result = (
+            <td>This user has no project affiliations</td>
+          );
+        }
         break;
       case 'Username':
         url = loris.BaseURL + '/user_accounts/edit_user/' + row.Username;

modules/user_accounts/php/useraccountrowprovisioner.class.inc

@@ -44,20 +44,14 @@ class UserAccountRowProvisioner extends \LORIS\Data\Provisioners\DBRowProvisione
      */
     public function getInstance($row) : \LORIS\Data\DataInstance
     {
-        $pids = array_map(
-            function (string $pid) : \ProjectID {
-                return new \ProjectID($pid);
-            },
-            explode(',', $row['projectIds']),
-        );
-
-        $cids = array_map(
-            function (string $cid) : \CenterID {
-                return new \CenterID($cid);
-            },
-            explode(',', $row['centerIds']),
-        );
-
+        $cids = [];
+        $pids = [];
+        if (isset($row['centerIds'])) {
+            $cids = explode(',', $row['centerIds']);
+        }
+        if (isset($row['projectIds'])) {
+            $pids = explode(',', $row['projectIds']);
+        }
         $row['centerIds']  = $cids;
         $row['projectIds'] = $pids;
         return new UserAccountRow($row, $cids, $pids);