]> git.koha-community.org Git - koha.git/commit
Bug 7961 - Local cover images should support CSV link files
authorDobrica Pavlinusic <dpavlin@rot13.org>
Sat, 14 Apr 2012 09:39:20 +0000 (11:39 +0200)
committerChris Cormack <chrisc@catalyst.net.nz>
Wed, 20 Jun 2012 09:20:17 +0000 (21:20 +1200)
commit453e12b5e4c6c816e9a15857f3ca2c812c50c1cf
tree8e0cadae9b89422754df5a9970bb657ac49b72e3
parent43b98c01d6d4793423469cd7562912649a2cd391
Bug 7961 - Local cover images should support CSV link files

Corrent code doesn't have support for filenames which contain spaces
or commans which breaks CSV files saved from spreadsheet similar to:

  12345, "conver image, with spaces.jpg"

This patch tweaks file parsing a bit. We are always splitting line to
only two values (to support commas as part of filename) and removing
spaces only on beginning and end of filename (to cover space after
comma in CSV example above while preserving spaces in filename).

With this change only invalid character in picture filename left
are quotes (") which are commonly used to quote strings with spaces.

Covers added will be logged in action_log, using CATALOGUING / MODIFY
action (which is shown as "Catalog" in tools > Log viewer)

Test scenario:

1. collect pictures with spaces and commas in name
2. dump file list into CSV file and add biblio number as first column
   (name of file is idlink.txt or datalink.txt)
3. create zip with CSV file and pictures
4. verify that all pictures got uploaded and linked to biblio records
5. verify that modification log includes cover image name

Signed-off-by: Olugbenga Adara <gbengaadara@yahoo.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
tools/upload-cover-image.pl