Merge pull request #94 from jolicode/tweaks

Fix permissions on Windows notifiers binaries

Author: pyrech

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changes between versions
 
+## Not yet released
+
+* Fixed permissions on Windows notifiers binaries
+
 ## 2.5.0 (2022-12-24)
 
 * Dropped support for PHP 7.4

bin/notifu/notifu.exe

@@ -31,4 +31,4 @@
 
 $result = $notifier->send($notification);
 
-echo 'Notification ', $result ? 'successfully sent' : 'failed', ' with ', get_class($notifier), \PHP_EOL;
+echo 'Notification ', $result ? 'successfully sent' : 'failed', ' with ', $notifier::class, \PHP_EOL;

bin/snoreToast/snoretoast-x86.exe

@@ -19,9 +19,6 @@
  */
 class AppleScriptNotifier extends CliBasedNotifier
 {
-    /**
-     * {@inheritdoc}
-     */
     public function isSupported(): bool
     {
         if (OsHelper::isMacOS() && version_compare(OsHelper::getMacOSVersion(), '10.9.0', '>=')) {
@@ -31,25 +28,16 @@ public function isSupported(): bool
         return false;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getBinary(): string
     {
         return 'osascript';
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getPriority(): int
     {
         return static::PRIORITY_LOW;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function getCommandLineArguments(Notification $notification): array
     {
         $script = 'display notification "' . str_replace('"', '\\"', $notification->getBody()) . '"';

example/index.php

@@ -30,9 +30,6 @@ abstract class CliBasedNotifier implements Notifier
      */
     private int $support = self::SUPPORT_UNKNOWN;
 
-    /**
-     * {@inheritdoc}
-     */
     public function isSupported(): bool
     {
         if (self::SUPPORT_UNKNOWN !== $this->support) {
@@ -56,9 +53,6 @@ public function isSupported(): bool
         return false;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function send(Notification $notification): bool
     {
         if (!$notification->getBody()) {

src/Notifier/AppleScriptNotifier.php

@@ -18,25 +18,16 @@
  */
 class GrowlNotifyNotifier extends CliBasedNotifier
 {
-    /**
-     * {@inheritdoc}
-     */
     public function getBinary(): string
     {
         return 'growlnotify';
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getPriority(): int
     {
         return static::PRIORITY_HIGH;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function getCommandLineArguments(Notification $notification): array
     {
         $arguments = [

src/Notifier/CliBasedNotifier.php

@@ -19,25 +19,16 @@
  */
 class KDialogNotifier extends CliBasedNotifier
 {
-    /**
-     * {@inheritdoc}
-     */
     public function getBinary(): string
     {
         return 'kdialog';
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getPriority(): int
     {
         return static::PRIORITY_HIGH;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function getCommandLineArguments(Notification $notification): array
     {
         $arguments = [];

src/Notifier/GrowlNotifyNotifier.php

@@ -20,57 +20,36 @@
  */
 class NotifuNotifier extends CliBasedNotifier implements BinaryProvider
 {
-    /**
-     * {@inheritdoc}
-     */
     public function getBinary(): string
     {
         return 'notifu';
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getPriority(): int
     {
         return static::PRIORITY_LOW;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function canBeUsed(): bool
     {
         return OsHelper::isWindows() && OsHelper::isWindowsSeven();
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getRootDir(): string
     {
         return \dirname(__DIR__, 2) . '/bin/notifu';
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getEmbeddedBinary(): string
     {
         return 'notifu.exe';
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getExtraFiles(): array
     {
         return [];
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function getCommandLineArguments(Notification $notification): array
     {
         $arguments = [

src/Notifier/KDialogNotifier.php

@@ -19,25 +19,16 @@
  */
 class NotifySendNotifier extends CliBasedNotifier
 {
-    /**
-     * {@inheritdoc}
-     */
     public function getBinary(): string
     {
         return 'notify-send';
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getPriority(): int
     {
         return static::PRIORITY_MEDIUM;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function getCommandLineArguments(Notification $notification): array
     {
         $arguments = [];