Restore attribute cards on External Id canvas (CFM-345) (#133)

* Add test for ext_identity_source_records on External Id canvas (CFM-345) * Restore attribute cards on External Id canvas. Improve read-only form-list styling. (CFM-345)
COmanage · Oct 11, 2023 · 23f9182 · 23f9182
1 parent 8008f1e
commit 23f9182
Show file tree

Hide file tree

Showing 4 changed files with 35 additions and 28 deletions.
diff --git a/app/src/Model/Table/ExternalIdentitiesTable.php b/app/src/Model/Table/ExternalIdentitiesTable.php
@@ -131,18 +131,14 @@ public function initialize(array $config): void {
       'Addresses',
       'AdHocAttributes',
       'EmailAddresses',
+      'ExtIdentitySourceRecords' => ['ExternalIdentitySources'],
       'Identifiers',
       'Names',
       'Pronouns',
       'TelephoneNumbers',
       'Urls'
     ]);
 
-    $this->setViewContains([
-      'PrimaryName',
-      'ExtIdentitySourceRecords' => ['ExternalIdentitySources']
-    ]);
-
     $this->setAutoViewVars([
       'statuses' => [
         'type' => 'enum',

diff --git a/app/src/View/Helper/FieldHelper.php b/app/src/View/Helper/FieldHelper.php
@@ -220,7 +220,7 @@ public function dateControl(string $fieldName, string $dateType=DateTypeEnum::St
           $controlCode = '<time>' . $entity->$fieldName->i18nFormat("yyyy-MM-dd HH:mm:ss", $this->getView()->get('vv_tz')) . '</time>';
         }
       } else {
-        $controlCode = __d('information', 'notset');
+        $controlCode = '<div class="not-set">' . __d('information', 'notset') . '</div>';
       }
       // Return this to the generic control() function
       return $this->control($fieldName, $coptions, ctrlCode: $controlCode, labelIsTextOnly: true);

diff --git a/app/templates/ExternalIdentities/fields.inc b/app/templates/ExternalIdentities/fields.inc
@@ -27,28 +27,30 @@
 
 // This view will not support add/edit
 if($vv_action == 'add' || $vv_action == 'edit' || $vv_action == 'view') {
-  print __d(
-    'information', 
-    'ExternalIdentities.source',
-    [
-      $this->Html->link(
-        $vv_obj->ext_identity_source_records[0]->external_identity_source->description,
-        [
-          'controller'  => 'external-identity-sources',
-          'action'      => 'edit',
-          $vv_obj->ext_identity_source_records[0]->external_identity_source->id
-        ]
-      ),
-      $this->Html->link(
-        __d('operation', 'view.a', [__d('controller', 'ExtIdentitySourceRecords', 1)]),
-        [
-          'controller'  => 'ext-identity-source-records',
-          'action'      => 'view',
-          $vv_obj->ext_identity_source_records[0]->id
-        ]
-      )
-    ]
-  );
+  if(!empty($vv_obj->ext_identity_source_records[0])) {
+    print __d(
+      'information',
+      'ExternalIdentities.source',
+      [
+        $this->Html->link(
+          $vv_obj->ext_identity_source_records[0]->external_identity_source->description,
+          [
+            'controller' => 'external-identity-sources',
+            'action' => 'edit',
+            $vv_obj->ext_identity_source_records[0]->external_identity_source->id
+          ]
+        ),
+        $this->Html->link(
+          __d('operation', 'view.a', [__d('controller', 'ExtIdentitySourceRecords', 1)]),
+          [
+            'controller' => 'ext-identity-source-records',
+            'action' => 'view',
+            $vv_obj->ext_identity_source_records[0]->id
+          ]
+        )
+      ]
+    );
+  }
 
   print $this->Field->control('status', ['empty' => false]);
 

diff --git a/app/webroot/css/co-base.css b/app/webroot/css/co-base.css
@@ -1200,6 +1200,9 @@ ul.form-list {
   clear: both;
   padding: 0;
 }
+body.view ul.form-list {
+  border-bottom: 1px solid var(--cmg-color-bg-005);
+}
 .cm-time-picker-vals ul li,
 ul.form-list li {
   background-color: var(--cmg-color-body-bg);
@@ -1231,6 +1234,9 @@ ul.form-list li.fields-submit {
   border-left: none;
   border-right: none;
 }
+ul.form-list li.fields-submit .field:hover {
+  background-color: var(--cmg-color-body-bg);
+}
 ul.form-list .field-name {
   display: block;
 }
@@ -1273,6 +1279,9 @@ ul.form-list select {
   background-color: var(--cmg-color-body-bg);
   border-color: var(--cmg-color-bg-007);
 }
+ul.form-list .not-set {
+  padding: 4px 6px;
+}
 .form-select:disabled {
   background-color: var(--cmg-color-bg-009);
 }