redelivre / login-cidadao

src/LoginCidadao/NotificationBundle/Entity/Notification.php

Doc Comments +6 added lines

@@ -385,6 +385,9 @@ 
         return $this->level;
     }
 
+    /**
+     * @param integer $level
+     */
     public function setLevel($level)
     {
         switch ($level) {
@@ -426,6 +429,9 @@ 
         return $this->htmlTemplate;
     }
 
+    /**
+     * @param string $var
+     */
     public function parseHtmlTemplate($var)
     {
         $cplaces = array('%title%' => $this->title, '%shorttext%' => $this->shortText);

src/LoginCidadao/NotificationBundle/Entity/NotificationRepository.php

Doc Comments +6 added lines

@@ -53,6 +53,9 @@ 
         }
     }
 
+    /**
+     * @param PersonInterface $person
+     */
     public function getTotalUnreadGroupByClient($person)
     {
         $qb = $this->getEntityManager()->createQueryBuilder('n')
@@ -70,6 +73,9 @@ 
                 ->getResult();
     }
 
+    /**
+     * @param PersonInterface $person
+     */
     public function getTotalUnread($person)
     {
         return $this->getEntityManager()->createQueryBuilder('n')

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Doctrine\ORM\EntityRepository;
 use LoginCidadao\CoreBundle\Entity\Person;
-use Doctrine\ORM\Query;
 use LoginCidadao\CoreBundle\Model\PersonInterface;
 use LoginCidadao\OAuthBundle\Model\ClientInterface;
 

Indentation +3 added lines, -3 removed lines

@@ -12,8 +12,8 @@ 
 {
 
     public function findNextNotifications(Person $person, $items = 8,
-                                          $lastId = 0,
-                                          ClientInterface $client = null)
+                                            $lastId = 0,
+                                            ClientInterface $client = null)
     {
         $qb = $this->getEntityManager()->createQueryBuilder()
             ->select('n')
@@ -75,7 +75,7 @@ 
         return $this->getEntityManager()->createQueryBuilder('n')
                 ->select('COUNT(n.id) total')
                 ->from('LoginCidadaoNotificationBundle:Notification',
-                       'n')
+                        'n')
                 ->where('n.person = :person')
                 ->andWhere('n.readDate IS NULL')
                 ->setParameter('person', $person)

src/LoginCidadao/NotificationBundle/Handler/AuthenticatedNotificationHandler.php

Unused Use Statements -7 removed lines

@@ -2,17 +2,10 @@
 
 namespace LoginCidadao\NotificationBundle\Handler;
 
-use Doctrine\Common\Persistence\ObjectManager;
-use Symfony\Component\Form\FormFactoryInterface;
 use LoginCidadao\NotificationBundle\Handler\NotificationHandlerInterface;
-use LoginCidadao\NotificationBundle\Form\NotificationType;
-use LoginCidadao\NotificationBundle\Model\NotificationInterface;
-use LoginCidadao\NotificationBundle\Exception\InvalidFormException;
 use LoginCidadao\CoreBundle\Model\PersonInterface;
 use LoginCidadao\OAuthBundle\Model\ClientInterface;
 use LoginCidadao\NotificationBundle\Model\CategoryInterface;
-use LoginCidadao\NotificationBundle\Entity\PersonNotificationOption;
-use LoginCidadao\NotificationBundle\Model\NotificationSettings;
 use Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException;
 
 class AuthenticatedNotificationHandler implements AuthenticatedNotificationHandlerInterface

Indentation +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
     }
 
     public function getAllFromClient(ClientInterface $client, $limit = 5,
-                                     $offset = 0, $orderby = null)
+                                        $offset = 0, $orderby = null)
     {
         return $this->handler->getAllFromPersonByClient($this->person, $client,
                                                         $limit, $offset,
@@ -78,10 +78,10 @@ 
     }
 
     public function getGroupedSettings(ClientInterface $client = null,
-                                       CategoryInterface $category = null)
+                                        CategoryInterface $category = null)
     {
         return $this->handler->getGroupedSettings($this->person, $client,
-                                                  $category);
+                                                    $category);
     }
 
     public function countUnread()

src/LoginCidadao/NotificationBundle/Handler/NotificationHandler.php

Doc Comments +3 added lines

@@ -372,6 +372,9 @@
         return $this->oauthDefaultClient;
     }
 
+    /**
+     * @param resource $ch
+     */
     protected function registerFailedCallback(NotificationInterface $notification,
                                               ObjectManager $om, $ch,
                                               $curlResponse)

Indentation +17 added lines, -17 removed lines

@@ -66,16 +66,16 @@ 
     }
 
     public function getAllFromPerson(PersonInterface $person, $limit = 5,
-                                     $offset = 0, $orderby = null)
+                                        $offset = 0, $orderby = null)
     {
         return $this->repository->findBy(array('person' => $person), $orderby,
                 $limit, $offset);
     }
 
     public function getAllFromPersonByClient(PersonInterface $person,
-                                             ClientInterface $client,
-                                             $limit = 5, $offset = 0,
-                                             $orderby = null)
+                                                ClientInterface $client,
+                                                $limit = 5, $offset = 0,
+                                                $orderby = null)
     {
         return $this->repository->findBy(array('person' => $person, 'sender' => $client),
                 $orderby, $limit, $offset);
@@ -114,8 +114,8 @@ 
      * @throws \LoginCidadao\CoreBundle\Exception\Notification\InvalidFormException
      */
     private function processForm(NotificationInterface $notification,
-                                 array $parameters, $method = "PUT",
-                                 PersonInterface $person = null)
+                                    array $parameters, $method = "PUT",
+                                    PersonInterface $person = null)
     {
         $form = $this->formFactory->create($this->notificationType,
             $notification, compact('method'));
@@ -194,7 +194,7 @@ 
     }
 
     public function initializeSettings(PersonInterface $person,
-                                       ClientInterface $client = null)
+                                        ClientInterface $client = null)
     {
         $om               = $this->om;
         $categoriesRepo   = $om->getRepository('LoginCidadaoNotificationBundle:Category');
@@ -266,8 +266,8 @@ 
     }
 
     public function getGroupedSettings(PersonInterface $person,
-                                       ClientInterface $client = null,
-                                       CategoryInterface $category = null)
+                                        ClientInterface $client = null,
+                                        CategoryInterface $category = null)
     {
         $settings = new NotificationSettings();
 
@@ -290,8 +290,8 @@ 
     }
 
     public function getAllFromPersonIdOffset(PersonInterface $person,
-                                             $limit = 5, $offset = 0,
-                                             ClientInterface $client = null)
+                                                $limit = 5, $offset = 0,
+                                                ClientInterface $client = null)
     {
         return $this->repository->findNextNotifications($person, $limit,
                 $offset, $client);
@@ -348,9 +348,9 @@ 
     }
 
     public static function renderEmailByCategory($category,
-                                                 $replacePlaceholders = null,
-                                                 $replaceTitle = null,
-                                                 $replaceShortText = null)
+                                                    $replacePlaceholders = null,
+                                                    $replaceTitle = null,
+                                                    $replaceShortText = null)
     {
         return self::_renderHtml($category->getMailTemplate(),
                 $replacePlaceholders, $replaceTitle, $replaceShortText);
@@ -373,8 +373,8 @@ 
     }
 
     protected function registerFailedCallback(NotificationInterface $notification,
-                                              ObjectManager $om, $ch,
-                                              $curlResponse)
+                                                ObjectManager $om, $ch,
+                                                $curlResponse)
     {
         $info = curl_getinfo($ch);
 
@@ -393,7 +393,7 @@ 
     }
 
     private function sendCallback(NotificationInterface $notification,
-                                  ObjectManager $om)
+                                    ObjectManager $om)
     {
         if ($notification->getCallbackUrl()) {
             $secret            = $notification->getCategory()->getClient()->getSecret();

src/LoginCidadao/NotificationBundle/Handler/NotificationHandlerInterface.php

Doc Comments +3 added lines, -4 removed lines

@@ -48,8 +48,8 @@ 
      * Same as getAllFromPerson but uses the notification's id as the offset.
      *
      * @param PersonInterface $person
-     * @param type $limit
-     * @param type $offset the id of the last notification received
+     * @param integer $limit
+     * @param integer $offset the id of the last notification received
      * @param ClientInterface $client optionally filters by OAuth Client
      *
      * @return NotificationInterface[]
@@ -113,7 +113,7 @@ 
      * Retrieves a person's settings.
      *
      * @param PersonInterface   $person
-     * @param CategoryInterface $client optionally filter by category
+     * @param ClientInterface $client optionally filter by category
      * @param ClientInterface $client optionally filter by client
      *
      * @return PersonNotificationOption[]
@@ -184,7 +184,6 @@ 
     /**
      * Return a person's number of unread notifications grouping by Client
      * @param PersonInterface $person
-     * @param ClientInterface $client
      *
      * @return array
      */

Indentation +9 added lines, -9 removed lines

@@ -42,7 +42,7 @@ 
      * @return NotificationInterface[]
      */
     public function getAllFromPerson(PersonInterface $person, $limit = 5,
-                                     $offset = 0, $orderby = null);
+                                        $offset = 0, $orderby = null);
 
     /**
      * Same as getAllFromPerson but uses the notification's id as the offset.
@@ -55,8 +55,8 @@ 
      * @return NotificationInterface[]
      */
     public function getAllFromPersonIdOffset(PersonInterface $person,
-                                             $limit = 5, $offset = 0,
-                                             ClientInterface $client = null);
+                                                $limit = 5, $offset = 0,
+                                                ClientInterface $client = null);
 
     /**
      * Get a list of an user's Notifications restricted by Client.
@@ -70,9 +70,9 @@ 
      * @return NotificationInterface[]
      */
     public function getAllFromPersonByClient(PersonInterface $person,
-                                             ClientInterface $client,
-                                             $limit = 5, $offset = 0,
-                                             $orderby = null);
+                                                ClientInterface $client,
+                                                $limit = 5, $offset = 0,
+                                                $orderby = null);
 
     /**
      * Post Notification, creates a new Notification
@@ -142,7 +142,7 @@ 
      * @return boolean
      */
     public function initializeSettings(PersonInterface $person,
-                                       ClientInterface $client);
+                                        ClientInterface $client);
 
     /**
      * Mark the specified range of IDs as read.
@@ -163,8 +163,8 @@ 
      * @return NotificationSettings PersonNotificationOptions grouped by Client
      */
     public function getGroupedSettings(PersonInterface $person,
-                                       ClientInterface $client = null,
-                                       CategoryInterface $category = null);
+                                        ClientInterface $client = null,
+                                        CategoryInterface $category = null);
 
     /**
      *

src/LoginCidadao/NotificationBundle/Helper/NotificationsHelper.php

Doc Comments +13 added lines

@@ -69,6 +69,9 @@ 
         return $this->em->getRepository("LoginCidadaoNotificationBundle:Notification");
     }
 
+    /**
+     * @return \LoginCidadao\CoreBundle\Model\PersonInterface
+     */
     public function getUser()
     {
         return $this->context->getToken()->getUser();
@@ -101,6 +104,16 @@ 
         }
     }
 
+    /**
+     * @param string $title
+     * @param string $shortText
+     * @param string $text
+     * @param string $icon
+     * @param Notification $notification
+     * @param Placeholder[] $parameters
+     *
+     * @return NotificationInterface
+     */
     private function getDefaultNotification(Person $person, $title, $shortText,
                                             $text, $icon, Category $category,
                                             $notification, $parameters = null)

Unused Use Statements -5 removed lines

@@ -10,12 +10,7 @@
 use LoginCidadao\NotificationBundle\Entity\Notification;
 use LoginCidadao\NotificationBundle\Entity\Category;
 use LoginCidadao\NotificationBundle\Exception\MissingCategoryException;
-use Symfony\Component\Serializer\Normalizer\GetSetMethodNormalizer;
-use Symfony\Component\Serializer\Encoder\JsonEncoder;
-use Symfony\Component\Serializer\Serializer;
-use LoginCidadao\NotificationBundle\Form\NotificationType;
 use JMS\Serializer\SerializationContext;
-use LoginCidadao\NotificationBundle\Handler\NotificationHandler;
 use LoginCidadao\NotificationBundle\Entity\Placeholder;
 
 class NotificationsHelper

src/LoginCidadao/NotificationBundle/Model/BroadcastPlaceholder.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace LoginCidadao\NotificationBundle\Model;
 
 use LoginCidadao\NotificationBundle\Entity\Placeholder;
-use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\ORM\EntityManager;
 
 class BroadcastPlaceholder

Indentation +3 added lines, -3 removed lines

@@ -34,13 +34,13 @@
 
     public function getValue()
     {
-      return $this->value;
+        return $this->value;
     }
 
     public function setValue($value)
     {
-      $this->value = $value;
-      return $this;
+        $this->value = $value;
+        return $this;
     }
     
     public function getName()

src/LoginCidadao/OAuthBundle/Entity/Client.php

Doc Comments +5 added lines, -2 removed lines

@@ -214,6 +214,9 @@ 
         return $this->name;
     }
 
+    /**
+     * @param string $description
+     */
     public function setDescription($description)
     {
         $this->description = $description;
@@ -382,8 +385,8 @@ 
 
     /**
      * Sets the Unique Id of the Entity.
-     * @param string $id the entity UID
-     * @return AbstractUniqueEntity
+     * @param string $uid the entity UID
+     * @return Client
      */
     public function setUid($uid = null)
     {

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use LoginCidadao\NotificationBundle\Entity\Notification;
 use Symfony\Component\HttpFoundation\File\File;
 use Symfony\Component\Validator\Constraints as Assert;
-use JMS\Serializer\Tests\Fixtures\Publisher;
 use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;
 use JMS\Serializer\Annotation as JMS;
 use OAuth2\OAuth2;

src/LoginCidadao/OAuthBundle/Entity/Organization.php

Doc Comments +2 added lines, -2 removed lines

@@ -139,7 +139,7 @@ 
 
     /**
      * @param PersonInterface[] $members
-     * @return \Organization
+     * @return Organization
      */
     public function setMembers(array $members)
     {
@@ -158,7 +158,7 @@ 
 
     /**
      * @param \DateTime $verifiedAt
-     * @return \Organization
+     * @return Organization
      */
     public function setVerifiedAt($verifiedAt)
     {