Completed
Pull Request — master (#605)
by Zack
05:45
created
includes/class-common.php 1 patch
Doc Comments   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -167,7 +167,6 @@  discard block
 block discarded – undo
167 167
 	 * Returns the list of available forms
168 168
 	 *
169 169
 	 * @access public
170
-	 * @param mixed $form_id
171 170
 	 * @return array Empty array if GFAPI isn't available or no forms. Otherwise, associative array with id, title keys
172 171
 	 */
173 172
 	public static function get_forms() {
@@ -188,7 +187,7 @@  discard block
 block discarded – undo
188 187
 	 * Return array of fields' id and label, for a given Form ID
189 188
 	 *
190 189
 	 * @access public
191
-	 * @param string|array $form_id (default: '') or $form object
190
+	 * @param string|array $form (default: '') or $form object
192 191
 	 * @return array
193 192
 	 */
194 193
 	public static function get_form_fields( $form = '', $add_default_properties = false, $include_parent_field = true ) {
Please login to merge, or discard this patch.
includes/fields/class-gravityview-field.php 1 patch
Doc Comments   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
 	 * @param bool $url_encode Whether to URL-encode output
176 176
 	 * @param bool $esc_html Whether to apply `esc_html()` to output
177 177
 	 *
178
-	 * @return mixed
178
+	 * @return string
179 179
 	 */
180 180
 	public function replace_merge_tag( $matches = array(), $text = '', $form = array(), $entry = array(), $url_encode = false, $esc_html = false ) {
181 181
 
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
 	 *
203 203
 	 * @since 1.8.4
204 204
 	 *
205
-	 * @param array $existing_merge_tags
205
+	 * @param array $custom_merge_tags
206 206
 	 * @param int $form_id GF Form ID
207 207
 	 * @param GF_Field[] $fields Array of fields in the form
208 208
 	 * @param string $element_id The ID of the input that Merge Tags are being used on
@@ -341,10 +341,10 @@  discard block
 block discarded – undo
341 341
 	 * </pre>
342 342
 	 *
343 343
 	 * @param  [type]      $field_options [description]
344
-	 * @param  [type]      $template_id   [description]
345
-	 * @param  [type]      $field_id      [description]
346
-	 * @param  [type]      $context       [description]
347
-	 * @param  [type]      $input_type    [description]
344
+	 * @param  string      $template_id   [description]
345
+	 * @param  string      $field_id      [description]
346
+	 * @param  string      $context       [description]
347
+	 * @param  string      $input_type    [description]
348 348
 	 * @return [type]                     [description]
349 349
 	 */
350 350
 	function field_options( $field_options, $template_id, $field_id, $context, $input_type ) {
Please login to merge, or discard this patch.
includes/fields/class-gravityview-fields.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -77,7 +77,7 @@
 block discarded – undo
77 77
 	/**
78 78
 	 * Alias for get_instance()
79 79
 	 *
80
-	 * @param $field_name
80
+	 * @param string $field_name
81 81
 	 *
82 82
 	 * @return GravityView_Field
83 83
 	 */
Please login to merge, or discard this patch.
includes/fields/class-gravityview-field-transaction-type.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@
 block discarded – undo
69 69
 	 *
70 70
 	 * @since 1.16
71 71
 	 *
72
-	 * @param int|string $value Number value for the field
72
+	 * @param string $value Number value for the field
73 73
 	 *
74 74
 	 * @return string Based on $value; `1`: "One-Time Payment"; `2`: "Subscription"
75 75
 	 */
Please login to merge, or discard this patch.
includes/helper-functions.php 1 patch
Doc Comments   +2 added lines, -3 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
  *
29 29
  * We could do this in a cleaner fashion, but this prevents a lot of code duplication, checking for URL structure, etc.
30 30
  *
31
- * @param int|WP_Post $id        Optional. Post ID or post object. Default current post.
31
+ * @param integer $id        Optional. Post ID or post object. Default current post.
32 32
  *
33 33
  * @return array URL args, if exists. Empty array if not.
34 34
  */
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
  * @since 1.15 Added $object param
136 136
  *
137 137
  * @param string $file_path Full path to a file
138
- * @param mixed $object Pass pseudo-global to the included file
138
+ * @param GravityView_Edit_Entry_Render $object Pass pseudo-global to the included file
139 139
  * @return string Included file contents
140 140
  */
141 141
 function gravityview_ob_include( $file_path, $object = NULL ) {
@@ -310,7 +310,6 @@  discard block
 block discarded – undo
310 310
  * Do a _very_ basic match for second-level TLD domains, like `.co.uk`
311 311
  *
312 312
  * Ideally, we'd use https://github.com/jeremykendall/php-domain-parser to check for this, but it's too much work for such a basic functionality. Maybe if it's needed more in the future. So instead, we use [Basic matching regex](http://stackoverflow.com/a/12372310).
313
- * @param  string $domain Domain to check if it's a TLD or subdomain
314 313
  * @return string         Extracted domain if it has a subdomain
315 314
  */
316 315
 function _gravityview_strip_subdomain( $string_maybe_has_subdomain ) {
Please login to merge, or discard this patch.
includes/admin/metaboxes/class-gravityview-admin-metaboxes.php 1 patch
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -291,7 +291,6 @@
 block discarded – undo
291 291
 	 * Render shortcode hint in the Publish metabox
292 292
 	 *
293 293
 	 * @access public
294
-	 * @param object $post
295 294
 	 * @return void
296 295
 	 */
297 296
 	function render_shortcode_hint() {
Please login to merge, or discard this patch.
includes/class-admin-approve-entries.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
 	 *
171 171
 	 * @uses  GravityView_frontend::get_search_criteria() Convert the $_POST search request into a properly formatted request.
172 172
 	 * @access public
173
-	 * @return void|boolean
173
+	 * @return null|false
174 174
 	 */
175 175
 	public function process_bulk_action() {
176 176
 		if ( ! class_exists( 'RGForms' ) ) {
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
 	 * @param array|boolean $entries If array, array of entry IDs that are to be updated. If true: update all entries.
253 253
 	 * @param int $approved Approved status. If `0`: unapproved, if not empty, `Approved`
254 254
 	 * @param int $form_id The Gravity Forms Form ID
255
-	 * @return boolean|void
255
+	 * @return false|null
256 256
 	 */
257 257
 	private static function update_bulk( $entries, $approved, $form_id ) {
258 258
 
@@ -470,7 +470,7 @@  discard block
 block discarded – undo
470 470
 	 * @access public
471 471
 	 * @static
472 472
 	 * @param mixed $form GF Form or Form ID
473
-	 * @return false|null|string Returns the input ID of the approved field. Returns NULL if no approved fields were found. Returns false if $form_id wasn't set.
473
+	 * @return integer Returns the input ID of the approved field. Returns NULL if no approved fields were found. Returns false if $form_id wasn't set.
474 474
 	 */
475 475
 	static public function get_approved_column( $form ) {
476 476
 
Please login to merge, or discard this patch.
includes/class-admin.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -311,7 +311,7 @@
 block discarded – undo
311 311
 	 * Remove any style or script non-registered in the no conflict mode
312 312
 	 * @todo  Move this to GravityView_Admin_Views
313 313
 	 * @param  WP_Dependencies $wp_objects        Object of WP_Styles or WP_Scripts
314
-	 * @param  array $required_objects   List of registered script/style handles
314
+	 * @param  string[] $required_objects   List of registered script/style handles
315 315
 	 * @param  string $type              Either 'styles' or 'scripts'
316 316
 	 * @return void
317 317
 	 */
Please login to merge, or discard this patch.
includes/class-data.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -113,6 +113,9 @@
 block discarded – undo
113 113
 		return ( sizeof( $ids ) === 1 ) ? $ids[0] : $ids;
114 114
 	}
115 115
 
116
+	/**
117
+	 * @return GravityView_View_Data
118
+	 */
116 119
 	public static function getInstance( $passed_post = NULL ) {
117 120
 
118 121
 		if( empty( self::$instance ) ) {
Please login to merge, or discard this patch.