[Koha-bugs] [Bug 17499] Koha objects for messaging preferences
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Wed Apr 22 11:31:53 CEST 2020
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17499
--- Comment #80 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
Lari, I have rebased the patches.
Wondering about find_with_message_name
Koha::Patron::Message::Preferences->find_with_message_name({
borrowernumber => 123,
message_name => 'Hold_Filled',
});
Should not we prefer something like:
$patron->messaging_preferences->filter_by_message_name('Hold_Filled');
or even:
$patron->messaging_preferences->search({ message_name => 'Hold_Filled' });
ie why is it a class method?
Same for the setter, there is only 1 set_default_messaging_preferences, why
cannot we add set messaging preferences directly from $patron?
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list