[Bug 4995] New: extended attributes replication from LDAP
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4995 Summary: extended attributes replication from LDAP Change sponsored?: --- Product: Koha Version: rel_3_0 Platform: All OS/Version: All Status: NEW Severity: enhancement Priority: P5 Component: Authentication AssignedTo: gmcharlt@gmail.com ReportedBy: dpavlin@rot13.org QAContact: koha-bugs@lists.koha-community.org CC: dpavlin@rot13.org Estimated Hours: 0.0 Documentation for extended attributes is somewhat lacking, and my last patch in my previous bug report just fixed case without any extended attributes in LDAP. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4995 --- Comment #1 from Dobrica Pavlinusic <dpavlin@rot13.org> --- Created attachment 2360 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=2360 fix mapping of extended attribute code and ldap attributes -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4995 Dobrica Pavlinusic <dpavlin@rot13.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |critical -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4995 Dobrica Pavlinusic <dpavlin@rot13.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 |PATCH-Sent -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4995 --- Comment #2 from Dobrica Pavlinusic <dpavlin@rot13.org> --- This approach still require additional entry in koha-conf.xml mapping like this: <JMBG is="hrEduPersonUniqueNumber_JMBG"></JMBG> It's quite easy to add code to match existing types in extended attributes to ldap attribute names directly, so that it doesn't require change in koha-conf.xml, but I can see also argumentation against it along the lines of "it's really nice to have all LDAP configuration in one place in koha-conf.xml and I like ability to rename fields". -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4995 Dobrica Pavlinusic <dpavlin@rot13.org> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=4256 -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4995 dev_patches@biblibre.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED CC| |dev_patches@biblibre.com Resolution| |FIXED --- Comment #3 from dev_patches@biblibre.com 2010-09-09 17:52:59 UTC --- Applied -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4995 Dobrica Pavlinusic <dpavlin@rot13.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED | --- Comment #4 from Dobrica Pavlinusic <dpavlin@rot13.org> 2011-07-14 20:20:06 UTC --- I can't see this patch applied in current master. It would bring current LDAP implementation into fully-functional state, and still applies cleanly to master. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4995 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris@bigballofwax.co.nz Patch Status|--- |Needs Signoff -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4995 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |RESOLVED Patch Status|Needs Signoff |Patch Pushed Resolution| |FIXED --- Comment #5 from Chris Cormack <chris@bigballofwax.co.nz> 2011-07-28 22:12:30 UTC --- It is there in master already, applying the patch causes this conflict <<<<<<< HEAD my @attributes = grep{my $key=$_; any{$_ eq $key}@types;} keys %borrower; my $extended_patron_attributes; @{$extended_patron_attributes} = map { { code => $_, value => $borrower{$_} } } @attributes; ======= my @attributes = grep{my $key=$_; any{$_->{code} eq $key}@types;} keys %borrower; my $extended_patron_attributes = [ map{{code=>$_,value=>$borrower{$_}}}@attributes ];
> Bug 4995 - extended attributes replication from LDAP
-- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org