[Koha-bugs] [Bug 10363] There is no package for authorised values.

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10363

Bernardo Gonzalez Kriegel <bgkriegel at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #30585|0                           |1
        is obsolete|                            |

--- Comment #35 from Bernardo Gonzalez Kriegel <bgkriegel at gmail.com> ---
Created attachment 30712
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=30712&action=edit
[SIGNED-OFF] Bug 10363: Add a package for authorised values

Currently, the authorised values are anarchically managed :)
We have to have a better way to manage them.
I propose a package in order to execute the sql queries into a package
instead of directly into the pl script.

This development adds:
- A new unique key (category and authorised values) for the
  authorised_values table.
- 1 new package Koha::AuthorisedValue
- 1 new unit test file in order to test these 2 packages.

It is a code source evolution, but there is no change for the final user.

Test plan:
1/ Apply the patches
2/ Check that existing authorised values is displayed on your admin
page (admin/authorised_values.pl).
3/ Update an existing one
4/ Create a new category
5/ Create a new AV for this category
6/ Create another one with the same value, an error should be occur.
7/ Delete one AV.
8/ Click everywhere on the authorised values admin page and try to find
a bug.
9/ Launch t/db_dependent/AuthorisedValues.t and check that all tests
pass (1 sql error appears, it is normal).

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Work as described following test plan, can't find regressions,
can create, update and delete AV and categories
Test pass
No koha-qa errors

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.