@@ -5,251 +5,251 @@ |
||
5 | 5 | */ |
6 | 6 | class GravityView_Change_Entry_Creator { |
7 | 7 | |
8 | - function __construct() { |
|
9 | - |
|
10 | - /** |
|
11 | - * @since 1.5.1 |
|
12 | - */ |
|
13 | - add_action('gform_user_registered', array( $this, 'assign_new_user_to_lead'), 10, 4 ); |
|
14 | - |
|
15 | - // ONLY ADMIN FROM HERE ON. |
|
16 | - if( !is_admin() ) { return; } |
|
17 | - |
|
18 | - /** |
|
19 | - * @filter `gravityview_disable_change_entry_creator` Disable the Change Entry Creator functionality |
|
20 | - * @since 1.7.4 |
|
21 | - * @param boolean $disable Disable the Change Entry Creator functionality. Default: false. |
|
22 | - */ |
|
23 | - if( apply_filters('gravityview_disable_change_entry_creator', false ) ) { |
|
24 | - return; |
|
25 | - } |
|
26 | - |
|
27 | - /** |
|
28 | - * Use `init` to fix bbPress warning |
|
29 | - * @see https://bbpress.trac.wordpress.org/ticket/2309 |
|
30 | - */ |
|
31 | - add_action('init', array( $this, 'load'), 100 ); |
|
32 | - |
|
33 | - add_action('plugins_loaded', array( $this, 'prevent_conflicts') ); |
|
34 | - |
|
35 | - } |
|
36 | - |
|
37 | - /** |
|
38 | - * When an user is created using the User Registration add-on, assign the entry to them |
|
39 | - * |
|
40 | - * @since 1.5.1 |
|
41 | - * @uses RGFormsModel::update_lead_property() Modify the entry `created_by` field |
|
42 | - * @param int $user_id WordPress User ID |
|
43 | - * @param array $config User registration feed configuration |
|
44 | - * @param array $entry GF Entry array |
|
45 | - * @param string $password User password |
|
46 | - * @return void |
|
47 | - */ |
|
48 | - function assign_new_user_to_lead( $user_id, $config, $entry = array(), $password = '' ) { |
|
49 | - |
|
50 | - /** |
|
51 | - * Disable assigning the new user to the entry by returning false. |
|
52 | - * @param int $user_id WordPress User ID |
|
53 | - * @param array $config User registration feed configuration |
|
54 | - * @param array $entry GF Entry array |
|
55 | - */ |
|
56 | - $assign_to_lead = apply_filters( 'gravityview_assign_new_user_to_entry', true, $user_id, $config, $entry ); |
|
57 | - |
|
58 | - // If filter returns false, do not process |
|
59 | - if( empty( $assign_to_lead ) ) { |
|
60 | - return; |
|
61 | - } |
|
62 | - |
|
63 | - // Update the entry. The `false` prevents checking Akismet; `true` disables the user updated hook from firing |
|
64 | - $result = RGFormsModel::update_lead_property( $entry['id'], 'created_by', $user_id, false, true ); |
|
65 | - |
|
66 | - if( empty( $result ) ) { |
|
67 | - $status = __('Error', 'gravityview'); |
|
68 | - } else { |
|
69 | - $status = __('Success', 'gravityview'); |
|
70 | - } |
|
71 | - |
|
72 | - $note = sprintf( _x('%s: Assigned User ID #%d as the entry creator.', 'First parameter: Success or error of the action. Second: User ID number', 'gravityview'), $status, $user_id ); |
|
73 | - |
|
74 | - do_action( 'gravityview_log_debug', 'GravityView_Change_Entry_Creator[assign_new_user_to_lead] - '.$note ); |
|
75 | - |
|
76 | - /** |
|
77 | - * @filter `gravityview_disable_change_entry_creator_note` Disable adding a note when changing the entry creator |
|
78 | - * @since 1.21.5 |
|
79 | - * @param boolean $disable Disable the Change Entry Creator note. Default: false. |
|
80 | - */ |
|
81 | - if( apply_filters('gravityview_disable_change_entry_creator_note', false ) ) { |
|
82 | - return; |
|
83 | - } |
|
84 | - |
|
85 | - GravityView_Entry_Notes::add_note( $entry['id'], -1, 'GravityView', $note, 'gravityview' ); |
|
8 | + function __construct() { |
|
9 | + |
|
10 | + /** |
|
11 | + * @since 1.5.1 |
|
12 | + */ |
|
13 | + add_action('gform_user_registered', array( $this, 'assign_new_user_to_lead'), 10, 4 ); |
|
14 | + |
|
15 | + // ONLY ADMIN FROM HERE ON. |
|
16 | + if( !is_admin() ) { return; } |
|
17 | + |
|
18 | + /** |
|
19 | + * @filter `gravityview_disable_change_entry_creator` Disable the Change Entry Creator functionality |
|
20 | + * @since 1.7.4 |
|
21 | + * @param boolean $disable Disable the Change Entry Creator functionality. Default: false. |
|
22 | + */ |
|
23 | + if( apply_filters('gravityview_disable_change_entry_creator', false ) ) { |
|
24 | + return; |
|
25 | + } |
|
26 | + |
|
27 | + /** |
|
28 | + * Use `init` to fix bbPress warning |
|
29 | + * @see https://bbpress.trac.wordpress.org/ticket/2309 |
|
30 | + */ |
|
31 | + add_action('init', array( $this, 'load'), 100 ); |
|
32 | + |
|
33 | + add_action('plugins_loaded', array( $this, 'prevent_conflicts') ); |
|
34 | + |
|
35 | + } |
|
36 | + |
|
37 | + /** |
|
38 | + * When an user is created using the User Registration add-on, assign the entry to them |
|
39 | + * |
|
40 | + * @since 1.5.1 |
|
41 | + * @uses RGFormsModel::update_lead_property() Modify the entry `created_by` field |
|
42 | + * @param int $user_id WordPress User ID |
|
43 | + * @param array $config User registration feed configuration |
|
44 | + * @param array $entry GF Entry array |
|
45 | + * @param string $password User password |
|
46 | + * @return void |
|
47 | + */ |
|
48 | + function assign_new_user_to_lead( $user_id, $config, $entry = array(), $password = '' ) { |
|
49 | + |
|
50 | + /** |
|
51 | + * Disable assigning the new user to the entry by returning false. |
|
52 | + * @param int $user_id WordPress User ID |
|
53 | + * @param array $config User registration feed configuration |
|
54 | + * @param array $entry GF Entry array |
|
55 | + */ |
|
56 | + $assign_to_lead = apply_filters( 'gravityview_assign_new_user_to_entry', true, $user_id, $config, $entry ); |
|
57 | + |
|
58 | + // If filter returns false, do not process |
|
59 | + if( empty( $assign_to_lead ) ) { |
|
60 | + return; |
|
61 | + } |
|
62 | + |
|
63 | + // Update the entry. The `false` prevents checking Akismet; `true` disables the user updated hook from firing |
|
64 | + $result = RGFormsModel::update_lead_property( $entry['id'], 'created_by', $user_id, false, true ); |
|
65 | + |
|
66 | + if( empty( $result ) ) { |
|
67 | + $status = __('Error', 'gravityview'); |
|
68 | + } else { |
|
69 | + $status = __('Success', 'gravityview'); |
|
70 | + } |
|
71 | + |
|
72 | + $note = sprintf( _x('%s: Assigned User ID #%d as the entry creator.', 'First parameter: Success or error of the action. Second: User ID number', 'gravityview'), $status, $user_id ); |
|
73 | + |
|
74 | + do_action( 'gravityview_log_debug', 'GravityView_Change_Entry_Creator[assign_new_user_to_lead] - '.$note ); |
|
75 | + |
|
76 | + /** |
|
77 | + * @filter `gravityview_disable_change_entry_creator_note` Disable adding a note when changing the entry creator |
|
78 | + * @since 1.21.5 |
|
79 | + * @param boolean $disable Disable the Change Entry Creator note. Default: false. |
|
80 | + */ |
|
81 | + if( apply_filters('gravityview_disable_change_entry_creator_note', false ) ) { |
|
82 | + return; |
|
83 | + } |
|
84 | + |
|
85 | + GravityView_Entry_Notes::add_note( $entry['id'], -1, 'GravityView', $note, 'gravityview' ); |
|
86 | 86 | |
87 | - } |
|
87 | + } |
|
88 | 88 | |
89 | - /** |
|
90 | - * Disable previous functionality; use this one as the canonical. |
|
91 | - * @return void |
|
92 | - */ |
|
93 | - function prevent_conflicts() { |
|
89 | + /** |
|
90 | + * Disable previous functionality; use this one as the canonical. |
|
91 | + * @return void |
|
92 | + */ |
|
93 | + function prevent_conflicts() { |
|
94 | 94 | |
95 | - // Plugin that was provided here: |
|
96 | - // @link https://gravityview.co/support/documentation/201991205/ |
|
97 | - remove_action("gform_entry_info", 'gravityview_change_entry_creator_form', 10 ); |
|
98 | - remove_action("gform_after_update_entry", 'gravityview_update_entry_creator', 10 ); |
|
99 | - |
|
100 | - // Disable for Gravity Forms Add-ons 3.6.2 and lower |
|
101 | - if( class_exists( 'KWS_GF_Change_Lead_Creator' ) ) { |
|
95 | + // Plugin that was provided here: |
|
96 | + // @link https://gravityview.co/support/documentation/201991205/ |
|
97 | + remove_action("gform_entry_info", 'gravityview_change_entry_creator_form', 10 ); |
|
98 | + remove_action("gform_after_update_entry", 'gravityview_update_entry_creator', 10 ); |
|
99 | + |
|
100 | + // Disable for Gravity Forms Add-ons 3.6.2 and lower |
|
101 | + if( class_exists( 'KWS_GF_Change_Lead_Creator' ) ) { |
|
102 | 102 | |
103 | - $Old_Lead_Creator = new KWS_GF_Change_Lead_Creator; |
|
103 | + $Old_Lead_Creator = new KWS_GF_Change_Lead_Creator; |
|
104 | 104 | |
105 | - // Now, no validation is required in the methods; let's hook in. |
|
106 | - remove_action('admin_init', array( $Old_Lead_Creator, 'set_screen_mode' ) ); |
|
107 | - |
|
108 | - remove_action("gform_entry_info", array( $Old_Lead_Creator, 'add_select' ), 10 ); |
|
109 | - |
|
110 | - remove_action("gform_after_update_entry", array( $Old_Lead_Creator, 'update_entry_creator' ), 10 ); |
|
111 | - } |
|
112 | - |
|
113 | - } |
|
114 | - |
|
115 | - /** |
|
116 | - * @since 3.6.3 |
|
117 | - * @return void |
|
118 | - */ |
|
119 | - function load() { |
|
105 | + // Now, no validation is required in the methods; let's hook in. |
|
106 | + remove_action('admin_init', array( $Old_Lead_Creator, 'set_screen_mode' ) ); |
|
107 | + |
|
108 | + remove_action("gform_entry_info", array( $Old_Lead_Creator, 'add_select' ), 10 ); |
|
109 | + |
|
110 | + remove_action("gform_after_update_entry", array( $Old_Lead_Creator, 'update_entry_creator' ), 10 ); |
|
111 | + } |
|
112 | + |
|
113 | + } |
|
114 | + |
|
115 | + /** |
|
116 | + * @since 3.6.3 |
|
117 | + * @return void |
|
118 | + */ |
|
119 | + function load() { |
|
120 | 120 | |
121 | - // Does GF exist? |
|
122 | - if( !class_exists('GFCommon') ) { |
|
123 | - return; |
|
124 | - } |
|
121 | + // Does GF exist? |
|
122 | + if( !class_exists('GFCommon') ) { |
|
123 | + return; |
|
124 | + } |
|
125 | 125 | |
126 | - // Can the user edit entries? |
|
127 | - if( ! GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_entries' ) ) ) { |
|
128 | - return; |
|
129 | - } |
|
126 | + // Can the user edit entries? |
|
127 | + if( ! GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_entries' ) ) ) { |
|
128 | + return; |
|
129 | + } |
|
130 | 130 | |
131 | - // If screen mode isn't set, then we're in the wrong place. |
|
132 | - if( empty( $_REQUEST['screen_mode'] ) ) { |
|
133 | - return; |
|
134 | - } |
|
131 | + // If screen mode isn't set, then we're in the wrong place. |
|
132 | + if( empty( $_REQUEST['screen_mode'] ) ) { |
|
133 | + return; |
|
134 | + } |
|
135 | 135 | |
136 | - // Now, no validation is required in the methods; let's hook in. |
|
137 | - add_action('admin_init', array( &$this, 'set_screen_mode' ) ); |
|
136 | + // Now, no validation is required in the methods; let's hook in. |
|
137 | + add_action('admin_init', array( &$this, 'set_screen_mode' ) ); |
|
138 | 138 | |
139 | - add_action("gform_entry_info", array( &$this, 'add_select' ), 10, 2); |
|
139 | + add_action("gform_entry_info", array( &$this, 'add_select' ), 10, 2); |
|
140 | 140 | |
141 | - add_action("gform_after_update_entry", array( &$this, 'update_entry_creator' ), 10, 2); |
|
141 | + add_action("gform_after_update_entry", array( &$this, 'update_entry_creator' ), 10, 2); |
|
142 | 142 | |
143 | - } |
|
143 | + } |
|
144 | 144 | |
145 | - /** |
|
146 | - * Allows for edit links to work with a link instead of a form (GET instead of POST) |
|
147 | - * @return void |
|
148 | - */ |
|
149 | - function set_screen_mode() { |
|
145 | + /** |
|
146 | + * Allows for edit links to work with a link instead of a form (GET instead of POST) |
|
147 | + * @return void |
|
148 | + */ |
|
149 | + function set_screen_mode() { |
|
150 | 150 | |
151 | - // If $_GET['screen_mode'] is set to edit, set $_POST value |
|
152 | - if( rgget('screen_mode') === 'edit' ) { |
|
153 | - $_POST["screen_mode"] = 'edit'; |
|
154 | - } |
|
151 | + // If $_GET['screen_mode'] is set to edit, set $_POST value |
|
152 | + if( rgget('screen_mode') === 'edit' ) { |
|
153 | + $_POST["screen_mode"] = 'edit'; |
|
154 | + } |
|
155 | 155 | |
156 | - } |
|
156 | + } |
|
157 | 157 | |
158 | - /** |
|
159 | - * When the entry creator is changed, add a note to the entry |
|
160 | - * @param array $form GF entry array |
|
161 | - * @param int $entry_id Entry ID |
|
162 | - * @return void |
|
163 | - */ |
|
164 | - function update_entry_creator($form, $entry_id) { |
|
165 | - global $current_user; |
|
158 | + /** |
|
159 | + * When the entry creator is changed, add a note to the entry |
|
160 | + * @param array $form GF entry array |
|
161 | + * @param int $entry_id Entry ID |
|
162 | + * @return void |
|
163 | + */ |
|
164 | + function update_entry_creator($form, $entry_id) { |
|
165 | + global $current_user; |
|
166 | 166 | |
167 | - // Update the entry |
|
168 | - $created_by = absint( rgpost('created_by') ); |
|
167 | + // Update the entry |
|
168 | + $created_by = absint( rgpost('created_by') ); |
|
169 | 169 | |
170 | - RGFormsModel::update_lead_property( $entry_id, 'created_by', $created_by ); |
|
170 | + RGFormsModel::update_lead_property( $entry_id, 'created_by', $created_by ); |
|
171 | 171 | |
172 | - // If the creator has changed, let's add a note about who it used to be. |
|
173 | - $originally_created_by = rgpost('originally_created_by'); |
|
172 | + // If the creator has changed, let's add a note about who it used to be. |
|
173 | + $originally_created_by = rgpost('originally_created_by'); |
|
174 | 174 | |
175 | - // If there's no owner and there didn't used to be, keep going |
|
176 | - if( empty( $originally_created_by ) && empty( $created_by ) ) { |
|
177 | - return; |
|
178 | - } |
|
175 | + // If there's no owner and there didn't used to be, keep going |
|
176 | + if( empty( $originally_created_by ) && empty( $created_by ) ) { |
|
177 | + return; |
|
178 | + } |
|
179 | 179 | |
180 | - // If the values have changed |
|
181 | - if( absint( $originally_created_by ) !== absint( $created_by ) ) { |
|
180 | + // If the values have changed |
|
181 | + if( absint( $originally_created_by ) !== absint( $created_by ) ) { |
|
182 | 182 | |
183 | - $user_data = get_userdata($current_user->ID); |
|
183 | + $user_data = get_userdata($current_user->ID); |
|
184 | 184 | |
185 | - $user_format = _x('%s (ID #%d)', 'The name and the ID of users who initiated changes to entry ownership', 'gravityview'); |
|
185 | + $user_format = _x('%s (ID #%d)', 'The name and the ID of users who initiated changes to entry ownership', 'gravityview'); |
|
186 | 186 | |
187 | - $original_name = $created_by_name = esc_attr_x( 'No User', 'To show that the entry was unassigned from an actual user to no user.', 'gravityview'); |
|
187 | + $original_name = $created_by_name = esc_attr_x( 'No User', 'To show that the entry was unassigned from an actual user to no user.', 'gravityview'); |
|
188 | 188 | |
189 | - if( !empty( $originally_created_by ) ) { |
|
190 | - $originally_created_by_user_data = get_userdata($originally_created_by); |
|
191 | - $original_name = sprintf( $user_format, $originally_created_by_user_data->display_name, $originally_created_by_user_data->ID ); |
|
192 | - } |
|
189 | + if( !empty( $originally_created_by ) ) { |
|
190 | + $originally_created_by_user_data = get_userdata($originally_created_by); |
|
191 | + $original_name = sprintf( $user_format, $originally_created_by_user_data->display_name, $originally_created_by_user_data->ID ); |
|
192 | + } |
|
193 | 193 | |
194 | - if( !empty( $created_by ) ) { |
|
195 | - $created_by_user_data = get_userdata($created_by); |
|
196 | - $created_by_name = sprintf( $user_format, $created_by_user_data->display_name, $created_by_user_data->ID ); |
|
197 | - } |
|
194 | + if( !empty( $created_by ) ) { |
|
195 | + $created_by_user_data = get_userdata($created_by); |
|
196 | + $created_by_name = sprintf( $user_format, $created_by_user_data->display_name, $created_by_user_data->ID ); |
|
197 | + } |
|
198 | 198 | |
199 | - GravityView_Entry_Notes::add_note( $entry_id, $current_user->ID, $user_data->display_name, sprintf( __('Changed entry creator from %s to %s', 'gravityview'), $original_name, $created_by_name ), 'note' ); |
|
200 | - } |
|
199 | + GravityView_Entry_Notes::add_note( $entry_id, $current_user->ID, $user_data->display_name, sprintf( __('Changed entry creator from %s to %s', 'gravityview'), $original_name, $created_by_name ), 'note' ); |
|
200 | + } |
|
201 | 201 | |
202 | - } |
|
202 | + } |
|
203 | 203 | |
204 | - /** |
|
205 | - * Output the select to change the entry creator |
|
206 | - * @param int $form_id GF Form ID |
|
207 | - * @param array $entry GF entry array |
|
208 | - * @return void |
|
209 | - */ |
|
210 | - function add_select($form_id, $entry ) { |
|
204 | + /** |
|
205 | + * Output the select to change the entry creator |
|
206 | + * @param int $form_id GF Form ID |
|
207 | + * @param array $entry GF entry array |
|
208 | + * @return void |
|
209 | + */ |
|
210 | + function add_select($form_id, $entry ) { |
|
211 | 211 | |
212 | - if( rgpost('screen_mode') !== 'edit' ) { |
|
213 | - return; |
|
214 | - } |
|
212 | + if( rgpost('screen_mode') !== 'edit' ) { |
|
213 | + return; |
|
214 | + } |
|
215 | 215 | |
216 | - $created_by_id = rgar( $entry, 'created_by' ); |
|
216 | + $created_by_id = rgar( $entry, 'created_by' ); |
|
217 | 217 | |
218 | - $users = GVCommon::get_users( 'change_entry_creator' ); |
|
218 | + $users = GVCommon::get_users( 'change_entry_creator' ); |
|
219 | 219 | |
220 | - $is_created_by_in_users = wp_list_filter( $users, array( 'ID' => $created_by_id ) ); |
|
220 | + $is_created_by_in_users = wp_list_filter( $users, array( 'ID' => $created_by_id ) ); |
|
221 | 221 | |
222 | - // Make sure that the entry creator is included in the users list. If not, add them. |
|
223 | - if ( ! empty( $created_by_id ) && empty( $is_created_by_in_users ) ) { |
|
222 | + // Make sure that the entry creator is included in the users list. If not, add them. |
|
223 | + if ( ! empty( $created_by_id ) && empty( $is_created_by_in_users ) ) { |
|
224 | 224 | |
225 | - if ( $created_by_user = GVCommon::get_users( 'change_entry_creator', array( 'include' => $created_by_id ) ) ) { |
|
226 | - $users = array_merge( $users, $created_by_user ); |
|
227 | - } |
|
228 | - } |
|
225 | + if ( $created_by_user = GVCommon::get_users( 'change_entry_creator', array( 'include' => $created_by_id ) ) ) { |
|
226 | + $users = array_merge( $users, $created_by_user ); |
|
227 | + } |
|
228 | + } |
|
229 | 229 | |
230 | - $output = '<label for="change_created_by">'; |
|
231 | - $output .= esc_html__('Change Entry Creator:', 'gravityview'); |
|
232 | - $output .= '</label>'; |
|
230 | + $output = '<label for="change_created_by">'; |
|
231 | + $output .= esc_html__('Change Entry Creator:', 'gravityview'); |
|
232 | + $output .= '</label>'; |
|
233 | 233 | |
234 | - // If there are users who are not being shown, show a warning. |
|
235 | - // TODO: Use AJAX instead of <select> |
|
236 | - $count_users = count_users(); |
|
237 | - if( sizeof( $users ) < $count_users['total_users'] ) { |
|
238 | - $output .= '<p><i class="dashicons dashicons-warning"></i> ' . sprintf( esc_html__( 'The displayed list of users has been trimmed due to the large number of users. %sLearn how to remove this limit%s.', 'gravityview' ), '<a href="https://docs.gravityview.co/article/251-i-only-see-some-users-in-the-change-entry-creator-dropdown" rel="external">', '</a>' ) . '</p>'; |
|
239 | - } |
|
234 | + // If there are users who are not being shown, show a warning. |
|
235 | + // TODO: Use AJAX instead of <select> |
|
236 | + $count_users = count_users(); |
|
237 | + if( sizeof( $users ) < $count_users['total_users'] ) { |
|
238 | + $output .= '<p><i class="dashicons dashicons-warning"></i> ' . sprintf( esc_html__( 'The displayed list of users has been trimmed due to the large number of users. %sLearn how to remove this limit%s.', 'gravityview' ), '<a href="https://docs.gravityview.co/article/251-i-only-see-some-users-in-the-change-entry-creator-dropdown" rel="external">', '</a>' ) . '</p>'; |
|
239 | + } |
|
240 | 240 | |
241 | - $output .= '<select name="created_by" id="change_created_by" class="widefat">'; |
|
242 | - $output .= '<option value="' . selected( $entry['created_by'], '0', false ) . '"> — '.esc_attr_x( 'No User', 'No user assigned to the entry', 'gravityview').' — </option>'; |
|
243 | - foreach($users as $user) { |
|
244 | - $output .= '<option value="'. $user->ID .'"'. selected( $entry['created_by'], $user->ID, false ).'>'.esc_attr( $user->display_name.' ('.$user->user_nicename.')' ).'</option>'; |
|
245 | - } |
|
246 | - $output .= '</select>'; |
|
247 | - $output .= '<input name="originally_created_by" value="'.esc_attr( $entry['created_by'] ).'" type="hidden" />'; |
|
241 | + $output .= '<select name="created_by" id="change_created_by" class="widefat">'; |
|
242 | + $output .= '<option value="' . selected( $entry['created_by'], '0', false ) . '"> — '.esc_attr_x( 'No User', 'No user assigned to the entry', 'gravityview').' — </option>'; |
|
243 | + foreach($users as $user) { |
|
244 | + $output .= '<option value="'. $user->ID .'"'. selected( $entry['created_by'], $user->ID, false ).'>'.esc_attr( $user->display_name.' ('.$user->user_nicename.')' ).'</option>'; |
|
245 | + } |
|
246 | + $output .= '</select>'; |
|
247 | + $output .= '<input name="originally_created_by" value="'.esc_attr( $entry['created_by'] ).'" type="hidden" />'; |
|
248 | 248 | |
249 | - unset( $is_created_by_in_users, $created_by_user, $users, $created_by_id, $count_users ); |
|
249 | + unset( $is_created_by_in_users, $created_by_user, $users, $created_by_id, $count_users ); |
|
250 | 250 | |
251 | - echo $output; |
|
252 | - } |
|
251 | + echo $output; |
|
252 | + } |
|
253 | 253 | |
254 | 254 | } |
255 | 255 |
@@ -10,17 +10,17 @@ discard block |
||
10 | 10 | /** |
11 | 11 | * @since 1.5.1 |
12 | 12 | */ |
13 | - add_action('gform_user_registered', array( $this, 'assign_new_user_to_lead'), 10, 4 ); |
|
13 | + add_action( 'gform_user_registered', array( $this, 'assign_new_user_to_lead' ), 10, 4 ); |
|
14 | 14 | |
15 | 15 | // ONLY ADMIN FROM HERE ON. |
16 | - if( !is_admin() ) { return; } |
|
16 | + if ( ! is_admin() ) { return; } |
|
17 | 17 | |
18 | 18 | /** |
19 | 19 | * @filter `gravityview_disable_change_entry_creator` Disable the Change Entry Creator functionality |
20 | 20 | * @since 1.7.4 |
21 | 21 | * @param boolean $disable Disable the Change Entry Creator functionality. Default: false. |
22 | 22 | */ |
23 | - if( apply_filters('gravityview_disable_change_entry_creator', false ) ) { |
|
23 | + if ( apply_filters( 'gravityview_disable_change_entry_creator', false ) ) { |
|
24 | 24 | return; |
25 | 25 | } |
26 | 26 | |
@@ -28,9 +28,9 @@ discard block |
||
28 | 28 | * Use `init` to fix bbPress warning |
29 | 29 | * @see https://bbpress.trac.wordpress.org/ticket/2309 |
30 | 30 | */ |
31 | - add_action('init', array( $this, 'load'), 100 ); |
|
31 | + add_action( 'init', array( $this, 'load' ), 100 ); |
|
32 | 32 | |
33 | - add_action('plugins_loaded', array( $this, 'prevent_conflicts') ); |
|
33 | + add_action( 'plugins_loaded', array( $this, 'prevent_conflicts' ) ); |
|
34 | 34 | |
35 | 35 | } |
36 | 36 | |
@@ -56,33 +56,33 @@ discard block |
||
56 | 56 | $assign_to_lead = apply_filters( 'gravityview_assign_new_user_to_entry', true, $user_id, $config, $entry ); |
57 | 57 | |
58 | 58 | // If filter returns false, do not process |
59 | - if( empty( $assign_to_lead ) ) { |
|
59 | + if ( empty( $assign_to_lead ) ) { |
|
60 | 60 | return; |
61 | 61 | } |
62 | 62 | |
63 | 63 | // Update the entry. The `false` prevents checking Akismet; `true` disables the user updated hook from firing |
64 | - $result = RGFormsModel::update_lead_property( $entry['id'], 'created_by', $user_id, false, true ); |
|
64 | + $result = RGFormsModel::update_lead_property( $entry[ 'id' ], 'created_by', $user_id, false, true ); |
|
65 | 65 | |
66 | - if( empty( $result ) ) { |
|
67 | - $status = __('Error', 'gravityview'); |
|
66 | + if ( empty( $result ) ) { |
|
67 | + $status = __( 'Error', 'gravityview' ); |
|
68 | 68 | } else { |
69 | - $status = __('Success', 'gravityview'); |
|
69 | + $status = __( 'Success', 'gravityview' ); |
|
70 | 70 | } |
71 | 71 | |
72 | - $note = sprintf( _x('%s: Assigned User ID #%d as the entry creator.', 'First parameter: Success or error of the action. Second: User ID number', 'gravityview'), $status, $user_id ); |
|
72 | + $note = sprintf( _x( '%s: Assigned User ID #%d as the entry creator.', 'First parameter: Success or error of the action. Second: User ID number', 'gravityview' ), $status, $user_id ); |
|
73 | 73 | |
74 | - do_action( 'gravityview_log_debug', 'GravityView_Change_Entry_Creator[assign_new_user_to_lead] - '.$note ); |
|
74 | + do_action( 'gravityview_log_debug', 'GravityView_Change_Entry_Creator[assign_new_user_to_lead] - ' . $note ); |
|
75 | 75 | |
76 | 76 | /** |
77 | 77 | * @filter `gravityview_disable_change_entry_creator_note` Disable adding a note when changing the entry creator |
78 | 78 | * @since 1.21.5 |
79 | 79 | * @param boolean $disable Disable the Change Entry Creator note. Default: false. |
80 | 80 | */ |
81 | - if( apply_filters('gravityview_disable_change_entry_creator_note', false ) ) { |
|
81 | + if ( apply_filters( 'gravityview_disable_change_entry_creator_note', false ) ) { |
|
82 | 82 | return; |
83 | 83 | } |
84 | 84 | |
85 | - GravityView_Entry_Notes::add_note( $entry['id'], -1, 'GravityView', $note, 'gravityview' ); |
|
85 | + GravityView_Entry_Notes::add_note( $entry[ 'id' ], -1, 'GravityView', $note, 'gravityview' ); |
|
86 | 86 | |
87 | 87 | } |
88 | 88 | |
@@ -94,20 +94,20 @@ discard block |
||
94 | 94 | |
95 | 95 | // Plugin that was provided here: |
96 | 96 | // @link https://gravityview.co/support/documentation/201991205/ |
97 | - remove_action("gform_entry_info", 'gravityview_change_entry_creator_form', 10 ); |
|
98 | - remove_action("gform_after_update_entry", 'gravityview_update_entry_creator', 10 ); |
|
97 | + remove_action( "gform_entry_info", 'gravityview_change_entry_creator_form', 10 ); |
|
98 | + remove_action( "gform_after_update_entry", 'gravityview_update_entry_creator', 10 ); |
|
99 | 99 | |
100 | 100 | // Disable for Gravity Forms Add-ons 3.6.2 and lower |
101 | - if( class_exists( 'KWS_GF_Change_Lead_Creator' ) ) { |
|
101 | + if ( class_exists( 'KWS_GF_Change_Lead_Creator' ) ) { |
|
102 | 102 | |
103 | 103 | $Old_Lead_Creator = new KWS_GF_Change_Lead_Creator; |
104 | 104 | |
105 | 105 | // Now, no validation is required in the methods; let's hook in. |
106 | - remove_action('admin_init', array( $Old_Lead_Creator, 'set_screen_mode' ) ); |
|
106 | + remove_action( 'admin_init', array( $Old_Lead_Creator, 'set_screen_mode' ) ); |
|
107 | 107 | |
108 | - remove_action("gform_entry_info", array( $Old_Lead_Creator, 'add_select' ), 10 ); |
|
108 | + remove_action( "gform_entry_info", array( $Old_Lead_Creator, 'add_select' ), 10 ); |
|
109 | 109 | |
110 | - remove_action("gform_after_update_entry", array( $Old_Lead_Creator, 'update_entry_creator' ), 10 ); |
|
110 | + remove_action( "gform_after_update_entry", array( $Old_Lead_Creator, 'update_entry_creator' ), 10 ); |
|
111 | 111 | } |
112 | 112 | |
113 | 113 | } |
@@ -119,26 +119,26 @@ discard block |
||
119 | 119 | function load() { |
120 | 120 | |
121 | 121 | // Does GF exist? |
122 | - if( !class_exists('GFCommon') ) { |
|
122 | + if ( ! class_exists( 'GFCommon' ) ) { |
|
123 | 123 | return; |
124 | 124 | } |
125 | 125 | |
126 | 126 | // Can the user edit entries? |
127 | - if( ! GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_entries' ) ) ) { |
|
127 | + if ( ! GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_entries' ) ) ) { |
|
128 | 128 | return; |
129 | 129 | } |
130 | 130 | |
131 | 131 | // If screen mode isn't set, then we're in the wrong place. |
132 | - if( empty( $_REQUEST['screen_mode'] ) ) { |
|
132 | + if ( empty( $_REQUEST[ 'screen_mode' ] ) ) { |
|
133 | 133 | return; |
134 | 134 | } |
135 | 135 | |
136 | 136 | // Now, no validation is required in the methods; let's hook in. |
137 | - add_action('admin_init', array( &$this, 'set_screen_mode' ) ); |
|
137 | + add_action( 'admin_init', array( &$this, 'set_screen_mode' ) ); |
|
138 | 138 | |
139 | - add_action("gform_entry_info", array( &$this, 'add_select' ), 10, 2); |
|
139 | + add_action( "gform_entry_info", array( &$this, 'add_select' ), 10, 2 ); |
|
140 | 140 | |
141 | - add_action("gform_after_update_entry", array( &$this, 'update_entry_creator' ), 10, 2); |
|
141 | + add_action( "gform_after_update_entry", array( &$this, 'update_entry_creator' ), 10, 2 ); |
|
142 | 142 | |
143 | 143 | } |
144 | 144 | |
@@ -149,8 +149,8 @@ discard block |
||
149 | 149 | function set_screen_mode() { |
150 | 150 | |
151 | 151 | // If $_GET['screen_mode'] is set to edit, set $_POST value |
152 | - if( rgget('screen_mode') === 'edit' ) { |
|
153 | - $_POST["screen_mode"] = 'edit'; |
|
152 | + if ( rgget( 'screen_mode' ) === 'edit' ) { |
|
153 | + $_POST[ "screen_mode" ] = 'edit'; |
|
154 | 154 | } |
155 | 155 | |
156 | 156 | } |
@@ -161,42 +161,42 @@ discard block |
||
161 | 161 | * @param int $entry_id Entry ID |
162 | 162 | * @return void |
163 | 163 | */ |
164 | - function update_entry_creator($form, $entry_id) { |
|
164 | + function update_entry_creator( $form, $entry_id ) { |
|
165 | 165 | global $current_user; |
166 | 166 | |
167 | 167 | // Update the entry |
168 | - $created_by = absint( rgpost('created_by') ); |
|
168 | + $created_by = absint( rgpost( 'created_by' ) ); |
|
169 | 169 | |
170 | 170 | RGFormsModel::update_lead_property( $entry_id, 'created_by', $created_by ); |
171 | 171 | |
172 | 172 | // If the creator has changed, let's add a note about who it used to be. |
173 | - $originally_created_by = rgpost('originally_created_by'); |
|
173 | + $originally_created_by = rgpost( 'originally_created_by' ); |
|
174 | 174 | |
175 | 175 | // If there's no owner and there didn't used to be, keep going |
176 | - if( empty( $originally_created_by ) && empty( $created_by ) ) { |
|
176 | + if ( empty( $originally_created_by ) && empty( $created_by ) ) { |
|
177 | 177 | return; |
178 | 178 | } |
179 | 179 | |
180 | 180 | // If the values have changed |
181 | - if( absint( $originally_created_by ) !== absint( $created_by ) ) { |
|
181 | + if ( absint( $originally_created_by ) !== absint( $created_by ) ) { |
|
182 | 182 | |
183 | - $user_data = get_userdata($current_user->ID); |
|
183 | + $user_data = get_userdata( $current_user->ID ); |
|
184 | 184 | |
185 | - $user_format = _x('%s (ID #%d)', 'The name and the ID of users who initiated changes to entry ownership', 'gravityview'); |
|
185 | + $user_format = _x( '%s (ID #%d)', 'The name and the ID of users who initiated changes to entry ownership', 'gravityview' ); |
|
186 | 186 | |
187 | - $original_name = $created_by_name = esc_attr_x( 'No User', 'To show that the entry was unassigned from an actual user to no user.', 'gravityview'); |
|
187 | + $original_name = $created_by_name = esc_attr_x( 'No User', 'To show that the entry was unassigned from an actual user to no user.', 'gravityview' ); |
|
188 | 188 | |
189 | - if( !empty( $originally_created_by ) ) { |
|
190 | - $originally_created_by_user_data = get_userdata($originally_created_by); |
|
189 | + if ( ! empty( $originally_created_by ) ) { |
|
190 | + $originally_created_by_user_data = get_userdata( $originally_created_by ); |
|
191 | 191 | $original_name = sprintf( $user_format, $originally_created_by_user_data->display_name, $originally_created_by_user_data->ID ); |
192 | 192 | } |
193 | 193 | |
194 | - if( !empty( $created_by ) ) { |
|
195 | - $created_by_user_data = get_userdata($created_by); |
|
194 | + if ( ! empty( $created_by ) ) { |
|
195 | + $created_by_user_data = get_userdata( $created_by ); |
|
196 | 196 | $created_by_name = sprintf( $user_format, $created_by_user_data->display_name, $created_by_user_data->ID ); |
197 | 197 | } |
198 | 198 | |
199 | - GravityView_Entry_Notes::add_note( $entry_id, $current_user->ID, $user_data->display_name, sprintf( __('Changed entry creator from %s to %s', 'gravityview'), $original_name, $created_by_name ), 'note' ); |
|
199 | + GravityView_Entry_Notes::add_note( $entry_id, $current_user->ID, $user_data->display_name, sprintf( __( 'Changed entry creator from %s to %s', 'gravityview' ), $original_name, $created_by_name ), 'note' ); |
|
200 | 200 | } |
201 | 201 | |
202 | 202 | } |
@@ -207,9 +207,9 @@ discard block |
||
207 | 207 | * @param array $entry GF entry array |
208 | 208 | * @return void |
209 | 209 | */ |
210 | - function add_select($form_id, $entry ) { |
|
210 | + function add_select( $form_id, $entry ) { |
|
211 | 211 | |
212 | - if( rgpost('screen_mode') !== 'edit' ) { |
|
212 | + if ( rgpost( 'screen_mode' ) !== 'edit' ) { |
|
213 | 213 | return; |
214 | 214 | } |
215 | 215 | |
@@ -228,23 +228,23 @@ discard block |
||
228 | 228 | } |
229 | 229 | |
230 | 230 | $output = '<label for="change_created_by">'; |
231 | - $output .= esc_html__('Change Entry Creator:', 'gravityview'); |
|
231 | + $output .= esc_html__( 'Change Entry Creator:', 'gravityview' ); |
|
232 | 232 | $output .= '</label>'; |
233 | 233 | |
234 | 234 | // If there are users who are not being shown, show a warning. |
235 | 235 | // TODO: Use AJAX instead of <select> |
236 | 236 | $count_users = count_users(); |
237 | - if( sizeof( $users ) < $count_users['total_users'] ) { |
|
237 | + if ( sizeof( $users ) < $count_users[ 'total_users' ] ) { |
|
238 | 238 | $output .= '<p><i class="dashicons dashicons-warning"></i> ' . sprintf( esc_html__( 'The displayed list of users has been trimmed due to the large number of users. %sLearn how to remove this limit%s.', 'gravityview' ), '<a href="https://docs.gravityview.co/article/251-i-only-see-some-users-in-the-change-entry-creator-dropdown" rel="external">', '</a>' ) . '</p>'; |
239 | 239 | } |
240 | 240 | |
241 | 241 | $output .= '<select name="created_by" id="change_created_by" class="widefat">'; |
242 | - $output .= '<option value="' . selected( $entry['created_by'], '0', false ) . '"> — '.esc_attr_x( 'No User', 'No user assigned to the entry', 'gravityview').' — </option>'; |
|
243 | - foreach($users as $user) { |
|
244 | - $output .= '<option value="'. $user->ID .'"'. selected( $entry['created_by'], $user->ID, false ).'>'.esc_attr( $user->display_name.' ('.$user->user_nicename.')' ).'</option>'; |
|
242 | + $output .= '<option value="' . selected( $entry[ 'created_by' ], '0', false ) . '"> — ' . esc_attr_x( 'No User', 'No user assigned to the entry', 'gravityview' ) . ' — </option>'; |
|
243 | + foreach ( $users as $user ) { |
|
244 | + $output .= '<option value="' . $user->ID . '"' . selected( $entry[ 'created_by' ], $user->ID, false ) . '>' . esc_attr( $user->display_name . ' (' . $user->user_nicename . ')' ) . '</option>'; |
|
245 | 245 | } |
246 | 246 | $output .= '</select>'; |
247 | - $output .= '<input name="originally_created_by" value="'.esc_attr( $entry['created_by'] ).'" type="hidden" />'; |
|
247 | + $output .= '<input name="originally_created_by" value="' . esc_attr( $entry[ 'created_by' ] ) . '" type="hidden" />'; |
|
248 | 248 | |
249 | 249 | unset( $is_created_by_in_users, $created_by_user, $users, $created_by_id, $count_users ); |
250 | 250 |