Changeset 8886

Timestamp:

05/27/2019 08:00:04 PM (5 years ago)

Author:

vedjain

Message:

WC Report: Use return b/c this action may be hooked to admin_init.

In r7902, we added a wp_die statement if nonce is not verified for Meetup Events report. However, when opening WordCamp Report, this method gets hooked to admin_init, and cause all reports that are loaded after it, like Meetup Details report to break.

Replacing wp_die with return makes the behavior consistent with other reports and fix this bug.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-reports/classes/report/class-meetup-events.php (modified) (1 diff)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-reports/classes/report/class-meetup-events.php

@@ -468 +468 @@
 
         if ( ! wp_verify_nonce( $nonce, 'run-report' ) || ! current_user_can( 'manage_network' ) ) {
-            wp_die( esc_html__( 'Sorry, you are not allowed to export this data.', 'wordcamporg' ) );
+            return;
         }