newscoop / plugin-NewscoopPaywallBundle

Entity/Subscription.php

Doc Comments +9 added lines, -9 removed lines

@@ -184,7 +184,7 @@ 
     /**
      * Get subscription type.
      *
-     * @return int
+     * @return string
      */
     public function getType()
     {
@@ -196,7 +196,7 @@ 
      *
      * @param int $type
      *
-     * @return int
+     * @return Subscription
      */
     public function setType($type)
     {
@@ -208,7 +208,7 @@ 
     /**
      * Get subscription price.
      *
-     * @return decimal
+     * @return integer
      */
     public function getPrice()
     {
@@ -244,7 +244,7 @@ 
      *
      * @param string $currency
      *
-     * @return string
+     * @return Subscription
      */
     public function setCurrency($currency)
     {
@@ -292,7 +292,7 @@ 
      *
      * @param bool $is_active
      *
-     * @return bool
+     * @return Subscription
      */
     public function setIsActive($is_active)
     {
@@ -304,7 +304,7 @@ 
     /**
      * Get subscription create date.
      *
-     * @return datetime
+     * @return string
      */
     public function getCreatedAt()
     {
@@ -314,7 +314,7 @@ 
     /**
      * Set subscription create date.
      *
-     * @param datetime $created_at
+     * @param \DateTime $created_at
      *
      * @return datetime
      */
@@ -340,7 +340,7 @@ 
      *
      * @param bool $is_default
      *
-     * @return bool
+     * @return Subscription
      */
     public function setIsDefault($is_default)
     {
@@ -362,7 +362,7 @@ 
     /**
      * Sets the value of ranges.
      *
-     * @param Doctrine\Common\Collections\ArrayCollection $ranges the ranges
+     * @param ArrayCollection $ranges the ranges
      *
      * @return self
      */

Entity/SubscriptionSpecification.php

Doc Comments +8 added lines, -8 removed lines

@@ -133,7 +133,7 @@ 
      *
      * @param int $publication
      *
-     * @return int
+     * @return SubscriptionSpecification
      */
     public function setPublication($publication)
     {
@@ -157,7 +157,7 @@ 
      *
      * @param int $issue
      *
-     * @return int
+     * @return SubscriptionSpecification
      */
     public function setIssue($issue)
     {
@@ -181,7 +181,7 @@ 
      *
      * @param int $section
      *
-     * @return int
+     * @return SubscriptionSpecification
      */
     public function setSection($section)
     {
@@ -205,7 +205,7 @@ 
      *
      * @param int $article
      *
-     * @return int
+     * @return SubscriptionSpecification
      */
     public function setArticle($article)
     {
@@ -229,7 +229,7 @@ 
      *
      * @param bool $is_active
      *
-     * @return bool
+     * @return SubscriptionSpecification
      */
     public function setIsActive($is_active)
     {
@@ -241,7 +241,7 @@ 
     /**
      * Get specification create date.
      *
-     * @return datetime
+     * @return string
      */
     public function getCreatedAt()
     {
@@ -251,9 +251,9 @@ 
     /**
      * Set specification create date.
      *
-     * @param datetime $created_at
+     * @param \DateTime $created_at
      *
-     * @return datetime
+     * @return SubscriptionSpecification
      */
     public function setCreatedAt(\DateTime $created_at)
     {

Entity/Trial.php

Doc Comments +6 added lines, -6 removed lines

@@ -126,7 +126,7 @@ 
      *
      * @param bool $hadTrial
      *
-     * @return bool
+     * @return Trial
      */
     public function setHadTrial($hadTrial)
     {
@@ -150,7 +150,7 @@ 
      *
      * @param datetime $finishTrial
      *
-     * @return datetime
+     * @return Trial
      */
     public function setFinishTrial($finishTrial)
     {
@@ -174,7 +174,7 @@ 
      *
      * @param bool $is_active
      *
-     * @return bool
+     * @return Trial
      */
     public function setIsActive($is_active)
     {
@@ -186,7 +186,7 @@ 
     /**
      * Get create date.
      *
-     * @return datetime
+     * @return string
      */
     public function getCreatedAt()
     {
@@ -196,9 +196,9 @@ 
     /**
      * Set create date.
      *
-     * @param datetime $created_at
+     * @param \DateTime $created_at
      *
-     * @return datetime
+     * @return Trial
      */
     public function setCreatedAt(\DateTime $created_at)
     {

Entity/UserSubscription.php

Doc Comments +18 added lines, -18 removed lines

@@ -263,7 +263,7 @@ 
     /**
      * Set subscription.
      *
-     * @param Newscoop\PaywallBundle\Entity\Subscription $subscription
+     * @param integer $subscription
      */
     public function setSubscription($subscription)
     {
@@ -309,7 +309,7 @@ 
      *
      * @param Newscoop\Entity\Publication $publication
      *
-     * @return Newscoop\Entity\Subscription
+     * @return UserSubscription
      */
     public function setPublication(Publication $publication)
     {
@@ -353,7 +353,7 @@ 
      *
      * @param float $toPay
      *
-     * @return Newscoop\Entity\Subscription
+     * @return UserSubscription
      */
     public function setToPay($toPay)
     {
@@ -377,7 +377,7 @@ 
      *
      * @param float $toPay
      *
-     * @return Newscoop\Entity\Subscription
+     * @return UserSubscription
      */
     public function setPrice($toPay)
     {
@@ -401,7 +401,7 @@ 
      *
      * @param string $type
      *
-     * @return Newscoop\Entity\Subscription
+     * @return UserSubscription
      */
     public function setType($type)
     {
@@ -435,7 +435,7 @@ 
      *
      * @param bool $active
      *
-     * @return Newscoop\Entity\Subscription
+     * @return UserSubscription
      */
     public function setActive($active)
     {
@@ -470,7 +470,7 @@ 
     /**
      * Set currency.
      *
-     * @return string
+     * @return UserSubscription
      */
     public function setCurrency($currency)
     {
@@ -492,7 +492,7 @@ 
     /**
      * Set trial.
      *
-     * @return Trial
+     * @return UserSubscription
      */
     public function setTrial($trial)
     {
@@ -504,7 +504,7 @@ 
     /**
      * Get create date.
      *
-     * @return datetime
+     * @return \DateTime
      */
     public function getCreatedAt()
     {
@@ -514,9 +514,9 @@ 
     /**
      * Set create date.
      *
-     * @param datetime $created_at
+     * @param \DateTime $created_at
      *
-     * @return datetime
+     * @return UserSubscription
      */
     public function setCreatedAt(\DateTime $created_at)
     {
@@ -538,9 +538,9 @@ 
     /**
      * Set expire date.
      *
-     * @param datetime $expire_at
+     * @param \DateTime $expire_at
      *
-     * @return datetime
+     * @return UserSubscription
      */
     public function setExpireAt(\DateTime $expire_at = null)
     {
@@ -564,7 +564,7 @@ 
      *
      * @param bool $is_active
      *
-     * @return bool
+     * @return UserSubscription
      */
     public function setIsActive($is_active)
     {
@@ -624,7 +624,7 @@ 
     /**
      * Gets the value of updated.
      *
-     * @return DateTime
+     * @return \DateTime
      */
     public function getUpdated()
     {
@@ -634,7 +634,7 @@ 
     /**
      * Sets the value of updated.
      *
-     * @param DateTime $updated the updated
+     * @param \DateTime $updated the updated
      *
      * @return self
      */
@@ -984,7 +984,7 @@ 
     /**
      * Gets the value of startsAt.
      *
-     * @return DateTime
+     * @return \DateTime
      */
     public function getStartsAt()
     {
@@ -994,7 +994,7 @@ 
     /**
      * Sets the value of startsAt.
      *
-     * @param DateTime $startsAt the starts at
+     * @param \DateTime $startsAt the starts at
      *
      * @return self
      */

Spacing +1 added lines, -1 removed lines

@@ -795,7 +795,7 @@
             return $this->modifications;
         }
 
-        return $this->modifications->filter(function (Modification $modification) use ($type) {
+        return $this->modifications->filter(function(Modification $modification) use ($type) {
             return $type === $modification->getLabel();
         });
     }

EventListener/NotificationListener.php

Doc Comments +3 added lines

@@ -139,6 +139,9 @@
         }
     }
 
+    /**
+     * @return string
+     */
     private function getUser($subscription)
     {
         $user = null;

Serializer/CurrencyContextHandler.php

Doc Comments -1 removed lines

@@ -21,7 +21,6 @@
     /**
      * Construct.
      *
-     * @param CurrencyConverter $converter
      */
     public function __construct(CurrencyContextInterface $context)
     {

Services/NotificationsService.php

Doc Comments +3 added lines, -3 removed lines

@@ -79,8 +79,6 @@ 
      * Sends email notifications.
      *
      * @param string                                $code             Email message type
-     * @param Newscoop\Entity\User                  $user             User object
-     * @param PaywallBundle\Entity\UserSubscription $userSubscription User's subscription
      */
     public function sendNotification($code, array $recipients = array(), array $data = array())
     {
@@ -124,6 +122,9 @@ 
         );
     }
 
+    /**
+     * @param string $code
+     */
     private function loadProperMessageTemplateBy($code)
     {
         switch ($code) {
@@ -192,7 +193,6 @@ 
      * notifications to users.
      *
      * @param \DateTime $now                Current date time
-     * @param int       $subscriptionsCount Subscriptions count
      */
     public function processExpiringSubscriptions($now, $notify, $count = 0, $days = 7)
     {

Services/OrderService.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      * Calculates prices including all discounts.
      *
      * @param array  $items    array of subscription identifiers and its periods
-     * @param string $surrency currency
+     * @param string $currency currency
      *
      * @return OrderInterface
      */

Form/Type/OrderItemType.php

Spacing +3 added lines, -3 removed lines

@@ -56,7 +56,7 @@ 
             ),
         ));
 
-        $formModifier = function (FormInterface $form, Subscription $subscription = null) {
+        $formModifier = function(FormInterface $form, Subscription $subscription = null) {
             $periods = null === $subscription ? array() : $subscription->getRanges();
 
             $form->add('duration', 'entity', array(
@@ -73,7 +73,7 @@ 
 
         $builder->addEventListener(
             FormEvents::PRE_SET_DATA,
-            function (FormEvent $event) use ($formModifier) {
+            function(FormEvent $event) use ($formModifier) {
                 $item = $event->getData();
                 $subscription = null;
                 if ($item) {
@@ -85,7 +85,7 @@ 
 
         $builder->get('subscription')->addEventListener(
             FormEvents::POST_SUBMIT,
-            function (FormEvent $event) use ($formModifier) {
+            function(FormEvent $event) use ($formModifier) {
                 $subscription = $event->getForm()->getData();
                 $formModifier($event->getForm()->getParent(), $subscription);
             }