{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":6492705,"defaultBranch":"master","name":"Koha","ownerLogin":"mrenvoize","currentUserCanPush":false,"isFork":true,"isEmpty":false,"createdAt":"2012-11-01T16:17:34.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/557612?v=4","public":true,"private":false,"isOrgOwned":false},"refInfo":{"name":"","listCacheKey":"v0:1714662341.0","currentOid":""},"activityList":{"items":[{"before":"5f031219ba0f242cab75cd07d4fcdf30af91d177","after":"003cfc71ce4bdb83015c2dfeecb13b7d095459c8","ref":"refs/heads/main","pushedAt":"2024-05-24T15:07:51.000Z","pushType":"push","commitsCount":15,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 31981: (follow-up) Fix typo in class name\n\n.debt_gaurantors > .debt_guarantors\n\nSigned-off-by: Katrin Fischer ","shortMessageHtmlLink":"Bug 31981: (follow-up) Fix typo in class name"}},{"before":"b7aa395b6b9be134267baad6270bb459c9042443","after":"5cb492a6cdcdb349f4877bffa2d8c32d63948d94","ref":"refs/heads/23.05.x","pushedAt":"2024-05-23T16:39:05.000Z","pushType":"push","commitsCount":20,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36033: DBRev 23.05.11.001\n\nSigned-off-by: Lucas Gass ","shortMessageHtmlLink":"Bug 36033: DBRev 23.05.11.001"}},{"before":"0d4ee5319875439d2309bfae5d7f0054d2d02194","after":"5f031219ba0f242cab75cd07d4fcdf30af91d177","ref":"refs/heads/main","pushedAt":"2024-05-23T16:38:54.000Z","pushType":"push","commitsCount":16,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36904: Fix batch->ill_batch\n\nThis was missed when renaming follow-ups were added to bug 30719\n\n1) Enable ILL, install FreeForm and checkout the current compatible branch with main\n bash <(curl -s https://raw.githubusercontent.com/ammopt/koha-ill-dev/master/start-ill-dev.sh)\n cd /kohadevbox/koha/Koha/Illbackends/FreeForm\n git checkout reorganize_ILL\n2) Visit ILL module:\n http://localhost:8081/cgi-bin/koha/ill/ill-requests.pl\n3) Type whatever search in the tiny 'Search' input box directly above the table\n4) Notice you get an error. Apply patch. Repeat.\n\nSigned-off-by: David Nind \nSigned-off-by: Tomas Cohen Arazi \nSigned-off-by: Katrin Fischer ","shortMessageHtmlLink":"Bug 36904: Fix batch->ill_batch"}},{"before":"99b47d08728f196ce44a1d39fca13d311a719af7","after":"789bdc2a489ebdc8984299add2b9081965b75605","ref":"refs/heads/22.11.x","pushedAt":"2024-05-23T10:20:34.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Revert \"Bug 36532: Protect opac-dismiss-message.pl from malicious usages\"\n\nThis reverts commit 2278d229e899cd279f62addd8275365718ad8cbb.","shortMessageHtmlLink":"Revert \"Bug 36532: Protect opac-dismiss-message.pl from malicious usa…"}},{"before":"40900343673c280d092de9c57399a77c113b41ba","after":"0d4ee5319875439d2309bfae5d7f0054d2d02194","ref":"refs/heads/main","pushedAt":"2024-05-22T16:41:01.000Z","pushType":"push","commitsCount":19,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36665: (follow-up) Rephrase system preference description\n\n... and replace some branches with libraries.\n\nSigned-off-by: Katrin Fischer ","shortMessageHtmlLink":"Bug 36665: (follow-up) Rephrase system preference description"}},{"before":"2884226fe27cd3e4d5f4a070405c047183fed881","after":"40900343673c280d092de9c57399a77c113b41ba","ref":"refs/heads/main","pushedAt":"2024-05-21T13:16:48.000Z","pushType":"push","commitsCount":59,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36892: Fix label for 'is_standing' input\n\nTest plan:\n\n1) Go to acqui/histsearch.pl and look for \"search_children_too\" with\n dev tools console.\n2) On 'is_standing' input, the same label is used\n3) Apply this patch and reload it\n4) Now it's correct\n\nSponsored by: BibLibre\n\nSigned-off-by: Owen Leonard \nSigned-off-by: Pedro Amorim \nSigned-off-by: Katrin Fischer ","shortMessageHtmlLink":"Bug 36892: Fix label for 'is_standing' input"}},{"before":"3604df19920174c23a7385f7fb2cf5f62361e788","after":"e67a8dcb1999c638b5721df98e2b8ef6c4c1283c","ref":"refs/heads/22.05.x","pushedAt":"2024-05-15T07:22:52.000Z","pushType":"push","commitsCount":12,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Update release notes for 22.05.21 release\n\nSigned-off-by: Wainui Witika-Park ","shortMessageHtmlLink":"Update release notes for 22.05.21 release"}},{"before":"72cead50b49efaad6349cf653b970e6f5b610475","after":"b7aa395b6b9be134267baad6270bb459c9042443","ref":"refs/heads/23.05.x","pushedAt":"2024-05-15T07:22:33.000Z","pushType":"push","commitsCount":29,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36053: Correct input name for item replacement fields\n\nTo test:\n1 - Follow test plan and use sample record from bug 35912 ( or similar)\n2 - Confirm that when the order is saved the replacement price is $0.00\n3 - Apply patch, restart_all\n4 - Delete previous orders, and stage and add to basket again\n5 - Confirm that upon adding orders to basket the replacement price is saved correctly\n\nSigned-off-by: Brendan Lawlor \n\nSigned-off-by: Marcel de Rooy \nSigned-off-by: Katrin Fischer \n(cherry picked from commit 94da493d0eaaec80e42258ec421eb7c5ca1a0dd2)\nSigned-off-by: Fridolin Somers \n(cherry picked from commit f53018dce0fa3db1afacb5914e8ee2e57794296d)\nSigned-off-by: Lucas Gass ","shortMessageHtmlLink":"Bug 36053: Correct input name for item replacement fields"}},{"before":"fffb3665eadec04792b055e9ba04152b3b36d6dd","after":"2884226fe27cd3e4d5f4a070405c047183fed881","ref":"refs/heads/main","pushedAt":"2024-05-15T07:22:22.000Z","pushType":"push","commitsCount":15,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36772: OPAC Self checkout accepts wrong or partial barcodes\n\n When using the opac trusted checkout feature it is possible to enter non existant or partial barcodes to check out items.\n\n Testplan:\n a) set System preference OpacTrustedCheckout to Allow\n b) go to http://localhost:8080/ and login with koha / koha\n c) click Self Checkout in navigation bar\n d) enter barcode 1234\n e) result: Item '39999000011234' was checked out\n\n apply patch and reload the page\n\n a) now enter barcode 1234\n b) result: Item '1234' not found\n c) enter 39999000011234\n d) result: Item '39999000011234' was checked out\n\nSigned-off-by: Matt Blenkinsop \nSigned-off-by: Martin Renvoize \nSigned-off-by: Katrin Fischer ","shortMessageHtmlLink":"Bug 36772: OPAC Self checkout accepts wrong or partial barcodes"}},{"before":"2eab2949fc50c752c3bf950c887e54845353e900","after":"99b47d08728f196ce44a1d39fca13d311a719af7","ref":"refs/heads/22.11.x","pushedAt":"2024-05-13T12:13:17.000Z","pushType":"push","commitsCount":9,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Update release notes for 22.11.17 release\n\nSigned-off-by: Frédéric Demians ","shortMessageHtmlLink":"Update release notes for 22.11.17 release"}},{"before":"c74169ba27c2e12473239b15358e428bc05da9f5","after":"fffb3665eadec04792b055e9ba04152b3b36d6dd","ref":"refs/heads/main","pushedAt":"2024-05-13T12:12:55.000Z","pushType":"push","commitsCount":24,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36723: Add musical presentation to Elasticsearch index mappings\n\nTesting plan:\n\n1. Start up KTD with Elasticsearch: ktd --es8 up\n2. Apply the patch and restart everything (restart_all).\n3. Reset the mappings: Administration > Catalog > Search engine configuration (Elasticsearch) > Reset mappings (at the bottom of the page)\n4. Reindex: koha-elasticsearch --rebuild -d -b -a kohadev\n5. Alternative to steps 3 and 4: reset_all\n6. Update the visibility for 254$a in the default framework so that it is visible in the Editor (OPAC, Staff interface should already be selected).\n7. Add a new record using the default framework and put a term in 254$a, such as 'Full score'.\n\nSponsored-by: Education Services Australia SCIS\nSigned-off-by: David Nind \nSigned-off-by: Katrin Fischer ","shortMessageHtmlLink":"Bug 36723: Add musical presentation to Elasticsearch index mappings"}},{"before":"b1944c78954ef00873ba0a8bc9c1da1bf4ec9577","after":"c74169ba27c2e12473239b15358e428bc05da9f5","ref":"refs/heads/main","pushedAt":"2024-05-13T09:43:24.000Z","pushType":"push","commitsCount":44,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36794: Illegitimate modification of biblionumber subfield content (999 $c)\n\nIt happens that librarians, by mistake, open a biblio editor putting in the\nURL, by hand, the biblionumber prefixed with a blank (e.g.\n.../addbiblio.pl?biblionumber= 123 -- mind the space before 123).\nIn such a case the editor opens with the right biblio record (i.e. 123)\nbut, after saving the record, the content of the biblionumber MARC\nfield (999 $c for a standard MARC 21 installation) results modified and\ncontains additional initial blanks.\n\nMoreover, while using ES and making a search for the record (with title,\nauthor etc.) we get two records on the result list (instead of one).\n\nThis is because in the addbiblio.pl script $biblionumber is taken (and\ncontinuously used) directly from CGI parameter, without any\nvalidation and/or correction.\n\nTest plan:\n==========\n0. Have a test installation with ES.\n1. Open a biblio record in the editor with an added space before\n biblionumber value, e.g.:\n http://ktd:8081/cgi-bin/koha/cataloguing/addbiblio.pl?biblionumber= 123\n Save the record.\n2. From the Normal view choose Save -> MARCXML. Open the saved file in\n your favourite editor. You should see, at the end, something like:\n \n 123\n (mind the space before 123).\n This is not right.\n3. Make a search with the title or author's name from the record (e.g.\n Henning Mankell for the record 123 from the default ktd data set).\n You should get two records instead of one (while using ES).\n4. Apply the patch, restart_all. Repeat p. 1 and 2 with a different\n biblionumber. Notice the unchanged (i.e. without spaces) value\n of 999 $c subfield in the exported record and only one record\n as a result of a search.\n\nWNC amended patch - rebased, added conditional in case no bib, moved comments to their own lines\n\nSponsored-by: Ignatianum University in Cracow\nSigned-off-by: Roman Dolny \nSigned-off-by: Nick Clemens \nSigned-off-by: Katrin Fischer ","shortMessageHtmlLink":"Bug 36794: Illegitimate modification of biblionumber subfield content…"}},{"before":"650065fcd2fc58491d589541449b240abdf1de6e","after":"b1944c78954ef00873ba0a8bc9c1da1bf4ec9577","ref":"refs/heads/main","pushedAt":"2024-05-10T14:22:52.000Z","pushType":"push","commitsCount":51,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36757: (QA follow-up) Fix for action on different triggers\n\nThis patch updates the 'title' and 'comment count' links to action the\nsame ticket detail/update modal as the 'details' button.\n\nSigned-off-by: Martin Renvoize \nSigned-off-by: Paul Derscheid \nSigned-off-by: Katrin Fischer ","shortMessageHtmlLink":"Bug 36757: (QA follow-up) Fix for action on different triggers"}},{"before":"05a979a8ae91748ade65ee47e0e8ba3ad18797c8","after":"650065fcd2fc58491d589541449b240abdf1de6e","ref":"refs/heads/main","pushedAt":"2024-05-09T15:08:07.000Z","pushType":"push","commitsCount":165,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 35138: (follow-up) Mark DisplayLibraryFacets as zebra only\n\nThe tests are no longer relevant\n\nSigned-off-by: Martin Renvoize ","shortMessageHtmlLink":"Bug 35138: (follow-up) Mark DisplayLibraryFacets as zebra only"}},{"before":null,"after":"05a979a8ae91748ade65ee47e0e8ba3ad18797c8","ref":"refs/heads/main","pushedAt":"2024-05-02T15:05:41.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 25159: DBIC schema update\n\nSigned-off-by: Katrin Fischer ","shortMessageHtmlLink":"Bug 25159: DBIC schema update"}},{"before":"0adad5a5853407ed37b624d9ab6073f900d68a94","after":"3604df19920174c23a7385f7fb2cf5f62361e788","ref":"refs/heads/22.05.x","pushedAt":"2024-05-02T15:04:00.000Z","pushType":"push","commitsCount":84,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36511: Some scripts missing a dependency following Bug 24879\n\nThese files needed the addition of 'use C4::Auth qw( check_cookie_auth\n);'.\n\nTo test, apply the patch and restart services.\n\n- If necessary, enable the LocalCoverImages system preference.\n- Open the browser console and then the \"Network\" tab. You can click\n \"Images\" to filter for the correct kind of request.\n- Perform a catalog search. After the search has loaded, check that\n there are no 500 errors in the Network tab.\n\n- Go to Cataloging -> Label creator.\n- If necessary, create a label batch and add some items.\n- Export your batch and test both the \"Download as CSV\" and \"Download as\n XML\" links. Both should trigger the correct download.\n\n- Go to Serials -> Claims, and select a vendor with late issues.\n- Select all late issues and click \"Download selected claims\" at the\n bottom of the page.\n- Your CSV file should download correctly.\n\nThe file acqui/check_uniqueness.pl has been corrected as well but I'm\nnot sure how to test it!\n\nSigned-off-by: danyonsewell \n\nSigned-off-by: Jonathan Druart \nSigned-off-by: Katrin Fischer \n(cherry picked from commit 747f5132311ea51ea6babbfc92a775ac0c67f93a)\nSigned-off-by: Fridolin Somers \n(cherry picked from commit 01b22fb71d30f56d3102837b5c9b4cfdacbc9e76)\nSigned-off-by: Lucas Gass \nSigned-off-by: Wainui Witika-Park ","shortMessageHtmlLink":"Bug 36511: Some scripts missing a dependency following Bug 24879"}},{"before":"f09b359cb7d04001e2c13c194fe4324d5faf1837","after":"2eab2949fc50c752c3bf950c887e54845353e900","ref":"refs/heads/22.11.x","pushedAt":"2024-05-02T15:03:48.000Z","pushType":"push","commitsCount":34,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36511: Some scripts missing a dependency following Bug 24879\n\nThese files needed the addition of 'use C4::Auth qw( check_cookie_auth\n);'.\n\nTo test, apply the patch and restart services.\n\n- If necessary, enable the LocalCoverImages system preference.\n- Open the browser console and then the \"Network\" tab. You can click\n \"Images\" to filter for the correct kind of request.\n- Perform a catalog search. After the search has loaded, check that\n there are no 500 errors in the Network tab.\n\n- Go to Cataloging -> Label creator.\n- If necessary, create a label batch and add some items.\n- Export your batch and test both the \"Download as CSV\" and \"Download as\n XML\" links. Both should trigger the correct download.\n\n- Go to Serials -> Claims, and select a vendor with late issues.\n- Select all late issues and click \"Download selected claims\" at the\n bottom of the page.\n- Your CSV file should download correctly.\n\nThe file acqui/check_uniqueness.pl has been corrected as well but I'm\nnot sure how to test it!\n\nSigned-off-by: danyonsewell \n\nSigned-off-by: Jonathan Druart \nSigned-off-by: Katrin Fischer \n(cherry picked from commit 747f5132311ea51ea6babbfc92a775ac0c67f93a)\nSigned-off-by: Fridolin Somers \n(cherry picked from commit 01b22fb71d30f56d3102837b5c9b4cfdacbc9e76)\nSigned-off-by: Lucas Gass \n(cherry picked from commit 72cead50b49efaad6349cf653b970e6f5b610475)\nSigned-off-by: Frédéric Demians ","shortMessageHtmlLink":"Bug 36511: Some scripts missing a dependency following Bug 24879"}},{"before":"78f7aa74cf8c2c5362cfafc2267c4968e7e78d03","after":"72cead50b49efaad6349cf653b970e6f5b610475","ref":"refs/heads/23.05.x","pushedAt":"2024-05-02T15:03:31.000Z","pushType":"push","commitsCount":112,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 36511: Some scripts missing a dependency following Bug 24879\n\nThese files needed the addition of 'use C4::Auth qw( check_cookie_auth\n);'.\n\nTo test, apply the patch and restart services.\n\n- If necessary, enable the LocalCoverImages system preference.\n- Open the browser console and then the \"Network\" tab. You can click\n \"Images\" to filter for the correct kind of request.\n- Perform a catalog search. After the search has loaded, check that\n there are no 500 errors in the Network tab.\n\n- Go to Cataloging -> Label creator.\n- If necessary, create a label batch and add some items.\n- Export your batch and test both the \"Download as CSV\" and \"Download as\n XML\" links. Both should trigger the correct download.\n\n- Go to Serials -> Claims, and select a vendor with late issues.\n- Select all late issues and click \"Download selected claims\" at the\n bottom of the page.\n- Your CSV file should download correctly.\n\nThe file acqui/check_uniqueness.pl has been corrected as well but I'm\nnot sure how to test it!\n\nSigned-off-by: danyonsewell \n\nSigned-off-by: Jonathan Druart \nSigned-off-by: Katrin Fischer \n(cherry picked from commit 747f5132311ea51ea6babbfc92a775ac0c67f93a)\nSigned-off-by: Fridolin Somers \n(cherry picked from commit 01b22fb71d30f56d3102837b5c9b4cfdacbc9e76)\nSigned-off-by: Lucas Gass ","shortMessageHtmlLink":"Bug 36511: Some scripts missing a dependency following Bug 24879"}},{"before":"581fefab65a8a323ed0bdfcada2619e891419c6c","after":"f09b359cb7d04001e2c13c194fe4324d5faf1837","ref":"refs/heads/22.11.x","pushedAt":"2024-03-08T07:53:32.000Z","pushType":"push","commitsCount":20,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 35942: OPAC user can enroll several times to the same club [23.05.x]\n\nTest Plan:\n\n1) Create 3 clubs, 1 limited to library A, 1 limited to library B and one not limited\n2) Use a patron with home library A.\n3) Go to the opac-user page, \"Clubs\" tab show 0/2 (the one from library B is not listed)\n4) Browse to /cgi-bin/koha/svc/club/enroll?id=1\n5) Reload that page a couple times\n6) Note the patron is now enrolled in the same club multiple times\n7) Delete those enrollments\n8) Apply this patch\n9) Restart all the things!\n10) Repeat steps 2-7, note the lack of duplicate enrollments!\n11) Repeat steps 2-10 for the staff interface\n\nSigned-off-by: Jonathan Druart \nSigned-off-by: Martin Renvoize \nSigned-off-by: Lucas Gass \n(cherry picked from commit 9bdab108e22768b018b017ed7c0e0016270f2570)\nSigned-off-by: Frédéric Demians ","shortMessageHtmlLink":"Bug 35942: OPAC user can enroll several times to the same club [23.05.x]"}},{"before":"252ccbabc8ef67158878a19551d35734222ef04d","after":"78f7aa74cf8c2c5362cfafc2267c4968e7e78d03","ref":"refs/heads/23.05.x","pushedAt":"2024-03-08T07:53:22.000Z","pushType":"push","commitsCount":27,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Fix release notes for 23.05.09 part 2\n\nSigned-off-by: Lucas Gass ","shortMessageHtmlLink":"Fix release notes for 23.05.09 part 2"}},{"before":"fee4368cd81e4318bed14f0a082c978badca4256","after":"285a0fc74da5747922bcc28626066a7067e4fb66","ref":"refs/heads/master","pushedAt":"2024-03-08T07:53:10.000Z","pushType":"push","commitsCount":485,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 35357: Enhance unit tests\n\nSigned-off-by: Victor Grousset/tuxayo \n\nFixed typo in test output.\n\nSigned-off-by: Katrin Fischer ","shortMessageHtmlLink":"Bug 35357: Enhance unit tests"}},{"before":"ccd357e14dcd78467364939e371ece8b9e867cc1","after":"581fefab65a8a323ed0bdfcada2619e891419c6c","ref":"refs/heads/22.11.x","pushedAt":"2024-02-15T10:18:38.000Z","pushType":"push","commitsCount":4,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 34893: (RMaint follow-up) Test plan correction\n\nSigned-off-by: Martin Renvoize ","shortMessageHtmlLink":"Bug 34893: (RMaint follow-up) Test plan correction"}},{"before":"b507f9410892c7d35f31da1ddffa9f424a314802","after":"252ccbabc8ef67158878a19551d35734222ef04d","ref":"refs/heads/23.05.x","pushedAt":"2024-02-15T10:18:28.000Z","pushType":"push","commitsCount":35,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 35507: Add verbose parameters in Koha::Plugins\n\nYet another approach ;)\nDriemaal is scheepsrecht / Third time lucky..\n\nTest plan:\nRun prove $(git grep -l Koha::Plugin | grep -P \"^t\\/db\")\n\nSigned-off-by: Marcel de Rooy \nSigned-off-by: Martin Renvoize \nSigned-off-by: Katrin Fischer \n(cherry picked from commit d59bf322e9eb41b38ea35bf0e1a1955177c31cb3)\nSigned-off-by: Fridolin Somers \n(cherry picked from commit 4fb9fad791f10469eeb43b8f54399c1e37c5c6a6)\nSigned-off-by: Lucas Gass ","shortMessageHtmlLink":"Bug 35507: Add verbose parameters in Koha::Plugins"}},{"before":"12b6c0e67d39590da23a614fbe6d4aa4f18923aa","after":"fee4368cd81e4318bed14f0a082c978badca4256","ref":"refs/heads/master","pushedAt":"2024-02-15T10:18:19.000Z","pushType":"push","commitsCount":7,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 35930: Add guards for plugins_enabled\n\nThe 'new' method in Koha::Plugins returns undefined if\nplugins are disabled. Therefore, calls to this method\nmust be guarded by a check that plugins actually are enabled.\n\nTest plan:\n\n* Code inspection of patch, alternatively\n* Activate the ill system by installing a backend such as\n koha-illbackend-libris:\n https://github.com/Libriotech/koha-illbackend-libris\n* Make sure plugins are disabled in koha-conf.xml\n* In the staff interface, go to ILL requests.\n* The page should load without getting an error 500.\n\nPA amended commit message: This is not related to ILL backends being plugins or not\nThis is about ILL batches, where checking for metadata enrichment plugins was missing 'enable_plugins' guard\nAdditionally, unrelated to batches, it's also about ILLAvailability, where checking for ILL availabililty plugins was missing enable_plugins guard\n\nSigned-off-by: Pedro Amorim \nSigned-off-by: Tomas Cohen Arazi \nSigned-off-by: Hans Pålsson \nSigned-off-by: Martin Renvoize ","shortMessageHtmlLink":"Bug 35930: Add guards for plugins_enabled"}},{"before":"4451023126e032c1cd107f52aa2916d8a0a2af54","after":"8a9fb4541bee691090aa8b17bfd1aa119817f081","ref":"refs/heads/21.11.x","pushedAt":"2024-02-05T09:59:40.000Z","pushType":"push","commitsCount":9,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Update release notes for 21.11.26 release","shortMessageHtmlLink":"Update release notes for 21.11.26 release"}},{"before":"f646accf9e1034dce96c8fc70533e3540d3df7de","after":"0adad5a5853407ed37b624d9ab6073f900d68a94","ref":"refs/heads/22.05.x","pushedAt":"2024-02-05T09:59:29.000Z","pushType":"push","commitsCount":23,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Update release notes for 22.05.18 release\n\nSigned-off-by: Wainui Witika-Park ","shortMessageHtmlLink":"Update release notes for 22.05.18 release"}},{"before":"cf8e10c429ba48809b3fe1a35e20e1ff291fca51","after":"ccd357e14dcd78467364939e371ece8b9e867cc1","ref":"refs/heads/22.11.x","pushedAt":"2024-02-05T09:59:18.000Z","pushType":"push","commitsCount":24,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Update release notes for 22.11.14 release\n\nSigned-off-by: Frédéric Demians ","shortMessageHtmlLink":"Update release notes for 22.11.14 release"}},{"before":"45512e188da7da01c95e3c8d3e4ec66845e91a1f","after":"b507f9410892c7d35f31da1ddffa9f424a314802","ref":"refs/heads/23.05.x","pushedAt":"2024-02-05T09:59:00.000Z","pushType":"push","commitsCount":132,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 34479: Move patron selection history to a re-useable file\n\nThis patch takes some of the functionality for maintaining patron selections and moves it to a new file that can be used in other template files. It also introduces a new method for determining whether to delete the history after an operation is complete, along with an .inc file containing the checkbox that manages this\n\nTest plan:\n1) Navigate to Patrons and run a search\n2) Use the checkboxes to select some patrons and run the three different options in the menu bar: Add to patron list, Merge selected patrons, Batch patron modification.\n3) For each operation, you should see a checkbox asking if you want to \"Keep patrons selected for anew operation\". N.B. For adding patrons to a list, you will only see this when selecting to add them to a new list\n4) When you run the operations, if you select the checkbox to keep the patrons then when you return tho the patron search, those patrons should all be still selected.\n5) If you don't check the box, when you return to the search, your patron selection history should be empty and no patrons should be selected\nN.B. If you have run a merge operation and elected to keep the patron history, you will only keep the patron who was kept\n\nSigned-off-by: Sharon Dugdale \n\nSigned-off-by: Jonathan Druart \nSigned-off-by: Michael Adamyk \nSigned-off-by: Katrin Fischer \n(cherry picked from commit 21e2ee11fba6230b77e8b6376b0a903951170646)\nSigned-off-by: Fridolin Somers \n(cherry picked from commit 8ea3e73edc558db15befa4bc50350b65036e3c4a)\nSigned-off-by: Lucas Gass ","shortMessageHtmlLink":"Bug 34479: Move patron selection history to a re-useable file"}},{"before":"fb1c01452db88b87fb41cfea7f8f49f7aa080694","after":"12b6c0e67d39590da23a614fbe6d4aa4f18923aa","ref":"refs/heads/master","pushedAt":"2024-02-05T09:58:42.000Z","pushType":"push","commitsCount":383,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 34893: Unit tests for C4::Auth::checkpw\n\nThis patch introduces some tests on the current (and new) behavior for\nthe `checkpw` function.\n\nI needed it to better understand if an edge case was actually possible\n(it wasn't).\n\nFound a really minor annoyance for the internal check with expired\npassword not returning the $patron object for consistency with the other\nuse cases.\n\nI think this method deserves (at least) changing the return value to a\nsane data structure. But that's not target for backporting to stable\nreleases. So a separate bug.\n\nSigned-off-by: Tomas Cohen Arazi \nSigned-off-by: Katrin Fischer ","shortMessageHtmlLink":"Bug 34893: Unit tests for C4::Auth::checkpw"}},{"before":"e381372944088f0b75ad5a2b2d026780e8718636","after":"4451023126e032c1cd107f52aa2916d8a0a2af54","ref":"refs/heads/21.11.x","pushedAt":"2023-11-28T12:03:46.000Z","pushType":"push","commitsCount":11,"pusher":{"login":"mrenvoize","name":"Martin Renvoize","path":"/mrenvoize","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/557612?s=80&v=4"},"commit":{"message":"Bug 35174: update control files for koha-l10n dependency package\n\n to test...\n\n - apply patch\n - build koha-common package\n - install koha-common package, and confirm koha-l10n package is installed","shortMessageHtmlLink":"Bug 35174: update control files for koha-l10n dependency package"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAEUypfdQA","startCursor":null,"endCursor":null}},"title":"Activity · mrenvoize/Koha"}