Distinguish authenticated/unauthenticated ORCID IDs. #783

Author: fbacall

app/assets/images/ORCID-iD_icon_unauth_vector.svg

@@ -5,4 +5,19 @@ def self.user_profile_resource_limit
     30
   end
 
+  def orcid_link(profile, *opts)
+    content_tag(:span) do
+      if profile.orcid_authenticated?
+        concat image_tag('ORCID-iD_icon_vector.svg', size: 16)
+        concat ' '
+        concat external_link(profile.orcid.sub(OrcidValidator::ORCID_PREFIX, ''), profile.orcid, *opts)
+      else profile.orcid.present?
+        concat image_tag('ORCID-iD_icon_unauth_vector.svg', size: 16)
+        concat ' '
+        concat external_link(profile.orcid.sub(OrcidValidator::ORCID_PREFIX, ''), profile.orcid, *opts)
+        concat ' (Unauthenticated)'
+      end
+    end
+  end
+
 end

app/assets/images/ORCID-iD_icon_vector.svg

@@ -21,7 +21,7 @@
     <!-- Field: orcid -->
     <% if @trainer.orcid.present? %>
       <div class="url-wrap">
-        <%= external_link @trainer.orcid, @trainer.orcid, class: 'h5', target: '_blank', rel: 'noopener', track: true %>
+        <%= orcid_link(@trainer, class: 'h5', track: true) %>
       </div>
     <% end %>
 

app/helpers/users_helper.rb

@@ -50,7 +50,7 @@
         <% if @user.profile.orcid.blank? %>
           <span class="empty">None specified</span>
         <% else %>
-          <%= link_to @user.profile.orcid, @user.profile.orcid, rel: 'nofollow', target: '_blank' %>
+          <%= orcid_link(@user.profile) %>
         <% end %>
       </p>
 

app/views/trainers/show.html.erb

@@ -0,0 +1,5 @@
+class AddOrcidAuthenticatedToProfiles < ActiveRecord::Migration[7.2]
+  def change
+    add_column :profiles, :orcid_authenticated, :boolean, default: false
+  end
+end

app/views/users/show.html.erb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.2].define(version: 2025_03_25_151745) do
+ActiveRecord::Schema[7.2].define(version: 2025_10_13_115523) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
 
@@ -452,6 +452,7 @@
     t.string "social_media", default: [], array: true
     t.string "type", default: "Profile"
     t.string "fields", default: [], array: true
+    t.boolean "orcid_authenticated", default: false
     t.index ["slug"], name: "index_profiles_on_slug", unique: true
   end
 

db/migrate/20251013115523_add_orcid_authenticated_to_profiles.rb

@@ -68,6 +68,7 @@ trainer_one_profile:
   expertise_technical: ['python', 'R']
   public: true
   type: Trainer
+  orcid_authenticated: true
 
 trainer_two_profile:
   user: private_user
@@ -76,13 +77,14 @@ trainer_two_profile:
   image_url:
   email: samael@research.org
   website: ''
-  orcid: ''
+  orcid: https://orcid.org/0000-0002-0048-3300
   description: none
   location: 'Sydney, Australia'
   experience: much
   expertise_technical: ['java', 'python', 'ruby']
   public: false
   type: Profile
+  orcid_authenticated: false
 
 admin_trainer_profile:
   user: admin_trainer

db/schema.rb

@@ -0,0 +1,20 @@
+require 'test_helper'
+
+class UsersHelperTest < ActionView::TestCase
+  include ApplicationHelper
+
+  test 'displays authenticated orcid link' do
+    profile = profiles(:trainer_one_profile)
+    l = orcid_link(profile)
+    assert_includes l, 'ORCID-iD_icon_vector.svg'
+    assert_not_includes l, 'Unauthenticated'
+  end
+
+  test 'displays unauthenticated orcid link' do
+    profile = profiles(:trainer_two_profile)
+    l = orcid_link(profile)
+    assert_includes l, 'ORCID-iD_icon_unauth_vector.svg'
+    assert_includes l, 'Unauthenticated'
+  end
+
+end