Completed
Pull Request — master (#5433)
by Sam
19:38
created
tests/model/HierarchyTest.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -512,7 +512,7 @@
 block discarded – undo
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) {
Please login to merge, or discard this patch.
tests/parsers/ShortcodeParserTest.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -175,6 +175,10 @@
 block discarded – undo
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
 	}
Please login to merge, or discard this patch.
tests/security/GroupTest.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -175,6 +175,10 @@
 block discarded – undo
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(
Please login to merge, or discard this patch.
tests/security/MemberTest.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -913,6 +913,9 @@
 block discarded – undo
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');
Please login to merge, or discard this patch.
tests/view/ContentNegotiatorTest.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -5,6 +5,7 @@
 block discarded – undo
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);
Please login to merge, or discard this patch.
tests/view/SSViewerCacheBlockTest.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -28,6 +28,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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);
Please login to merge, or discard this patch.
thirdparty/jsmin/jsmin.php 1 patch
Doc Comments   +11 added lines patch added patch discarded remove patch
@@ -82,6 +82,10 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
   }
Please login to merge, or discard this patch.
thirdparty/jstree/_demo/_inc/class._database_i.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -97,6 +97,9 @@
 block discarded – undo
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
 	}
Please login to merge, or discard this patch.
thirdparty/jstree/_demo/_inc/class.tree.php 1 patch
Doc Comments   +16 added lines patch added patch discarded remove patch
@@ -61,9 +61,17 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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);
Please login to merge, or discard this patch.