Bug #10949

Adding a "Use For" Alternative title for a taxonomy term causes duplicate 'Untitled' to display in search/browse results

Added by Dan Gillean over 3 years ago. Updated about 3 years ago.

Status:VerifiedStart date:03/07/2017
Priority:MediumDue date:
Assignee:Dan Gillean% Done:

0%

Category:Taxonomy / Term
Target version:Release 2.4.0
Google Code Legacy ID: Tested version:2.3, 2.4
Sponsored:No Requires documentation:

Description

Originally reported in the AtoM user forum, 2017-03-07: https://groups.google.com/d/msg/ica-atom-users/eOpAOFc_zQo/Wgqxbz10BAAJ

To reproduce
  • Navigate to one of the taxonomies, e.g. Places
  • Create a new term, and add an alternate label in the "Use For" field. Save.
  • Navigate to the search/browse page for the taxonomy. Use the sort button to display the most recent results.

Resulting error
Term browse UF label field shows "untitled" preceding the UF term.

See screenshot below (also attached for permanence).

Expected result
No accidental duplication of blank UF row on save.

term-untitled.png (7.42 KB) Dan Gillean, 03/07/2017 02:58 PM

History

#1 Updated by Dan Gillean over 3 years ago

  • Description updated (diff)

Updated description to add link back to user forum thread

#2 Updated by Nick Wilkinson over 3 years ago

  • Assignee set to Mike Gale

#3 Updated by Dan Gillean over 3 years ago

Note - the reporting user has provided a suggested fix in the forum. I will ask if he wants to submit a PR, but you might want to take a look at the proposed solution in advance to see if it makes sense, and conforms to our preferred coding practices. See: https://groups.google.com/d/msg/ica-atom-users/eOpAOFc_zQo/e8thSEJqBQAJ

#4 Updated by Mike Gale over 3 years ago

  • Status changed from New to Code Review
  • Assignee changed from Mike Gale to José Raddaoui Marín

#5 Updated by Dan Gillean over 3 years ago

  • Target version set to Release 2.4.0

#6 Updated by José Raddaoui Marín over 3 years ago

  • Status changed from Code Review to Feedback
  • Assignee changed from José Raddaoui Marín to Mike Gale

Added a small comment in the PR, but it looks good

#7 Updated by Mike Gale over 3 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from Mike Gale to Dan Gillean

#8 Updated by Dan Gillean about 3 years ago

  • Status changed from QA/Review to Verified

Also available in: Atom PDF