https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42095
Brendan Lawlor <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #2 from Brendan Lawlor <[email protected]> --- (In reply to Oleg Vasylenko from comment #1) > Possible solution is to find in unimarc_field_4XX.pl the string > elsif ( $op eq "do_search" ) { > and change it to: > elsif ( $op eq "do_search" || $op eq "cud-do_search" ) { See also this bug 40307 I am interested in this type of bug because I am working on updating the coding guideline for 5.1 This op was prefixed with cud- by a script as a result of bug 34478. One one hand if the request does not result in making any changes to the database, then it doesn't necessarily need to be POST and it could be changed to GET. GET could be preferred in cases where it would be convenient for users to be able to bookmark/share a url. POST would be preferred for scalability when the form needs to be able to send many parameters/values that would exceed the character limit in a url. In these cases there are examples in the codebase of ops like cud-do_search, cud-search, and cud-form I don't know enough about this unimarc plugin to know which solution is better for this case, but the similar bug 40307 has a patch changing the request to GET. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
