irishdan / NotificationBundle

DatabaseNotificationManager.php

Doc Comments +6 added lines

@@ -17,6 +17,9 @@ 
         $this->databaseConfiguration = $databaseConfiguration;
     }
 
+    /**
+     * @param \DateTime $now
+     */
     public function setReadAtDate(DatabaseNotificationInterface $notification, $now = null, $flush = true)
     {
         if (empty($now)) {
@@ -31,6 +34,9 @@ 
         }
     }
 
+    /**
+     * @param \DateTime $now
+     */
     public function setUsersNotificationsAsRead(NotifiableInterface $notifiable, $now = null)
     {
         $entity = $this->notificationEntityName();

Command/CreateDatabaseNotificationCommand.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 
         // @TODO: Add existing bundle validation
         $question->setValidator(['Sensio\Bundle\GeneratorBundle\Command\Validators', 'validateBundleName']);
-        $question->setNormalizer(function ($value) {
+        $question->setNormalizer(function($value) {
             return $value ? trim($value) : '';
         });
         $question->setMaxAttempts(2);

NotificationCollection.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     public function markAsRead()
     {
         // @TODO: sort it.
-        $this->entities->forAll(function ($notification) {
+        $this->entities->forAll(function($notification) {
             $notification->markAsRead();
         });
     }

Command/CreateNotificationCommand.php

Spacing +6 added lines, -6 removed lines

@@ -62,7 +62,7 @@ 
         // @TODO: Add existing bundle validation
         $question->setValidator(['Sensio\Bundle\GeneratorBundle\Command\Validators', 'validateBundleName']);
         $question->setNormalizer(
-            function ($value) {
+            function($value) {
                 return $value ? trim($value) : '';
             }
         );
@@ -85,7 +85,7 @@ 
             $input->getOption('notification_name')
         );
         $question->setValidator(
-            function ($answer) {
+            function($answer) {
                 // Should only contain letters.
                 $valid = preg_match('/^[a-zA-Z]+$/', $answer);
                 if (!$valid) {
@@ -98,7 +98,7 @@ 
             }
         );
         $question->setNormalizer(
-            function ($value) {
+            function($value) {
                 return $value ? trim($value) : '';
             }
         );
@@ -164,19 +164,19 @@ 
             ), 'yes'
         );
         $question->setNormalizer(
-            function ($value) {
+            function($value) {
                 return $value[0] == 'y' ? 'y' : 'n';
             }
         );
 
         $question->setValidator(
-            function ($answer) {
+            function($answer) {
                 // Should only contain letters.
                 $allowed = [
                     'y',
                     'n',
                 ];
-                $valid   = in_array($answer, $allowed);
+                $valid = in_array($answer, $allowed);
                 if (!$valid) {
                     throw new \RuntimeException(
                         'Only allowed value are ' . implode(', ', $allowed)

Doc Comments +12 added lines

@@ -155,6 +155,10 @@ 
         $questionHelper->writeGeneratorSummary($output, []);
     }
 
+    /**
+     * @param \Sensio\Bundle\GeneratorBundle\Command\Helper\QuestionHelper $questionHelper
+     * @param InputInterface $input
+     */
     protected function createYesNoQuestion($questionHelper, $input, $channel)
     {
         $question = new Question(
@@ -164,12 +168,20 @@ 
             ), 'yes'
         );
         $question->setNormalizer(
+
+            /**
+             * @param \Sensio\Bundle\GeneratorBundle\Command\Helper\QuestionHelper $value
+             */
             function ($value) {
                 return $value[0] == 'y' ? 'y' : 'n';
             }
         );
 
         $question->setValidator(
+
+            /**
+             * @param \Sensio\Bundle\GeneratorBundle\Command\Helper\QuestionHelper $answer
+             */
             function ($answer) {
                 // Should only contain letters.
                 $allowed = [

Dispatcher/DatabaseMessageDispatcher.php

Doc Comments +3 added lines

@@ -38,6 +38,9 @@
         return false;
     }
 
+    /**
+     * @param MessageInterface $data
+     */
     protected function createDatabaseNotificationEntity($data)
     {
         if ($this->accessor === null) {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace IrishDan\NotificationBundle\Dispatcher;
 
 use Doctrine\ORM\EntityManager;
-use IrishDan\NotificationBundle\Message\BaseMessage;
 use IrishDan\NotificationBundle\Message\MessageInterface;
 use Symfony\Component\PropertyAccess\PropertyAccess;
 

Dispatcher/NexmoMessageDispatcher.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
         $messageData  = $message->getMessageData();
 
         if (empty($this->client)) {
-            $credentials  = new Client\Credentials\Basic(
+            $credentials = new Client\Credentials\Basic(
                 $this->nexmoConfiguration['api_key'],
                 $this->nexmoConfiguration['api_secret']
             );

Broadcast/Broadcaster.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 
 use IrishDan\NotificationBundle\Channel\ChannelInterface;
-use IrishDan\NotificationBundle\Notification\NotifiableInterface;
 use IrishDan\NotificationBundle\Notification\NotificationInterface;
 
 class Broadcaster

Formatter/SlackWebhookFormatter.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use IrishDan\NotificationBundle\Message\Message;
 use IrishDan\NotificationBundle\Notification\NotificationInterface;
 use IrishDan\NotificationBundle\Slackable;
-use IrishDan\NotificationBundle\Textable;
 
 class SlackWebhookFormatter implements MessageFormatterInterface
 {

NotificationManager.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
         $this->channelManager = $channelManager;
     }
 
-    public function broadcast(NotificationInterface $notificatin, array $broadcasters  = null) {
+    public function broadcast(NotificationInterface $notificatin, array $broadcasters = null) {
 
     }