search

UCSCLibrary / ucsc-library-digital-collections

A rails app based on Hyrax to be used as a repository for UCSC library digital collections.
1 stars 2 forks source link

(2)Aerials metadata: Backslashes appearing in front of semi-colons #567

Closed rmjaffe closed 1 year ago

rmjaffe commented 1 year ago

Describe the bug

In reviewing the aerials metadata, I've noticed that where semi-colons appear within the text of a description they are proceeded by a . Before bulkrax, we had used semi-colons as delimiters and backslashes (\;) as needed to escape them out. After we adopted | as our delimiter, I removed the backslashes from the ingest spreadsheets, nonetheless backslashes are still appearing. Not clear on why this is happening.

To reproduce

Steps to reproduce the behavior:

  1. Go to a work within the aerials collection, for example: http://digitalcollections-staging-sandbox.library.ucsc.edu/records/2b88qc57q
  2. Look at the contents of the description property
  3. See the \;

Expected behavior

No backslashes!

Screenshots

Example 1 What I'm seeing in sandbox:

Screen Shot 2023-02-07 at 3 12 19 PM

What's on the ingest spreadsheet: title: 1937 Santa Cruz County and Monterey County Flight 13-ABG 5-88 inherited description (parent work=1937 Santa Cruz County and Monterey County Flight 13-ABG): Santa Cruz County and Monterey County Flight 13-ABG: Salinas River Valley, from the Monterey Bay to San Ardo; including Watsonville and the Pajaro Valley, Elkhorn Slough, Salinas, Chualar, Gonzales, Soledad, Greenfield, King City, San Lucas, and San Ardo.

Example 2 What I'm seeing in sandbox:

Screen Shot 2023-02-07 at 3 12 36 PM

What's on the ingest spreadsheet: Screen Shot 2023-02-07 at 3 31 26 PM

Acceptance criteria

Done means:

snehagunduraoUL commented 1 year ago

This is a user error. The excel sheet that I used was an old one I guess. It already had the 'backslash' in it. Tested again with example2 above and here is the result. http://digitalcollections-staging-sandbox.library.ucsc.edu/concern/works/8910jv00x?locale=en

rmjaffe commented 1 year ago

Good to know -- thanks for investigating and testing! I'll keep any eye out for this to see if it becomes an issue again.

rmjaffe commented 1 year ago

Given that it doesn't seem to be real issue, should we close this out and move it to done?

snehagunduraoUL commented 1 year ago

@rmjaffe Yes. Sounds good to me.