Bug 11861 - Add ISSN column in serials claims table
Summary: Add ISSN column in serials claims table
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Serials (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Julian Maurice
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 17703
  Show dependency tree / graph
 
Reported: 2014-02-27 10:26 UTC by Julian Maurice
Modified: 2016-11-30 13:33 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Bug 11861: Add ISSN column in serials claims table (2.58 KB, patch)
2014-02-27 10:27 UTC, Julian Maurice 		Details | Diff | Splinter Review
[PASSED QA] Bug 11861: Add ISSN column in serials claims table (2.78 KB, patch)
2014-03-12 09:12 UTC, Katrin Fischer 		Details | Diff | Splinter Review
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Julian Maurice 2014-02-27 10:26:39 UTC 
Add ISSN column in serials claims table
Comment 1 Julian Maurice 2014-02-27 10:27:10 UTC Comment hidden (obsolete)
Comment 2 Julian Maurice 2014-02-27 12:48:27 UTC 
Test plan:
1/ Go to serials claims page
2/ Choose a vendor and submit
3/ You should see a ISSN column, filled with subscription ISSN
Comment 3 Koha Team University Lyon 3 2014-03-11 15:35:13 UTC 
ok for me. I go to the claims page and the table I can see a column with the ISSN.
Comment 4 Katrin Fischer 2014-03-12 09:12:20 UTC 
Created attachment 26143 [details] [review]
[PASSED QA] Bug 11861: Add ISSN column in serials claims table

Signed-off-by: Koha Team Lyon 3 <koha@univ-lyon3.fr>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Added Sign off line.
Passes all tests and QA script, including t/db_dependent/Serials.t
Comment 5 Katrin Fischer 2014-03-12 09:13:24 UTC 
This is a rather small change and we have a simple test for the routine in t/db_dependent/Serials.t, but some more tests would be good.
Comment 6 Galen Charlton 2014-03-12 18:03:13 UTC 
Pushed to master.  Thanks, Julian!