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 (vote)
Assignee: Julian Maurice
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 17703
  Show dependency treegraph
 
Reported: 2014-02-27 10:26 UTC by Julian Maurice
Modified: 2016-11-30 13:33 UTC (History)
4 users (show)

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


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

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!