Skip to content

Commit 8b40661

Browse files
committed
Merge branch 'main' into develop
2 parents db94ee8 + 67ee01a commit 8b40661

2 files changed

Lines changed: 14 additions & 0 deletions

File tree

classes/admin/class-page-settings.php

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,13 @@ public function store_settings_form_options() {
133133
\wp_send_json_error( [ 'message' => \esc_html__( 'You do not have permission to update settings.', 'progress-planner' ) ] );
134134
}
135135

136+
// Check the nonce.
137+
\check_admin_referer( 'progress_planner' );
138+
139+
if ( ! \current_user_can( 'manage_options' ) ) {
140+
\wp_send_json_error( [ 'message' => \esc_html__( 'You do not have permission to update settings.', 'progress-planner' ) ] );
141+
}
142+
136143
// Use check_ajax_referer instead of check_admin_referer for AJAX handlers.
137144
// check_admin_referer is designed for form submissions, not AJAX requests.
138145
if ( ! \check_ajax_referer( 'progress_planner', 'nonce', false ) ) {

classes/suggested-tasks/providers/class-email-sending.php

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -263,6 +263,13 @@ public function ajax_test_email_sending() {
263263
\wp_send_json_error( [ 'message' => \esc_html__( 'You do not have permission to test email sending.', 'progress-planner' ) ] );
264264
}
265265

266+
// Check the nonce.
267+
\check_admin_referer( 'progress_planner' );
268+
269+
if ( ! $this->capability_required() ) {
270+
\wp_send_json_error( [ 'message' => \esc_html__( 'You do not have permission to test email sending.', 'progress-planner' ) ] );
271+
}
272+
266273
// Use check_ajax_referer for AJAX handlers.
267274
if ( ! \check_ajax_referer( 'progress_planner', 'nonce', false ) ) {
268275
\wp_send_json_error( [ 'message' => \esc_html__( 'Invalid nonce.', 'progress-planner' ) ] );

0 commit comments

Comments
 (0)