Merge pull request #8 from MESH-Research/develop

Add logic to populate username in form with default from environment
MESH-Research · Jun 23, 2021 · 017725c · 017725c
2 parents 4d0da60 + a3212c1
commit 017725c
Show file tree

Hide file tree

Showing 6 changed files with 26 additions and 4 deletions.
diff --git a/Config/Schema/schema.xml b/Config/Schema/schema.xml
@@ -40,6 +40,7 @@
     <field name="ldap_bind_password" type="C" size="64" />
     <field name="ldap_basedn" type="C" size="128" />
     <field name="hc_idp_scope" type="C" size="64" />
+    <field name="username_env_variable" type="C" size="256" />
     <field name="debug" type="L" />
     <field name="created" type="T" />
     <field name="modified" type="T" />

diff --git a/Controller/HumanitiesCommonsIdpEnrollerCoPetitionsController.php b/Controller/HumanitiesCommonsIdpEnrollerCoPetitionsController.php
@@ -223,7 +223,12 @@ protected function execute_plugin_checkEligibility($id, $onFinish) {
 
     // GET, fall through to display view
     ( $debug ? $this->log($logPrefix . "received GET so displaying form to collect username") : null);
+
     $this->set('vv_efwid', $efwid);
+
+    if(!empty($config['HumanitiesCommonsIdpEnroller']['username_env_variable'])) {
+      $this->set('vv_suggested_username', getenv($config['HumanitiesCommonsIdpEnroller']['username_env_variable']));
+    }
   }
 
   /**

diff --git a/Lib/lang.php b/Lib/lang.php
@@ -56,6 +56,8 @@
   'pl.humanitiescommonsidpenroller.config.ldap_basedn.desc'  => 'Base DN to provision accounts under',
   'pl.humanitiescommonsidpenroller.config.hc_idp_scope'  => 'Humanities Commons IdP Scope',
   'pl.humanitiescommonsidpenroller.config.hc_idp_scope.desc'  => 'Scope asserted by the Humanities Commons IdP (hcommons.org)',
+  'pl.humanitiescommonsidpenroller.config.username_env_variable'  => 'Username Environment Variable Default',
+  'pl.humanitiescommonsidpenroller.config.username_env_variable.desc'  => 'Environment variable used to set username default in form',
   'pl.humanitiescommonsidpenroller.config.debug'       => 'Debug',
   'pl.humanitiescommonsidpenroller.config.debug.desc'  => 'Toggle for debug mode',
   'pl.humanitiescommonsidpenroller.config.menu'        => 'HC IdP Enroller Configuration',

diff --git a/Model/HumanitiesCommonsIdpEnroller.php b/Model/HumanitiesCommonsIdpEnroller.php
@@ -74,7 +74,12 @@ class HumanitiesCommonsIdpEnroller extends AppModel {
       'rule' => 'notBlank',
       'required' => true,
       'allowEmpty' => false
-    )
+    ),
+    'username_env_variable' => array(
+      'rule' => array('validateInput'),
+      'required' => false,
+      'allowEmpty' => true
+    ),
   );
 
   /**

diff --git a/View/HumanitiesCommonsIdpEnrollerCoPetitions/check_eligibility.ctp b/View/HumanitiesCommonsIdpEnrollerCoPetitions/check_eligibility.ctp
@@ -84,9 +84,9 @@
                 $args['type'] = 'text';
                 $args['label'] = false;
                 $args['required'] = 'required';
-                if( $current_enrollment_flow_cou == 'MSU' ) {
-                  $args['value'] = getenv('netid');
-		}
+                if(!empty($vv_suggested_username)) {
+                  $args['value'] = $vv_suggested_username;
+                }
                 print $this->Form->input('username', $args);
               ?>
               <span id="username-error" class="field-error"></span>

diff --git a/View/HumanitiesCommonsIdpEnrollers/fields.inc b/View/HumanitiesCommonsIdpEnrollers/fields.inc
@@ -94,6 +94,15 @@
         <?php print ($e ? $this->Form->input('hc_idp_scope', array('size' => 50)) : Sanitize::html($humanities_commons_idp_enrollers[0]['HumanitiesCommonsIdpEnroller']['hc_idp_scope'])); ?>
       </td>
     </tr>
+    <tr class="line<?php print ($l % 2); $l++; ?>">
+      <td>
+        <strong class="fieldTitle"><?php print _txt('pl.humanitiescommonsidpenroller.config.username_env_variable'); ?></strong><br />
+        <font class="desc"><?php print _txt('pl.humanitiescommonsidpenroller.config.username_env_variable.desc'); ?></font>
+      </td>
+      <td>
+        <?php print ($e ? $this->Form->input('username_env_variable', array('size' => 50)) : Sanitize::html($humanities_commons_idp_enrollers[0]['HumanitiesCommonsIdpEnroller']['username_env_variable'])); ?>
+      </td>
+    </tr>
     <tr class="line<?php print ($l % 2); $l++; ?>">
       <td>
         <strong class="fieldTitle"><?php print _txt('pl.humanitiescommonsidpenroller.config.debug'); ?></strong><font class="required">*</font><br />