https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37893
Kyle M Hall (khall) <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA --- Comment #54 from Kyle M Hall (khall) <[email protected]> --- For the institutions editor: It appears that deleting an institution will delete all accounts connected to that institution ( which makes sense ) with no warning. It would be best if the editor either warned that "the following accounts will be deleted: act1, act2, etc." or simply prevent deletion of institutions with SIP accounts. It would also be nice if the institution "name" field were limited to a branchcode pulldown since Koha can no longer handle institutions that are not branchcodes. This is a bit out of scope so it's not a blocker. At the very least the tooltip should state that unexpected behavior can occur if it is *not* a valid branchcode. The policy defaults should all default to "yes" in my mind, but I'm open to arguments against it. Retries field should be limited to a max of 999 retries, as the field is 3 characters long. This is a blocker and a minor fix I think. "Staff userid for SIP2 authentication" could be using the patron search in the same way that circulation does to fill in the userid from a list of options. Also not a blocker but would be really nice to have. For the accounts editor: "Convert nonprinting characters" defaults to NULL for imported accounts, but defaults to empty string for created accounts with no option for NULL. The only value I've ever used for this is a space, as it is usually used to convert things like newlines for devices that cannot handle them. This could be either changed to a text field with a checkbox, or a pulldown with options for NULL and space ( and could be added to in the future ), or even a list of authorized values. The simplest solution would be to have empty string convert to NULL. Any of these solutions is acceptable IMO. "Format due date" should have a tooltip explaining it will format the due date in the current DateFormat syspref. "Holds get captured" may need a tool tip? Not a blocker, but I'm not sure how well understood this one will be. "Previous checkout block checkout" should read "Block checkout of items previously or currently checked out by a patron", or maybe "Block previous checkouts" with a tooltip since that is quite verbose. "Send patron home library in AF" can be deprecated but is totally out of scope for this bug. It was for Overdrive but they no longer use this feature ( though we'll need to verify they don't have legacy customers using it if we do ). They now use custom fields of the patron's branchcode. Terminator should default to CRLF. Patron attribute could use a pulldown of attribute types. Could use a tooltip at the least. It would be nice if delimiter, encoding and terminator had the defaults inserted into the database instead of being NULL in the database and showing as empty cells in the accounts table. SIP communication itself appears to work great! -- You are receiving this mail because: 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/
