@@ -, +, @@ empty 1 - Set some messaging preferences for at least two categories 2 - Enter a new patron 3 - Confirm switching categories correctly defaults the messaging preferences 4 - Edit the preferences and switch to another category with defaults 5 - Confirm the warning appears, and preferences are set if confirmed and ignored if not 6 - Set to a category with defaults, uncheck all, and save 7 - Ooof, patron is defaulted 8 - Apply patch 9 - Repeat 1-5 and confirm nothing has changed --- C4/Form/MessagingPreferences.pm | 9 --------- 1 file changed, 9 deletions(-) --- a/C4/Form/MessagingPreferences.pm +++ a/C4/Form/MessagingPreferences.pm @@ -96,15 +96,6 @@ sub handle_form_action { } C4::Members::Messaging::SetMessagingPreference( $updater ); - - if ($query->param( $option->{'message_attribute_id'})){ - $prefs_set = 1; - } - } - if (! $prefs_set && $insert){ - # this is new borrower, and we have no preferences set, use the defaults - $target_params->{categorycode} = $categorycode; - C4::Members::Messaging::SetMessagingPreferencesFromDefaults( $target_params ); } # show the success message $template->param( settings_updated => 1 ); --