sunnysideup / silverstripe-campaignmonitor

code/api/CampaignMonitorAPIConnector.php

Doc Comments +18 added lines, -10 removed lines

@@ -80,6 +80,7 @@ 
      * turn debug on or off
      *
      * @param Boolean
+     * @param boolean $b
      */
     public function setDebug($b)
     {
@@ -160,6 +161,8 @@ 
     /**
      * returns the result or NULL in case of an error
      * @param CS_REST_Wrapper_Result
+     * @param CS_REST_Wrapper_Result $result
+     * @param string $apiCall
      * @return Mixed | Null
      */
     protected function returnResult($result, $apiCall, $description)
@@ -495,7 +498,8 @@ 
      * @param string $listID - list ID
      * @param string $type - type of custom field
      * @param string $title - field type
-     * @param array $options - options for dropdown field type
+     * @param string[] $options - options for dropdown field type
+     * @param boolean $visible
      *
      * @return CS_REST_Wrapper_Result A successful response will be the key of the newly created custom field
      */
@@ -604,7 +608,7 @@ 
      * Gets all active subscribers added since the given date
      *
      * @param Int $listID
-     * @param string $daysAgo The date to start getting subscribers from
+     * @param integer $daysAgo The date to start getting subscribers from
      * @param int $page The page number to get
      * @param int $pageSize The number of records per page
      * @param string $sortByField ('EMAIL', 'NAME', 'DATE')
@@ -657,7 +661,7 @@ 
      * Gets all unconfirmed subscribers added since the given date
      *
      * @param Int $listID
-     * @param string $daysAgo The date to start getting subscribers from
+     * @param integer $daysAgo The date to start getting subscribers from
      * @param int $page The page number to get
      * @param int $pageSize The number of records per page
      * @param string $sortByField ('EMAIL', 'NAME', 'DATE')
@@ -710,7 +714,7 @@ 
      * Gets all bounced subscribers who have bounced out since the given date
      *
      * @param Int $listID
-     * @param string $daysAgo The date to start getting subscribers from
+     * @param integer $daysAgo The date to start getting subscribers from
      * @param int $page The page number to get
      * @param int $pageSize The number of records per page
      * @param string $sortByField ('EMAIL', 'NAME', 'DATE')
@@ -764,7 +768,7 @@ 
      * Gets all unsubscribed subscribers who have unsubscribed since the given date
      *
      * @param Int $listID
-     * @param string $daysAgo The date to start getting subscribers from
+     * @param integer $daysAgo The date to start getting subscribers from
      * @param int $page The page number to get
      * @param int $pageSize The number of records per page
      * @param string $sortByField ('EMAIL', 'NAME', 'DATE')
@@ -823,7 +827,8 @@ 
      * @param string $confirmationSuccessPage - The page to redirect subscribers to when they confirm their subscription
      * @param string $unsubscribeSetting - Unsubscribe setting must be CS_REST_LIST_UNSUBSCRIBE_SETTING_ALL_CLIENT_LISTS or CS_REST_LIST_UNSUBSCRIBE_SETTING_ONLY_THIS_LIST.  See the documentation for details: http://www.campaignmonitor.com/api/lists/#creating_a_list
      * @param boolean $addUnsubscribesToSuppList -  When UnsubscribeSetting is CS_REST_LIST_UNSUBSCRIBE_SETTING_ALL_CLIENT_LISTS, whether unsubscribes from this list should be added to the suppression list.
-     * @param boolean $acrubActiveWithSuppList - When UnsubscribeSetting is CS_REST_LIST_UNSUBSCRIBE_SETTING_ALL_CLIENT_LISTS, whether active subscribers should be scrubbed against the suppression list.
+     * @param boolean $scrubActiveWithSuppList - When UnsubscribeSetting is CS_REST_LIST_UNSUBSCRIBE_SETTING_ALL_CLIENT_LISTS, whether active subscribers should be scrubbed against the suppression list.
+     * @param CS_REST_Wrapper_Result $listID
      *
      * @return CS_REST_Wrapper_Result A successful response will be empty
      */
@@ -1133,7 +1138,7 @@ 
      * Gets all unsubscribes recorded for a campaign since the provided date
      *
      * @param int $campaignID ID of the Campaign
-     * @param string $daysAgo The date to start getting subscribers from
+     * @param integer $daysAgo The date to start getting subscribers from
      * @param int $page The page number to get
      * @param int $pageSize The number of records per page
      * @param string $sortByField ('EMAIL', 'NAME', 'DATE')
@@ -1239,7 +1244,7 @@ 
      * @param Array $customFields
      * @param array $customFields The subscriber details to use during creation.
      * @param boolean $resubscribe Whether we should resubscribe this subscriber if they already exist in the list
-     * @param boolean $RestartSubscriptionBasedAutoResponders Whether we should restart subscription based auto responders which are sent when the subscriber first subscribes to a list.
+     * @param boolean $restartSubscriptionBasedAutoResponders Whether we should restart subscription based auto responders which are sent when the subscriber first subscribes to a list.
      *
      * NOTE that for the custom fields they need to be formatted like this:
      *    Array(
@@ -1353,9 +1358,9 @@ 
      * given email not existing in the list.
      *
      * @param Int $listID
-     * @param ArraySet $memberSet - list of mebers
+     * @param ArraySet $membersSet - list of mebers
      * @param array $customFields The subscriber details to use during creation. Each array item needs to have the same key as the member ID - e.g. array( 123 => array( [custom fields here] ), 456 => array( [custom fields here] ) )
-     * @param $resubscribe Whether we should resubscribe any existing subscribers
+     * @param boolean $resubscribe Whether we should resubscribe any existing subscribers
      * @param $queueSubscriptionBasedAutoResponders By default, subscription based auto responders do not trigger during an import. Pass a value of true to override this behaviour
      * @param $restartSubscriptionBasedAutoResponders By default, subscription based auto responders will not be restarted
      *
@@ -1420,6 +1425,7 @@ 
     /**
      * @param Int $listID
      * @param Member | String $member - email address or Member Object
+     * @param DataObject|null $member
      *
      * @return CS_REST_Wrapper_Result A successful response will be empty
      */
@@ -1518,6 +1524,7 @@ 
      *
      * @param Int $listID
      * @param Member | String $member
+     * @param DataObject|null $member
      *
      * @return Boolean
      */
@@ -1587,6 +1594,7 @@ 
      *
      * @param Int $listID
      * @param Member | String $member
+     * @param DataObject|null $member
      *
      * @return CS_REST_Wrapper_Result A successful response will be an object of the form
      * {

code/CampaignMonitorSignupPage.php

Doc Comments +2 added lines, -3 removed lines

@@ -579,7 +579,6 @@ 
 
     /**
      * action subscription form
-     * @param Array $array
      * @param Form $form
      *
      * return redirect
@@ -859,7 +858,7 @@ 
      * same as $this->CampaignMonitorCampaigns()
      * but sorted correctly.
      *
-     * @return DataSet of CampaignMonitorCampaigns
+     * @return DataList|null of CampaignMonitorCampaigns
      */
     public function PreviousCampaignMonitorCampaigns()
     {
@@ -916,7 +915,7 @@ 
     /**
      * returns a bunch of stats about a campaign
      * IF the user is an admin AND a campaign is selected
-     * @return String (html) | false
+     * @return null|false (html) | false
      */
     public function CampaignStats()
     {

code/control/CampaignMonitorAPIConnector_TestController.php

Doc Comments +1 added lines

@@ -64,6 +64,7 @@
      * link for controller
      * we add baseURL to make it work for all set ups.
      * @var String
+     * @param string $action
      */
     public function Link($action = null)
     {

code/decorators/CampaignMonitorMemberDOD.php

Doc Comments +1 added lines, -3 removed lines

@@ -146,8 +146,6 @@ 
 
     /**
      * action subscription form
-     * @param CampaignMonitorSignUpPage $page
-     * @param Array $array
      * @param Form $form
      *
      * return string: can be subscribe / unsubscribe / error
@@ -207,7 +205,7 @@ 
     /**
      * is this user currently signed up to one or more newsletters
      *
-     * @return Boolean
+     * @return boolean|null
      */
     public function IsCampaignMonitorSubscriber()
     {

code/model/CampaignMonitorCampaignStyle.php

Doc Comments +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public function getFoldersToSearch()
     {
@@ -70,7 +70,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public function getCSSFoldersToSearch()
     {