@@ -67,7 +67,7 @@ discard block |
||
67 | 67 | |
68 | 68 | parent::add_hooks(); |
69 | 69 | |
70 | - if( gravityview()->request->is_admin( '', null ) ) { |
|
70 | + if ( gravityview()->request->is_admin( '', null ) ) { |
|
71 | 71 | |
72 | 72 | // Make Yoast metabox go down to the bottom please. |
73 | 73 | add_filter( 'wpseo_metabox_prio', array( $this, 'return_low' ) ); |
@@ -92,8 +92,8 @@ discard block |
||
92 | 92 | */ |
93 | 93 | public function disable_content_analysis( $options ) { |
94 | 94 | |
95 | - $options['keyword_analysis_active'] = false; |
|
96 | - $options['content_analysis_active'] = false; |
|
95 | + $options[ 'keyword_analysis_active' ] = false; |
|
96 | + $options[ 'content_analysis_active' ] = false; |
|
97 | 97 | |
98 | 98 | return $options; |
99 | 99 | } |
@@ -112,8 +112,8 @@ discard block |
||
112 | 112 | global $pagenow; |
113 | 113 | |
114 | 114 | // New View page |
115 | - if( $pagenow === 'post-new.php' ) { |
|
116 | - $options['hideeditbox-gravityview'] = true; |
|
115 | + if ( $pagenow === 'post-new.php' ) { |
|
116 | + $options[ 'hideeditbox-gravityview' ] = true; |
|
117 | 117 | } |
118 | 118 | |
119 | 119 | return $options; |
@@ -38,7 +38,7 @@ discard block |
||
38 | 38 | */ |
39 | 39 | $is_page = false; |
40 | 40 | |
41 | - if( function_exists( '\get_current_screen' ) || function_exists( 'get_current_screen' ) ) { |
|
41 | + if ( function_exists( '\get_current_screen' ) || function_exists( 'get_current_screen' ) ) { |
|
42 | 42 | $current_screen = \get_current_screen(); |
43 | 43 | } else { |
44 | 44 | $current_screen = false; |
@@ -51,13 +51,13 @@ discard block |
||
51 | 51 | $is_page = 'single'; |
52 | 52 | } elseif ( $is_gv_settings = 'gravityview_page_gravityview_settings' === $current_screen->id ) { |
53 | 53 | $is_page = 'settings'; |
54 | - } elseif( $is_extensions = 'gravityview_page_gv-admin-installer' === $current_screen->id ) { |
|
54 | + } elseif ( $is_extensions = 'gravityview_page_gv-admin-installer' === $current_screen->id ) { |
|
55 | 55 | $is_page = 'downloads'; |
56 | - } elseif( $is_changelog = 'gravityview_page_gv-changelog' === $current_screen->id ) { |
|
56 | + } elseif ( $is_changelog = 'gravityview_page_gv-changelog' === $current_screen->id ) { |
|
57 | 57 | $is_page = 'changelog'; |
58 | - } elseif( $is_getting_started = 'gravityview_page_gv-getting-started' === $current_screen->id ) { |
|
58 | + } elseif ( $is_getting_started = 'gravityview_page_gv-getting-started' === $current_screen->id ) { |
|
59 | 59 | $is_page = 'getting-started'; |
60 | - } elseif( $is_credits = 'gravityview_page_gv-credits' === $current_screen->id ) { |
|
60 | + } elseif ( $is_credits = 'gravityview_page_gv-credits' === $current_screen->id ) { |
|
61 | 61 | $is_page = 'credits'; |
62 | 62 | } |
63 | 63 | } |
@@ -40,8 +40,7 @@ discard block |
||
40 | 40 | $tab = __( 'Edit Entry', 'gravityview' ); |
41 | 41 | $context = 'edit'; |
42 | 42 | break; |
43 | - case ( $gravityview->request->is_entry( $gravityview->view->form ? $gravityview->view->form->ID : 0 ) ): |
|
44 | - $tab = __( 'Single Entry', 'gravityview' ); |
|
43 | + case ( $gravityview->request->is_entry( $gravityview->view->form ? $gravityview->view->form->ID : 0 ) ) : $tab = __( 'Single Entry', 'gravityview' ); |
|
45 | 44 | $context = 'single'; |
46 | 45 | break; |
47 | 46 | default: |
@@ -61,7 +60,7 @@ discard block |
||
61 | 60 | $action_text = sprintf( esc_html__( 'Add fields to %s', 'gravityview' ), $tab ); |
62 | 61 | $message = esc_html__( 'You can only see this message because you are able to edit this View.', 'gravityview' ); |
63 | 62 | |
64 | - $image = sprintf( '<img alt="%s" src="%s" style="margin-top: 10px;" />', $tab, esc_url( plugins_url( sprintf( 'assets/images/tab-%s.png', $context ), GRAVITYVIEW_FILE ) ) ); |
|
63 | + $image = sprintf( '<img alt="%s" src="%s" style="margin-top: 10px;" />', $tab, esc_url( plugins_url( sprintf( 'assets/images/tab-%s.png', $context ), GRAVITYVIEW_FILE ) ) ); |
|
65 | 64 | $output = sprintf( '<h3>%s <strong><a href="%s">%s</a></strong></h3><p>%s</p>', $title, esc_url( $edit_link ), $action_text, $message ); |
66 | 65 | |
67 | 66 | echo \GVCommon::generate_notice( $output . $image, 'gv-error error', 'edit_gravityview', $gravityview->view->ID ); |
@@ -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 ( false === $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 | - gravityview()->log->debug( 'GravityView_Change_Entry_Creator[assign_new_user_to_lead] - {note}', array( 'note' => $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 ( false === $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 | + gravityview()->log->debug( 'GravityView_Change_Entry_Creator[assign_new_user_to_lead] - {note}', array( 'note' => $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( \GV\Utils::_GET( 'screen_mode' ) === 'edit' ) { |
|
153 | - $_POST["screen_mode"] = 'edit'; |
|
154 | - } |
|
151 | + // If $_GET['screen_mode'] is set to edit, set $_POST value |
|
152 | + if( \GV\Utils::_GET( '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( \GV\Utils::_POST( 'created_by') ); |
|
167 | + // Update the entry |
|
168 | + $created_by = absint( \GV\Utils::_POST( '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 = \GV\Utils::_POST( 'originally_created_by' ); |
|
172 | + // If the creator has changed, let's add a note about who it used to be. |
|
173 | + $originally_created_by = \GV\Utils::_POST( '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( \GV\Utils::_POST( 'screen_mode' ) !== 'edit' ) { |
|
213 | - return; |
|
214 | - } |
|
212 | + if( \GV\Utils::_POST( 'screen_mode' ) !== 'edit' ) { |
|
213 | + return; |
|
214 | + } |
|
215 | 215 | |
216 | - $created_by_id = \GV\Utils::get( $entry, 'created_by' ); |
|
216 | + $created_by_id = \GV\Utils::get( $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,20 +56,20 @@ 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 | 66 | if ( false === $result ) { |
67 | - $status = __('Error', 'gravityview'); |
|
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 | 74 | gravityview()->log->debug( 'GravityView_Change_Entry_Creator[assign_new_user_to_lead] - {note}', array( 'note' => $note ) ); |
75 | 75 | |
@@ -78,11 +78,11 @@ discard block |
||
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( \GV\Utils::_GET( 'screen_mode' ) === 'edit' ) { |
|
153 | - $_POST["screen_mode"] = 'edit'; |
|
152 | + if ( \GV\Utils::_GET( 'screen_mode' ) === 'edit' ) { |
|
153 | + $_POST[ "screen_mode" ] = 'edit'; |
|
154 | 154 | } |
155 | 155 | |
156 | 156 | } |
@@ -161,11 +161,11 @@ 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( \GV\Utils::_POST( 'created_by') ); |
|
168 | + $created_by = absint( \GV\Utils::_POST( 'created_by' ) ); |
|
169 | 169 | |
170 | 170 | RGFormsModel::update_lead_property( $entry_id, 'created_by', $created_by ); |
171 | 171 | |
@@ -173,30 +173,30 @@ discard block |
||
173 | 173 | $originally_created_by = \GV\Utils::_POST( '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( \GV\Utils::_POST( 'screen_mode' ) !== 'edit' ) { |
|
212 | + if ( \GV\Utils::_POST( '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 |