[Koha-bugs] [Bug 15381] Move the authority types related code to Koha::Authority::Type[s] - part 2
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Mon Dec 21 09:33:21 CET 2015
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15381
Frédéric Demians <frederic at tamil.fr> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |frederic at tamil.fr
--- Comment #4 from Frédéric Demians <frederic at tamil.fr> ---
For me what happens here is the same field as what you have observed with
system preferences. We have information serialized in the relational DB model
(so a lot of linked tables) which could be more easily represented in other
data structures. Koha is requesting ad nauseam multiple tables. I fear that
your patches are just overcomplexifying something which is already far too
complex. It's becoming byzantines. Finally, you're creating multiples classes
to access data found in one table (auth_types) which can contain up to 10
records.
On bug 5572, I've tried to simplify the authorities merge() function. On the
road, I introduced a new function that gather info from auth_type,
auth_subfield_structure, and marc_subfield_structure. A data structure
represents all authority types. It can be something like that:
GEO:
type: GEO
tag: 151
letters: 68abvxyz
bibtags: 691
PERSO_NAME:
type: PERSO_NAME
tag: 100
letters: 68abcdefghjklmnopqrstvxyz
THEME:
type: THEME
tag: 150
letters: 68abvxyz
bibtags: 690|691
This can be cached. From that, we have all info needed by Koha code
manipulating authorities, without having to go back to DB tables containing
authorities definitions.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list