@@ -253,6 +253,7 @@ discard block |
||
253 | 253 | * Returns on the basis of the module package which type of add this is. Can be either add, extend, or upgrade |
254 | 254 | * |
255 | 255 | * @param object modulepackage An modulepackage object |
256 | + * @param Intraface_modules_modulepackage_ModulePackage $modulepackage |
|
256 | 257 | * |
257 | 258 | * @return string either 'add', 'extend' or 'upgrade' |
258 | 259 | */ |
@@ -295,6 +296,7 @@ discard block |
||
295 | 296 | * Returns the end date of the last modulepacke in a given group |
296 | 297 | * |
297 | 298 | * @param object modulepackage |
299 | + * @return string |
|
298 | 300 | */ |
299 | 301 | public function getLastEndDateInGroup($modulepackage) |
300 | 302 | { |
@@ -400,6 +402,8 @@ discard block |
||
400 | 402 | * |
401 | 403 | * @param object modulepackage module package |
402 | 404 | * @param string duration Duration as either date 'dd-mm-yyyy' or 'yyyy-mm-dd' or 'X month' where X is any whole number |
405 | + * @param Intraface_modules_modulepackage_ModulePackage $modulepackage |
|
406 | + * @param string $duration |
|
403 | 407 | * |
404 | 408 | * @return object Action object with the actions needed to be processed |
405 | 409 | */ |
@@ -470,6 +474,8 @@ discard block |
||
470 | 474 | * |
471 | 475 | * @param object modulepackage |
472 | 476 | * @param string duration Duration as either date 'dd-mm-yyyy' or 'yyyy-mm-dd' or 'X month' where X is any whole number |
477 | + * @param Intraface_modules_modulepackage_ModulePackage $modulepackage |
|
478 | + * @param string $duration |
|
473 | 479 | * |
474 | 480 | * @return object Action object with the actions needed to be performed |
475 | 481 | */ |
@@ -628,7 +634,7 @@ discard block |
||
628 | 634 | /** |
629 | 635 | * Returns the possible types of status' |
630 | 636 | * |
631 | - * @return array status types |
|
637 | + * @return string[] status types |
|
632 | 638 | */ |
633 | 639 | static public function getStatusTypes() |
634 | 640 | { |
@@ -234,7 +234,7 @@ |
||
234 | 234 | /** |
235 | 235 | * Returns order details from order identifier |
236 | 236 | * |
237 | - * @param string $order_identifier |
|
237 | + * @param string $order_identification |
|
238 | 238 | * @return array |
239 | 239 | */ |
240 | 240 | public function getOrderDetails($order_identification) |
@@ -200,7 +200,7 @@ discard block |
||
200 | 200 | /** |
201 | 201 | * Puts subscribers in a queue to the newsletter |
202 | 202 | * |
203 | - * @return boolean |
|
203 | + * @return null|boolean |
|
204 | 204 | */ |
205 | 205 | function queue() |
206 | 206 | { |
@@ -306,7 +306,7 @@ discard block |
||
306 | 306 | * |
307 | 307 | * @param integer $id Contact id |
308 | 308 | * |
309 | - * @return object |
|
309 | + * @return Contact |
|
310 | 310 | */ |
311 | 311 | function getContact($id) |
312 | 312 | { |
@@ -57,6 +57,9 @@ discard block |
||
57 | 57 | return $db->numRows(); |
58 | 58 | } |
59 | 59 | |
60 | + /** |
|
61 | + * @param string $code |
|
62 | + */ |
|
60 | 63 | function findByCode($code) |
61 | 64 | { |
62 | 65 | // kernel og kode |
@@ -73,6 +76,9 @@ discard block |
||
73 | 76 | return new NewsletterSubscriber(new NewsletterList($this->kernel, $db->f('list_id')), $db->f('id')); |
74 | 77 | } |
75 | 78 | |
79 | + /** |
|
80 | + * @param string $email |
|
81 | + */ |
|
76 | 82 | function findByListAndEmail($list, $email) |
77 | 83 | { |
78 | 84 | $email = safeToDb($email); |
@@ -33,6 +33,9 @@ discard block |
||
33 | 33 | return $this->getProvider($provider, $value); |
34 | 34 | } |
35 | 35 | |
36 | + /** |
|
37 | + * @param integer $provider |
|
38 | + */ |
|
36 | 39 | public function findByProvider($provider) |
37 | 40 | { |
38 | 41 | if (!in_array($provider, $this->implemented_providers)) { |
@@ -42,6 +45,9 @@ discard block |
||
42 | 45 | return $this->getProvider($provider); |
43 | 46 | } |
44 | 47 | |
48 | + /** |
|
49 | + * @param integer $value |
|
50 | + */ |
|
45 | 51 | public function findByTransactionNumber($value) |
46 | 52 | { |
47 | 53 | $db = new DB_Sql; |
@@ -8,6 +8,9 @@ |
||
8 | 8 | |
9 | 9 | class OnlinePaymentDefault extends OnlinePayment |
10 | 10 | { |
11 | + /** |
|
12 | + * @param integer $id |
|
13 | + */ |
|
11 | 14 | function __construct($kernel, $id) { |
12 | 15 | |
13 | 16 | parent::__construct($kernel, $id); |
@@ -95,7 +95,7 @@ |
||
95 | 95 | * |
96 | 96 | * @param string $action Which action to perform? |
97 | 97 | * |
98 | - * @return void |
|
98 | + * @return boolean |
|
99 | 99 | */ |
100 | 100 | function transactionAction($action) |
101 | 101 | { |
@@ -182,7 +182,7 @@ discard block |
||
182 | 182 | * returns possible status types |
183 | 183 | * @todo: duplicate in Procurement class |
184 | 184 | * |
185 | - * @return array status types |
|
185 | + * @return string[] status types |
|
186 | 186 | */ |
187 | 187 | public function getStatusTypes() |
188 | 188 | { |
@@ -197,7 +197,7 @@ discard block |
||
197 | 197 | * returns the possible regions where procurement is bought |
198 | 198 | * @todo: duplicate in Procurement class |
199 | 199 | * |
200 | - * @return array possible regions |
|
200 | + * @return string[] possible regions |
|
201 | 201 | */ |
202 | 202 | public function getRegionTypes() |
203 | 203 | { |
@@ -21,7 +21,7 @@ |
||
21 | 21 | * @todo $doctrine should not be optional |
22 | 22 | * |
23 | 23 | * @param object $doctrine Doctrine_Connection |
24 | - * @return void |
|
24 | + * @return integer |
|
25 | 25 | */ |
26 | 26 | public function __construct($doctrine = NULL) |
27 | 27 | { |