Commit caaf74e8 authored by Ranjit Sahoo's avatar Ranjit Sahoo Committed by Artem Sylchuk
Browse files

Issue #3356586 by Ranjit1032002, wstocker, artem_sylchuk: Call to a member... 

Issue #3356586 by Ranjit1032002, wstocker, artem_sylchuk: Call to a member function get() on null in private_message_form_user_form_alter()
parent 3b5338b0

private_message.module

+7 −4
Original line number Diff line number Diff line
@@ -320,8 +320,9 @@ function private_message_form_user_form_alter(array &$form, FormStateInterface $ 
      '#title' => t('Receive notification of private messages'),

      '#default_value' => $default_value,

    ];



    if (!empty($user_data)) {

      $user_setting = $user_data->get('private_message', $user->id(), 'notify_when_using');

    }

    // If the user has not set a value, the system-wide default is used.

    $default_value = is_null($user_setting) ? $config->get('notify_when_using') : $user_setting;

    $form['private_messages']['notify_when_using'] = [
@@ -353,7 +354,9 @@ function private_message_form_user_form_alter(array &$form, FormStateInterface $ 
      86400 => t('1 day'),

    ];

    

    if (!empty($user_data)) {

      $user_setting = (int) $user_data->get('private_message', $user->id(), 'number_of_seconds_considered_away');

    }

    // If the user has not set a value, the system-wide default is used.

    $default_value = is_null($user_setting) ? $config->get('number_of_seconds_considered_away') : $user_setting;

    // The system default used by administrators is a free value, whereas users