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

--- Comment #14 from Jonathan Druart <[email protected]> 
---
1. 
Patches applied on top of the whole tree

    #   Failed test 'globally mandatory attributes tests'
    #   at t/db_dependent/Koha/Patron.t line 661.
    # Looks like you failed 1 test of 15.
t/db_dependent/Koha/Patron.t .. 7/7 
#   Failed test 'extended_attributes'
#   at t/db_dependent/Koha/Patron.t line 664.
Can't locate object method "value" via package
"Koha::Exceptions::Patron::MissingMandatoryExtendedAttribute" at
t/db_dependent/Koha/Patron.t line 650.


2.
Why Koha::Patron::Attribute->type does not look like

  return Koha::Patron::Attribute::Type->_new_from_dbic( $self->_result->code );

3. Additionally I don't agree with the "cache", simply because the following
code will be wrong:

  $attr->type($new_code)->store;
  my $type = $attr->type; # will get the old type

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to