Bug #10633

Investigate potential Ubuntu 16.04 compatibility issues with CSV import

Added by Nick Wilkinson over 3 years ago. Updated over 3 years ago.

Status:VerifiedStart date:12/02/2016
Priority:MediumDue date:
Assignee:Nick Wilkinson% Done:

0%

Category:CSV import
Target version:Release 2.3.1
Google Code Legacy ID: Tested version:
Sponsored:No Requires documentation:

test-ISAD-2.3-import.csv Magnifier (4.24 KB) Dan Gillean, 12/13/2016 02:23 PM

form-error.png (60.5 KB) Dan Gillean, 12/13/2016 02:35 PM

History

#1 Updated by David Hume over 3 years ago

  • Status changed from New to In progress
  • Assignee set to Steve Breker

Created Digital Ocean VM 'atom1604test' (16.04.1 x64, 4GB/60GB as 'atomtest', San Francisco 1 data center).

Access by ssh keys, created sbreker, also system admins, David J for Hylas, Jesus at home, including for artefactual account created.

- Dave H.

#2 Updated by Steve Breker over 3 years ago

  • Status changed from In progress to Code Review
  • Assignee changed from Steve Breker to Nick Wilkinson

PR ready for code review.

This change corrects the csv import bug where fields are missing from the record after import when running AtoM under Ubuntu 16.04/PHP 7.

https://github.com/artefactual/atom/pull/478

#3 Updated by Nick Wilkinson over 3 years ago

  • Assignee changed from Nick Wilkinson to Jesús García Crespo

#4 Updated by Jesús García Crespo over 3 years ago

  • Status changed from Code Review to Feedback
  • Assignee changed from Jesús García Crespo to Steve Breker

#5 Updated by Steve Breker over 3 years ago

This fix addresses the issue where fields were missing from records imported via CSV under 16.04.

- for ISAD imports, Level of Description was not being imported correctly.
- for RAD imports, radGeneralMaterialDesignation was not being imported correctly.

#6 Updated by Steve Breker over 3 years ago

This fix will need to be cherry picked for stable/2.3.x.

#7 Updated by Steve Breker over 3 years ago

Testing in qa/2.4.x under Ubuntu 14.04: The reported issue did not occur under 14.04. Verify that these fields continue to be imported.
Testing qa/2.4.x under Ubuntu 16.04: The reported issue occurs under 16.04. The fields in question will not be imported when this patch is not in place.

I have configured a clean AtoM instance based on qa/2.4.x to test this fix here:

http://198.199.111.25/

This server was configured by DaveH and is running Ubuntu 16.04.

#8 Updated by Steve Breker over 3 years ago

  • Copied to Bug #10675: PDF derivatives not generating correctly on Ubuntu 16.04 added

#9 Updated by Steve Breker over 3 years ago

  • Copied to deleted (Bug #10675: PDF derivatives not generating correctly on Ubuntu 16.04)

#10 Updated by Steve Breker over 3 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from Steve Breker to Nick Wilkinson

#11 Updated by Nick Wilkinson over 3 years ago

  • Assignee changed from Nick Wilkinson to Dan Gillean

#12 Updated by Dan Gillean over 3 years ago

  • File test-ISAD-2.3-import.csvMagnifier added
  • Status changed from QA/Review to Feedback
  • Assignee changed from Dan Gillean to Steve Breker

Welp, can't test the CSV import issues because import does not seem to be working at all. In fact I noticed that right now it looks like the old 2.2 import page, rather than the new import page - not sure if this is actually a 2.3 install, but Steve's notes above said it was from qa/2.4.x so I expected something else. In any case, when I click import, the page reloads to a "Page not found" message and nothing is imported. Attaching the sample file I was using: a standard ISAD 2.3 CSV template, with one additional row in it.

Will continue testing the PDF derivs issue in the meantime.

#13 Updated by Dan Gillean over 3 years ago

Also, tried to create a new description - clicking the Create button did nothing, could not save the description. I checked all the permissions settings for the group and the demo user - doesn't look like anything has been changed. Using Firefox 50.0.2 on Windows 10. Will try again w a different browser - no update on this thread means no joy.

#14 Updated by Dan Gillean over 3 years ago

No Joy in Chrome either. Adding a screenshot of the browser error returned in case it's helpful.

#15 Updated by Steve Breker over 3 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from Steve Breker to Dan Gillean

I did not clear the cache(es) since my previous test install. All is cleared up now and ready to test.

#16 Updated by Dan Gillean over 3 years ago

  • Subject changed from Investigate potential Ubuntu 16.04 compatibility issues to Investigate potential Ubuntu 16.04 compatibility issues with CSV import
  • Category set to CSV import
  • Status changed from QA/Review to Verified
  • Target version set to Release 2.3.1

#17 Updated by Steve Breker over 3 years ago

  • Assignee changed from Dan Gillean to Nick Wilkinson

Cherry-picked to stable/2.3.x.

Also available in: Atom PDF