CFM-121_ORCID_Source_Plugin_review

app/plugins/OrcidSource/resources/locales/en_US/orcid_source.po

@@ -55,7 +55,7 @@ msgstr "Access token not configured (try resaving configuration)"
 msgid "error.param.notfound"
 msgstr "{0} was not found"
 
-msgid "error.orcid_source.no_orcid"
+msgid "error.response.no_orcid"
 msgstr "ORCID identifier missing from response."
 
 msgid "field.OrcidSources.api_type"

app/plugins/OrcidSource/src/Controller/OrcidSourceCollectorsController.php

@@ -138,7 +138,7 @@ public function dispatch(string $id) {
 
                 // Use the response and save the data to petitions table
                 if(empty($response->orcid)) {
-                    throw new \RuntimeException(__d('orcid_source', 'error.orcid_source.no_orcid'));
+                    throw new \RuntimeException(__d('orcid_source', 'error.response.no_orcid'));
                 }
                 $this->set('vv_orcid', $response->orcid);
                 $this->set('vv_token', $response);

app/plugins/OrcidSource/src/Model/Table/OrcidSourceCollectorsTable.php

@@ -91,9 +91,8 @@ public function initialize(array $config): void {
 
         $this->setAutoViewVars([
             'externalIdentitySources' => [
-                'type' => 'select',
-                'model' => 'ExternalIdentitySources',
-                'where' => ['plugin' => 'OrcidSource.OrcidSources']
+                'type' => 'plugin',
+                'model' => 'OrcidSource.OrcidSources',
             ]
         ]);
 

app/resources/locales/en_US/field.po

@@ -210,9 +210,6 @@ msgstr "Order"
 msgid "organization"
 msgstr "Organization"
 
-msgid "other.value"
-msgstr "Other Value"
-
 msgid "parameters"
 msgstr "Parameters"