January 2017 - Koha-bugs - lists.koha-community.org

[Bug 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
by bugzilla-daemon＠bugs.koha-community.org 09 Jan '17

09 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #58429|0 |1 is obsolete| | --- Comment #61 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 58714 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58714&action=ed… Bug 13757: (followup) Regression tests for ->approve changes Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
by bugzilla-daemon＠bugs.koha-community.org 09 Jan '17

09 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #58428|0 |1 is obsolete| | --- Comment #60 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 58713 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58713&action=ed… Bug 13757: (followup) Only touch opac_editable attributes As reported by Owen, the members-update.pl was showing every attributes the patron has (display issue) instead of showing only those affected by the changes. This patch fixes this by filtering the patron's attributes by opac editability. It also fixes Koha::Patron::Modification->approve so it only clears the attributes with the updating 'code' and leaves the others untouched. As its been coded so far (until someone refactors it all) the Koha::Patron::Modification object needs to contain all the attributes for a specific code. And it comes from parsing the UI's input. Tests for Koha::Patron::Modification->approve to come. Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
by bugzilla-daemon＠bugs.koha-community.org 09 Jan '17

09 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #58703|0 |1 is obsolete| | --- Comment #59 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 58712 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58712&action=ed… Bug 13757: (followup) Staff interface changes This patch adds proper extended attributes display and handling on the patron modifications moderation page (members-update.pl). It also adds changes checking to the opac-memberentry.pl page so it only saves a modification request if there are changes (it only checked regular fields and not the extended ones). Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
by bugzilla-daemon＠bugs.koha-community.org 09 Jan '17

09 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #58356|0 |1 is obsolete| | --- Comment #58 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 58711 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58711&action=ed… Bug 13757: (followup) Remove warnings Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
by bugzilla-daemon＠bugs.koha-community.org 09 Jan '17

09 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #58355|0 |1 is obsolete| | --- Comment #57 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 58710 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58710&action=ed… Bug 13757: Add extended attributes to the patron modification Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
by bugzilla-daemon＠bugs.koha-community.org 09 Jan '17

09 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #58354|0 |1 is obsolete| | --- Comment #56 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 58709 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58709&action=ed… Bug 13757: DBIC update Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
by bugzilla-daemon＠bugs.koha-community.org 09 Jan '17

09 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #58353|0 |1 is obsolete| | --- Comment #55 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 58708 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58708&action=ed… Bug 13757: OPAC changes Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
by bugzilla-daemon＠bugs.koha-community.org 09 Jan '17

09 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #58352|0 |1 is obsolete| | --- Comment #54 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 58707 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58707&action=ed… Bug 13757: DB update Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
by bugzilla-daemon＠bugs.koha-community.org 09 Jan '17

09 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #58351|0 |1 is obsolete| | --- Comment #53 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 58706 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58706&action=ed… Bug 13757: Add the option to set patron attributes editable in the OPAC Note: this is a squashed version of the original patchset, because it was needed This patch adds an opac_editable property of borrower attribute types that can be set in the interface. I'm removing work on OPAC and will refactor it, keeping the author attribution. Test plan: 1. Repeat the following with a new and existing borrower attribute type: 2. Verify that "Editable in OPAC" can only be checked if "Display in OPAC" is checked. 3. Verify that this new property is correctly saved. Signed-off-by: Aleisha <aleishaamohia(a)hotmail.com> Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 13757] Make patron attributes editable in the opac if set to ' editable in OPAC'
by bugzilla-daemon＠bugs.koha-community.org 09 Jan '17

09 Jan '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #58357|0 |1 is obsolete| | --- Comment #52 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Created attachment 58703 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58703&action=ed… Bug 13757: (followup) Staff interface changes This patch adds proper extended attributes display and handling on the patron modifications moderation page (members-update.pl). It also adds changes checking to the opac-memberentry.pl page so it only saves a modification request if there are changes (it only checked regular fields and not the extended ones). Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> -- You are receiving this mail because: You are watching all bug changes.

1 0