@@ -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); |