Pipelines

COmanage · Apr 27, 2024 · 5bdfbdb · 5bdfbdb
1 parent 5f4a378
commit 5bdfbdb
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 28 deletions.
diff --git a/app/src/View/Helper/FieeldHelper.php b/app/src/View/Helper/FieeldHelper.php
@@ -313,7 +313,7 @@ public function formField(string $fieldName,
     // Cause any select (except status) to render with a blank option, even
     // if the field is required. This makes it clear when a value needs to be set.
     // Note this will be ignored for non-select controls.
-    if($fieldName === 'status') {
+    if(\in_array($fieldName, ['status', 'sync_status_on_delete'], true)) {
       $fieldArgs['empty'] = false;
     } elseif(isset($options['empty'])) {
       $fieldArgs['empty'] = $options['empty'];

diff --git a/app/templates/Pipelines/fields.inc b/app/templates/Pipelines/fields.inc
@@ -50,37 +50,46 @@
 <?php
 // This view does not support read-only
 if($vv_action == 'add' || $vv_action == 'edit') {
-  print $this->Field->control('description');
-
-  print $this->Field->control('status');
-
-  // Match Strategy
-  print $this->Field->control(
-    fieldName: 'match_strategy',
-    controlType: 'select',
-    options: [
-      'onChange' => 'updateGadgets()'
+  print $this->element('form/listItem', [
+    'arguments' => [
+      'fieldName' => 'description'
     ]
-  );
-
-  print $this->Field->control('match_email_address_type_id');
-
-  print $this->Field->control('match_identifier_type_id');
-
-  // Sync Strategy
-//  print $this->Field->control('sync_on_update');
-
-//  print $this->Field->control('sync_on_delete');
+  ]);
 
-  print $this->Field->control('sync_status_on_delete', ['empty' => false]);
-
-  print $this->Field->control('sync_cou_id');
-
-  print $this->Field->control('sync_replace_cou_id');
+  print $this->element('form/listItem', [
+    'arguments' => [
+      'fieldName' => 'status'
+    ]
+  ]);
 
-  print $this->Field->control('sync_affiliation_type_id');
+  // Match Strategy
+  print $this->element('form/listItem', [
+    'arguments' => [
+      'fieldName' => 'match_strategy',
+      'options' => [
+        'onChange' => 'updateGadgets()'
+      ],
+      'fieldType' => 'select'
+    ]
+  ]);
 
-  print $this->Field->control('sync_identifier_type_id');
+  foreach([
+            'match_email_address_type_id',
+            'match_identifier_type_id',
+//            'sync_on_update',
+//            'sync_on_delete',
+            'sync_status_on_delete',
+            'sync_cou_id',
+            'sync_replace_cou_id',
+            'sync_affiliation_type_id',
+            'sync_identifier_type_id',
+          ] as $field) {
+    print $this->element('form/listItem', [
+      'arguments' => [
+        'fieldName' => $field,
+      ]
+    ]);
+  }
 
   // Connections
 //XXX