@@ -42,14 +42,14 @@ discard block |
||
42 | 42 | */ |
43 | 43 | public static function by_id( $form_id ) { |
44 | 44 | $form = \GFAPI::get_form( $form_id ); |
45 | - if ( !$form ) { |
|
45 | + if ( ! $form ) { |
|
46 | 46 | return null; |
47 | 47 | } |
48 | 48 | |
49 | 49 | $self = new self(); |
50 | 50 | $self->form = $form; |
51 | 51 | |
52 | - $self->ID = $self->form['id']; |
|
52 | + $self->ID = $self->form[ 'id' ]; |
|
53 | 53 | |
54 | 54 | return $self; |
55 | 55 | } |
@@ -91,15 +91,15 @@ discard block |
||
91 | 91 | |
92 | 92 | /** The offset and limit */ |
93 | 93 | if ( ! empty( $offset->limit ) ) { |
94 | - $paging['page_size'] = $offset->limit; |
|
94 | + $paging[ 'page_size' ] = $offset->limit; |
|
95 | 95 | } |
96 | 96 | |
97 | 97 | if ( ! empty( $offset->offset ) ) { |
98 | - $paging['offset'] = $offset->offset; |
|
98 | + $paging[ 'offset' ] = $offset->offset; |
|
99 | 99 | } |
100 | 100 | |
101 | 101 | foreach ( \GFAPI::get_entries( $form->ID, $search_criteria, $sorting, $paging ) as $entry ) { |
102 | - $entries->add( \GV\GF_Entry::by_id( $entry['id'] ) ); |
|
102 | + $entries->add( \GV\GF_Entry::by_id( $entry[ 'id' ] ) ); |
|
103 | 103 | } |
104 | 104 | |
105 | 105 | return $entries; |
@@ -131,7 +131,7 @@ discard block |
||
131 | 131 | * @return bool Whether the offset exists or not. |
132 | 132 | */ |
133 | 133 | public function offsetExists( $offset ) { |
134 | - return isset( $this->form[$offset] ); |
|
134 | + return isset( $this->form[ $offset ] ); |
|
135 | 135 | } |
136 | 136 | |
137 | 137 | /** |
@@ -148,7 +148,7 @@ discard block |
||
148 | 148 | * @return mixed The value of the requested form data. |
149 | 149 | */ |
150 | 150 | public function offsetGet( $offset ) { |
151 | - return $this->form[$offset]; |
|
151 | + return $this->form[ $offset ]; |
|
152 | 152 | } |
153 | 153 | |
154 | 154 | /** |
@@ -42,14 +42,14 @@ discard block |
||
42 | 42 | */ |
43 | 43 | public static function by_id( $entry_id ) { |
44 | 44 | $entry = \GFAPI::get_entry( $entry_id ); |
45 | - if ( !$entry ) { |
|
45 | + if ( ! $entry ) { |
|
46 | 46 | return null; |
47 | 47 | } |
48 | 48 | |
49 | 49 | $self = new self(); |
50 | 50 | $self->entry = $entry; |
51 | 51 | |
52 | - $self->ID = $self->entry['id']; |
|
52 | + $self->ID = $self->entry[ 'id' ]; |
|
53 | 53 | |
54 | 54 | return $self; |
55 | 55 | } |
@@ -63,7 +63,7 @@ discard block |
||
63 | 63 | * @return bool Whether the offset exists or not. |
64 | 64 | */ |
65 | 65 | public function offsetExists( $offset ) { |
66 | - return isset( $this->entry[$offset] ); |
|
66 | + return isset( $this->entry[ $offset ] ); |
|
67 | 67 | } |
68 | 68 | |
69 | 69 | /** |
@@ -80,7 +80,7 @@ discard block |
||
80 | 80 | * @return mixed The value of the requested entry data. |
81 | 81 | */ |
82 | 82 | public function offsetGet( $offset ) { |
83 | - return $this->entry[$offset]; |
|
83 | + return $this->entry[ $offset ]; |
|
84 | 84 | } |
85 | 85 | |
86 | 86 | /** |
@@ -39,7 +39,7 @@ discard block |
||
39 | 39 | */ |
40 | 40 | $default_settings = apply_filters( 'gravityview_default_args', array( |
41 | 41 | 'id' => array( |
42 | - 'label' => __('View ID', 'gravityview'), |
|
42 | + 'label' => __( 'View ID', 'gravityview' ), |
|
43 | 43 | 'type' => 'number', |
44 | 44 | 'group' => 'default', |
45 | 45 | 'value' => NULL, |
@@ -47,7 +47,7 @@ discard block |
||
47 | 47 | 'show_in_shortcode' => false, |
48 | 48 | ), |
49 | 49 | 'page_size' => array( |
50 | - 'label' => __('Number of entries per page', 'gravityview'), |
|
50 | + 'label' => __( 'Number of entries per page', 'gravityview' ), |
|
51 | 51 | 'type' => 'number', |
52 | 52 | 'class' => 'small-text', |
53 | 53 | 'group' => 'default', |
@@ -55,7 +55,7 @@ discard block |
||
55 | 55 | 'show_in_shortcode' => true, |
56 | 56 | ), |
57 | 57 | 'offset' => array( |
58 | - 'label' => __('Offset entries starting from', 'gravityview'), |
|
58 | + 'label' => __( 'Offset entries starting from', 'gravityview' ), |
|
59 | 59 | 'type' => 'number', |
60 | 60 | 'class' => 'small-text', |
61 | 61 | 'group' => 'default', |
@@ -79,8 +79,8 @@ discard block |
||
79 | 79 | ), |
80 | 80 | 'admin_show_all_statuses' => array( |
81 | 81 | 'label' => __( 'Show all entries to administrators', 'gravityview' ), |
82 | - 'desc' => __('Administrators will be able to see entries with any approval status.', 'gravityview'), |
|
83 | - 'tooltip' => __('Logged-out visitors and non-administrators will only see approved entries, while administrators will see entries with all statuses. This makes it easier for administrators to moderate entries from a View.', 'gravityview'), |
|
82 | + 'desc' => __( 'Administrators will be able to see entries with any approval status.', 'gravityview' ), |
|
83 | + 'tooltip' => __( 'Logged-out visitors and non-administrators will only see approved entries, while administrators will see entries with all statuses. This makes it easier for administrators to moderate entries from a View.', 'gravityview' ), |
|
84 | 84 | 'requires' => 'show_only_approved', |
85 | 85 | 'type' => 'checkbox', |
86 | 86 | 'group' => 'default', |
@@ -105,40 +105,40 @@ discard block |
||
105 | 105 | 'user_edit' => array( |
106 | 106 | 'label' => __( 'Allow User Edit', 'gravityview' ), |
107 | 107 | 'group' => 'default', |
108 | - 'desc' => __('Allow logged-in users to edit entries they created.', 'gravityview'), |
|
108 | + 'desc' => __( 'Allow logged-in users to edit entries they created.', 'gravityview' ), |
|
109 | 109 | 'value' => 0, |
110 | - 'tooltip' => __('Display "Edit Entry" fields to non-administrator users if they created the entry. Edit Entry fields will always be displayed to site administrators.', 'gravityview'), |
|
110 | + 'tooltip' => __( 'Display "Edit Entry" fields to non-administrator users if they created the entry. Edit Entry fields will always be displayed to site administrators.', 'gravityview' ), |
|
111 | 111 | 'type' => 'checkbox', |
112 | 112 | 'show_in_shortcode' => true, |
113 | 113 | ), |
114 | 114 | 'user_delete' => array( |
115 | 115 | 'label' => __( 'Allow User Delete', 'gravityview' ), |
116 | 116 | 'group' => 'default', |
117 | - 'desc' => __('Allow logged-in users to delete entries they created.', 'gravityview'), |
|
117 | + 'desc' => __( 'Allow logged-in users to delete entries they created.', 'gravityview' ), |
|
118 | 118 | 'value' => 0, |
119 | - 'tooltip' => __('Display "Delete Entry" fields to non-administrator users if they created the entry. Delete Entry fields will always be displayed to site administrators.', 'gravityview'), |
|
119 | + 'tooltip' => __( 'Display "Delete Entry" fields to non-administrator users if they created the entry. Delete Entry fields will always be displayed to site administrators.', 'gravityview' ), |
|
120 | 120 | 'type' => 'checkbox', |
121 | 121 | 'show_in_shortcode' => true, |
122 | 122 | ), |
123 | 123 | 'sort_field' => array( |
124 | - 'label' => __('Sort by field', 'gravityview'), |
|
124 | + 'label' => __( 'Sort by field', 'gravityview' ), |
|
125 | 125 | 'type' => 'select', |
126 | 126 | 'value' => '', |
127 | 127 | 'group' => 'sort', |
128 | 128 | 'options' => array( |
129 | - '' => __( 'Default', 'gravityview'), |
|
130 | - 'date_created' => __( 'Date Created', 'gravityview'), |
|
129 | + '' => __( 'Default', 'gravityview' ), |
|
130 | + 'date_created' => __( 'Date Created', 'gravityview' ), |
|
131 | 131 | ), |
132 | 132 | 'show_in_shortcode' => true, |
133 | 133 | ), |
134 | 134 | 'sort_direction' => array( |
135 | - 'label' => __('Sort direction', 'gravityview'), |
|
135 | + 'label' => __( 'Sort direction', 'gravityview' ), |
|
136 | 136 | 'type' => 'select', |
137 | 137 | 'value' => 'ASC', |
138 | 138 | 'group' => 'sort', |
139 | 139 | 'options' => array( |
140 | - 'ASC' => __('ASC', 'gravityview'), |
|
141 | - 'DESC' => __('DESC', 'gravityview'), |
|
140 | + 'ASC' => __( 'ASC', 'gravityview' ), |
|
141 | + 'DESC' => __( 'DESC', 'gravityview' ), |
|
142 | 142 | //'RAND' => __('Random', 'gravityview'), |
143 | 143 | ), |
144 | 144 | 'show_in_shortcode' => true, |
@@ -154,69 +154,69 @@ discard block |
||
154 | 154 | 'show_in_template' => array( 'default_table' ), |
155 | 155 | ), |
156 | 156 | 'start_date' => array( |
157 | - 'label' => __('Filter by Start Date', 'gravityview'), |
|
157 | + 'label' => __( 'Filter by Start Date', 'gravityview' ), |
|
158 | 158 | 'class' => 'gv-datepicker', |
159 | - 'desc' => __('Show entries submitted after this date. Supports relative dates, such as "-1 week" or "-1 month".', 'gravityview' ), |
|
159 | + 'desc' => __( 'Show entries submitted after this date. Supports relative dates, such as "-1 week" or "-1 month".', 'gravityview' ), |
|
160 | 160 | 'type' => 'text', |
161 | 161 | 'value' => '', |
162 | 162 | 'group' => 'filter', |
163 | 163 | 'show_in_shortcode' => true, |
164 | 164 | ), |
165 | 165 | 'end_date' => array( |
166 | - 'label' => __('Filter by End Date', 'gravityview'), |
|
166 | + 'label' => __( 'Filter by End Date', 'gravityview' ), |
|
167 | 167 | 'class' => 'gv-datepicker', |
168 | - 'desc' => __('Show entries submitted before this date. Supports relative dates, such as "now" or "-3 days".', 'gravityview' ), |
|
168 | + 'desc' => __( 'Show entries submitted before this date. Supports relative dates, such as "now" or "-3 days".', 'gravityview' ), |
|
169 | 169 | 'type' => 'text', |
170 | 170 | 'value' => '', |
171 | 171 | 'group' => 'filter', |
172 | 172 | 'show_in_shortcode' => true, |
173 | 173 | ), |
174 | 174 | 'class' => array( |
175 | - 'label' => __('CSS Class', 'gravityview'), |
|
176 | - 'desc' => __('CSS class to add to the wrapping HTML container.', 'gravityview'), |
|
175 | + 'label' => __( 'CSS Class', 'gravityview' ), |
|
176 | + 'desc' => __( 'CSS class to add to the wrapping HTML container.', 'gravityview' ), |
|
177 | 177 | 'group' => 'default', |
178 | 178 | 'type' => 'text', |
179 | 179 | 'value' => '', |
180 | 180 | 'show_in_shortcode' => false, |
181 | 181 | ), |
182 | 182 | 'search_value' => array( |
183 | - 'label' => __('Search Value', 'gravityview'), |
|
184 | - 'desc' => __('Define a default search value for the View', 'gravityview'), |
|
183 | + 'label' => __( 'Search Value', 'gravityview' ), |
|
184 | + 'desc' => __( 'Define a default search value for the View', 'gravityview' ), |
|
185 | 185 | 'type' => 'text', |
186 | 186 | 'value' => '', |
187 | 187 | 'group' => 'filter', |
188 | 188 | 'show_in_shortcode' => false, |
189 | 189 | ), |
190 | 190 | 'search_field' => array( |
191 | - 'label' => __('Search Field', 'gravityview'), |
|
192 | - 'desc' => __('If Search Value is set, you can define a specific field to search in. Otherwise, all fields will be searched.', 'gravityview'), |
|
191 | + 'label' => __( 'Search Field', 'gravityview' ), |
|
192 | + 'desc' => __( 'If Search Value is set, you can define a specific field to search in. Otherwise, all fields will be searched.', 'gravityview' ), |
|
193 | 193 | 'type' => 'number', |
194 | 194 | 'value' => '', |
195 | 195 | 'group' => 'filter', |
196 | 196 | 'show_in_shortcode' => false, |
197 | 197 | ), |
198 | 198 | 'single_title' => array( |
199 | - 'label' => __('Single Entry Title', 'gravityview'), |
|
199 | + 'label' => __( 'Single Entry Title', 'gravityview' ), |
|
200 | 200 | 'type' => 'text', |
201 | - 'desc' => __('When viewing a single entry, change the title of the page to this setting. Otherwise, the title will not change between the Multiple Entries and Single Entry views.', 'gravityview'), |
|
201 | + 'desc' => __( 'When viewing a single entry, change the title of the page to this setting. Otherwise, the title will not change between the Multiple Entries and Single Entry views.', 'gravityview' ), |
|
202 | 202 | 'group' => 'default', |
203 | 203 | 'value' => '', |
204 | 204 | 'show_in_shortcode' => false, |
205 | 205 | 'full_width' => true, |
206 | 206 | ), |
207 | 207 | 'back_link_label' => array( |
208 | - 'label' => __('Back Link Label', 'gravityview'), |
|
208 | + 'label' => __( 'Back Link Label', 'gravityview' ), |
|
209 | 209 | 'group' => 'default', |
210 | - 'desc' => __('The text of the link that returns to the multiple entries view.', 'gravityview'), |
|
210 | + 'desc' => __( 'The text of the link that returns to the multiple entries view.', 'gravityview' ), |
|
211 | 211 | 'type' => 'text', |
212 | 212 | 'value' => '', |
213 | 213 | 'show_in_shortcode' => false, |
214 | 214 | 'full_width' => true, |
215 | 215 | ), |
216 | 216 | 'embed_only' => array( |
217 | - 'label' => __('Prevent Direct Access', 'gravityview'), |
|
217 | + 'label' => __( 'Prevent Direct Access', 'gravityview' ), |
|
218 | 218 | 'group' => 'default', |
219 | - 'desc' => __('Only allow access to this View when embedded using the shortcode.', 'gravityview'), |
|
219 | + 'desc' => __( 'Only allow access to this View when embedded using the shortcode.', 'gravityview' ), |
|
220 | 220 | 'type' => 'checkbox', |
221 | 221 | 'value' => '', |
222 | 222 | 'show_in_shortcode' => false, |
@@ -238,19 +238,19 @@ discard block |
||
238 | 238 | // By default, we only want the key => value pairing, not the whole array. |
239 | 239 | if ( ! $detailed ) { |
240 | 240 | $defaults = array(); |
241 | - foreach( $default_settings as $key => $value ) { |
|
242 | - $defaults[ $key ] = $value['value']; |
|
241 | + foreach ( $default_settings as $key => $value ) { |
|
242 | + $defaults[ $key ] = $value[ 'value' ]; |
|
243 | 243 | } |
244 | 244 | return $defaults; |
245 | 245 | |
246 | 246 | // But sometimes, we want all the details. |
247 | 247 | } else { |
248 | - foreach ($default_settings as $key => $value) { |
|
248 | + foreach ( $default_settings as $key => $value ) { |
|
249 | 249 | |
250 | 250 | // If the $group argument is set for the method, |
251 | 251 | // ignore any settings that aren't in that group. |
252 | 252 | if ( ! empty( $group ) && is_string( $group ) ) { |
253 | - if ( empty( $value['group'] ) || $value['group'] !== $group ) { |
|
253 | + if ( empty( $value[ 'group' ] ) || $value[ 'group' ] !== $group ) { |
|
254 | 254 | unset( $default_settings[ $key ] ); |
255 | 255 | } |
256 | 256 | } |
@@ -26,7 +26,7 @@ discard block |
||
26 | 26 | * @return void |
27 | 27 | */ |
28 | 28 | public function set( $key, $value ) { |
29 | - $this->settings[$key] = $value; |
|
29 | + $this->settings[ $key ] = $value; |
|
30 | 30 | } |
31 | 31 | |
32 | 32 | /** |
@@ -40,7 +40,7 @@ discard block |
||
40 | 40 | * @return mixed|null |
41 | 41 | */ |
42 | 42 | public function get( $key, $default = null ) { |
43 | - return isset( $this->settings[$key] ) ? $this->settings[$key] : $default; |
|
43 | + return isset( $this->settings[ $key ] ) ? $this->settings[ $key ] : $default; |
|
44 | 44 | } |
45 | 45 | |
46 | 46 | /** |
@@ -144,19 +144,19 @@ |
||
144 | 144 | public function from_configuration( $configuration ) { |
145 | 145 | $configuration = wp_parse_args( $configuration, $this->as_configuration() ); |
146 | 146 | |
147 | - $this->ID = $configuration['id']; |
|
148 | - $this->label = $configuration['label']; |
|
149 | - $this->show_label = $configuration['show_label'] == '1'; |
|
150 | - $this->custom_label = $configuration['custom_label']; |
|
151 | - $this->custom_class = $configuration['custom_class']; |
|
152 | - $this->cap = $configuration['only_loggedin'] == '1' ? $configuration['only_loggedin_cap'] : ''; |
|
153 | - $this->search_filter = $configuration['search_filter'] == '1'; |
|
154 | - $this->show_as_link = $configuration['show_as_link'] == '1'; |
|
147 | + $this->ID = $configuration[ 'id' ]; |
|
148 | + $this->label = $configuration[ 'label' ]; |
|
149 | + $this->show_label = $configuration[ 'show_label' ] == '1'; |
|
150 | + $this->custom_label = $configuration[ 'custom_label' ]; |
|
151 | + $this->custom_class = $configuration[ 'custom_class' ]; |
|
152 | + $this->cap = $configuration[ 'only_loggedin' ] == '1' ? $configuration[ 'only_loggedin_cap' ] : ''; |
|
153 | + $this->search_filter = $configuration[ 'search_filter' ] == '1'; |
|
154 | + $this->show_as_link = $configuration[ 'show_as_link' ] == '1'; |
|
155 | 155 | |
156 | 156 | /** Shared among all field types (sort of). */ |
157 | 157 | $shared_configuration_keys = array( |
158 | 158 | 'id', 'label', 'show_label', 'custom_label', 'custom_class', |
159 | - 'only_loggedin' ,'only_loggedin_cap', 'search_filter', 'show_as_link', |
|
159 | + 'only_loggedin', 'only_loggedin_cap', 'search_filter', 'show_as_link', |
|
160 | 160 | ); |
161 | 161 | |
162 | 162 | /** Everything else goes into the properties for now. @todo subclasses! */ |
@@ -117,7 +117,7 @@ discard block |
||
117 | 117 | public function set_template_data( array $data, $var_name = 'data' ) { |
118 | 118 | global $wp_query; |
119 | 119 | |
120 | - $wp_query->query_vars[ $var_name ] = (object) $data; |
|
120 | + $wp_query->query_vars[ $var_name ] = (object)$data; |
|
121 | 121 | } |
122 | 122 | |
123 | 123 | /** |
@@ -130,8 +130,8 @@ discard block |
||
130 | 130 | public function unset_template_data() { |
131 | 131 | global $wp_query; |
132 | 132 | |
133 | - if ( isset( $wp_query->query_vars['data'] ) ) { |
|
134 | - unset( $wp_query->query_vars['data'] ); |
|
133 | + if ( isset( $wp_query->query_vars[ 'data' ] ) ) { |
|
134 | + unset( $wp_query->query_vars[ 'data' ] ); |
|
135 | 135 | } |
136 | 136 | } |
137 | 137 | |
@@ -148,9 +148,9 @@ discard block |
||
148 | 148 | protected function get_template_file_names( $slug, $name ) { |
149 | 149 | $templates = array(); |
150 | 150 | if ( isset( $name ) ) { |
151 | - $templates[] = $slug . '-' . $name . '.php'; |
|
151 | + $templates[ ] = $slug . '-' . $name . '.php'; |
|
152 | 152 | } |
153 | - $templates[] = $slug . '.php'; |
|
153 | + $templates[ ] = $slug . '.php'; |
|
154 | 154 | |
155 | 155 | /** |
156 | 156 | * Allow template choices to be filtered. |
@@ -188,7 +188,7 @@ discard block |
||
188 | 188 | $located = false; |
189 | 189 | |
190 | 190 | // Remove empty entries. |
191 | - $template_names = array_filter( (array) $template_names ); |
|
191 | + $template_names = array_filter( (array)$template_names ); |
|
192 | 192 | $template_paths = $this->get_template_paths(); |
193 | 193 | |
194 | 194 | // Try to find a template file. |
@@ -233,7 +233,7 @@ discard block |
||
233 | 233 | |
234 | 234 | // Only add this conditionally, so non-child themes don't redundantly check active theme twice. |
235 | 235 | if ( is_child_theme() ) { |
236 | - $file_paths[1] = trailingslashit( get_stylesheet_directory() ) . $theme_directory; |
|
236 | + $file_paths[ 1 ] = trailingslashit( get_stylesheet_directory() ) . $theme_directory; |
|
237 | 237 | } |
238 | 238 | |
239 | 239 | /** |
@@ -5,7 +5,7 @@ |
||
5 | 5 | } |
6 | 6 | |
7 | 7 | /** The future branch of GravityView requires PHP 5.3+ namespaces and SPL. */ |
8 | -if ( version_compare( phpversion(), '5.3' , '<' ) ) { |
|
8 | +if ( version_compare( phpversion(), '5.3', '<' ) ) { |
|
9 | 9 | return false; |
10 | 10 | |
11 | 11 | /** Tests with a suppressed future. */ |
@@ -81,7 +81,7 @@ discard block |
||
81 | 81 | /** |
82 | 82 | * @see RGFormsModel::update_lead_property() Trigger when any entry property changes |
83 | 83 | */ |
84 | - foreach( $this->lead_db_columns as $column ) { |
|
84 | + foreach ( $this->lead_db_columns as $column ) { |
|
85 | 85 | add_action( 'gform_update_' . $column, array( $this, 'entry_status_changed' ), 10, 3 ); |
86 | 86 | } |
87 | 87 | |
@@ -114,9 +114,9 @@ discard block |
||
114 | 114 | return; |
115 | 115 | } |
116 | 116 | |
117 | - do_action( 'gravityview_log_debug', __METHOD__ . ' adding form ' . $entry['form_id'] . ' to blacklist because entry #' . $lead_id . ' was deleted', array( 'value' => $property_value, 'previous' => $previous_value ) ); |
|
117 | + do_action( 'gravityview_log_debug', __METHOD__ . ' adding form ' . $entry[ 'form_id' ] . ' to blacklist because entry #' . $lead_id . ' was deleted', array( 'value' => $property_value, 'previous' => $previous_value ) ); |
|
118 | 118 | |
119 | - $this->blacklist_add( $entry['form_id'] ); |
|
119 | + $this->blacklist_add( $entry[ 'form_id' ] ); |
|
120 | 120 | } |
121 | 121 | |
122 | 122 | /** |
@@ -129,9 +129,9 @@ discard block |
||
129 | 129 | */ |
130 | 130 | public function entry_updated( $form, $lead_id ) { |
131 | 131 | |
132 | - do_action( 'gravityview_log_debug', 'GravityView_Cache[entry_updated] adding form ' . $form['id'] . ' to blacklist because entry #' . $lead_id . ' was updated' ); |
|
132 | + do_action( 'gravityview_log_debug', 'GravityView_Cache[entry_updated] adding form ' . $form[ 'id' ] . ' to blacklist because entry #' . $lead_id . ' was updated' ); |
|
133 | 133 | |
134 | - $this->blacklist_add( $form['id'] ); |
|
134 | + $this->blacklist_add( $form[ 'id' ] ); |
|
135 | 135 | } |
136 | 136 | |
137 | 137 | /** |
@@ -146,9 +146,9 @@ discard block |
||
146 | 146 | */ |
147 | 147 | public function entry_created( $entry, $form ) { |
148 | 148 | |
149 | - do_action( 'gravityview_log_debug', 'GravityView_Cache[entry_created] adding form ' . $form['id'] . ' to blacklist because entry #' . $entry['id'] . ' was created' ); |
|
149 | + do_action( 'gravityview_log_debug', 'GravityView_Cache[entry_created] adding form ' . $form[ 'id' ] . ' to blacklist because entry #' . $entry[ 'id' ] . ' was created' ); |
|
150 | 150 | |
151 | - $this->blacklist_add( $form['id'] ); |
|
151 | + $this->blacklist_add( $form[ 'id' ] ); |
|
152 | 152 | } |
153 | 153 | |
154 | 154 | /** |
@@ -164,9 +164,9 @@ discard block |
||
164 | 164 | return; |
165 | 165 | } |
166 | 166 | |
167 | - do_action( 'gravityview_log_debug', 'GravityView_Cache[entry_added] adding form ' . $form['id'] . ' to blacklist because entry #' . $entry['id'] . ' was added' ); |
|
167 | + do_action( 'gravityview_log_debug', 'GravityView_Cache[entry_added] adding form ' . $form[ 'id' ] . ' to blacklist because entry #' . $entry[ 'id' ] . ' was added' ); |
|
168 | 168 | |
169 | - $this->blacklist_add( $form['id'] ); |
|
169 | + $this->blacklist_add( $form[ 'id' ] ); |
|
170 | 170 | } |
171 | 171 | |
172 | 172 | /** |
@@ -185,7 +185,7 @@ discard block |
||
185 | 185 | // Normally just one form, but supports multiple forms |
186 | 186 | // |
187 | 187 | // Array of IDs 12, 5, 14 would result in `f-12f-5f-14` |
188 | - $forms = 'f:' . implode( '&f:', (array) $form_ids ); |
|
188 | + $forms = 'f:' . implode( '&f:', (array)$form_ids ); |
|
189 | 189 | |
190 | 190 | // Prefix for transient keys |
191 | 191 | // Now the prefix would be: `gv-cache-f-12f-5f-14` |
@@ -242,7 +242,7 @@ discard block |
||
242 | 242 | ) ); |
243 | 243 | |
244 | 244 | // Add the passed form IDs |
245 | - $blacklist = array_merge( (array) $blacklist, $form_ids ); |
|
245 | + $blacklist = array_merge( (array)$blacklist, $form_ids ); |
|
246 | 246 | |
247 | 247 | // Don't duplicate |
248 | 248 | $blacklist = array_unique( $blacklist ); |
@@ -262,7 +262,7 @@ discard block |
||
262 | 262 | |
263 | 263 | $blacklist = get_option( self::BLACKLIST_OPTION_NAME, array() ); |
264 | 264 | |
265 | - $updated_list = array_diff( $blacklist, (array) $form_ids ); |
|
265 | + $updated_list = array_diff( $blacklist, (array)$form_ids ); |
|
266 | 266 | |
267 | 267 | do_action( 'gravityview_log_debug', 'GravityView_Cache[blacklist_remove] Removing form IDs from cache blacklist', array( |
268 | 268 | '$form_ids' => $form_ids, |
@@ -295,7 +295,7 @@ discard block |
||
295 | 295 | return false; |
296 | 296 | } |
297 | 297 | |
298 | - foreach ( (array) $form_ids as $form_id ) { |
|
298 | + foreach ( (array)$form_ids as $form_id ) { |
|
299 | 299 | |
300 | 300 | if ( in_array( $form_id, $blacklist ) ) { |
301 | 301 | |
@@ -367,7 +367,7 @@ discard block |
||
367 | 367 | * @filter `gravityview_cache_time_{$filter_name}` Modify the cache time for a type of cache |
368 | 368 | * @param int $time_in_seconds Default: `DAY_IN_SECONDS` |
369 | 369 | */ |
370 | - $cache_time = (int) apply_filters( 'gravityview_cache_time_' . $filter_name, DAY_IN_SECONDS ); |
|
370 | + $cache_time = (int)apply_filters( 'gravityview_cache_time_' . $filter_name, DAY_IN_SECONDS ); |
|
371 | 371 | |
372 | 372 | do_action( 'gravityview_log_debug', 'GravityView_Cache[set] Setting cache with transient key ' . $this->key . ' for ' . $cache_time . ' seconds' ); |
373 | 373 | |
@@ -403,7 +403,7 @@ discard block |
||
403 | 403 | return; |
404 | 404 | } |
405 | 405 | |
406 | - foreach ( (array) $form_ids as $form_id ) { |
|
406 | + foreach ( (array)$form_ids as $form_id ) { |
|
407 | 407 | |
408 | 408 | $key = $this->get_cache_key_prefix( $form_id ); |
409 | 409 | |
@@ -517,7 +517,7 @@ discard block |
||
517 | 517 | |
518 | 518 | if ( GVCommon::has_cap( 'edit_gravityviews' ) ) { |
519 | 519 | |
520 | - if ( isset( $_GET['cache'] ) || isset( $_GET['nocache'] ) ) { |
|
520 | + if ( isset( $_GET[ 'cache' ] ) || isset( $_GET[ 'nocache' ] ) ) { |
|
521 | 521 | |
522 | 522 | do_action( 'gravityview_log_debug', 'GravityView_Cache[use_cache] Not using cache: ?cache or ?nocache is in the URL' ); |
523 | 523 | |
@@ -546,7 +546,7 @@ discard block |
||
546 | 546 | */ |
547 | 547 | $use_cache = apply_filters( 'gravityview_use_cache', $use_cache, $this ); |
548 | 548 | |
549 | - return (boolean) $use_cache; |
|
549 | + return (boolean)$use_cache; |
|
550 | 550 | } |
551 | 551 | |
552 | 552 | } |
@@ -45,7 +45,7 @@ discard block |
||
45 | 45 | public static function process_modifiers( $value, $merge_tag, $modifier, $field, $raw_value ) { |
46 | 46 | |
47 | 47 | // No modifier was set or the raw value was empty |
48 | - if( 'all_fields' === $merge_tag || '' === $modifier || ! is_string( $raw_value ) || '' === $raw_value ) { |
|
48 | + if ( 'all_fields' === $merge_tag || '' === $modifier || ! is_string( $raw_value ) || '' === $raw_value ) { |
|
49 | 49 | return $value; |
50 | 50 | } |
51 | 51 | |
@@ -61,9 +61,9 @@ discard block |
||
61 | 61 | foreach ( $gv_modifiers as $gv_modifier => $method ) { |
62 | 62 | |
63 | 63 | // Only match the regex if it's the first modifer; this allows us to enforce our own modifier structure |
64 | - preg_match( '/^' . $gv_modifier .'/ism', $modifier, $matches ); |
|
64 | + preg_match( '/^' . $gv_modifier . '/ism', $modifier, $matches ); |
|
65 | 65 | |
66 | - if( ! empty( $matches ) ) { |
|
66 | + if ( ! empty( $matches ) ) { |
|
67 | 67 | // The called method is passed the raw value and the full matches array |
68 | 68 | $return = self::$method( $raw_value, $matches ); |
69 | 69 | break; |
@@ -87,7 +87,7 @@ discard block |
||
87 | 87 | */ |
88 | 88 | private static function modifier_timestamp( $raw_value, $matches ) { |
89 | 89 | |
90 | - if( empty( $matches[0] ) ) { |
|
90 | + if ( empty( $matches[ 0 ] ) ) { |
|
91 | 91 | return $raw_value; |
92 | 92 | } |
93 | 93 | |
@@ -111,7 +111,7 @@ discard block |
||
111 | 111 | */ |
112 | 112 | private static function modifier_wpautop( $raw_value, $matches ) { |
113 | 113 | |
114 | - if( empty( $matches[0] ) || ! function_exists( 'wpautop' ) ) { |
|
114 | + if ( empty( $matches[ 0 ] ) || ! function_exists( 'wpautop' ) ) { |
|
115 | 115 | return $raw_value; |
116 | 116 | } |
117 | 117 | |
@@ -135,11 +135,11 @@ discard block |
||
135 | 135 | */ |
136 | 136 | private static function modifier_maxwords( $raw_value, $matches ) { |
137 | 137 | |
138 | - if( ! is_string( $raw_value ) || empty( $matches[1] ) || ! function_exists( 'wp_trim_words' ) ) { |
|
138 | + if ( ! is_string( $raw_value ) || empty( $matches[ 1 ] ) || ! function_exists( 'wp_trim_words' ) ) { |
|
139 | 139 | return $raw_value; |
140 | 140 | } |
141 | 141 | |
142 | - $max = intval( $matches[1] ); |
|
142 | + $max = intval( $matches[ 1 ] ); |
|
143 | 143 | |
144 | 144 | $more_placeholder = '[GVMORE]'; |
145 | 145 | |
@@ -171,7 +171,7 @@ discard block |
||
171 | 171 | * @param bool $esc_html Pass return value through `esc_html()` |
172 | 172 | * @return string Text with variables maybe replaced |
173 | 173 | */ |
174 | - public static function replace_variables($text, $form = array(), $entry = array(), $url_encode = false, $esc_html = true ) { |
|
174 | + public static function replace_variables( $text, $form = array(), $entry = array(), $url_encode = false, $esc_html = true ) { |
|
175 | 175 | |
176 | 176 | // TODO: This needs to be fixed in `/future/` branch |
177 | 177 | if ( is_a( $form, '\GV\Form' ) ) { |
@@ -202,9 +202,9 @@ discard block |
||
202 | 202 | * @internal Fixed $form['title'] in Gravity Forms |
203 | 203 | * @see https://github.com/gravityforms/gravityforms/pull/27/files |
204 | 204 | */ |
205 | - $form['title'] = isset( $form['title'] ) ? $form['title'] : ''; |
|
206 | - $form['id'] = isset( $form['id'] ) ? $form['id'] : ''; |
|
207 | - $form['fields'] = isset( $form['fields'] ) ? $form['fields'] : array(); |
|
205 | + $form[ 'title' ] = isset( $form[ 'title' ] ) ? $form[ 'title' ] : ''; |
|
206 | + $form[ 'id' ] = isset( $form[ 'id' ] ) ? $form[ 'id' ] : ''; |
|
207 | + $form[ 'fields' ] = isset( $form[ 'fields' ] ) ? $form[ 'fields' ] : array(); |
|
208 | 208 | |
209 | 209 | return GFCommon::replace_variables( $text, $form, $entry, $url_encode, $esc_html ); |
210 | 210 | } |
@@ -224,7 +224,7 @@ discard block |
||
224 | 224 | * |
225 | 225 | * @return mixed |
226 | 226 | */ |
227 | - public static function replace_gv_merge_tags( $text, $form = array(), $entry = array(), $url_encode = false, $esc_html = false ) { |
|
227 | + public static function replace_gv_merge_tags( $text, $form = array(), $entry = array(), $url_encode = false, $esc_html = false ) { |
|
228 | 228 | |
229 | 229 | /** |
230 | 230 | * This prevents the gform_replace_merge_tags filter from being called twice, as defined in: |
@@ -232,7 +232,7 @@ discard block |
||
232 | 232 | * @see GFCommon::replace_variables_prepopulate() |
233 | 233 | * @todo Remove eventually: Gravity Forms fixed this issue in 1.9.14 |
234 | 234 | */ |
235 | - if( false === $form ) { |
|
235 | + if ( false === $form ) { |
|
236 | 236 | return $text; |
237 | 237 | } |
238 | 238 | |
@@ -264,7 +264,7 @@ discard block |
||
264 | 264 | 'diff' => in_array( 'diff', $exploded ), // {date_created:diff} |
265 | 265 | 'raw' => in_array( 'raw', $exploded ), // {date_created:raw} |
266 | 266 | 'timestamp' => in_array( 'timestamp', $exploded ), // {date_created:timestamp} |
267 | - 'time' => in_array( 'time', $exploded ), // {date_created:time} |
|
267 | + 'time' => in_array( 'time', $exploded ), // {date_created:time} |
|
268 | 268 | ); |
269 | 269 | |
270 | 270 | $formatted_date = GVCommon::format_date( $date_created, $atts ); |
@@ -327,14 +327,14 @@ discard block |
||
327 | 327 | preg_match_all( "/{get:(.*?)}/ism", $text, $matches, PREG_SET_ORDER ); |
328 | 328 | |
329 | 329 | // If there are no matches OR the Entry `created_by` isn't set or is 0 (no user) |
330 | - if( empty( $matches ) ) { |
|
330 | + if ( empty( $matches ) ) { |
|
331 | 331 | return $text; |
332 | 332 | } |
333 | 333 | |
334 | 334 | foreach ( $matches as $match ) { |
335 | 335 | |
336 | - $full_tag = $match[0]; |
|
337 | - $property = $match[1]; |
|
336 | + $full_tag = $match[ 0 ]; |
|
337 | + $property = $match[ 1 ]; |
|
338 | 338 | |
339 | 339 | $value = stripslashes_deep( rgget( $property ) ); |
340 | 340 | |
@@ -358,7 +358,7 @@ discard block |
||
358 | 358 | * @since 1.15 |
359 | 359 | * @param bool $esc_html Whether to esc_html() the value. Default: `true` |
360 | 360 | */ |
361 | - $esc_html = apply_filters('gravityview/merge_tags/get/esc_html/' . $property, true ); |
|
361 | + $esc_html = apply_filters( 'gravityview/merge_tags/get/esc_html/' . $property, true ); |
|
362 | 362 | |
363 | 363 | $value = $esc_html ? esc_html( $value ) : $value; |
364 | 364 | |
@@ -369,7 +369,7 @@ discard block |
||
369 | 369 | * @param[in] array $form Gravity Forms form array |
370 | 370 | * @param[in] array $entry Entry array |
371 | 371 | */ |
372 | - $value = apply_filters('gravityview/merge_tags/get/value/' . $property, $value, $text, $form, $entry ); |
|
372 | + $value = apply_filters( 'gravityview/merge_tags/get/value/' . $property, $value, $text, $form, $entry ); |
|
373 | 373 | |
374 | 374 | $text = str_replace( $full_tag, $value, $text ); |
375 | 375 | } |