[Bug 43397] New: SIP accounts with a blank AE field template send an empty patron name
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 Bug ID: 43397 Summary: SIP accounts with a blank AE field template send an empty patron name Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: SIP2 Assignee: koha-bugs@lists.koha-community.org Reporter: kyle@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org Target Milestone: --- Bug 37893 moved SIP accounts from SIPconfig.xml into the database. The new SIP account editor initializes every text field to an empty string rather than null, so an account created through the staff interface gets ae_field_template = '' instead of NULL which is defined and so the empty template is used thus giving an empty AE field in responses for newly created accounts! -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 Kyle M Hall (khall) <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 --- Comment #1 from Kyle M Hall (khall) <kyle@bywatersolutions.com> --- Created attachment 204331 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=204331&action=edit Bug 43397: Add unit tests Patch from commit d0f0b2e -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 --- Comment #2 from Kyle M Hall (khall) <kyle@bywatersolutions.com> --- Created attachment 204332 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=204332&action=edit Bug 43397: SIP accounts with a blank AE field template send an empty patron name Bug 37893 moved SIP accounts from SIPconfig.xml into the database. The new SIP account editor initializes every text field to an empty string rather than null, so an account created through the staff interface gets ae_field_template = '' instead of NULL which is defined and so the empty template is used thus giving an empty AE field in responses for newly created accounts! Test Plan: 1) Create a SIP account in the staff interface leaving the AE field template blank 2) Restart all the things! 3) Send a patron status or patron information request for a valid patron 4) Note the AE field in the response is empty 5) Apply this patch 6) Restart all the things! 7) Repeat the request, note the AE field now contains the patron's name! -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 Kyle M Hall (khall) <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |kyle@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 Kyle M Hall (khall) <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |normal -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #204331|0 |1 is obsolete| | Attachment #204332|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 --- Comment #3 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Created attachment 207234 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=207234&action=edit Bug 43397: Add unit tests Signed-off-by: Rae Schumann <rschumann@utah.gov> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 --- Comment #4 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Created attachment 207235 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=207235&action=edit Bug 43397: SIP accounts with a blank AE field template send an empty patron name Bug 37893 moved SIP accounts from SIPconfig.xml into the database. The new SIP account editor initializes every text field to an empty string rather than null, so an account created through the staff interface gets ae_field_template = '' instead of NULL which is defined and so the empty template is used thus giving an empty AE field in responses for newly created accounts! Test Plan: 1) Create a SIP account in the staff interface leaving the AE field template blank 2) Restart all the things! 3) Send a patron status or patron information request for a valid patron 4) Note the AE field in the response is empty 5) Apply this patch 6) Restart all the things! 7) Repeat the request, note the AE field now contains the patron's name! Signed-off-by: Rae Schumann <rschumann@utah.gov> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |jacob.omara@openfifth.co.uk |y.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 Jacob O'Mara <jacob.omara@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 Jacob O'Mara <jacob.omara@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #207234|0 |1 is obsolete| | Attachment #207235|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 --- Comment #5 from Jacob O'Mara <jacob.omara@openfifth.co.uk> --- Created attachment 208135 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=208135&action=edit Bug 43397: Add unit tests Signed-off-by: Rae Schumann <rschumann@utah.gov> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 --- Comment #6 from Jacob O'Mara <jacob.omara@openfifth.co.uk> --- Created attachment 208136 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=208136&action=edit Bug 43397: SIP accounts with a blank AE field template send an empty patron name Bug 37893 moved SIP accounts from SIPconfig.xml into the database. The new SIP account editor initializes every text field to an empty string rather than null, so an account created through the staff interface gets ae_field_template = '' instead of NULL which is defined and so the empty template is used thus giving an empty AE field in responses for newly created accounts! Test Plan: 1) Create a SIP account in the staff interface leaving the AE field template blank 2) Restart all the things! 3) Send a patron status or patron information request for a valid patron 4) Note the AE field in the response is empty 5) Apply this patch 6) Restart all the things! 7) Repeat the request, note the AE field now contains the patron's name! Signed-off-by: Rae Schumann <rschumann@utah.gov> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |26.11.00 released in| | --- Comment #7 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Pushed to main for 26.11 with consent from Pedro whilst he's on annual leave -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43397 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org