Completed
Pull Request — master (#49)
by
unknown
02:21
created
code/tasks/ContentReviewEmails.php 1 patch
Doc Comments   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -94,6 +94,8 @@  discard block
 block discarded – undo
94 94
      * The 'days before due' value for each of these is configurable in settings.
95 95
      * ie. a value of 30 for the 'first reminder' setting means a page with a review date exactly 30 days from due, will
96 96
      * be present in the email sent to the configured address.
97
+     * @param ArrayList $firstReminderPages
98
+     * @param ArrayList $secondReminderPages
97 99
      */
98 100
     protected function notifyTeam($firstReminderPages, $secondReminderPages) {
99 101
         // Prepare variables
@@ -128,7 +130,7 @@  discard block
 block discarded – undo
128 130
 
129 131
     /**
130 132
      * @param int           $ownerID
131
-     * @param array|SS_List $pages
133
+     * @param SS_List $pages
132 134
      * @param string        $type
133 135
      */
134 136
     protected function notifyOwner($ownerID, SS_List $pages, $type)
@@ -166,6 +168,7 @@  discard block
 block discarded – undo
166 168
      *
167 169
      * @param SiteConfig $config
168 170
      * @param array List of safe template variables to expose to this template
171
+     * @param string $type
169 172
      *
170 173
      * @return HTMLText
171 174
      */
@@ -196,7 +199,6 @@  discard block
 block discarded – undo
196 199
      * @param Member     $recipient
197 200
      * @param SiteConfig $config
198 201
      * @param SS_List    $pages
199
-     * @param string     $type
200 202
      *
201 203
      * @return array
202 204
      */
Please login to merge, or discard this patch.