Bug #13201

Browse Physical storage page does not show values that have not been translated

Added by David Juhasz about 1 year ago. Updated 12 months ago.

Status:VerifiedStart date:10/10/2019
Priority:MediumDue date:
Assignee:-% Done:

0%

Category:I18NEstimated time:2.00 hours
Target version:Release 2.6.0
Google Code Legacy ID: Tested version:2.6
Sponsored:Yes Requires documentation:

Description

To reproduce

  1. Login as a user with permissions to edit physical storage data
  2. Switch the AtoM UI to a culture other than the default culture
  3. Click "Manage > Physical storage" from the menus
  4. Click "Add new" at the bottom of the page
  5. Enter form data (e.g. name, location) and click "Save"
  6. Switch the AtoM UI to another culture
  7. Click "Manage > Physical storage" from the menus, and find the new item you added

Resulting error

  1. The "location" field is blank

Expected result

  1. The Browse Physical storage page should fall back to the source culture value for name and location if there is no translation in the current UI culture

issue13201_browse_physical_storage_no_fallback.png - Screenshot - highlighted row was entered in French with name, location and type (35.1 KB) David Juhasz, 10/10/2019 05:42 PM


Related issues

Related to Access to Memory (AtoM) - Feature #13183: Add CSV import for physical locations Verified 09/25/2019 09/25/2019

History

#2 Updated by David Juhasz 12 months ago

  • Related to Feature #13183: Add CSV import for physical locations added

#3 Updated by David Juhasz 12 months ago

  • Description updated (diff)

The problem with rows having the wrong source_culture was limited to using the csv:physicalobject-import task to create the record, and I've fixed that issue in the import code. I've removed the source_culture problem from the issue description.

#4 Updated by David Juhasz 12 months ago

  • Status changed from New to Code Review

One line fix ready for code review: https://github.com/artefactual/atom/pull/988

#5 Updated by Dan Gillean 12 months ago

  • Status changed from Code Review to QA/Review
  • Target version set to Release 2.6.0

#6 Updated by David Juhasz 12 months ago

Merged to qa/2.6.x

#7 Updated by Dan Gillean 12 months ago

  • Status changed from QA/Review to Verified
  • Sponsored changed from No to Yes

Looks good!

Also available in: Atom PDF