{"push_id":2886167336,"size":19,"distinct_size":19,"ref":"refs/heads/18.05.x","head":"145eef6a9037b93cd8902f0a77582fcadbf517bc","before":"3017d592a10da23fc12e6a94ba70d7ff9e7d7d94","commits":[{"sha":"637469974c3290a43ac259642df0fefdfe2d369f","author":{"name":"Tomas Cohen Arazi","email":"63ac9f6777fb741d62c33cd73a438a8ce82c7d4a@theke.io"},"message":"Bug 21285: Add missing select2x2.png file\n\nThis file is part of the Select2 jquery plugin, but wasn't added when\nthe use of the plugin was introduced in Koha.\n\nTo test:\n- On a high DPI screen (retina mac?) on Chrome or Safari\n- Edit an item\n=> FAIL: dropdowns using Select2 (like 'Shelving location') don't display the icons (down arrow and cross) and a default value is wrongly selected\n=> FAIL: the inspector says the requested select2x2.png file returned a 400 error.\n- Apply this patch\n- Reload the edit page\n=> SUCCESS: Things display correctly! Dropdowns without selected values\nby default!\n=> SUCCESS: No errors fetching select2x2.png\n- Sign off :-D\n\nSigned-off-by: Jesse Maseto <jesse@bywatersolutions.com>\n\nSigned-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit 235654a3cfa5b6e323489408ddc7875281bb1f50)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/637469974c3290a43ac259642df0fefdfe2d369f"},{"sha":"89f6aacecc787affd07856d6c092f21bacab7993","author":{"name":"David Cook","email":"12c544e20ab381c052e45e12406c394ee619015a@prosentient.com.au"},"message":"Bug 13779: sessionID declared twice in C4::Auth::checkauth()\n\nThe bug is that $sessionID is declared twice in C4::Auth::checkauth().\nAt the moment, this doesn't actually create a problem, because no\none seems to be using the $sessionID which is returned by checkauth(),\nexcept in the case of opac/external/overdrive/auth.pl which skips\nthe second declartion as it doesn't require auth.\n\nThis patch removes the redefining of the $sessionID variable.\n\nIn terms of testing, try logging in with a username and password\nand see if it works. The only risk this patch would pose is breaking\nauth I would think, since nothing is actually using the return value\nfrom checkauth() for $sessionID.\n\nNOTE:\nIt was initially defined near the top of the function (~line 791).\nI believe the scoping would mean the correct version of $sessionID\nwould be used in the latter lines for the unset'ing.\nI have skimmed code to see if the sessionID return value is used.\nI did not test overdrive, as I do not know how. However, this is\nthe only area, I think this could possibly break. This change makes\nsense to me.\n\nQA: Please test overdrive.\nopac/external/overdrive/auth.pl only checks if the value is set, so\n   this patch might fix/break something there?\nopac/svc/overdrive same kind of check.\n\nSigned-off-by: Mark Tompsett <mtompset@hotmail.com>\n\nSigned-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>\nThis is a (tiny) code improvement. Now we consistently return the session\nid that is also stored in the returned cookie. (Which can be an 'anon'\nsession.)\nFact is that Koha almost everywhere ignores the returned session id and\nsometimes gets the session from the cookie (obviously). The session id is\nalso passed to the template by get_template_and_user but never used in\ntemplates.\nAs mentioned, the two overdrive scripts are the exception. But since both\ntest on both $user && $sessionID, they will not choke on an anynonomous\nsession id without userid. So theoretically fine, but not tested.\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit 331af8521b5fba4b5553c182888a4017b2ce9c3a)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/89f6aacecc787affd07856d6c092f21bacab7993"},{"sha":"ee2d5ae04dc4c5fed9f09c9067c3de0037cdba8e","author":{"name":"Marcel de Rooy","email":"a4ed5d8b9e88a5c4401757374b1604745a5e46b6@rijksmuseum.nl"},"message":"Bug 13779: (QA follow-up) Correct checkauth call in offline_circ/download.pl\n\nScript offline_circ/download.pl does not use the return values of\ncheckauth. So should not assign them to bad chosen variables that do\nnot correspond with checkauth but with get_template_and_user.\n\nTest plan:\nLook for $template, $loggedinuser, $cookie, $flags in the code.\n\nSigned-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit bf7082f854a1db9eb5b7eb77e5f450d018d8e07b)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/ee2d5ae04dc4c5fed9f09c9067c3de0037cdba8e"},{"sha":"7fe692bf0ec4dd6744c81c068c400927ccc0c54d","author":{"name":"Pasi Kallinen","email":"7b512aa5acd92ce973869573b500589b80ee7f83@joensuu.fi"},"message":"Bug 21248: Fix COinS carp in MARC details page on unknown record\n\nLooking up a nonexistent record in the MARC details -page causes\nthe COinS to carp. The fix is trivial, just move the COinS call\ncouple lines, after the record existence check.\n\nTEST PLAN\n---------\n1) reset_all for kohadevbox\n2) log in to staff client - admin/admin\n3) search catalogue for green\n4) tweak the top URL to have a bad biblionumber (eg. 9999)\n5) cat /var/log/koha/kohadev/plack-err.log\n   -- floody! with errors.\n6) echo | sudo tee /var/log/koha/kohadev/plack-err.log\n7) restart_all\n8) repeat steps 2 through 5.\n   -- just nice starting messages, not floody.\n9) run koha qa test tools\n\nSigned-off-by: Mark Tompsett <mtompset@hotmail.com>\n\nSigned-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n\n(cherry picked from commit 030047da428521c248868e3ee5d655afd3bed05b)\n\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/7fe692bf0ec4dd6744c81c068c400927ccc0c54d"},{"sha":"903b880f330026d476d761901a2099972159a94c","author":{"name":"Owen Leonard","email":"9116c67f61a8025cb8cf35425d522a2ad0f46d71@myacpl.org"},"message":"Bug 21127: Remove jqTransform jQuery plugin from the OPAC\n\nThis patch removes the jqTransform jQuery plugin from the OPAC. It was\nadded for the CCSR theme, which no longer exists.\n\nhttp://git.koha-community.org/gitweb/?p=koha.git;a=commit;h=c7c2e9efbde273f2b77786316b7502a8847072bd\n\nThere will be no visible changes to the OPAC after applying the patch.\nSearch the Koha codebase for references to jqTransform and confirm that\nnone exist.\n\nSigned-off-by: Mark Tompsett <mtompset@hotmail.com>\n\nSigned-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit 83e515820abb6e2fa8ddf30f61b3f4972a48ed10)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/903b880f330026d476d761901a2099972159a94c"},{"sha":"9f221f9ae8b16846bd3b9b35b19d4b5434050f95","author":{"name":"Charlotte Cordwell","email":"6088a3dffc46b1a98ef8f78031f29a06b76ce45f@gmail.com"},"message":"Bug 19179: Email option for SMSSendDriver is not documented\n\n'Email' is not documented as a valid setting.\n\n1) Apply patch.\n2) Go to Administration --> search SMSSendDriver\n3) Check if SMSSendDriver has this message is at the end of the box:\n   \"If you would prefer to receive SMS via E-mail set SMSSendDriver to Email\"\n\nSigned-off-by:  Michal Denar <black23@gmail.coml>\n\nSigned-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit 2d6cf8bd7a0e290c8bb947ed1880f1e906d0e74c)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/9f221f9ae8b16846bd3b9b35b19d4b5434050f95"},{"sha":"31db0813d36f21a0c898b75841a3e34364619a84","author":{"name":"Marcel de Rooy","email":"a4ed5d8b9e88a5c4401757374b1604745a5e46b6@rijksmuseum.nl"},"message":"Bug 19179: (QA follow-up) Replace second E-mail\n\nBetter write here exactly what we need to type in the pref text box.\nSince E-mail will not work and Email will :)\n\nSigned-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit b0ef7174fe601bbf39408e455365ea563949b4bb)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/31db0813d36f21a0c898b75841a3e34364619a84"},{"sha":"b8b006fd73381b0be9aac2432aebae07cfffca7d","author":{"name":"Jonathan Druart","email":"90effa772b5c8a7a3420e6b7ea56e265888c5743@bugs.koha-community.org"},"message":"Bug 21355: Prevent GetDailyQuote.t to fail when run after 23:00\n\nTests will fail (Jenkins run #554) if run after 23:00\n\n #   Failed test 'Id is correct'\n #   at t/db_dependent/Koha/GetDailyQuote.t line 72.\n #          got: 6\n #     expected: 8\n\n #   Failed test 'Timestamp 2018-09-15 00:44:46 is correct'\n #   at t/db_dependent/Koha/GetDailyQuote.t line 74.\n #          got: '2018-09-14 23:44:46'\n #     expected: '2018-09-15 00:44:46'\n # Looks like you failed 2 tests of 12.\n [23:44:46] t/db_dependent/Koha/GetDailyQuote.t\n Dubious, test returned 2 (wstat 512, 0x200)\n\nAdding only 1 second will hugely reduce the risk of failure here.\n\nIt also updates a useless test (column 'quote' does not exist)\n\nSigned-off-by: Tomas Cohen Arazi <tomascohen@theke.io>\n\nTested:\n- Run:\n  $ sudo date -s 2018-09-15T23:10:00\n  $ kshell\n k$ prove t/db_dependent/Koha/GetDailyQuote.t\n=> FAIL: Tests fail as patch author mentions\n- Apply the patch\n k$ prove t/db_dependent/Koha/GetDailyQuote.t\n=> SUCCESS: Tests pass!\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit ca887754ead4d6207811d751229c4a4dbcf4af80)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/b8b006fd73381b0be9aac2432aebae07cfffca7d"},{"sha":"06f0209840d290bb1a7340f4b1af8d55e932f7c1","author":{"name":"Fridolin Somers","email":"028c9373aaf61b41f9c30e69951b05554a2bb781@biblibre.com"},"message":"Bug 20765: Fix search for items by acqdate in label batch\n\nIn Tools >  Label creator, when creating a batch you can add items using a search.\n\nThis search has filters on acquisition date.\nActually those filters does not work.\nThe create a search using CCL syntaxe to query Zebra search engine.\n\nThe cause is that operator \"ge\" or \"le\" must be before \"st-date-normalized\" in CCL query.\nLooks like it is because of \"st-date-normalized\" uses \"r=o\" in ccl.properties.\n\nTest plan :\n1) Check in advance search you have indexed acquisition date\n2) Go to Tools > Label creator\n3) Click on New > Label batch\n4) Click on Add item(s)\n5) Enter a relevent date in \"Added on or after date\" and in \"Added on or before date\"\n6) Click on \"Search\"\n=> SUCCESS: You get results\n7) Apply the patch\n8) Retry the 'Add item(s)' step\n=> SUCCESS: Patch doesn't change behaviour for old Zebra\n9) Get rid of the patch on your working branch\n10) Use the latest Zebra:\n- /etc/apt/sources.list.d/indexdata.list:\ndeb http://ftp.indexdata.dk/debian jessie main\n- Add the repo key:\n  $ wget http://ftp.indexdata.dk/debian/indexdata.asc -O- | sudo apt-key add -\n- Upgrade:\n  $ sudo apt update; sudo apt dist-upgrade\n  $ sudo koha-rebuild-zebra -f -a -b -v kohadev\n  $ restart_all\n11) Retry the 'Add item(s)' step\n=> FAIL: No results!\n9) Apply the patch, retry\n=> SUCCESS: Search results!\n- Sign off :-D\n\nSigned-off-by: Maryse Simard <maryse.simard@inlibro.com>\nFollowed the test plan and it works.\nSigned-off-by: Tomas Cohen Arazi <tomascohen@theke.io>\nIt doesn't break with 2.0.59-1 (stretch/jessie), and fixes the problem\nfor 2.1.X.\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit a6f8eac4edb6463d8ff16928d6f2cddcf4dfbfc2)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/06f0209840d290bb1a7340f4b1af8d55e932f7c1"},{"sha":"40536fedd39a0a22c3e93a00721b573d66392e09","author":{"name":"Pasi Kallinen","email":"7b512aa5acd92ce973869573b500589b80ee7f83@joensuu.fi"},"message":"Bug 21288: Slowness in acquisition caused by GetInvoices\n\nThe SQL query done by GetInvoices is quite slow on our production server,\ntaking about a minute.\n\nIn the query, there's this join:\n\n  LEFT JOIN subscription ON biblio.biblionumber = subscription.biblionumber\n\nbut there's no index on subscription.biblionumber. Adding the index speeds\nup the query so it only takes up about 3 seconds.\n\nSigned-off-by: Pasi Kallinen <pasi.kallinen@joensuu.fi>\n\nSigned-off-by: Mark Tompsett <mtompset@hotmail.com>\nSigned-off-by: Chris Cormack <chris@bigballofwax.co.nz>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit cb406f72b0266c97ac904754cc9637035000f5ca)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/40536fedd39a0a22c3e93a00721b573d66392e09"},{"sha":"44ce16d0069fdff7b487753277c11d368a083ccb","author":{"name":"Nick Clemens","email":"75ef9faee755c70589550b513ad881e5a603182c@bywatersolutions.com"},"message":"Bug 21288: DBRev 18.05.03.002\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit 11e513f83f0c5fdd0d47de14b7c09635faf58ae4)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/44ce16d0069fdff7b487753277c11d368a083ccb"},{"sha":"d3d43360b04864f13e7a2d6fdf37e3bcf4cf1e83","author":{"name":"Tomas Cohen Arazi","email":"63ac9f6777fb741d62c33cd73a438a8ce82c7d4a@theke.io"},"message":"Bug 20777: Remove unused accountlines.dispute field\n\nSigned-off-by: Tomas Cohen Arazi <tomascohen@theke.io>\n\nSigned-off-by: Mark Tompsett <mtompset@hotmail.com>\n\nSigned-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit f9359675e2f6a8b89ce471ac5f090eef33be3305)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/d3d43360b04864f13e7a2d6fdf37e3bcf4cf1e83"},{"sha":"b2bfea26a72a13ea6b5bb33e7777ebd604d8f015","author":{"name":"Tomas Cohen Arazi","email":"63ac9f6777fb741d62c33cd73a438a8ce82c7d4a@theke.io"},"message":"Bug 20777: DBIC update\n\nSigned-off-by: Tomas Cohen Arazi <tomascohen@theke.io>\n\nSigned-off-by: Mark Tompsett <mtompset@hotmail.com>\n\nSigned-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit b64f1808f1682c5e132e0ddbc19ae5869a55bd99)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/b2bfea26a72a13ea6b5bb33e7777ebd604d8f015"},{"sha":"6fe2ebd1e465b5458a26aa736a02c944ba9b8f9d","author":{"name":"Tomas Cohen Arazi","email":"63ac9f6777fb741d62c33cd73a438a8ce82c7d4a@theke.io"},"message":"Bug 20777: (follow-up) Remove 'dispute' reference in /credits endpoint\n\nSigned-off-by: Tomas Cohen Arazi <tomascohen@theke.io>\n\nSigned-off-by: Mark Tompsett <mtompset@hotmail.com>\n\nSigned-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit 2d94834807851c3325f26572c7a228903034741c)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/6fe2ebd1e465b5458a26aa736a02c944ba9b8f9d"},{"sha":"9659b2f447ef21a71b3d3698fba126be526a8849","author":{"name":"Nick Clemens","email":"75ef9faee755c70589550b513ad881e5a603182c@bywatersolutions.com"},"message":"Bug 20777: DBRev 18.05.03.003\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit 2847ac4b4de239a430e6e9d20f6e895435c56d7b)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/9659b2f447ef21a71b3d3698fba126be526a8849"},{"sha":"c28d8f0600806f9930d115b2fdbccc21f5e86436","author":{"name":"Jonathan Druart","email":"90effa772b5c8a7a3420e6b7ea56e265888c5743@bugs.koha-community.org"},"message":"Bug 21360: Prevent IssueSlip.t to fail when run at 23:59\n\nNot sure what happened on bug 15029 (Make the IssueSlip tests pass even if\nlaunched after 23:00) but we should have tested with 23:59, not 23:50...\n\nkoha_1       |     # Looks like you planned 3 tests but ran 2.\nkoha_1       |\nkoha_1       | #   Failed test 'Issue slip'\nkoha_1       | #   at t/db_dependent/Members/IssueSlip.t line 294.\nkoha_1       | Label not found for \"last SKIP\" at /usr/share/perl5/Test/More.pm line 1370.\nkoha_1       | # Looks like your test exited with 255 just after 1.\nkoha_1       | [23:59:10] t/db_dependent/Members/IssueSlip.t\nkoha_1       | Dubious, test returned 255 (wstat 65280, 0xff00)\nkoha_1       | Failed 3/3 subtests\n\nTest plan:\n0/ Don't apply this patch\n1/ sudo date -s \"2018-09-15 23:59:00\";prove t/db_dependent/Members/IssueSlip.t\n=> You should get the error\n2/ Apply this patch and repeat 1.\n=> Tests should pass now!\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit 60788555b3d79902ccef9f20cf76d4c764efcba0)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/c28d8f0600806f9930d115b2fdbccc21f5e86436"},{"sha":"ad16c9f5b20525303eb631107883cb37e36de8cd","author":{"name":"Nick Clemens","email":"75ef9faee755c70589550b513ad881e5a603182c@bywatersolutions.com"},"message":"Bug 21262: Do not format numbers for editing if too big\n\nThis patch copies the code from bug 15770 to the format_for_editing\nsub.\n\nIt also corrects schema issue, listing the purpose for two acq columns\ninstead of claing unused\n\nTo test:\n1 - Edit a fund\n2 - Set warning at amount to: 100000000000000\n3 - Save budget\n4 - Try to edit - internal server error\n5 - Apply patch\n6 - Should be able to edit budget\n7 - Number should appear unformatted\n\nSigned-off-by: Christopher Brannon <cbrannon@cdalibrary.org>\n\nSigned-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>\n2 quick notes:\n1. Code must be moved to a subroutine if reused later\n2. It would have prefered to have the comment of the 2 DB columns on a separate patch\n(even separate bug report). As it's minor and that the patch will certainly backported\nI will not block it.\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit 7a84297ac4b29f4bdaf1bc4d01def71eb924d4fa)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/ad16c9f5b20525303eb631107883cb37e36de8cd"},{"sha":"c82595a4fa5c93ba5220ab0c95bc296f70fa5a78","author":{"name":"Caroline Cyr La Rose","email":"8b0c7a0dfc90def3027a6a938ed2701d29741026@inlibro.com"},"message":"Bug 21356: Add Missing space in parcel.tt\n\nThis patch adds a space between the user's name and 'On:'\nin parcel.tt.\n\nTo test:\n\n1) Go to Acquisitions\n2) Search for a vendor\n3) Create a new basket\n4) Add an order to the basket\n5) Close the basket\n6) Receive the order\n7) In the page title 'Receipt summary' notice there is no space\nbetween the 'Received by' user name and 'On:'\n8) Apply the patch\n9) Refresh the page, there should now be a space\n\nSigned-off-by: Fridolin Somers <fridolin.somers@biblibre.com>\n\nSigned-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit d56c19138c36b79ba7d46c0a65c92469bf6fd12c)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/c82595a4fa5c93ba5220ab0c95bc296f70fa5a78"},{"sha":"145eef6a9037b93cd8902f0a77582fcadbf517bc","author":{"name":"Owen Leonard","email":"9116c67f61a8025cb8cf35425d522a2ad0f46d71@myacpl.org"},"message":"Bug 13692: series link is taking you to just 800a instead of whole 800\n\nThis patch modifies the OPAC and staff client's XSL template for the\ndetail page so that series links based on a traced 490 tag use both\nauthor and series title from the 800 tag.\n\nTo test you should have multiple titles in the same series with matching\n490 and 800 tags in each record. Apply the patch and restart Plack if\nnecessary.\n\nView the detail page for one of the series' titles. The \"Series:\" link\nshould link to a search which uses both series title and author.\nClicking the link should return the correct results.\n\nTest both the OPAC and staff client.\n\nSigned-off-by: John Sterbenz <jsterben@umich.edu>\nSigned-off-by: Chris Cormack <chris@bigballofwax.co.nz>\n\nSigned-off-by: Nick Clemens <nick@bywatersolutions.com>\n(cherry picked from commit cf5a92461835725fe44bb726c98bcfbf909dfdc8)\nSigned-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>","distinct":true,"url":"https://api.github.com/repos/colinsc/koha/commits/145eef6a9037b93cd8902f0a77582fcadbf517bc"}]}
{"actor":{"display_login":"colinsc"}}