Changeset 7734 for sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-reports/classes/report/class-base.php

Timestamp:

10/16/2018 05:30:06 PM (7 years ago)

Author:

vedjain

Message:

WordCamp Report: Add reporting for meetup applications.

This commit adds functionality for downloading meetup application reports from admin dashboard. Also refactors WordCamp report classes
to make code DRY.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-reports/classes/report/class-base.php (modified) (2 diffs)

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

@@ -135 +135 @@
 
     /**
-     * Determine the data fields safelist based on the context of the report.
-     *
-     * @return array The list of fields that are safe to include.
-     */
-    protected function get_data_fields_safelist() {
-        $safelist = $this->public_data_fields;
-
-        if ( false === $this->options['public'] ) {
-            $safelist = array_merge( $safelist, $this->private_data_fields );
-        }
-
-        return $safelist;
-    }
-
-    /**
      * Filter the report data prior to caching and compiling.
      *
@@ -315 +300 @@
         return new \WP_REST_Response( $response_data );
     }
+
+    /**
+     * Determine the data fields safelist based on the context of the report.
+     *
+     * @return array The list of fields that are safe to include.
+     */
+    public function get_data_fields_safelist() {
+        $safelist = $this->public_data_fields;
+
+        if ( false === $this->options['public'] ) {
+            $safelist = array_merge( $safelist, $this->private_data_fields );
+        }
+
+        return $safelist;
+    }
 }