Fixes usage of icon

nunomaduro · Feb 16, 2018 · d555a6e · d555a6e
1 parent f4002ba
commit d555a6e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,10 +4,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
-## [1.3.2] - 2018-01-24
+## [2.0.0] - 2018-02-16
 ### Added
 - Support to Laravel 5.6
 
+### Removed
+- Trait `Notifications`.
+
 ## [1.3.0] - 2017-08-30
 ### Added
 - Bumps Laravel version

diff --git a/src/LaravelDesktopNotifierServiceProvider.php b/src/LaravelDesktopNotifierServiceProvider.php
@@ -11,8 +11,8 @@
 
 namespace NunoMaduro\LaravelDesktopNotifier;
 
-use Joli\JoliNotif\NotifierFactory;
 use Illuminate\Console\Command;
+use Joli\JoliNotif\NotifierFactory;
 use Illuminate\Support\ServiceProvider;
 use NunoMaduro\LaravelDesktopNotifier\Contracts\Notifier as NotifierContract;
 use NunoMaduro\LaravelDesktopNotifier\Contracts\Notification as NotificationContract;
@@ -43,8 +43,11 @@ function (string $text, string $body, $icon = null) {
 
                 $notification = $this->app[Contracts\Notification::class]
                     ->setTitle($text)
-                    ->setBody($body)
-                    ->setIcon($icon);
+                    ->setBody($body);
+
+                if (! empty($icon)) {
+                    $notification->setIcon($icon);
+                }
 
                 $notifier->send($notification);
             }