Changeset 2277 for sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/includes/wordcamp-budgets.php

Timestamp:

01/12/2016 04:36:29 PM (11 years ago)

Author:

iandunn

Message:

WordCamp Budgets: Move payment-request script enqueing into module.

Since it's not shared across modules, it should live in the Payment Requests module.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/includes/wordcamp-budgets.php (modified) (2 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/includes/wordcamp-budgets.php

@@ -11 +11 @@
          */
         public function __construct() {
-                add_action( 'admin_enqueue_scripts',  array( $this, 'enqueue_assets' ), 11 );
+                add_action( 'admin_enqueue_scripts',  array( $this, 'enqueue_common_assets' ), 11 );
         }
 
         /**
-         * Enqueue scripts and stylesheets
+         * Enqueue scripts and stylesheets common to all modules
          */
-        public function enqueue_assets( $hook ) {
-                global $post;
-
+        public function enqueue_common_assets() {
                 // todo setup grunt to concat/minify js and css?
-
-                // Register our assets
-                wp_register_script(
-                        'payment-requests',
-                        plugins_url( 'javascript/payment-requests.js', __DIR__ ),
-                        array( 'jquery', 'jquery-ui-datepicker', 'media-upload', 'media-views' ),
-                        self::VERSION,
-                        true
-                );
-
-                wp_register_script(
-                        'wcp-attached-files',
-                        plugins_url( 'javascript/attached-files.js', __DIR__ ),
-                        array( 'payment-requests', 'backbone', 'wp-util' ),
-                        self::VERSION,
-                        true
-                );
 
                 // Let's still include our .css file even if these are unavailable.
@@ -51 +32 @@
                         self::VERSION
                 );
-
-                // Enqueue our assets if they're needed on the current screen
-                $current_screen = get_current_screen();
-
-                if ( in_array( $current_screen->id, array( 'edit-wcp_payment_request', 'wcp_payment_request' ) ) ) {
-                        wp_enqueue_script( 'payment-requests' );
-                        wp_enqueue_style( 'wordcamp-budgets' );
-
-                        if ( in_array( $current_screen->id, array( 'wcp_payment_request' ) ) && isset( $post->ID ) ) {
-                                wp_enqueue_media( array( 'post' => $post->ID ) );
-                                wp_enqueue_script( 'wcp-attached-files' );
-                        }
-
-                        wp_localize_script(
-                                'payment-requests',
-                                'wcpLocalizedStrings',          // todo merge into wordcampBudgets var
-                                array(
-                                        'uploadModalTitle'  => __( 'Attach Supporting Documentation', 'wordcamporg' ),
-                                        'uploadModalButton' => __( 'Attach Files', 'wordcamporg' ),
-                                )
-                        );
-                }
         }