Provide a "deleted" action and view for deletion within iframe views (CFM-392)

app/resources/locales/en_US/information.po

@@ -87,6 +87,9 @@ msgstr "Attribute Modal"
 msgid "global.attributes"
 msgstr "Attributes"
 
+msgid "global.attributes.none"
+msgstr "There are no attributes of this type"
+
 msgid "global.records.none"
 msgstr "There are no records to display."
 

app/resources/locales/en_US/result.po

@@ -81,6 +81,9 @@ msgstr "Added {0} to group {1} via nesting of group {2} ({3})"
 msgid "GroupMembers.deleted"
 msgstr "Removed {0} from group {1}"
 
+msgid "GroupMembers.deleted.generic"
+msgstr "Removed member from group"
+
 msgid "GroupMembers.deleted.nesting"
 msgstr "Removed {0} from group {1} via nesting of group {2} ({3})"
 

app/src/Controller/AppController.php

@@ -362,7 +362,8 @@ protected function primaryLinkLookup(): void
 
     // At the end we need to have a Primary Link
     if(empty($this->cur_pl->value)
-      && !$this->$modelsName->allowEmptyPrimaryLink($this->request->getParam('action'))) {
+      && !$this->$modelsName->allowEmptyPrimaryLink($this->request->getParam('action'))
+      && $this->request->getParam('action') != 'deleted') {
       throw new \RuntimeException(__d('error', 'primary_link'));
     }
   }
@@ -531,7 +532,7 @@ protected function populateAvailableCos() {
    */
 
   protected function setCO() {
-    if($this->cur_co) {
+    if($this->cur_co || $this->request->getParam('action') == 'deleted') {
       // Nothing to do...
       return;
     }

app/src/Controller/GroupMembersController.php

@@ -84,4 +84,18 @@ public function add() {
 
     return parent::add();
   }
+
+  /**
+   * Handle the deleted action for a Group Member.
+   * This is used to set the view title.
+   * 
+   * @since  COmanage Registry v5.0.0
+   */
+
+  public function deleted() {
+    // Override the standard title
+    $this->set('vv_title', __d('result','GroupMembers.deleted.generic'));
+
+    return parent::deleted();
+  }
 }

app/src/Controller/MVEAController.php

@@ -48,7 +48,7 @@ public function beforeFilter(\Cake\Event\EventInterface $event) {
     // $this->name = Models
     $modelsName = $this->name;
 
-    if(!$this->request->is('restful')) {
+    if(!$this->request->is('restful') && $this->request->getParam('action') != 'deleted') {
       // Provide additional hints to BreadcrumbsComponent. This needs to be here
       // and not in beforeRender because the component beforeRender will run first.
 
@@ -143,7 +143,7 @@ public function beforeRender(\Cake\Event\EventInterface $event) {
     // field = model (or model_name)
     $fieldName = Inflector::underscore(Inflector::singularize($modelsName));
 
-    if(!$this->request->is('restful')) {
+    if(!$this->request->is('restful') && $this->request->getParam('action') != 'deleted') {
       // If there is a default type setting for this model, pass it to the view
       if($this->$modelsName->getSchema()->hasColumn('type_id')) {
         $defaultTypeField = "default_" . $fieldName . "_type_id";

app/src/Controller/NamesController.php

@@ -50,7 +50,7 @@ class NamesController extends MVEAController {
    */
 
   public function beforeRender(\Cake\Event\EventInterface $event) {
-    if(!$this->request->is('restful')) {
+    if(!$this->request->is('restful') && $this->request->getParam('action') != 'deleted') {
       // Get the set of permitted name fields to pass to the view.
       // (We don't need required name fields since FormHelper will handle that.)
 

app/src/Controller/StandardController.php

@@ -33,6 +33,7 @@
 use Cake\ORM\TableRegistry;
 use InvalidArgumentException;
 use \Cake\Http\Exception\BadRequestException;
+use Cake\Utility\Inflector;
 use \App\Lib\Enum\ProvisioningContextEnum;
 use \App\Lib\Enum\SuspendableStatusEnum;
 use \App\Lib\Util\{StringUtilities, FunctionUtilities};
@@ -169,8 +170,10 @@ public function beforeRender(\Cake\Event\EventInterface $event) {
     $table = $this->$modelsName;
 
     // Provide some hints to the views
-    $this->getFieldTypes();
-    $this->getRequiredFields();
+    if($this->request->getParam('action') != 'deleted') {
+      $this->getFieldTypes();
+      $this->getRequiredFields();
+    }
 
     // Set the display field as a view var to make it available to the views
     $this->set('vv_display_field', $table->getDisplayField());
@@ -307,6 +310,23 @@ public function delete($id) {
     return $this->redirect(['action' => 'edit', $id]);
   }
 
+  /**
+   * Handle a deleted action for a Standard object.
+   *
+   * @since  COmanage Registry v5.0.0
+   */
+
+  public function deleted() {
+    // Set the title when not set at the individual controller
+    if(empty($this->viewBuilder()->getVar('vv_title'))) {
+      $modelsName = $this->name;
+      $fieldName = Inflector::singularize($modelsName);
+      $this->set('vv_title', __d('result', 'deleted.a', [$fieldName]));
+    }
+    // Render the view
+    $this->render('/Standard/deleted');
+  }
+
   /**
    * Handle an edit action for a Standard object.
    *
@@ -451,8 +471,11 @@ public function generateRedirect($entity) {
         $redirectGoal = 'index';
       }
     }
-
-    if($redirectGoal == 'self'
+
+    if($redirectGoal == 'deleted') {
+      // Immediately redirect to the (mostly blank) deleted view
+      return $this->redirect(['action' => 'deleted']);
+    } elseif($redirectGoal == 'self'
        && $entity
        && in_array($this->request->getParam('action'), ['add', 'edit'])) {
       // We typically want to redirect to the edit view of the record,

app/src/Controller/TelephoneNumbersController.php

@@ -49,7 +49,7 @@ class TelephoneNumbersController extends MVEAController {
    */
 
   public function beforeRender(\Cake\Event\EventInterface $event) {
-    if(!$this->request->is('restful')) {
+    if(!$this->request->is('restful') && $this->request->getParam('action') != 'deleted') {
       $CoSettings = TableRegistry::getTableLocator()->get('CoSettings');
 
       $settings = $CoSettings->find()->where(['co_id' => $this->getCOID()])->firstOrFail();

app/src/Lib/Traits/PrimaryLinkTrait.php

@@ -525,7 +525,7 @@ public function setPrimaryLink($fields) {
    */
 
   public function setRedirectGoal(string $goal, string $action='*') {
-    if(!in_array($goal, ['index', 'pluggableLink', 'primaryLink', 'self', 'special'])) {
+    if(!in_array($goal, ['deleted', 'index', 'pluggableLink', 'primaryLink', 'self', 'special'])) {
       throw new \InvalidArgumentException(__d('error', 'invalid', [$goal]));
     }
 

app/src/Model/Table/AdHocAttributesTable.php

@@ -87,6 +87,7 @@ public function initialize(array $config): void {
     $this->setPrimaryLink(['external_identity_id', 'external_identity_role_id', 'person_id', 'person_role_id']);
     $this->setRequiresCO(true);
     $this->setRedirectGoal('self');
+    $this->setRedirectGoal(action: 'delete', goal: 'deleted');
     $this->setAllowLookupPrimaryLink(['unfreeze']);
     $this->setEditContains(['ExternalIdentities', 'ExternalIdentityRoles', 'SourceAdHocAttributes']);
 
@@ -103,7 +104,8 @@ public function initialize(array $config): void {
       // Actions that operate over a table (ie: do not require an $id)
       'table' => [
         'add' =>      ['platformAdmin', 'coAdmin'],
-        'index' =>    ['platformAdmin', 'coAdmin']
+        'index' =>    ['platformAdmin', 'coAdmin'],
+        'deleted' =>  ['platformAdmin', 'coAdmin']
       ]
     ]);
   }

app/src/Model/Table/AddressesTable.php

@@ -104,6 +104,7 @@ public function initialize(array $config): void {
     // Models that AcceptCoId should be expicitly added to StandardApiController::initialize()
     $this->setAcceptsCoId(true);
     $this->setRedirectGoal('self');
+    $this->setRedirectGoal(action: 'delete', goal: 'deleted');
     $this->setAllowLookupPrimaryLink(['unfreeze']);
     $this->setEditContains(['ExternalIdentities', 'ExternalIdentityRoles', 'SourceAddresses']);
 
@@ -131,7 +132,8 @@ public function initialize(array $config): void {
       // Actions that operate over a table (ie: do not require an $id)
       'table' => [
         'add' =>      ['platformAdmin', 'coAdmin'],
-        'index' =>    ['platformAdmin', 'coAdmin']
+        'index' =>    ['platformAdmin', 'coAdmin'],
+        'deleted' =>  ['platformAdmin', 'coAdmin']
       ]
     ]);
   }

app/src/Model/Table/EmailAddressesTable.php

@@ -109,6 +109,7 @@ public function initialize(array $config): void {
     $this->setAllowLookupPrimaryLink(['primary']);
     $this->setRequiresCO(true);
     $this->setRedirectGoal('self');
+    $this->setRedirectGoal(action: 'delete', goal: 'deleted');
     $this->setAllowLookupPrimaryLink(['forceVerify', 'unfreeze']);
     $this->setEditContains(['ExternalIdentities', 'SourceEmailAddresses']);
 
@@ -133,7 +134,8 @@ public function initialize(array $config): void {
       // Actions that operate over a table (ie: do not require an $id)
       'table' => [
         'add' =>      ['platformAdmin', 'coAdmin'],
-        'index' =>    ['platformAdmin', 'coAdmin']
+        'index' =>    ['platformAdmin', 'coAdmin'],
+        'deleted' =>  ['platformAdmin', 'coAdmin']
       ]
     ]);
   }

app/src/Model/Table/GroupMembersTable.php

@@ -62,7 +62,7 @@ class GroupMembersTable extends Table {
    */
   public function getLayout(string $action = ''): string {
     return match($action) {
-      'add','edit','view' => 'iframe',  
+      'add','edit','view','deleted' => 'iframe',  
       default => 'default'
     };
   }
@@ -93,6 +93,7 @@ public function initialize(array $config): void {
     $this->setPrimaryLink(['group_id', 'person_id']);
     $this->setRequiresCO(true);
     $this->setRedirectGoal('self');
+    $this->setRedirectGoal(action: 'delete', goal: 'deleted');
 
     $this->setEditContains(['Groups', 'People.PrimaryName']);
     $this->setViewContains(['Groups', 'People.PrimaryName']);
@@ -125,7 +126,8 @@ public function initialize(array $config): void {
       // Actions that operate over a table (ie: do not require an $id)
       'table' => [
         'add' =>      ['platformAdmin', 'coAdmin'],
-        'index' =>    ['platformAdmin', 'coAdmin']
+        'index' =>    ['platformAdmin', 'coAdmin'],
+        'deleted' =>  ['platformAdmin', 'coAdmin']
       ]
     ]);
 

app/src/Model/Table/IdentifiersTable.php

@@ -119,6 +119,7 @@ public function initialize(array $config): void {
     $this->setPrimaryLink(['external_identity_id', 'group_id', 'person_id']);
     $this->setRequiresCO(true);
     $this->setRedirectGoal('self');
+    $this->setRedirectGoal(action: 'delete', goal: 'deleted');
     $this->setAllowLookupPrimaryLink(['unfreeze']);
     $this->setEditContains(['ExternalIdentities', 'SourceIdentifiers']);
 
@@ -146,7 +147,8 @@ public function initialize(array $config): void {
       // Actions that operate over a table (ie: do not require an $id)
       'table' => [
         'add' =>      ['platformAdmin', 'coAdmin'],
-        'index' =>    ['platformAdmin', 'coAdmin']
+        'index' =>    ['platformAdmin', 'coAdmin'],
+        'deleted' =>  ['platformAdmin', 'coAdmin']
       ],
       // Related models whose permissions we'll need, typically for table views
       'related' => [

app/src/Model/Table/NamesTable.php

@@ -110,6 +110,7 @@ public function initialize(array $config): void {
     // Models that AcceptCoId should be explicitly added to StandardApiController::initialize()
     $this->setAcceptsCoId(true);
     $this->setRedirectGoal('self');
+    $this->setRedirectGoal(action: 'delete', goal: 'deleted');
     $this->setEditContains(['ExternalIdentities', 'SourceNames']);
 
     $this->setAutoViewVars([
@@ -137,7 +138,8 @@ public function initialize(array $config): void {
       // Actions that operate over a table (ie: do not require an $id)
       'table' => [
         'add' =>      ['platformAdmin', 'coAdmin'],
-        'index' =>    ['platformAdmin', 'coAdmin']
+        'index' =>    ['platformAdmin', 'coAdmin'],
+        'deleted' =>  ['platformAdmin', 'coAdmin']
       ]
     ]);
   }

app/src/Model/Table/PronounsTable.php

@@ -96,6 +96,7 @@ public function initialize(array $config): void {
     $this->setPrimaryLink(['external_identity_id', 'person_id']);
     $this->setRequiresCO(true);
     $this->setRedirectGoal('self');
+    $this->setRedirectGoal(action: 'delete', goal: 'deleted');
     $this->setAllowLookupPrimaryLink(['unfreeze']);
     $this->setEditContains(['ExternalIdentities', 'SourcePronouns']);
 
@@ -123,7 +124,8 @@ public function initialize(array $config): void {
       // Actions that operate over a table (ie: do not require an $id)
       'table' => [
         'add' =>      ['platformAdmin', 'coAdmin'],
-        'index' =>    ['platformAdmin', 'coAdmin']
+        'index' =>    ['platformAdmin', 'coAdmin'],
+        'deleted' =>  ['platformAdmin', 'coAdmin']
       ]
     ]);
   }

app/src/Model/Table/TelephoneNumbersTable.php

@@ -104,6 +104,7 @@ public function initialize(array $config): void {
     // Models that AcceptCoId should be expicitly added to StandardApiController::initialize()
     $this->setAcceptsCoId(true);
     $this->setRedirectGoal('self');
+    $this->setRedirectGoal(action: 'delete', goal: 'deleted');
     $this->setAllowLookupPrimaryLink(['unfreeze']);
     $this->setEditContains(['ExternalIdentities', 'ExternalIdentityRoles', 'SourceTelephoneNumbers']);
 
@@ -128,7 +129,8 @@ public function initialize(array $config): void {
       // Actions that operate over a table (ie: do not require an $id)
       'table' => [
         'add' =>      ['platformAdmin', 'coAdmin'],
-        'index' =>    ['platformAdmin', 'coAdmin']
+        'index' =>    ['platformAdmin', 'coAdmin'],
+        'deleted' =>  ['platformAdmin', 'coAdmin']
       ]
     ]);
   }

app/src/Model/Table/UrlsTable.php

@@ -96,6 +96,7 @@ public function initialize(array $config): void {
     $this->setPrimaryLink(['external_identity_id', 'person_id']);
     $this->setRequiresCO(true);
     $this->setRedirectGoal('self');
+    $this->setRedirectGoal(action: 'delete', goal: 'deleted');
     $this->setAllowLookupPrimaryLink(['unfreeze']);
     $this->setEditContains(['ExternalIdentities', 'ExternalIdentityRoles', 'SourceUrls']);
 
@@ -119,7 +120,8 @@ public function initialize(array $config): void {
       // Actions that operate over a table (ie: do not require an $id)
       'table' => [
         'add' =>      ['platformAdmin', 'coAdmin'],
-        'index' =>    ['platformAdmin', 'coAdmin']
+        'index' =>    ['platformAdmin', 'coAdmin'],
+        'deleted' =>  ['platformAdmin', 'coAdmin']
       ]
     ]);
   }

app/src/View/Helper/VueHelper.php

@@ -61,6 +61,7 @@ class VueHelper extends Helper {
       'unverified'
     ],
     'information' => [
+      'global.attributes.none',
       'global.value.none',
       'record',
       'report.for',

app/templates/EmailAddresses/fields-nav.inc

@@ -33,17 +33,19 @@ $subnav = [
   'active' => 'person', // default for person. 'external identities' are special cased
   'subActive' => 'email_addresses'
 ];
-
-$topLinks = [
-  [
-    'icon' => 'verified_user',
-    'order' => 'Default',
-    'label' => __d('operation', 'EmailAddresses.verify.force'),
-    'if' => 'notVerified',
-    'link' => [
-      'action' => 'forceVerify',
-      $vv_obj->id
+
+if($vv_action != 'add') {
+  $topLinks = [
+    [
+      'icon' => 'verified_user',
+      'order' => 'Default',
+      'label' => __d('operation', 'EmailAddresses.verify.force'),
+      'if' => 'notVerified',
+      'link' => [
+        'action' => 'forceVerify',
+        $vv_obj->id
+      ],
+      'class' => ''
     ],
-    'class' => ''
-  ],
-];
+  ];
+}