Summary: | Creating a user via LDAP doesn't populate message preferences | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | Patrons | Assignee: | Bugs List <koha-bugs> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | gmcharlt, kyle.m.hall, tomascohen, ulrich.kleiber |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
BUG 12304: LDAP auto-provisioning set default messaging preferences
BUG 12304: LDAP auto-provisioning set default messaging preferences [SIGNED OFF] BUG 12304: LDAP auto-provisioning set default messaging preferences BUG 12304: LDAP auto-provisioning set default messaging preferences [PASSED QA] BUG 12304: LDAP auto-provisioning set default messaging preferences |
Description
Katrin Fischer
2014-05-21 13:26:51 UTC
Looking at the code and with Martin's help I think adding this after the AddMember call in the LDAP code could solve the problem: C4::Members::Messaging::SetMessagingPreferencesFromDefaults( { borrowernumber => $borrower{'borrowernumber'}, categorycode => $borrower{'categorycode'} } ); Untested so far. Created attachment 28409 [details] [review] BUG 12304: LDAP auto-provisioning set default messaging preferences LDAP auto-provisioning should set default messaging preferences upon creation of a user. Comment on attachment 28409 [details] [review] BUG 12304: LDAP auto-provisioning set default messaging preferences This is awesome, thx Martin! Created attachment 28417 [details] [review] BUG 12304: LDAP auto-provisioning set default messaging preferences LDAP auto-provisioning should set default messaging preferences upon creation of a user. Manually applied to 3.12.9 and it works beautifully in test and production. Thanks Martin! Created attachment 28855 [details] [review] [SIGNED OFF] BUG 12304: LDAP auto-provisioning set default messaging preferences LDAP auto-provisioning should set default messaging preferences upon creation of a user. Signed-off-by: Ulrich Kleiber <ulrich.kleiber@bsz-bw.de> Manually applied to 3.12.9 and it works beautifully in test and production. Created attachment 28980 [details] [review] BUG 12304: LDAP auto-provisioning set default messaging preferences LDAP auto-provisioning should set default messaging preferences upon creation of a user. Signed-off-by: Ulrich Kleiber <ulrich.kleiber@bsz-bw.de> Manually applied to 3.12.9 and it works beautifully in test and production. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> This is also a problem in current master! Created attachment 29114 [details] [review] [PASSED QA] BUG 12304: LDAP auto-provisioning set default messaging preferences LDAP auto-provisioning should set default messaging preferences upon creation of a user. Signed-off-by: Ulrich Kleiber <ulrich.kleiber@bsz-bw.de> Manually applied to 3.12.9 and it works beautifully in test and production. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Patch pushed to master. Thanks Martin! Pushed to 3.16.x for inclusion in 3.16.2. |