@@ -59,7 +59,7 @@ discard block |
||
59 | 59 | public function get_content( $output, $entry = array(), $field_settings = array(), $field = array() ) { |
60 | 60 | |
61 | 61 | /** Overridden by a template. */ |
62 | - if( ! empty( $field['field_path'] ) ) { return $output; } |
|
62 | + if ( ! empty( $field[ 'field_path' ] ) ) { return $output; } |
|
63 | 63 | |
64 | 64 | return $this->get_string_from_value( $output ); |
65 | 65 | } |
@@ -78,11 +78,11 @@ discard block |
||
78 | 78 | switch ( intval( $value ) ) { |
79 | 79 | case self::ONE_TIME_PAYMENT: |
80 | 80 | default: |
81 | - $return = __('One-Time Payment', 'gravityview'); |
|
81 | + $return = __( 'One-Time Payment', 'gravityview' ); |
|
82 | 82 | break; |
83 | 83 | |
84 | 84 | case self::SUBSCRIPTION: |
85 | - $return = __('Subscription', 'gravityview'); |
|
85 | + $return = __( 'Subscription', 'gravityview' ); |
|
86 | 86 | break; |
87 | 87 | } |
88 | 88 |
@@ -72,7 +72,7 @@ |
||
72 | 72 | * |
73 | 73 | * @param mixed $value The value in. |
74 | 74 | * |
75 | - * @return mixed The value out. |
|
75 | + * @return string The value out. |
|
76 | 76 | */ |
77 | 77 | public function get_value( $value ) { |
78 | 78 | return $this->get_string_from_value( $value ); |
@@ -40,7 +40,6 @@ |
||
40 | 40 | * Get the default date format for a field based on the field ID and the time format setting |
41 | 41 | * |
42 | 42 | * @since 1.16.4 |
43 | - |
|
44 | 43 | * @param string $date_format The Gravity Forms date format for the field. Default: "mdy" |
45 | 44 | * @param int $field_id The ID of the field. Used to figure out full date/day/month/year |
46 | 45 | * |
@@ -31,11 +31,11 @@ discard block |
||
31 | 31 | |
32 | 32 | public function field_options( $field_options, $template_id, $field_id, $context, $input_type, $form_id ) { |
33 | 33 | |
34 | - if( 'edit' === $context ) { |
|
34 | + if ( 'edit' === $context ) { |
|
35 | 35 | return $field_options; |
36 | 36 | } |
37 | 37 | |
38 | - $this->add_field_support('date_display', $field_options ); |
|
38 | + $this->add_field_support( 'date_display', $field_options ); |
|
39 | 39 | |
40 | 40 | return $field_options; |
41 | 41 | } |
@@ -83,7 +83,7 @@ discard block |
||
83 | 83 | $field_input_id = gravityview_get_input_id_from_id( $field_id ); |
84 | 84 | |
85 | 85 | $date_field_output = ''; |
86 | - switch( $field_input_id ) { |
|
86 | + switch ( $field_input_id ) { |
|
87 | 87 | case 1: |
88 | 88 | $date_field_output = \GV\Utils::get( $parsed_date, 'month' ); |
89 | 89 | break; |
@@ -50,7 +50,7 @@ |
||
50 | 50 | public function edit_entry_fix_uid_fields( $fields ) { |
51 | 51 | |
52 | 52 | /** @var GF_Field $field */ |
53 | - foreach( $fields as &$field ) { |
|
53 | + foreach ( $fields as &$field ) { |
|
54 | 54 | if ( 'uid' === $field->type ) { |
55 | 55 | |
56 | 56 | // Replace GF_Field with GF_Field_Text, copying all the data from $field |
@@ -53,7 +53,7 @@ discard block |
||
53 | 53 | */ |
54 | 54 | $hide_coupon_fields = apply_filters( 'gravityview/edit_entry/hide-coupon-fields', $has_transaction_data ); |
55 | 55 | |
56 | - return (bool) $hide_coupon_fields; |
|
56 | + return (bool)$hide_coupon_fields; |
|
57 | 57 | } |
58 | 58 | |
59 | 59 | /** |
@@ -69,7 +69,7 @@ discard block |
||
69 | 69 | public function edit_entry_field_blacklist( $blacklist = array(), $entry = array() ) { |
70 | 70 | |
71 | 71 | if ( $this->should_hide_coupon_fields( $entry ) ) { |
72 | - $blacklist[] = 'coupon'; |
|
72 | + $blacklist[ ] = 'coupon'; |
|
73 | 73 | } |
74 | 74 | |
75 | 75 | return $blacklist; |
@@ -107,7 +107,7 @@ discard block |
||
107 | 107 | // No coupons match the codes provided |
108 | 108 | $discounts = gf_coupons()->get_coupons_by_codes( $coupon_codes, $form ); |
109 | 109 | |
110 | - if( ! $discounts ) { |
|
110 | + if ( ! $discounts ) { |
|
111 | 111 | return $value; |
112 | 112 | } |
113 | 113 | |
@@ -116,7 +116,7 @@ discard block |
||
116 | 116 | * @see GF_Field_Coupon::get_field_input |
117 | 117 | */ |
118 | 118 | $_POST = ! isset( $_POST ) ? array() : $_POST; |
119 | - $_POST[ 'gf_coupons_' . $form['id'] ] = json_encode( (array) $discounts ); |
|
119 | + $_POST[ 'gf_coupons_' . $form[ 'id' ] ] = json_encode( (array)$discounts ); |
|
120 | 120 | $_POST[ 'input_' . $field->id ] = implode( ',', $coupon_codes ); |
121 | 121 | |
122 | 122 | return $value; |
@@ -13,7 +13,7 @@ discard block |
||
13 | 13 | ?> |
14 | 14 | <label for="<?php echo $this->get_field_id(); ?>" class="<?php echo $this->get_label_class(); ?>"><?php |
15 | 15 | |
16 | - echo '<span class="gv-label">'.$this->get_field_label().'</span>'; |
|
16 | + echo '<span class="gv-label">' . $this->get_field_label() . '</span>'; |
|
17 | 17 | |
18 | 18 | echo $this->get_tooltip() . $this->get_field_desc(); |
19 | 19 | |
@@ -26,14 +26,14 @@ discard block |
||
26 | 26 | |
27 | 27 | function render_input( $override_input = null ) { |
28 | 28 | |
29 | - if( isset( $override_input ) ) { |
|
29 | + if ( isset( $override_input ) ) { |
|
30 | 30 | echo $override_input; |
31 | 31 | return; |
32 | 32 | } |
33 | 33 | |
34 | 34 | ?> |
35 | 35 | <select name="<?php echo esc_attr( $this->name ); ?>[]" id="<?php echo $this->get_field_id(); ?>" multiple="multiple"> |
36 | - <?php foreach( $this->field['options'] as $value => $label ) : ?> |
|
36 | + <?php foreach ( $this->field[ 'options' ] as $value => $label ) : ?> |
|
37 | 37 | <option value="<?php echo esc_attr( $value ); ?>" <?php selected( in_array( $value, (array)$this->value ), true, true ); ?>><?php echo esc_html( $label ); ?></option> |
38 | 38 | <?php endforeach; ?> |
39 | 39 | </select> |
@@ -324,7 +324,7 @@ discard block |
||
324 | 324 | } |
325 | 325 | |
326 | 326 | |
327 | - if( GFCommon::is_product_field( $field['type'] ) ){ |
|
327 | + if( GFCommon::is_product_field( $field['type'] ) ) { |
|
328 | 328 | $has_product_fields = true; |
329 | 329 | } |
330 | 330 | |
@@ -384,7 +384,7 @@ discard block |
||
384 | 384 | |
385 | 385 | $fields = array(); |
386 | 386 | |
387 | - foreach ( $extra_fields as $key => $field ){ |
|
387 | + foreach ( $extra_fields as $key => $field ) { |
|
388 | 388 | if ( ! empty( $only_default_column ) && ! empty( $field['is_default_column'] ) ) { |
389 | 389 | $fields[ $key ] = array( 'label' => $field['label'], 'type' => 'entry_meta' ); |
390 | 390 | } |
@@ -978,7 +978,7 @@ discard block |
||
978 | 978 | $form = GFAPI::get_form( $form ); |
979 | 979 | } |
980 | 980 | |
981 | - if ( class_exists( 'GFFormsModel' ) ){ |
|
981 | + if ( class_exists( 'GFFormsModel' ) ) { |
|
982 | 982 | return GFFormsModel::get_field( $form, $field_id ); |
983 | 983 | } else { |
984 | 984 | return null; |
@@ -1025,7 +1025,7 @@ discard block |
||
1025 | 1025 | $shortcodes = array(); |
1026 | 1026 | |
1027 | 1027 | preg_match_all( '/' . get_shortcode_regex() . '/s', $content, $matches, PREG_SET_ORDER ); |
1028 | - if ( empty( $matches ) ){ |
|
1028 | + if ( empty( $matches ) ) { |
|
1029 | 1029 | return false; |
1030 | 1030 | } |
1031 | 1031 |
@@ -270,7 +270,7 @@ discard block |
||
270 | 270 | * Return array of fields' id and label, for a given Form ID |
271 | 271 | * |
272 | 272 | * @access public |
273 | - * @param string|array $form_id (default: '') or $form object |
|
273 | + * @param string|array $form (default: '') or $form object |
|
274 | 274 | * @param bool $add_default_properties |
275 | 275 | * @param bool $include_parent_field |
276 | 276 | * @return array |
@@ -1012,7 +1012,7 @@ discard block |
||
1012 | 1012 | * |
1013 | 1013 | * @param array $form Gravity Forms form array |
1014 | 1014 | * @param string $field_id ID of the field. If an input, full input ID (like `1.3`) |
1015 | - * @param string|array $field_value Raw value of the field. |
|
1015 | + * @param string $field_value Raw value of the field. |
|
1016 | 1016 | * @return string |
1017 | 1017 | */ |
1018 | 1018 | public static function get_field_label( $form = array(), $field_id = '', $field_value = '' ) { |
@@ -1515,7 +1515,7 @@ discard block |
||
1515 | 1515 | * |
1516 | 1516 | * Do the same than parse_str without max_input_vars limitation: |
1517 | 1517 | * Parses $string as if it were the query string passed via a URL and sets variables in the current scope. |
1518 | - * @param $string string string to parse (not altered like in the original parse_str(), use the second parameter!) |
|
1518 | + * @param string $string string string to parse (not altered like in the original parse_str(), use the second parameter!) |
|
1519 | 1519 | * @param $result array If the second parameter is present, variables are stored in this variable as array elements |
1520 | 1520 | * @return bool true or false if $string is an empty string |
1521 | 1521 | * @since 1.5.3 |
@@ -1750,9 +1750,9 @@ discard block |
||
1750 | 1750 | * @param string $message Message body (required) |
1751 | 1751 | * @param string $from_name Displayed name of the sender |
1752 | 1752 | * @param string $message_format If "html", sent text as `text/html`. Otherwise, `text/plain`. Default: "html". |
1753 | - * @param string|array $attachments Optional. Files to attach. {@see wp_mail()} for usage. Default: "". |
|
1753 | + * @param string $attachments Optional. Files to attach. {@see wp_mail()} for usage. Default: "". |
|
1754 | 1754 | * @param array|false $entry Gravity Forms entry array, related to the email. Default: false. |
1755 | - * @param array|false $notification Gravity Forms notification that triggered the email. {@see GFCommon::send_notification}. Default:false. |
|
1755 | + * @param boolean $notification Gravity Forms notification that triggered the email. {@see GFCommon::send_notification}. Default:false. |
|
1756 | 1756 | */ |
1757 | 1757 | public static function send_email( $from, $to, $bcc, $reply_to, $subject, $message, $from_name = '', $message_format = 'html', $attachments = '', $entry = false, $notification = false ) { |
1758 | 1758 |
@@ -310,12 +310,12 @@ discard block |
||
310 | 310 | } |
311 | 311 | |
312 | 312 | /** |
313 | - * @hack |
|
314 | - * In case of email/email confirmation, the input for email has the same id as the parent field |
|
315 | - */ |
|
313 | + * @hack |
|
314 | + * In case of email/email confirmation, the input for email has the same id as the parent field |
|
315 | + */ |
|
316 | 316 | if( 'email' === $field['type'] && false === strpos( $input['id'], '.' ) ) { |
317 | - continue; |
|
318 | - } |
|
317 | + continue; |
|
318 | + } |
|
319 | 319 | $fields["{$input['id']}"] = array( |
320 | 320 | 'label' => \GV\Utils::get( $input, 'label' ), |
321 | 321 | 'customLabel' => \GV\Utils::get( $input, 'customLabel' ), |
@@ -1488,7 +1488,7 @@ discard block |
||
1488 | 1488 | ), |
1489 | 1489 | ); |
1490 | 1490 | |
1491 | - $fields = $date_created + $fields; |
|
1491 | + $fields = $date_created + $fields; |
|
1492 | 1492 | |
1493 | 1493 | $blacklist_field_types = apply_filters( 'gravityview_blacklist_field_types', $blacklist, NULL ); |
1494 | 1494 | |
@@ -1516,13 +1516,13 @@ discard block |
||
1516 | 1516 | |
1517 | 1517 | } |
1518 | 1518 | |
1519 | - /** |
|
1520 | - * @filter `gravityview/common/sortable_fields` Filter the sortable fields |
|
1521 | - * @since 1.12 |
|
1522 | - * @param array $fields Sub-set of GF form fields that are sortable |
|
1523 | - * @param int $formid The Gravity Forms form ID that the fields are from |
|
1524 | - */ |
|
1525 | - $fields = apply_filters( 'gravityview/common/sortable_fields', $fields, $formid ); |
|
1519 | + /** |
|
1520 | + * @filter `gravityview/common/sortable_fields` Filter the sortable fields |
|
1521 | + * @since 1.12 |
|
1522 | + * @param array $fields Sub-set of GF form fields that are sortable |
|
1523 | + * @param int $formid The Gravity Forms form ID that the fields are from |
|
1524 | + */ |
|
1525 | + $fields = apply_filters( 'gravityview/common/sortable_fields', $fields, $formid ); |
|
1526 | 1526 | |
1527 | 1527 | return $fields; |
1528 | 1528 | } |
@@ -1814,26 +1814,26 @@ discard block |
||
1814 | 1814 | } |
1815 | 1815 | |
1816 | 1816 | |
1817 | - /** |
|
1818 | - * Display updated/error notice |
|
1819 | - * |
|
1820 | - * @since 1.19.2 Added $cap and $object_id parameters |
|
1821 | - * |
|
1822 | - * @param string $notice text/HTML of notice |
|
1823 | - * @param string $class CSS class for notice (`updated` or `error`) |
|
1824 | - * @param string $cap [Optional] Define a capability required to show a notice. If not set, displays to all caps. |
|
1825 | - * |
|
1826 | - * @return string |
|
1827 | - */ |
|
1828 | - public static function generate_notice( $notice, $class = '', $cap = '', $object_id = null ) { |
|
1829 | - |
|
1830 | - // If $cap is defined, only show notice if user has capability |
|
1831 | - if( $cap && ! GVCommon::has_cap( $cap, $object_id ) ) { |
|
1832 | - return ''; |
|
1833 | - } |
|
1834 | - |
|
1835 | - return '<div class="gv-notice '.gravityview_sanitize_html_class( $class ) .'">'. $notice .'</div>'; |
|
1836 | - } |
|
1817 | + /** |
|
1818 | + * Display updated/error notice |
|
1819 | + * |
|
1820 | + * @since 1.19.2 Added $cap and $object_id parameters |
|
1821 | + * |
|
1822 | + * @param string $notice text/HTML of notice |
|
1823 | + * @param string $class CSS class for notice (`updated` or `error`) |
|
1824 | + * @param string $cap [Optional] Define a capability required to show a notice. If not set, displays to all caps. |
|
1825 | + * |
|
1826 | + * @return string |
|
1827 | + */ |
|
1828 | + public static function generate_notice( $notice, $class = '', $cap = '', $object_id = null ) { |
|
1829 | + |
|
1830 | + // If $cap is defined, only show notice if user has capability |
|
1831 | + if( $cap && ! GVCommon::has_cap( $cap, $object_id ) ) { |
|
1832 | + return ''; |
|
1833 | + } |
|
1834 | + |
|
1835 | + return '<div class="gv-notice '.gravityview_sanitize_html_class( $class ) .'">'. $notice .'</div>'; |
|
1836 | + } |
|
1837 | 1837 | |
1838 | 1838 | /** |
1839 | 1839 | * Inspired on \GFCommon::encode_shortcodes, reverse the encoding by replacing the ascii characters by the shortcode brackets |
@@ -125,8 +125,8 @@ discard block |
||
125 | 125 | |
126 | 126 | $form = false; |
127 | 127 | |
128 | - if( $entry ) { |
|
129 | - $form = GFAPI::get_form( $entry['form_id'] ); |
|
128 | + if ( $entry ) { |
|
129 | + $form = GFAPI::get_form( $entry[ 'form_id' ] ); |
|
130 | 130 | } |
131 | 131 | |
132 | 132 | return $form; |
@@ -202,7 +202,7 @@ discard block |
||
202 | 202 | } |
203 | 203 | } |
204 | 204 | |
205 | - return (bool) $has_transaction_data; |
|
205 | + return (bool)$has_transaction_data; |
|
206 | 206 | } |
207 | 207 | |
208 | 208 | /** |
@@ -240,7 +240,7 @@ discard block |
||
240 | 240 | |
241 | 241 | $results = GFAPI::get_entries( intval( $form_id ), $search_criteria, null, $paging ); |
242 | 242 | |
243 | - $result = ( ! empty( $results ) && ! empty( $results[0]['id'] ) ) ? $results[0]['id'] : null; |
|
243 | + $result = ( ! empty( $results ) && ! empty( $results[ 0 ][ 'id' ] ) ) ? $results[ 0 ][ 'id' ] : null; |
|
244 | 244 | |
245 | 245 | return $result; |
246 | 246 | } |
@@ -257,10 +257,10 @@ discard block |
||
257 | 257 | * |
258 | 258 | * @return array Empty array if GFAPI class isn't available or no forms. Otherwise, the array of Forms |
259 | 259 | */ |
260 | - public static function get_forms( $active = true, $trash = false ) { |
|
260 | + public static function get_forms( $active = true, $trash = false ) { |
|
261 | 261 | $forms = array(); |
262 | 262 | if ( class_exists( 'GFAPI' ) ) { |
263 | - if( 'any' === $active ) { |
|
263 | + if ( 'any' === $active ) { |
|
264 | 264 | $active_forms = GFAPI::get_forms( true, $trash ); |
265 | 265 | $inactive_forms = GFAPI::get_forms( false, $trash ); |
266 | 266 | $forms = array_merge( array_filter( $active_forms ), array_filter( $inactive_forms ) ); |
@@ -291,9 +291,9 @@ discard block |
||
291 | 291 | $has_post_fields = false; |
292 | 292 | |
293 | 293 | if ( $form ) { |
294 | - foreach ( $form['fields'] as $field ) { |
|
295 | - if ( $include_parent_field || empty( $field['inputs'] ) ) { |
|
296 | - $fields["{$field['id']}"] = array( |
|
294 | + foreach ( $form[ 'fields' ] as $field ) { |
|
295 | + if ( $include_parent_field || empty( $field[ 'inputs' ] ) ) { |
|
296 | + $fields[ "{$field[ 'id' ]}" ] = array( |
|
297 | 297 | 'label' => \GV\Utils::get( $field, 'label' ), |
298 | 298 | 'parent' => null, |
299 | 299 | 'type' => \GV\Utils::get( $field, 'type' ), |
@@ -302,10 +302,10 @@ discard block |
||
302 | 302 | ); |
303 | 303 | } |
304 | 304 | |
305 | - if ( $add_default_properties && ! empty( $field['inputs'] ) ) { |
|
306 | - foreach ( $field['inputs'] as $input ) { |
|
305 | + if ( $add_default_properties && ! empty( $field[ 'inputs' ] ) ) { |
|
306 | + foreach ( $field[ 'inputs' ] as $input ) { |
|
307 | 307 | |
308 | - if( ! empty( $input['isHidden'] ) ) { |
|
308 | + if ( ! empty( $input[ 'isHidden' ] ) ) { |
|
309 | 309 | continue; |
310 | 310 | } |
311 | 311 | |
@@ -313,10 +313,10 @@ discard block |
||
313 | 313 | * @hack |
314 | 314 | * In case of email/email confirmation, the input for email has the same id as the parent field |
315 | 315 | */ |
316 | - if( 'email' === $field['type'] && false === strpos( $input['id'], '.' ) ) { |
|
316 | + if ( 'email' === $field[ 'type' ] && false === strpos( $input[ 'id' ], '.' ) ) { |
|
317 | 317 | continue; |
318 | 318 | } |
319 | - $fields["{$input['id']}"] = array( |
|
319 | + $fields[ "{$input[ 'id' ]}" ] = array( |
|
320 | 320 | 'label' => \GV\Utils::get( $input, 'label' ), |
321 | 321 | 'customLabel' => \GV\Utils::get( $input, 'customLabel' ), |
322 | 322 | 'parent' => $field, |
@@ -328,7 +328,7 @@ discard block |
||
328 | 328 | } |
329 | 329 | |
330 | 330 | |
331 | - if( GFCommon::is_product_field( $field['type'] ) ){ |
|
331 | + if ( GFCommon::is_product_field( $field[ 'type' ] ) ) { |
|
332 | 332 | $has_product_fields = true; |
333 | 333 | } |
334 | 334 | |
@@ -342,7 +342,7 @@ discard block |
||
342 | 342 | * @since 1.7 |
343 | 343 | */ |
344 | 344 | if ( $has_post_fields ) { |
345 | - $fields['post_id'] = array( |
|
345 | + $fields[ 'post_id' ] = array( |
|
346 | 346 | 'label' => __( 'Post ID', 'gravityview' ), |
347 | 347 | 'type' => 'post_id', |
348 | 348 | ); |
@@ -355,11 +355,11 @@ discard block |
||
355 | 355 | foreach ( $payment_fields as $payment_field ) { |
356 | 356 | |
357 | 357 | // Either the field exists ($fields['shipping']) or the form explicitly contains a `shipping` field with numeric key |
358 | - if( isset( $fields["{$payment_field->name}"] ) || GFCommon::get_fields_by_type( $form, $payment_field->name ) ) { |
|
358 | + if ( isset( $fields[ "{$payment_field->name}" ] ) || GFCommon::get_fields_by_type( $form, $payment_field->name ) ) { |
|
359 | 359 | continue; |
360 | 360 | } |
361 | 361 | |
362 | - $fields["{$payment_field->name}"] = array( |
|
362 | + $fields[ "{$payment_field->name}" ] = array( |
|
363 | 363 | 'label' => $payment_field->label, |
364 | 364 | 'desc' => $payment_field->description, |
365 | 365 | 'type' => $payment_field->name, |
@@ -391,9 +391,9 @@ discard block |
||
391 | 391 | |
392 | 392 | $fields = array(); |
393 | 393 | |
394 | - foreach ( $extra_fields as $key => $field ){ |
|
395 | - if ( ! empty( $only_default_column ) && ! empty( $field['is_default_column'] ) ) { |
|
396 | - $fields[ $key ] = array( 'label' => $field['label'], 'type' => 'entry_meta' ); |
|
394 | + foreach ( $extra_fields as $key => $field ) { |
|
395 | + if ( ! empty( $only_default_column ) && ! empty( $field[ 'is_default_column' ] ) ) { |
|
396 | + $fields[ $key ] = array( 'label' => $field[ 'label' ], 'type' => 'entry_meta' ); |
|
397 | 397 | } |
398 | 398 | } |
399 | 399 | |
@@ -433,33 +433,33 @@ discard block |
||
433 | 433 | 'search_criteria' => null, |
434 | 434 | 'sorting' => null, |
435 | 435 | 'paging' => null, |
436 | - 'cache' => (isset( $passed_criteria['cache'] ) ? (bool) $passed_criteria['cache'] : true), |
|
436 | + 'cache' => ( isset( $passed_criteria[ 'cache' ] ) ? (bool)$passed_criteria[ 'cache' ] : true ), |
|
437 | 437 | 'context_view_id' => null, |
438 | 438 | ); |
439 | 439 | |
440 | 440 | $criteria = wp_parse_args( $passed_criteria, $search_criteria_defaults ); |
441 | 441 | |
442 | - if ( ! empty( $criteria['search_criteria']['field_filters'] ) ) { |
|
443 | - foreach ( $criteria['search_criteria']['field_filters'] as &$filter ) { |
|
442 | + if ( ! empty( $criteria[ 'search_criteria' ][ 'field_filters' ] ) ) { |
|
443 | + foreach ( $criteria[ 'search_criteria' ][ 'field_filters' ] as &$filter ) { |
|
444 | 444 | |
445 | 445 | if ( ! is_array( $filter ) ) { |
446 | 446 | continue; |
447 | 447 | } |
448 | 448 | |
449 | 449 | // By default, we want searches to be wildcard for each field. |
450 | - $filter['operator'] = empty( $filter['operator'] ) ? 'contains' : $filter['operator']; |
|
450 | + $filter[ 'operator' ] = empty( $filter[ 'operator' ] ) ? 'contains' : $filter[ 'operator' ]; |
|
451 | 451 | |
452 | 452 | /** |
453 | 453 | * @filter `gravityview_search_operator` Modify the search operator for the field (contains, is, isnot, etc) |
454 | 454 | * @param string $operator Existing search operator |
455 | 455 | * @param array $filter array with `key`, `value`, `operator`, `type` keys |
456 | 456 | */ |
457 | - $filter['operator'] = apply_filters( 'gravityview_search_operator', $filter['operator'], $filter ); |
|
457 | + $filter[ 'operator' ] = apply_filters( 'gravityview_search_operator', $filter[ 'operator' ], $filter ); |
|
458 | 458 | } |
459 | 459 | |
460 | 460 | // don't send just the [mode] without any field filter. |
461 | - if( count( $criteria['search_criteria']['field_filters'] ) === 1 && array_key_exists( 'mode' , $criteria['search_criteria']['field_filters'] ) ) { |
|
462 | - unset( $criteria['search_criteria']['field_filters']['mode'] ); |
|
461 | + if ( count( $criteria[ 'search_criteria' ][ 'field_filters' ] ) === 1 && array_key_exists( 'mode', $criteria[ 'search_criteria' ][ 'field_filters' ] ) ) { |
|
462 | + unset( $criteria[ 'search_criteria' ][ 'field_filters' ][ 'mode' ] ); |
|
463 | 463 | } |
464 | 464 | |
465 | 465 | } |
@@ -470,33 +470,33 @@ discard block |
||
470 | 470 | * Prepare date formats to be in Gravity Forms DB format; |
471 | 471 | * $passed_criteria may include date formats incompatible with Gravity Forms. |
472 | 472 | */ |
473 | - foreach ( array('start_date', 'end_date' ) as $key ) { |
|
473 | + foreach ( array( 'start_date', 'end_date' ) as $key ) { |
|
474 | 474 | |
475 | - if ( ! empty( $criteria['search_criteria'][ $key ] ) ) { |
|
475 | + if ( ! empty( $criteria[ 'search_criteria' ][ $key ] ) ) { |
|
476 | 476 | |
477 | 477 | // Use date_create instead of new DateTime so it returns false if invalid date format. |
478 | - $date = date_create( $criteria['search_criteria'][ $key ] ); |
|
478 | + $date = date_create( $criteria[ 'search_criteria' ][ $key ] ); |
|
479 | 479 | |
480 | 480 | if ( $date ) { |
481 | 481 | // Gravity Forms wants dates in the `Y-m-d H:i:s` format. |
482 | - $criteria['search_criteria'][ $key ] = $date->format( 'Y-m-d H:i:s' ); |
|
482 | + $criteria[ 'search_criteria' ][ $key ] = $date->format( 'Y-m-d H:i:s' ); |
|
483 | 483 | } else { |
484 | - gravityview()->log->error( '{key} Date format not valid:', array( 'key' => $key, $criteria['search_criteria'][ $key ] ) ); |
|
484 | + gravityview()->log->error( '{key} Date format not valid:', array( 'key' => $key, $criteria[ 'search_criteria' ][ $key ] ) ); |
|
485 | 485 | |
486 | 486 | // If it's an invalid date, unset it. Gravity Forms freaks out otherwise. |
487 | - unset( $criteria['search_criteria'][ $key ] ); |
|
487 | + unset( $criteria[ 'search_criteria' ][ $key ] ); |
|
488 | 488 | } |
489 | 489 | } |
490 | 490 | } |
491 | 491 | |
492 | - if ( empty( $criteria['context_view_id'] ) ) { |
|
492 | + if ( empty( $criteria[ 'context_view_id' ] ) ) { |
|
493 | 493 | // Calculate the context view id and send it to the advanced filter |
494 | 494 | if ( GravityView_frontend::getInstance()->getSingleEntry() ) { |
495 | - $criteria['context_view_id'] = GravityView_frontend::getInstance()->get_context_view_id(); |
|
495 | + $criteria[ 'context_view_id' ] = GravityView_frontend::getInstance()->get_context_view_id(); |
|
496 | 496 | } else if ( class_exists( 'GravityView_View_Data' ) && GravityView_View_Data::getInstance() && GravityView_View_Data::getInstance()->has_multiple_views() ) { |
497 | - $criteria['context_view_id'] = GravityView_frontend::getInstance()->get_context_view_id(); |
|
497 | + $criteria[ 'context_view_id' ] = GravityView_frontend::getInstance()->get_context_view_id(); |
|
498 | 498 | } else if ( 'delete' === GFForms::get( 'action' ) ) { |
499 | - $criteria['context_view_id'] = isset( $_GET['view_id'] ) ? intval( $_GET['view_id'] ) : null; |
|
499 | + $criteria[ 'context_view_id' ] = isset( $_GET[ 'view_id' ] ) ? intval( $_GET[ 'view_id' ] ) : null; |
|
500 | 500 | } |
501 | 501 | } |
502 | 502 | |
@@ -506,7 +506,7 @@ discard block |
||
506 | 506 | * @param array $form_ids Forms to search |
507 | 507 | * @param int $view_id ID of the view being used to search |
508 | 508 | */ |
509 | - $criteria = apply_filters( 'gravityview_search_criteria', $criteria, $form_ids, $criteria['context_view_id'] ); |
|
509 | + $criteria = apply_filters( 'gravityview_search_criteria', $criteria, $form_ids, $criteria[ 'context_view_id' ] ); |
|
510 | 510 | |
511 | 511 | return (array)$criteria; |
512 | 512 | } |
@@ -541,7 +541,7 @@ discard block |
||
541 | 541 | /** Reduce # of database calls */ |
542 | 542 | add_filter( 'gform_is_encrypted_field', '__return_false' ); |
543 | 543 | |
544 | - if ( ! empty( $criteria['cache'] ) ) { |
|
544 | + if ( ! empty( $criteria[ 'cache' ] ) ) { |
|
545 | 545 | |
546 | 546 | $Cache = new GravityView_Cache( $form_ids, $criteria ); |
547 | 547 | |
@@ -549,7 +549,7 @@ discard block |
||
549 | 549 | |
550 | 550 | // Still update the total count when using cached results |
551 | 551 | if ( ! is_null( $total ) ) { |
552 | - $total = GFAPI::count_entries( $form_ids, $criteria['search_criteria'] ); |
|
552 | + $total = GFAPI::count_entries( $form_ids, $criteria[ 'search_criteria' ] ); |
|
553 | 553 | } |
554 | 554 | |
555 | 555 | $return = $entries; |
@@ -571,9 +571,9 @@ discard block |
||
571 | 571 | $entries = apply_filters_ref_array( 'gravityview_before_get_entries', array( null, $criteria, $passed_criteria, &$total ) ); |
572 | 572 | |
573 | 573 | // No entries returned from gravityview_before_get_entries |
574 | - if( is_null( $entries ) ) { |
|
574 | + if ( is_null( $entries ) ) { |
|
575 | 575 | |
576 | - $entries = GFAPI::get_entries( $form_ids, $criteria['search_criteria'], $criteria['sorting'], $criteria['paging'], $total ); |
|
576 | + $entries = GFAPI::get_entries( $form_ids, $criteria[ 'search_criteria' ], $criteria[ 'sorting' ], $criteria[ 'paging' ], $total ); |
|
577 | 577 | |
578 | 578 | if ( is_wp_error( $entries ) ) { |
579 | 579 | gravityview()->log->error( '{error}', array( 'error' => $entries->get_error_message(), 'data' => $entries ) ); |
@@ -584,7 +584,7 @@ discard block |
||
584 | 584 | } |
585 | 585 | } |
586 | 586 | |
587 | - if ( ! empty( $criteria['cache'] ) && isset( $Cache ) ) { |
|
587 | + if ( ! empty( $criteria[ 'cache' ] ) && isset( $Cache ) ) { |
|
588 | 588 | |
589 | 589 | // Cache results |
590 | 590 | $Cache->set( $entries, 'entries' ); |
@@ -693,12 +693,12 @@ discard block |
||
693 | 693 | */ |
694 | 694 | $check_entry_display = apply_filters( 'gravityview/common/get_entry/check_entry_display', $check_entry_display, $entry ); |
695 | 695 | |
696 | - if( $check_entry_display ) { |
|
696 | + if ( $check_entry_display ) { |
|
697 | 697 | // Is the entry allowed |
698 | 698 | $entry = self::check_entry_display( $entry ); |
699 | 699 | } |
700 | 700 | |
701 | - if( is_wp_error( $entry ) ) { |
|
701 | + if ( is_wp_error( $entry ) ) { |
|
702 | 702 | gravityview()->log->error( '{error}', array( 'error' => $entry->get_error_message() ) ); |
703 | 703 | return false; |
704 | 704 | } |
@@ -732,12 +732,12 @@ discard block |
||
732 | 732 | |
733 | 733 | $value = false; |
734 | 734 | |
735 | - if( 'context' === $val1 ) { |
|
735 | + if ( 'context' === $val1 ) { |
|
736 | 736 | |
737 | 737 | $matching_contexts = array( $val2 ); |
738 | 738 | |
739 | 739 | // We allow for non-standard contexts. |
740 | - switch( $val2 ) { |
|
740 | + switch ( $val2 ) { |
|
741 | 741 | // Check for either single or edit |
742 | 742 | case 'singular': |
743 | 743 | $matching_contexts = array( 'single', 'edit' ); |
@@ -780,18 +780,18 @@ discard block |
||
780 | 780 | $json_val_1 = json_decode( $val1, true ); |
781 | 781 | $json_val_2 = json_decode( $val2, true ); |
782 | 782 | |
783 | - if( ! empty( $json_val_1 ) || ! empty( $json_val_2 ) ) { |
|
783 | + if ( ! empty( $json_val_1 ) || ! empty( $json_val_2 ) ) { |
|
784 | 784 | |
785 | 785 | $json_in = false; |
786 | - $json_val_1 = $json_val_1 ? (array) $json_val_1 : array( $val1 ); |
|
787 | - $json_val_2 = $json_val_2 ? (array) $json_val_2 : array( $val2 ); |
|
786 | + $json_val_1 = $json_val_1 ? (array)$json_val_1 : array( $val1 ); |
|
787 | + $json_val_2 = $json_val_2 ? (array)$json_val_2 : array( $val2 ); |
|
788 | 788 | |
789 | 789 | // For JSON, we want to compare as "in" or "not in" rather than "contains" |
790 | 790 | foreach ( $json_val_1 as $item_1 ) { |
791 | 791 | foreach ( $json_val_2 as $item_2 ) { |
792 | 792 | $json_in = self::matches_operation( $item_1, $item_2, 'is' ); |
793 | 793 | |
794 | - if( $json_in ) { |
|
794 | + if ( $json_in ) { |
|
795 | 795 | break 2; |
796 | 796 | } |
797 | 797 | } |
@@ -843,10 +843,10 @@ discard block |
||
843 | 843 | public static function check_entry_display( $entry, $view = null ) { |
844 | 844 | |
845 | 845 | if ( ! $entry || is_wp_error( $entry ) ) { |
846 | - return new WP_Error('entry_not_found', 'Entry was not found.', $entry ); |
|
846 | + return new WP_Error( 'entry_not_found', 'Entry was not found.', $entry ); |
|
847 | 847 | } |
848 | 848 | |
849 | - if ( empty( $entry['form_id'] ) ) { |
|
849 | + if ( empty( $entry[ 'form_id' ] ) ) { |
|
850 | 850 | return new WP_Error( 'form_id_not_set', '[apply_filters_to_entry] Entry is empty!', $entry ); |
851 | 851 | } |
852 | 852 | |
@@ -854,8 +854,8 @@ discard block |
||
854 | 854 | $view_form_id = $view->form->ID; |
855 | 855 | |
856 | 856 | if ( $view->joins ) { |
857 | - if ( in_array( (int)$entry['form_id'], array_keys( $view::get_joined_forms( $view->ID ) ), true ) ) { |
|
858 | - $view_form_id = $entry['form_id']; |
|
857 | + if ( in_array( (int)$entry[ 'form_id' ], array_keys( $view::get_joined_forms( $view->ID ) ), true ) ) { |
|
858 | + $view_form_id = $entry[ 'form_id' ]; |
|
859 | 859 | } |
860 | 860 | } |
861 | 861 | |
@@ -863,13 +863,13 @@ discard block |
||
863 | 863 | * Check whether the entry is in the entries subset by running a modified query. |
864 | 864 | */ |
865 | 865 | add_action( 'gravityview/view/query', $entry_subset_callback = function( &$query, $view, $request ) use ( $entry, $view_form_id ) { |
866 | - $_tmp_query = new \GF_Query( $view_form_id, array( |
|
866 | + $_tmp_query = new \GF_Query( $view_form_id, array( |
|
867 | 867 | 'field_filters' => array( |
868 | 868 | 'mode' => 'all', |
869 | 869 | array( |
870 | 870 | 'key' => 'id', |
871 | 871 | 'operation' => 'is', |
872 | - 'value' => $entry['id'] |
|
872 | + 'value' => $entry[ 'id' ] |
|
873 | 873 | ) |
874 | 874 | ) |
875 | 875 | ) ); |
@@ -879,14 +879,14 @@ discard block |
||
879 | 879 | /** @var \GF_Query $query */ |
880 | 880 | $query_parts = $query->_introspect(); |
881 | 881 | |
882 | - $query->where( \GF_Query_Condition::_and( $_tmp_query_parts['where'], $query_parts['where'] ) ); |
|
882 | + $query->where( \GF_Query_Condition::_and( $_tmp_query_parts[ 'where' ], $query_parts[ 'where' ] ) ); |
|
883 | 883 | |
884 | 884 | }, 10, 3 ); |
885 | 885 | |
886 | 886 | // Prevent page offset from being applied to the single entry query; it's used to return to the referring page number |
887 | 887 | add_filter( 'gravityview_search_criteria', $remove_pagenum = function( $criteria ) { |
888 | 888 | |
889 | - $criteria['paging'] = array( |
|
889 | + $criteria[ 'paging' ] = array( |
|
890 | 890 | 'offset' => 0, |
891 | 891 | 'page_size' => 25 |
892 | 892 | ); |
@@ -905,20 +905,20 @@ discard block |
||
905 | 905 | } |
906 | 906 | |
907 | 907 | // This entry is on a View with joins |
908 | - if ( $entries[0]->is_multi() ) { |
|
908 | + if ( $entries[ 0 ]->is_multi() ) { |
|
909 | 909 | |
910 | 910 | $multi_entry_ids = array(); |
911 | 911 | |
912 | - foreach ( $entries[0]->entries as $multi_entry ) { |
|
913 | - $multi_entry_ids[] = (int) $multi_entry->ID; |
|
912 | + foreach ( $entries[ 0 ]->entries as $multi_entry ) { |
|
913 | + $multi_entry_ids[ ] = (int)$multi_entry->ID; |
|
914 | 914 | } |
915 | 915 | |
916 | - if ( ! in_array( (int) $entry['id'], $multi_entry_ids, true ) ) { |
|
916 | + if ( ! in_array( (int)$entry[ 'id' ], $multi_entry_ids, true ) ) { |
|
917 | 917 | remove_action( 'gravityview/view/query', $entry_subset_callback ); |
918 | 918 | return new \WP_Error( 'failed_criteria', 'Entry failed search_criteria and field_filters' ); |
919 | 919 | } |
920 | 920 | |
921 | - } elseif ( (int) $entries[0]->ID !== (int) $entry['id'] ) { |
|
921 | + } elseif ( (int)$entries[ 0 ]->ID !== (int)$entry[ 'id' ] ) { |
|
922 | 922 | remove_action( 'gravityview/view/query', $entry_subset_callback ); |
923 | 923 | return new \WP_Error( 'failed_criteria', 'Entry failed search_criteria and field_filters' ); |
924 | 924 | } |
@@ -931,59 +931,59 @@ discard block |
||
931 | 931 | 'context_view_id' => $view ? $view->ID : null, |
932 | 932 | ) ); |
933 | 933 | |
934 | - if ( empty( $criteria['search_criteria'] ) || ! is_array( $criteria['search_criteria'] ) ) { |
|
934 | + if ( empty( $criteria[ 'search_criteria' ] ) || ! is_array( $criteria[ 'search_criteria' ] ) ) { |
|
935 | 935 | gravityview()->log->debug( '[apply_filters_to_entry] Entry approved! No search criteria found:', array( 'data' => $criteria ) ); |
936 | 936 | return $entry; |
937 | 937 | } |
938 | 938 | |
939 | 939 | // Make sure the current View is connected to the same form as the Entry |
940 | - if( ! empty( $criteria['context_view_id'] ) ) { |
|
941 | - $context_view_id = intval( $criteria['context_view_id'] ); |
|
940 | + if ( ! empty( $criteria[ 'context_view_id' ] ) ) { |
|
941 | + $context_view_id = intval( $criteria[ 'context_view_id' ] ); |
|
942 | 942 | $context_form_id = gravityview_get_form_id( $context_view_id ); |
943 | - if( intval( $context_form_id ) !== intval( $entry['form_id'] ) ) { |
|
944 | - return new WP_Error( 'view_id_not_match', sprintf( '[apply_filters_to_entry] Entry form ID does not match current View connected form ID:', $entry['form_id'] ), $criteria['context_view_id'] ); |
|
943 | + if ( intval( $context_form_id ) !== intval( $entry[ 'form_id' ] ) ) { |
|
944 | + return new WP_Error( 'view_id_not_match', sprintf( '[apply_filters_to_entry] Entry form ID does not match current View connected form ID:', $entry[ 'form_id' ] ), $criteria[ 'context_view_id' ] ); |
|
945 | 945 | } |
946 | 946 | } |
947 | 947 | |
948 | - $search_criteria = $criteria['search_criteria']; |
|
948 | + $search_criteria = $criteria[ 'search_criteria' ]; |
|
949 | 949 | |
950 | 950 | // check entry status |
951 | - if ( array_key_exists( 'status', $search_criteria ) && $search_criteria['status'] != $entry['status'] ) { |
|
952 | - return new WP_Error( 'status_not_valid', sprintf( '[apply_filters_to_entry] Entry status - %s - is not valid according to filter:', $entry['status'] ), $search_criteria ); |
|
951 | + if ( array_key_exists( 'status', $search_criteria ) && $search_criteria[ 'status' ] != $entry[ 'status' ] ) { |
|
952 | + return new WP_Error( 'status_not_valid', sprintf( '[apply_filters_to_entry] Entry status - %s - is not valid according to filter:', $entry[ 'status' ] ), $search_criteria ); |
|
953 | 953 | } |
954 | 954 | |
955 | 955 | // check entry date |
956 | 956 | // @todo: Does it make sense to apply the Date create filters to the single entry? |
957 | 957 | |
958 | 958 | // field_filters |
959 | - if ( empty( $search_criteria['field_filters'] ) || ! is_array( $search_criteria['field_filters'] ) ) { |
|
959 | + if ( empty( $search_criteria[ 'field_filters' ] ) || ! is_array( $search_criteria[ 'field_filters' ] ) ) { |
|
960 | 960 | gravityview()->log->debug( '[apply_filters_to_entry] Entry approved! No field filters criteria found:', array( 'data' => $search_criteria ) ); |
961 | 961 | return $entry; |
962 | 962 | } |
963 | 963 | |
964 | - $filters = $search_criteria['field_filters']; |
|
964 | + $filters = $search_criteria[ 'field_filters' ]; |
|
965 | 965 | |
966 | - $mode = array_key_exists( 'mode', $filters ) ? strtolower( $filters['mode'] ) : 'all'; |
|
966 | + $mode = array_key_exists( 'mode', $filters ) ? strtolower( $filters[ 'mode' ] ) : 'all'; |
|
967 | 967 | |
968 | - $mode = $mode ? : 'all'; // If mode is an empty string, assume it's 'all' |
|
968 | + $mode = $mode ?: 'all'; // If mode is an empty string, assume it's 'all' |
|
969 | 969 | |
970 | 970 | // Prevent the mode from being processed below |
971 | - unset( $filters['mode'] ); |
|
971 | + unset( $filters[ 'mode' ] ); |
|
972 | 972 | |
973 | - $form = self::get_form( $entry['form_id'] ); |
|
973 | + $form = self::get_form( $entry[ 'form_id' ] ); |
|
974 | 974 | |
975 | 975 | foreach ( $filters as $filter ) { |
976 | - $operator = isset( $filter['operator'] ) ? strtolower( $filter['operator'] ) : 'is'; |
|
976 | + $operator = isset( $filter[ 'operator' ] ) ? strtolower( $filter[ 'operator' ] ) : 'is'; |
|
977 | 977 | |
978 | - if ( ! isset( $filter['key'] ) ) { |
|
978 | + if ( ! isset( $filter[ 'key' ] ) ) { |
|
979 | 979 | gravityview()->log->debug( '[apply_filters_to_entry] Filter key not set, any field mode', array( 'filter' => $filter ) ); |
980 | 980 | /** |
981 | 981 | * This is a cross-field search. Let's start digging'. |
982 | 982 | */ |
983 | 983 | foreach ( \GV\Utils::get( $form, 'fields', array() ) as $field ) { |
984 | 984 | $field_value = GFFormsModel::get_lead_field_value( $entry, $field ); |
985 | - if ( $is_value_match = GravityView_GFFormsModel::is_value_match( $field_value, $filter['value'], $operator, $field ) ) { |
|
986 | - if ( 'any' === $mode) { |
|
985 | + if ( $is_value_match = GravityView_GFFormsModel::is_value_match( $field_value, $filter[ 'value' ], $operator, $field ) ) { |
|
986 | + if ( 'any' === $mode ) { |
|
987 | 987 | return $entry; // All good here |
988 | 988 | } // mode === 'all' |
989 | 989 | continue 2; // Next filter |
@@ -992,13 +992,13 @@ discard block |
||
992 | 992 | } |
993 | 993 | |
994 | 994 | if ( 'all' === $mode ) { |
995 | - return new WP_Error('failed_criteria', '[apply_filters_to_entry] Entry cannot be displayed. Failed a subcriterium for any field in ALL mode', $filter ); |
|
995 | + return new WP_Error( 'failed_criteria', '[apply_filters_to_entry] Entry cannot be displayed. Failed a subcriterium for any field in ALL mode', $filter ); |
|
996 | 996 | } |
997 | 997 | |
998 | 998 | continue; |
999 | 999 | } |
1000 | 1000 | |
1001 | - $k = $filter['key']; |
|
1001 | + $k = $filter[ 'key' ]; |
|
1002 | 1002 | |
1003 | 1003 | $field = self::get_field( $form, $k ); |
1004 | 1004 | |
@@ -1006,12 +1006,12 @@ discard block |
||
1006 | 1006 | $field_value = isset( $entry[ $k ] ) ? $entry[ $k ] : null; |
1007 | 1007 | $field = $k; |
1008 | 1008 | } else { |
1009 | - $field_value = GFFormsModel::get_lead_field_value( $entry, $field ); |
|
1009 | + $field_value = GFFormsModel::get_lead_field_value( $entry, $field ); |
|
1010 | 1010 | // If it's a complex field, then fetch the input's value, if exists at the current key. Otherwise, let GF handle it |
1011 | 1011 | $field_value = ( is_array( $field_value ) && isset( $field_value[ $k ] ) ) ? \GV\Utils::get( $field_value, $k ) : $field_value; |
1012 | 1012 | } |
1013 | 1013 | |
1014 | - $is_value_match = GravityView_GFFormsModel::is_value_match( $field_value, $filter['value'], $operator, $field ); |
|
1014 | + $is_value_match = GravityView_GFFormsModel::is_value_match( $field_value, $filter[ 'value' ], $operator, $field ); |
|
1015 | 1015 | |
1016 | 1016 | // Any match is all we need to know |
1017 | 1017 | if ( $is_value_match && 'any' === $mode ) { |
@@ -1020,7 +1020,7 @@ discard block |
||
1020 | 1020 | |
1021 | 1021 | // Any failed match is a total fail |
1022 | 1022 | if ( ! $is_value_match && 'all' === $mode ) { |
1023 | - return new WP_Error('failed_criteria', '[apply_filters_to_entry] Entry cannot be displayed. Failed a criterium for ALL mode', $filter ); |
|
1023 | + return new WP_Error( 'failed_criteria', '[apply_filters_to_entry] Entry cannot be displayed. Failed a criterium for ALL mode', $filter ); |
|
1024 | 1024 | } |
1025 | 1025 | } |
1026 | 1026 | |
@@ -1030,7 +1030,7 @@ discard block |
||
1030 | 1030 | gravityview()->log->debug( '[apply_filters_to_entry] Entry approved: all conditions were met' ); |
1031 | 1031 | return $entry; |
1032 | 1032 | } else { |
1033 | - return new WP_Error('failed_any_criteria', '[apply_filters_to_entry] Entry cannot be displayed. Failed all the criteria for ANY mode', $filters ); |
|
1033 | + return new WP_Error( 'failed_any_criteria', '[apply_filters_to_entry] Entry cannot be displayed. Failed all the criteria for ANY mode', $filters ); |
|
1034 | 1034 | } |
1035 | 1035 | |
1036 | 1036 | } |
@@ -1071,18 +1071,18 @@ discard block |
||
1071 | 1071 | * Gravity Forms code to adjust date to locally-configured Time Zone |
1072 | 1072 | * @see GFCommon::format_date() for original code |
1073 | 1073 | */ |
1074 | - $date_gmt_time = mysql2date( 'G', $date_string ); |
|
1074 | + $date_gmt_time = mysql2date( 'G', $date_string ); |
|
1075 | 1075 | $date_local_timestamp = GFCommon::get_local_timestamp( $date_gmt_time ); |
1076 | 1076 | |
1077 | - $format = \GV\Utils::get( $atts, 'format' ); |
|
1078 | - $is_human = ! empty( $atts['human'] ); |
|
1079 | - $is_diff = ! empty( $atts['diff'] ); |
|
1080 | - $is_raw = ! empty( $atts['raw'] ); |
|
1081 | - $is_timestamp = ! empty( $atts['timestamp'] ); |
|
1082 | - $include_time = ! empty( $atts['time'] ); |
|
1077 | + $format = \GV\Utils::get( $atts, 'format' ); |
|
1078 | + $is_human = ! empty( $atts[ 'human' ] ); |
|
1079 | + $is_diff = ! empty( $atts[ 'diff' ] ); |
|
1080 | + $is_raw = ! empty( $atts[ 'raw' ] ); |
|
1081 | + $is_timestamp = ! empty( $atts[ 'timestamp' ] ); |
|
1082 | + $include_time = ! empty( $atts[ 'time' ] ); |
|
1083 | 1083 | |
1084 | 1084 | // If we're using time diff, we want to have a different default format |
1085 | - if( empty( $format ) ) { |
|
1085 | + if ( empty( $format ) ) { |
|
1086 | 1086 | /* translators: %s: relative time from now, used for generic date comparisons. "1 day ago", or "20 seconds ago" */ |
1087 | 1087 | $format = $is_diff ? esc_html__( '%s ago', 'gravityview' ) : get_option( 'date_format' ); |
1088 | 1088 | } |
@@ -1090,7 +1090,7 @@ discard block |
||
1090 | 1090 | // If raw was specified, don't modify the stored value |
1091 | 1091 | if ( $is_raw ) { |
1092 | 1092 | $formatted_date = $date_string; |
1093 | - } elseif( $is_timestamp ) { |
|
1093 | + } elseif ( $is_timestamp ) { |
|
1094 | 1094 | $formatted_date = $date_local_timestamp; |
1095 | 1095 | } elseif ( $is_diff ) { |
1096 | 1096 | $formatted_date = sprintf( $format, human_time_diff( $date_gmt_time ) ); |
@@ -1124,7 +1124,7 @@ discard block |
||
1124 | 1124 | |
1125 | 1125 | $label = \GV\Utils::get( $field, 'label' ); |
1126 | 1126 | |
1127 | - if( floor( $field_id ) !== floatval( $field_id ) ) { |
|
1127 | + if ( floor( $field_id ) !== floatval( $field_id ) ) { |
|
1128 | 1128 | $label = GFFormsModel::get_choice_text( $field, $field_value, $field_id ); |
1129 | 1129 | } |
1130 | 1130 | |
@@ -1152,7 +1152,7 @@ discard block |
||
1152 | 1152 | $form = GFAPI::get_form( $form ); |
1153 | 1153 | } |
1154 | 1154 | |
1155 | - if ( class_exists( 'GFFormsModel' ) ){ |
|
1155 | + if ( class_exists( 'GFFormsModel' ) ) { |
|
1156 | 1156 | return GFFormsModel::get_field( $form, $field_id ); |
1157 | 1157 | } else { |
1158 | 1158 | return null; |
@@ -1199,19 +1199,19 @@ discard block |
||
1199 | 1199 | $shortcodes = array(); |
1200 | 1200 | |
1201 | 1201 | preg_match_all( '/' . get_shortcode_regex() . '/s', $content, $matches, PREG_SET_ORDER ); |
1202 | - if ( empty( $matches ) ){ |
|
1202 | + if ( empty( $matches ) ) { |
|
1203 | 1203 | return false; |
1204 | 1204 | } |
1205 | 1205 | |
1206 | 1206 | foreach ( $matches as $shortcode ) { |
1207 | - if ( $tag === $shortcode[2] ) { |
|
1207 | + if ( $tag === $shortcode[ 2 ] ) { |
|
1208 | 1208 | |
1209 | 1209 | // Changed this to $shortcode instead of true so we get the parsed atts. |
1210 | - $shortcodes[] = $shortcode; |
|
1210 | + $shortcodes[ ] = $shortcode; |
|
1211 | 1211 | |
1212 | - } else if ( isset( $shortcode[5] ) && $results = self::has_shortcode_r( $shortcode[5], $tag ) ) { |
|
1213 | - foreach( $results as $result ) { |
|
1214 | - $shortcodes[] = $result; |
|
1212 | + } else if ( isset( $shortcode[ 5 ] ) && $results = self::has_shortcode_r( $shortcode[ 5 ], $tag ) ) { |
|
1213 | + foreach ( $results as $result ) { |
|
1214 | + $shortcodes[ ] = $result; |
|
1215 | 1215 | } |
1216 | 1216 | } |
1217 | 1217 | } |
@@ -1243,7 +1243,7 @@ discard block |
||
1243 | 1243 | 'post_type' => 'gravityview', |
1244 | 1244 | 'posts_per_page' => 100, |
1245 | 1245 | 'meta_key' => '_gravityview_form_id', |
1246 | - 'meta_value' => (int) $form_id, |
|
1246 | + 'meta_value' => (int)$form_id, |
|
1247 | 1247 | ); |
1248 | 1248 | $args = wp_parse_args( $args, $defaults ); |
1249 | 1249 | $views = get_posts( $args ); |
@@ -1255,21 +1255,21 @@ discard block |
||
1255 | 1255 | |
1256 | 1256 | $data = unserialize( $view->meta_value ); |
1257 | 1257 | |
1258 | - if( ! $data || ! is_array( $data ) ) { |
|
1258 | + if ( ! $data || ! is_array( $data ) ) { |
|
1259 | 1259 | continue; |
1260 | 1260 | } |
1261 | 1261 | |
1262 | 1262 | foreach ( $data as $datum ) { |
1263 | - if ( ! empty( $datum[2] ) && (int) $datum[2] === (int) $form_id ) { |
|
1264 | - $joined_forms[] = $view->post_id; |
|
1263 | + if ( ! empty( $datum[ 2 ] ) && (int)$datum[ 2 ] === (int)$form_id ) { |
|
1264 | + $joined_forms[ ] = $view->post_id; |
|
1265 | 1265 | } |
1266 | 1266 | } |
1267 | 1267 | } |
1268 | 1268 | |
1269 | 1269 | if ( $joined_forms ) { |
1270 | - $joined_args = array( |
|
1270 | + $joined_args = array( |
|
1271 | 1271 | 'post_type' => 'gravityview', |
1272 | - 'posts_per_page' => $args['posts_per_page'], |
|
1272 | + 'posts_per_page' => $args[ 'posts_per_page' ], |
|
1273 | 1273 | 'post__in' => $joined_forms, |
1274 | 1274 | ); |
1275 | 1275 | $views = array_merge( $views, get_posts( $joined_args ) ); |
@@ -1425,7 +1425,7 @@ discard block |
||
1425 | 1425 | |
1426 | 1426 | $directory_widgets = wp_parse_args( $view_widgets, $defaults ); |
1427 | 1427 | |
1428 | - if( $json_decode ) { |
|
1428 | + if ( $json_decode ) { |
|
1429 | 1429 | $directory_widgets = gv_map_deep( $directory_widgets, 'gv_maybe_json_decode' ); |
1430 | 1430 | } |
1431 | 1431 | |
@@ -1441,7 +1441,7 @@ discard block |
||
1441 | 1441 | * @return string html |
1442 | 1442 | */ |
1443 | 1443 | public static function get_sortable_fields( $formid, $current = '' ) { |
1444 | - $output = '<option value="" ' . selected( '', $current, false ).'>' . esc_html__( 'Default', 'gravityview' ) .'</option>'; |
|
1444 | + $output = '<option value="" ' . selected( '', $current, false ) . '>' . esc_html__( 'Default', 'gravityview' ) . '</option>'; |
|
1445 | 1445 | |
1446 | 1446 | if ( empty( $formid ) ) { |
1447 | 1447 | return $output; |
@@ -1454,11 +1454,11 @@ discard block |
||
1454 | 1454 | $blacklist_field_types = apply_filters( 'gravityview_blacklist_field_types', array( 'list', 'textarea' ), null ); |
1455 | 1455 | |
1456 | 1456 | foreach ( $fields as $id => $field ) { |
1457 | - if ( in_array( $field['type'], $blacklist_field_types ) ) { |
|
1457 | + if ( in_array( $field[ 'type' ], $blacklist_field_types ) ) { |
|
1458 | 1458 | continue; |
1459 | 1459 | } |
1460 | 1460 | |
1461 | - $output .= '<option value="'. $id .'" '. selected( $id, $current, false ).'>'. esc_attr( $field['label'] ) .'</option>'; |
|
1461 | + $output .= '<option value="' . $id . '" ' . selected( $id, $current, false ) . '>' . esc_attr( $field[ 'label' ] ) . '</option>'; |
|
1462 | 1462 | } |
1463 | 1463 | } |
1464 | 1464 | |
@@ -1493,16 +1493,16 @@ discard block |
||
1493 | 1493 | $blacklist_field_types = apply_filters( 'gravityview_blacklist_field_types', $blacklist, NULL ); |
1494 | 1494 | |
1495 | 1495 | // TODO: Convert to using array_filter |
1496 | - foreach( $fields as $id => $field ) { |
|
1496 | + foreach ( $fields as $id => $field ) { |
|
1497 | 1497 | |
1498 | - if( in_array( $field['type'], $blacklist_field_types ) ) { |
|
1498 | + if ( in_array( $field[ 'type' ], $blacklist_field_types ) ) { |
|
1499 | 1499 | unset( $fields[ $id ] ); |
1500 | 1500 | } |
1501 | 1501 | |
1502 | 1502 | /** |
1503 | 1503 | * Merge date and time subfields. |
1504 | 1504 | */ |
1505 | - if ( in_array( $field['type'], array( 'date', 'time' ) ) && ! empty( $field['parent'] ) ) { |
|
1505 | + if ( in_array( $field[ 'type' ], array( 'date', 'time' ) ) && ! empty( $field[ 'parent' ] ) ) { |
|
1506 | 1506 | $fields[ intval( $id ) ] = array( |
1507 | 1507 | 'label' => \GV\Utils::get( $field, 'parent/label' ), |
1508 | 1508 | 'parent' => null, |
@@ -1552,14 +1552,14 @@ discard block |
||
1552 | 1552 | * @param int|array $field field key or field array |
1553 | 1553 | * @return boolean |
1554 | 1554 | */ |
1555 | - public static function is_field_numeric( $form = null, $field = '' ) { |
|
1555 | + public static function is_field_numeric( $form = null, $field = '' ) { |
|
1556 | 1556 | |
1557 | 1557 | if ( ! is_array( $form ) && ! is_array( $field ) ) { |
1558 | 1558 | $form = self::get_form( $form ); |
1559 | 1559 | } |
1560 | 1560 | |
1561 | 1561 | // If entry meta, it's a string. Otherwise, numeric |
1562 | - if( ! is_numeric( $field ) && is_string( $field ) ) { |
|
1562 | + if ( ! is_numeric( $field ) && is_string( $field ) ) { |
|
1563 | 1563 | $type = $field; |
1564 | 1564 | } else { |
1565 | 1565 | $type = self::get_field_type( $form, $field ); |
@@ -1573,9 +1573,9 @@ discard block |
||
1573 | 1573 | $numeric_types = apply_filters( 'gravityview/common/numeric_types', array( 'number', 'time' ) ); |
1574 | 1574 | |
1575 | 1575 | // Defer to GravityView_Field setting, if the field type is registered and `is_numeric` is true |
1576 | - if( $gv_field = GravityView_Fields::get( $type ) ) { |
|
1577 | - if( true === $gv_field->is_numeric ) { |
|
1578 | - $numeric_types[] = $gv_field->is_numeric; |
|
1576 | + if ( $gv_field = GravityView_Fields::get( $type ) ) { |
|
1577 | + if ( true === $gv_field->is_numeric ) { |
|
1578 | + $numeric_types[ ] = $gv_field->is_numeric; |
|
1579 | 1579 | } |
1580 | 1580 | } |
1581 | 1581 | |
@@ -1725,18 +1725,18 @@ discard block |
||
1725 | 1725 | $final_atts = array_filter( $final_atts ); |
1726 | 1726 | |
1727 | 1727 | // If the href wasn't passed as an attribute, use the value passed to the function |
1728 | - if ( empty( $final_atts['href'] ) && ! empty( $href ) ) { |
|
1729 | - $final_atts['href'] = $href; |
|
1728 | + if ( empty( $final_atts[ 'href' ] ) && ! empty( $href ) ) { |
|
1729 | + $final_atts[ 'href' ] = $href; |
|
1730 | 1730 | } |
1731 | 1731 | |
1732 | - $final_atts['href'] = esc_url_raw( $href ); |
|
1732 | + $final_atts[ 'href' ] = esc_url_raw( $href ); |
|
1733 | 1733 | |
1734 | 1734 | /** |
1735 | 1735 | * Fix potential security issue with target=_blank |
1736 | 1736 | * @see https://dev.to/ben/the-targetblank-vulnerability-by-example |
1737 | 1737 | */ |
1738 | - if( '_blank' === \GV\Utils::get( $final_atts, 'target' ) ) { |
|
1739 | - $final_atts['rel'] = trim( \GV\Utils::get( $final_atts, 'rel', '' ) . ' noopener noreferrer' ); |
|
1738 | + if ( '_blank' === \GV\Utils::get( $final_atts, 'target' ) ) { |
|
1739 | + $final_atts[ 'rel' ] = trim( \GV\Utils::get( $final_atts, 'rel', '' ) . ' noopener noreferrer' ); |
|
1740 | 1740 | } |
1741 | 1741 | |
1742 | 1742 | // Sort the attributes alphabetically, to help testing |
@@ -1748,7 +1748,7 @@ discard block |
||
1748 | 1748 | $output .= sprintf( ' %s="%s"', $attr, esc_attr( $value ) ); |
1749 | 1749 | } |
1750 | 1750 | |
1751 | - if( '' !== $output ) { |
|
1751 | + if ( '' !== $output ) { |
|
1752 | 1752 | $output = '<a' . $output . '>' . $anchor_text . '</a>'; |
1753 | 1753 | } |
1754 | 1754 | |
@@ -1775,7 +1775,7 @@ discard block |
||
1775 | 1775 | if ( is_array( $value ) && isset( $merged[ $key ] ) && is_array( $merged[ $key ] ) ) { |
1776 | 1776 | $merged[ $key ] = self::array_merge_recursive_distinct( $merged[ $key ], $value ); |
1777 | 1777 | } else if ( is_numeric( $key ) && isset( $merged[ $key ] ) ) { |
1778 | - $merged[] = $value; |
|
1778 | + $merged[ ] = $value; |
|
1779 | 1779 | } else { |
1780 | 1780 | $merged[ $key ] = $value; |
1781 | 1781 | } |
@@ -1808,7 +1808,7 @@ discard block |
||
1808 | 1808 | * `$context` is where are we using this information (e.g. change_entry_creator, search_widget ..) |
1809 | 1809 | * @param array $settings Settings array, with `number` key defining the # of users to display |
1810 | 1810 | */ |
1811 | - $get_users_settings = apply_filters( 'gravityview/get_users/'. $context, apply_filters( 'gravityview_change_entry_creator_user_parameters', $get_users_settings ) ); |
|
1811 | + $get_users_settings = apply_filters( 'gravityview/get_users/' . $context, apply_filters( 'gravityview_change_entry_creator_user_parameters', $get_users_settings ) ); |
|
1812 | 1812 | |
1813 | 1813 | return get_users( $get_users_settings ); |
1814 | 1814 | } |
@@ -1828,11 +1828,11 @@ discard block |
||
1828 | 1828 | public static function generate_notice( $notice, $class = '', $cap = '', $object_id = null ) { |
1829 | 1829 | |
1830 | 1830 | // If $cap is defined, only show notice if user has capability |
1831 | - if( $cap && ! GVCommon::has_cap( $cap, $object_id ) ) { |
|
1831 | + if ( $cap && ! GVCommon::has_cap( $cap, $object_id ) ) { |
|
1832 | 1832 | return ''; |
1833 | 1833 | } |
1834 | 1834 | |
1835 | - return '<div class="gv-notice '.gravityview_sanitize_html_class( $class ) .'">'. $notice .'</div>'; |
|
1835 | + return '<div class="gv-notice ' . gravityview_sanitize_html_class( $class ) . '">' . $notice . '</div>'; |
|
1836 | 1836 | } |
1837 | 1837 | |
1838 | 1838 | /** |
@@ -50,7 +50,7 @@ |
||
50 | 50 | * |
51 | 51 | * @return array Blacklist, with "total" added. If not edit context, original field blacklist. Otherwise, blacklist including total. |
52 | 52 | */ |
53 | - public function add_to_blacklist( $blacklist = array(), $context = NULL ){ |
|
53 | + public function add_to_blacklist( $blacklist = array(), $context = NULL ) { |
|
54 | 54 | |
55 | 55 | if( empty( $context ) || $context !== 'edit' ) { |
56 | 56 | return $blacklist; |
@@ -50,13 +50,13 @@ discard block |
||
50 | 50 | * |
51 | 51 | * @return array Blacklist, with "total" added. If not edit context, original field blacklist. Otherwise, blacklist including total. |
52 | 52 | */ |
53 | - public function add_to_blacklist( $blacklist = array(), $context = NULL ){ |
|
53 | + public function add_to_blacklist( $blacklist = array(), $context = NULL ) { |
|
54 | 54 | |
55 | - if( empty( $context ) || $context !== 'edit' ) { |
|
55 | + if ( empty( $context ) || $context !== 'edit' ) { |
|
56 | 56 | return $blacklist; |
57 | 57 | } |
58 | 58 | |
59 | - $blacklist[] = 'total'; |
|
59 | + $blacklist[ ] = 'total'; |
|
60 | 60 | |
61 | 61 | return $blacklist; |
62 | 62 | } |
@@ -74,7 +74,7 @@ discard block |
||
74 | 74 | */ |
75 | 75 | function edit_entry_recalculate_totals( $form = array(), $entry_id = 0, $Edit_Entry_Render = null ) { |
76 | 76 | |
77 | - $original_form = GFAPI::get_form( $form['id'] ); |
|
77 | + $original_form = GFAPI::get_form( $form[ 'id' ] ); |
|
78 | 78 | |
79 | 79 | $total_fields = GFCommon::get_fields_by_type( $original_form, 'total' ); |
80 | 80 | |
@@ -85,12 +85,12 @@ discard block |
||
85 | 85 | |
86 | 86 | /** @var GF_Field_Total $total_field */ |
87 | 87 | foreach ( $total_fields as $total_field ) { |
88 | - $entry["{$total_field->id}"] = GFCommon::get_order_total( $original_form, $entry ); |
|
88 | + $entry[ "{$total_field->id}" ] = GFCommon::get_order_total( $original_form, $entry ); |
|
89 | 89 | } |
90 | 90 | |
91 | 91 | $return_entry = GFAPI::update_entry( $entry ); |
92 | 92 | |
93 | - if( is_wp_error( $return_entry ) ) { |
|
93 | + if ( is_wp_error( $return_entry ) ) { |
|
94 | 94 | gravityview()->log->error( 'Updating the entry total fields failed', array( 'data' => $return_entry ) ); |
95 | 95 | } else { |
96 | 96 | gravityview()->log->debug( 'Updating the entry total fields succeeded' ); |
@@ -313,6 +313,7 @@ |
||
313 | 313 | * Add a callback for lazy loading/counting. |
314 | 314 | * |
315 | 315 | * @param callable $callback The callback to call when needed. |
316 | + * @param string $type |
|
316 | 317 | * |
317 | 318 | * @return void |
318 | 319 | */ |
@@ -104,11 +104,11 @@ discard block |
||
104 | 104 | |
105 | 105 | /** Call all lazy callbacks. */ |
106 | 106 | foreach ( $this->callbacks as $callback ) { |
107 | - if ( $callback[0] != 'count' ) { |
|
107 | + if ( $callback[ 0 ] != 'count' ) { |
|
108 | 108 | continue; |
109 | 109 | } |
110 | 110 | |
111 | - $total += $callback[1]( $this->filters ); |
|
111 | + $total += $callback[ 1 ]( $this->filters ); |
|
112 | 112 | } |
113 | 113 | |
114 | 114 | if ( ! $total ) { |
@@ -148,7 +148,7 @@ discard block |
||
148 | 148 | |
149 | 149 | foreach ( $this->all() as $entry ) { |
150 | 150 | $entry = $entry->as_entry(); |
151 | - $result[] = Utils::get( $entry, $key, null ); |
|
151 | + $result[ ] = Utils::get( $entry, $key, null ); |
|
152 | 152 | } |
153 | 153 | |
154 | 154 | return $result; |
@@ -206,11 +206,11 @@ discard block |
||
206 | 206 | |
207 | 207 | /** Call all lazy callbacks. */ |
208 | 208 | foreach ( $this->callbacks as $i => $callback ) { |
209 | - if ( $callback[0] != 'fetch' ) { |
|
209 | + if ( $callback[ 0 ] != 'fetch' ) { |
|
210 | 210 | continue; |
211 | 211 | } |
212 | 212 | |
213 | - $this->merge( $callback[1]( $this->filters, $this->sorts, $offset ) ); |
|
213 | + $this->merge( $callback[ 1 ]( $this->filters, $this->sorts, $offset ) ); |
|
214 | 214 | } |
215 | 215 | |
216 | 216 | $this->fetched = parent::count(); |
@@ -366,7 +366,7 @@ discard block |
||
366 | 366 | return; |
367 | 367 | } |
368 | 368 | |
369 | - $this->callbacks []= array( $type, $callback ); |
|
369 | + $this->callbacks [ ] = array( $type, $callback ); |
|
370 | 370 | } |
371 | 371 | |
372 | 372 | /** |
@@ -25,7 +25,7 @@ discard block |
||
25 | 25 | * @return void |
26 | 26 | */ |
27 | 27 | public function add( $value ) { |
28 | - $this->storage []= $value; |
|
28 | + $this->storage [ ] = $value; |
|
29 | 29 | } |
30 | 30 | |
31 | 31 | /** |
@@ -49,7 +49,7 @@ discard block |
||
49 | 49 | * @return void |
50 | 50 | */ |
51 | 51 | public function merge( \GV\Collection $collection ) { |
52 | - array_map( array( $this, 'add'), $collection->all() ); |
|
52 | + array_map( array( $this, 'add' ), $collection->all() ); |
|
53 | 53 | } |
54 | 54 | |
55 | 55 | /** |