Bug #11961

Location information is not included in reports when default culture is not en

Added by Dan Gillean over 1 year ago. Updated about 1 year ago.

Status:VerifiedStart date:02/07/2018
Priority:MediumDue date:
Assignee:-% Done:

0%

Category:Physical storage
Target version:Release 2.5.0
Google Code Legacy ID: Tested version:2.4, 2.5
Sponsored:No Requires documentation:

Description

  • In apps/qubit/config/settings.yml, set the default installation culture to another language, like "fr"
  • Remember to restart services and reindex
  • Create a new description with some item-level children in the French UI
  • Link the item-level children to new storage locations - ensure new locations have all info filled out (name, location, type)
  • Return to parent description, and generate a physical locations report (e.g. HTML - though same issue is present in CSV)
  • When it is generated, open the new report
Error encountered
  • Name on report is a raw URL, instead of a hyperlinked name
  • Location is not included in the report

See attached screenshot.

Expected result
  • Reporting works across cultures. If the descriptions are in french, the UI is in french, and the report request was submitted via the French interface, then there should be no conflicts, and the report should work.

storage-locations-report-error.png (37.4 KB) Dan Gillean, 02/07/2018 10:05 AM


Related issues

Related to Access to Memory (AtoM) - Bug #12053: Culture fallback and physical location data export: physi... Verified 03/20/2018

History

#1 Updated by Dan Gillean over 1 year ago

  • Related to Bug #12053: Culture fallback and physical location data export: physical storage data not included in export if it is created in another culture added

#2 Updated by Nick Wilkinson over 1 year ago

  • Assignee set to José Raddaoui Marín

Hi Radda, can you look into this please?

#3 Updated by Dan Gillean over 1 year ago

See the related issue as well - #12053

#4 Updated by José Raddaoui Marín over 1 year ago

  • Assignee changed from José Raddaoui Marín to Nick Wilkinson

Hi Nick, I think it's better if this is done alongside #12053

#5 Updated by Nick Wilkinson over 1 year ago

  • Assignee changed from Nick Wilkinson to Mike Cantelon

Hi Mike, assigning this to you since you're already looking into #12053.

#6 Updated by Mike Cantelon over 1 year ago

  • Status changed from New to Code Review
  • Assignee changed from Mike Cantelon to Nick Wilkinson

PR for CR: https://github.com/artefactual/atom/pull/716

I wasn't able to replicate the problem specified in this issue in 2.5, but there was an error preventing the report from being generated.

I'm going to see if I can replicate the issue in 2.4 and submit a separate PR for that as I don't think the problem the above PR fixes will be present in 2.4.

#7 Updated by Nick Wilkinson about 1 year ago

  • Assignee changed from Nick Wilkinson to José Raddaoui Marín

Hi Radda, passing to you for CR.

#8 Updated by José Raddaoui Marín about 1 year ago

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

Looks good.

#9 Updated by Mike Cantelon about 1 year ago

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

I can't replicate the problem in 2.4 either. Maybe something fixed it at some point? Hopefully.

#10 Updated by Dan Gillean about 1 year ago

  • Status changed from QA/Review to Verified
  • Assignee deleted (Dan Gillean)
  • Target version set to Release 2.5.0

Strangely, French accents don't display properly on the HTML report page - but that's a separate issue, and can be filed as one if needed. I repeated the test described above, and got hyperlinks and location names in the HTML report, so I think that the original issue has been addressed. I was able to generate the report when default culture was FR, and EN, so it seems like Mike's fix might have resolved a different regression! As such, marking this as verified for now. Thanks!

Also available in: Atom PDF