irishdan / NotificationBundle

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

DependencyInjection/NotificationExtension.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace IrishDan\NotificationBundle\DependencyInjection;
 
-use IrishDan\NotificationBundle\DependencyInjection\Factory\Broadcaster\SlackBroadcasterFactory;
 use Symfony\Component\DependencyInjection\Definition;
 use Symfony\Component\DependencyInjection\Reference;
 use Symfony\Component\HttpKernel\DependencyInjection\Extension;

Doc Comments +3 added lines

@@ -86,6 +86,9 @@
         }
     }
 
+    /**
+     * @param ContainerBuilder $container
+     */
     private function createBroadcaster($name, $broadcaster, $container)
     {
         // @TODO:

Command/CreateNotificationCommand.php

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 = [

DatabaseNotificationManager.php

Doc Comments +9 added lines

@@ -48,6 +48,9 @@ 
         return false;
     }
 
+    /**
+     * @param \DateTime $now
+     */
     public function setReadAtDate(DatabaseNotificationInterface $notification, $now = null, $flush = true)
     {
         if (empty($now)) {
@@ -63,6 +66,9 @@ 
         }
     }
 
+    /**
+     * @param \DateTime $now
+     */
     public function setUsersNotificationsAsRead(NotifiableInterface $notifiable, $now = null)
     {
         $entity = $this->notificationEntityName();
@@ -98,6 +104,9 @@ 
         return 0;
     }
 
+    /**
+     * @return string
+     */
     protected function notificationEntityName()
     {
         $config = $this->databaseConfiguration;