Skip to content
Snippets Groups Projects
Commit e48eb19d authored by Jay Friendly's avatar Jay Friendly
Browse files

Fixing notice showing up in the logs

parent 57e845b2
No related branches found
No related tags found
No related merge requests found
......@@ -11,7 +11,7 @@ use Drupal\private_message\Plugin\PrivateMessageConfigForm\PrivateMessageConfigF
use Symfony\Component\DependencyInjection\ContainerInterface;
/**
* Adds Private Message Nodejse settings to the Private Message settings page.
* Adds Private Message Nodejs settings to the Private Message settings page.
*
* @PrivateMessageConfigForm(
* id = "private_message_node_js_settings",
......@@ -195,8 +195,8 @@ class PrivateMessageNodeJsPrivateMessageConfigForm extends PrivateMessageConfigF
// If the user uploaded a icon, save it to a permanent location and set it
// as the icon path.
$fid = $values['browser_push_notifications']['browser_notification_settings_wrapper']['browser_notification_icon_upload'][0];
if (!empty($fid)) {
$fid = isset($values['browser_push_notifications']['browser_notification_settings_wrapper']['browser_notification_icon_upload'][0]) ? $values['browser_push_notifications']['browser_notification_settings_wrapper']['browser_notification_icon_upload'][0] : FALSE;
if ($fid) {
$file = $this->fileHandler->load($fid);
$filename = file_unmanaged_copy($file->getFileUri());
$settings->set('browser_notification_icon', $filename);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment