Skip to content

Commit

Permalink
Fix identifier display in jobs list (#260)
Browse files Browse the repository at this point in the history
**ISSUE**
We missed some `.first` methods from when identifier was an arrary
- now they return the first letter from the string.  This change
gives us the full identifier string we want :)
  • Loading branch information
mark-dce authored Sep 28, 2022
1 parent e9d6110 commit 0aeadb8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion app/lib/tenejo/csv_importer.rb
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ def create_or_update(node)

def ensure_thumbnails(node)
return unless node.class == Tenejo::PFWork
work = Work.where(identifier_ssi: node.identifier.first)&.last
work = Work.where(identifier_ssi: node.identifier)&.last
unless work
@logger.error "CSV Importer couldn't find Work with primary_id #{node&.identifier} to attach thumbnail"
return
Expand Down
2 changes: 1 addition & 1 deletion app/views/jobs/_child.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
<%= status_span_generator(node.status) %>
</td>
<td class="identifier">
<%= node.identifier.first %>
<%= node.identifier %>
</td>
<td class="lineno"><%= node.lineno %></td>
<td class="index"><%= order || raw("&ndash;") %></td>
Expand Down

0 comments on commit 0aeadb8

Please sign in to comment.