[Koha-bugs] [Bug 35959] Inconsistent hierarchy during C3 merge of class 'Koha::AuthorisedValue' (and a few other modules)

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

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35959

Victor Grousset/tuxayo <victor at tuxayo.net> changed:

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

--- Comment #20 from Victor Grousset/tuxayo <victor at tuxayo.net> ---
Created attachment 162481
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=162481&action=edit
Bug 35959: Resolve Inconsistent hierarchy-warn from 4 modules

Inspired by https://www.perlmonks.org/?node_id=1151073
Clearing @ISA did not work, but pushing the relations seems to.

Test plan:
1. Run `perl -cw Koha/AuthorisedValue.pm`
2. You should get "Inconsistent hierarchy during C3 merge"
3. Apply the patches
4. Run `perl -cw Koha/AuthorisedValue.pm`
5. No more "Inconsistent hierarchy during C3 merge"
6. Run qa tools (to see results of perl -cw). (Ignore warning when processing
files before patches.)
7. Run some tests that use those modules. (see the following
   "Signoff-off-by section")
8. Touch a few pages referencing those modules. (see the following
   "Signoff-off-by section")
9. See next patch's test plan

Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Tested:
prove t/db_dependent/Accounts.t t/db_dependent/Koha/Account/DebitType.t
prove t/db_dependent/AuthorisedValues.t t/db_dependent/Koha.t
t/db_dependent/Koha/Object.t
prove t/db_dependent/Koha/Patron.t t/db_dependent/Koha/Patron/Attribute.t
t/db_dependent/Koha/Patron/Attribute/Types.t
prove t/db_dependent/Koha/Objects.t t/db_dependent/Koha/Patron/Categories.t

admin/debit_types.pl
admin/authorised_values.pl
admin/patron-attr-types.pl
admin/categories.pl

Signed-off-by: Victor Grousset/tuxayo <victor at tuxayo.net>

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