- Koha-bugs - lists.koha-community.org

[Bug 14085] Search for biblionumber in acquisitions
by bugzilla-daemon＠bugs.koha-community.org 24 Feb '22

24 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14085 --- Comment #1 from Marjorie Barry-Vila <marjorie.barry-vila(a)collecto.ca> --- still valid in 21.05 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 0

[Bug 11989] Holds Queue Scheduling
by bugzilla-daemon＠bugs.koha-community.org 24 Feb '22

24 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11989 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen(a)gmail.com --- Comment #2 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- (In reply to Christopher Brannon from comment #1) > Seriously, this is good stuff here. :) +1 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 0

[Bug 22520] New: Be Elastic compliant 7.x and 8.x (_doc)
by bugzilla-daemon＠bugs.koha-community.org 24 Feb '22

24 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22520 Bug ID: 22520 Summary: Be Elastic compliant 7.x and 8.x (_doc) Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Searching - Elasticsearch Assignee: koha-bugs(a)lists.koha-community.org Reporter: claire.hernandez(a)biblibre.com Target Milestone: --- -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 13

[Bug 19749] New: Report who show all the reservations
by bugzilla-daemon＠bugs.koha-community.org 24 Feb '22

24 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19749 Bug ID: 19749 Summary: Report who show all the reservations Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: patrick.robitaille(a)collecto.ca QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com Hi, The report "Holds queue" don't show all the reservations. It show only the list of reservations made on the "next available items". It would be great to have in the circulation section a report who give the list of all the reservations for itmes on loan. It could be useful for librarys to have acces to a report who show all the reservations in the circulation's module. Patrick. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 10

[Bug 17684] New: add floating table header on acqui-home.pl
by bugzilla-daemon＠bugs.koha-community.org 24 Feb '22

24 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17684 Bug ID: 17684 Summary: add floating table header on acqui-home.pl Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Acquisitions Assignee: koha-bugs(a)lists.koha-community.org Reporter: marjorie.barry-vila(a)ccsr.qc.ca QA Contact: testopia(a)bugs.koha-community.org Hi, When we have a lot of funds displayed on acqui-home, it would be easier if table header was floating. Regards, Marjorie -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 5

[Bug 13620] Add item when modifying an order (items created when placing an order)
by bugzilla-daemon＠bugs.koha-community.org 24 Feb '22

24 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13620 Marjorie Barry-Vila <marjorie.barry-vila(a)collecto.ca> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |WORKSFORME Status|NEW |RESOLVED -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 0

[Bug 13613] Don't allow digest to be selected without a digest-able transport selected
by bugzilla-daemon＠bugs.koha-community.org 24 Feb '22

24 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |30175 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30175 [Bug 30175] Digest options not enabled when populating messaging preferences for a selected category during patron entry -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12029] Patrons should be able to delete their patron messages
by bugzilla-daemon＠bugs.koha-community.org 24 Feb '22

24 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12029 --- Comment #22 from Martin Renvoize <martin.renvoize(a)ptfs-europe.com> --- Comment on attachment 130516 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=130516 Bug 12029: Patrons should be able to delete their patron messages Review of attachment 130516: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=120…) ----------------------------------------------------------------- ::: installer/data/mysql/kohastructure.sql @@ +3739,4 @@ > `message` mediumtext COLLATE utf8mb4_unicode_ci NOT NULL COMMENT 'the text of the message', > `message_date` timestamp NOT NULL DEFAULT current_timestamp() COMMENT 'the date and time the message was written', > `manager_id` int(11) DEFAULT NULL COMMENT 'creator of message', > + `patron_read_date` timestamp NULL DEFAULT NULL COMMENT 'date and time the patron dismissed the message', Final additional note from my own review: Shouldn't this be a DATETIME rather than a TIMESTAMP field? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12029] Patrons should be able to delete their patron messages
by bugzilla-daemon＠bugs.koha-community.org 24 Feb '22

24 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12029 Martin Renvoize <martin.renvoize(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com --- Comment #21 from Martin Renvoize <martin.renvoize(a)ptfs-europe.com> --- Welcome Bill, This looks like a great new feature and I'm really glad the discussion continued past comment 4 to get the refinements I also felt were necessary. Regarding the actual submission, did you read the coding guidelines available https://wiki.koha-community.org/wiki/Coding_Guidelines ? These are what the QA people will base their most basic feedback upon and are 'hard fails' in most cases when not adhered to. They are voted upon by a council of community members and have good reasoning behind each (though they do not cover all cases and being updated fairly regularly as we come across various patterns.. for exapme the 'filter_by_' requirement is not yet a formal guideline, but will become one very soon) Beyond hard fails, the QA persons role is to help spot regressions and ensure coding of a minimum standard is maintained for submissions so we can maintain the features going forward. Such advice can be argued against, but should generally be considered constructive. So, taking Jonathans list. 1. Style, this may fail guidelines given our perlcritic requirements. It's a 1s fix anyway. 2. Will be in the guidelines imminently, a few minutes to fix. 3. Style, improved coding style, readability improvement. Not a hard fail, but constructive. 4. Hard fail, all module changes have required corresponding unit tests since 2017. If you are stuck here, just ask for some guidance. https://wiki.koha-community.org/wiki/Coding_Guidelines#PERL17:_Unit_tests_a… 5. This is an unwritten rule.. I've made a note to add it to the guidelines. It just makes submissions considerably simpler for maintainers to add to all the supported branches. 6. Style, no guidelines broken, but it's a fair comment to help improve the developer. Another 10s fix. 7. Style, this will be fixed at push. 8. Hard fail: https://wiki.koha-community.org/wiki/Coding_Guidelines#Licence 9. Superfluous comments confuse future maintenance and cost everybody time. 10. Style, developers do things in their own way. Whilst I agree an ajax method would have been preferable (I'd probably have gone for a new REST route myself, rather than svc), this isn't a hard fail but more of a guide for future developments. 11. This is highlighting some red flags for future maintenance again. This is the most 'human' comment here. To me, this code looks rather 'copy/paste', which is fine but there should be evidence that you've stripped out all unnecessary boilerplate and given the development some thought as to what it's actually doing and requires. Hope that helps clarify things. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 27296] New: Return claims should be filtered by default to show unresolved claims
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '22

23 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27296 Bug ID: 27296 Summary: Return claims should be filtered by default to show unresolved claims Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: Circulation Assignee: oleonard(a)myacpl.org Reporter: oleonard(a)myacpl.org QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com I would like to add a default filter on the table of claims shown on the checkout and patron detail pages. The table will be filtered by default to show only unresolved claims, with links to switch back and forth between this view and a view of all claims. -- You are receiving this mail because: You are watching all bug changes.

1 15