Review changes (#318)

COmanage · Jun 25, 2025 · bf485b4 · bf485b4
1 parent d1fe460
commit bf485b4
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 8 deletions.
diff --git a/app/plugins/OrcidSource/resources/locales/en_US/orcid_source.po b/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"

diff --git a/app/plugins/OrcidSource/src/Controller/OrcidSourceCollectorsController.php b/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);

diff --git a/app/plugins/OrcidSource/src/Model/Table/OrcidSourceCollectorsTable.php b/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',
             ]
         ]);
 

diff --git a/app/resources/locales/en_US/field.po b/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"