@@ -512,7 +512,7 @@ |
||
512 | 512 | * Get the HTML class attribute from a node in the sitetree |
513 | 513 | * |
514 | 514 | * @param $html |
515 | - * @param $node |
|
515 | + * @param DataObject|null $node |
|
516 | 516 | * @return string |
517 | 517 | */ |
518 | 518 | protected function getNodeClassFromTree($html, $node) { |
@@ -175,6 +175,10 @@ |
||
175 | 175 | $this->assertEquals('', $this->parser->parse('[test_shortcode][test_shortcode]')); |
176 | 176 | } |
177 | 177 | |
178 | + /** |
|
179 | + * @param string $a |
|
180 | + * @param string $b |
|
181 | + */ |
|
178 | 182 | protected function assertEqualsIgnoringWhitespace($a, $b, $message = null) { |
179 | 183 | $this->assertEquals(preg_replace('/\s+/', '', $a), preg_replace('/\s+/', '', $b), $message); |
180 | 184 | } |
@@ -175,6 +175,10 @@ |
||
175 | 175 | |
176 | 176 | class GroupTest_MemberForm extends Form { |
177 | 177 | |
178 | + /** |
|
179 | + * @param GroupTest $controller |
|
180 | + * @param string $name |
|
181 | + */ |
|
178 | 182 | public function __construct($controller, $name) { |
179 | 183 | $fields = singleton('GroupTest_Member')->getCMSFields(); |
180 | 184 | $actions = new FieldList( |
@@ -913,6 +913,9 @@ |
||
913 | 913 | */ |
914 | 914 | class MemberTest_ValidatorExtension extends DataExtension implements TestOnly { |
915 | 915 | |
916 | + /** |
|
917 | + * @param Member_Validator $validator |
|
918 | + */ |
|
916 | 919 | public function updateValidator(&$validator) { |
917 | 920 | $validator->addRequiredField('Surname'); |
918 | 921 | $validator->removeRequiredField('FirstName'); |
@@ -5,6 +5,7 @@ |
||
5 | 5 | /** |
6 | 6 | * Small helper to render templates from strings |
7 | 7 | * Cloned from SSViewerTest |
8 | + * @param string $templateString |
|
8 | 9 | */ |
9 | 10 | private function render($templateString, $data = null) { |
10 | 11 | $t = SSViewer::fromString($templateString); |
@@ -28,6 +28,9 @@ discard block |
||
28 | 28 | "Versioned('Stage', 'Live')" |
29 | 29 | ); |
30 | 30 | |
31 | + /** |
|
32 | + * @param string $entropy |
|
33 | + */ |
|
31 | 34 | public function setEntropy($entropy) { |
32 | 35 | $this->entropy = $entropy; |
33 | 36 | } |
@@ -53,6 +56,9 @@ discard block |
||
53 | 56 | SS_Cache::set_cache_lifetime('cacheblock', $cacheOn ? 600 : -1); |
54 | 57 | } |
55 | 58 | |
59 | + /** |
|
60 | + * @param string $template |
|
61 | + */ |
|
56 | 62 | protected function _runtemplate($template, $data = null) { |
57 | 63 | if ($data === null) $data = $this->data; |
58 | 64 | if (is_array($data)) $data = $this->data->customise($data); |
@@ -82,6 +82,10 @@ discard block |
||
82 | 82 | action treats a string as a single character. Wow! |
83 | 83 | action recognizes a regular expression if it is preceded by ( or , or =. |
84 | 84 | */ |
85 | + |
|
86 | + /** |
|
87 | + * @param integer $d |
|
88 | + */ |
|
85 | 89 | protected function action($d) { |
86 | 90 | switch($d) { |
87 | 91 | case 1: |
@@ -160,6 +164,9 @@ discard block |
||
160 | 164 | } |
161 | 165 | } |
162 | 166 | |
167 | + /** |
|
168 | + * @return string |
|
169 | + */ |
|
163 | 170 | protected function get() { |
164 | 171 | $c = $this->lookAhead; |
165 | 172 | $this->lookAhead = null; |
@@ -187,6 +194,10 @@ discard block |
||
187 | 194 | /* isAlphanum -- return true if the character is a letter, digit, underscore, |
188 | 195 | dollar sign, or non-ASCII character. |
189 | 196 | */ |
197 | + |
|
198 | + /** |
|
199 | + * @param string $c |
|
200 | + */ |
|
190 | 201 | protected function isAlphaNum($c) { |
191 | 202 | return ord($c) > 126 || $c === '\\' || preg_match('/^[\w\$]$/', $c) === 1; |
192 | 203 | } |
@@ -97,6 +97,9 @@ |
||
97 | 97 | return $return; |
98 | 98 | } |
99 | 99 | |
100 | + /** |
|
101 | + * @param boolean $index |
|
102 | + */ |
|
100 | 103 | function f($index) { |
101 | 104 | return stripslashes($this->row[$index]); |
102 | 105 | } |
@@ -61,9 +61,17 @@ discard block |
||
61 | 61 | return $path; |
62 | 62 | } |
63 | 63 | |
64 | + /** |
|
65 | + * @param integer $parent |
|
66 | + * @param integer $position |
|
67 | + */ |
|
64 | 68 | function _create($parent, $position) { |
65 | 69 | return $this->_move(0, $parent, $position); |
66 | 70 | } |
71 | + |
|
72 | + /** |
|
73 | + * @param integer $id |
|
74 | + */ |
|
67 | 75 | function _remove($id) { |
68 | 76 | if((int)$id === 1) { return false; } |
69 | 77 | $data = $this->_get_node($id); |
@@ -100,6 +108,10 @@ discard block |
||
100 | 108 | ); |
101 | 109 | return true; |
102 | 110 | } |
111 | + |
|
112 | + /** |
|
113 | + * @param integer $id |
|
114 | + */ |
|
103 | 115 | function _move($id, $ref_id, $position = 0, $is_copy = false) { |
104 | 116 | if((int)$ref_id === 0 || (int)$id === 1) { return false; } |
105 | 117 | $sql = array(); // Queries executed at the end |
@@ -238,6 +250,10 @@ discard block |
||
238 | 250 | if($is_copy) $this->_fix_copy($ind, $position); |
239 | 251 | return $node === false || $is_copy ? $ind : true; |
240 | 252 | } |
253 | + |
|
254 | + /** |
|
255 | + * @param integer $position |
|
256 | + */ |
|
241 | 257 | function _fix_copy($id, $position) { |
242 | 258 | $node = $this->_get_node($id); |
243 | 259 | $children = $this->_get_children($id, true); |