Changeset 7011 for sites/trunk/wp15.wordpress.net/public_html/content/plugins/wp15-meetup-events/wp15-meetup-events.php

Timestamp:

04/03/2018 08:46:45 PM (8 years ago)

Author:

iandunn

Message:

WP15: Use wp_localize_script() to pass all data to JS.

This will be more organized as we add more data in future commits, like localized strings. It also takes some logic out of the view file.

File:

• sites/trunk/wp15.wordpress.net/public_html/content/plugins/wp15-meetup-events/wp15-meetup-events.php (modified) (4 diffs)

sites/trunk/wp15.wordpress.net/public_html/content/plugins/wp15-meetup-events/wp15-meetup-events.php

@@ -177 +177 @@
         }
 
+        wp_enqueue_style(
+                'wp15-meetup-events',
+                plugins_url( 'wp15-meetup-events.css', __FILE__ ),
+                array(),
+                filemtime( __DIR__ . '/wp15-meetup-events.css' )
+        );
+
         wp_register_script(
                 'google-maps',
@@ -201 +208 @@
         );
 
-        wp_enqueue_style(
+        wp_localize_script(
                 'wp15-meetup-events',
-                plugins_url( 'wp15-meetup-events.css', __FILE__ ),
-                array(),
-                filemtime( __DIR__ . '/wp15-meetup-events.css' )
-        );
-}
-
-/**
- * Render the WP15 events shortcode.
- */
-function render_events_shortcode() {
-        $events = get_option( 'wp15_events' );
-
-        // This needs to be done on the fly, in order to use the date format for the visitor's locale.
-        foreach ( $events as & $event ) {
-                $event['time'] = get_local_formatted_date( $event['time'], $event['timezone'] );
-        }
-        unset( $event ); // Destroy the reference to restore expected behavior; see https://stackoverflow.com/a/4969286/450127.
-
-        usort( $events, __NAMESPACE__ . '\sort_events' );
-
-        $map_options = array(
+                'wp15MeetupEventsData',
+                array(
+                        'map_options' => get_map_options(),
+                        'events'      => get_formatted_events(),
+                )
+        );
+}
+
+
+/**
+ * Get the configuration for the Google Map of events.
+ *
+ * @return array
+ */
+function get_map_options() {
+        return array(
                 'mapContainer'            => 'wp15-events-map',
-                'mapMarkers'              => $events,
                 'markerIconBaseURL'       => plugins_url( '/images/', __FILE__ ),
                 'markerIcon'              => 'map-marker.svg',
@@ -235 +236 @@
                 'clusterIconHeight'       => 52,
         );
+}
+
+/**
+ * Format the WP15 events for presentation.
+ *
+ * @return array
+ */
+function get_formatted_events() {
+        $events = get_option( 'wp15_events' );
+
+        // This needs to be done on the fly, in order to use the date format for the visitor's locale.
+        foreach ( $events as & $event ) {
+                $event['time'] = get_local_formatted_date( $event['time'], $event['timezone'] );
+        }
+        unset( $event ); // Destroy the reference to restore expected behavior; see https://stackoverflow.com/a/4969286/450127.
+
+        usort( $events, __NAMESPACE__ . '\sort_events' );
+
+        return $events;
+}
+
+/**
+ * Sort events by their timestamp.
+ *
+ * @param array $a
+ * @param array $b
+ *
+ * @return int
+ */
+function sort_events( $a, $b ) {
+        if ( $a['time'] === $b['time'] ) {
+                return 0;
+        }
+
+        return $a['time'] > $b['time'];
+}
+
+/**
+ * Render the WP15 events shortcode.
+ */
+function render_events_shortcode() {
+        $events = get_formatted_events();
 
         ob_start();
@@ -240 +283 @@
         require_once( __DIR__ . '/views/events-list.php' );
         return ob_get_clean();
-}
-
-/**
- * Sort events by their timestamp.
- *
- * @param array $a
- * @param array $b
- *
- * @return int
- */
-function sort_events( $a, $b ) {
-        if ( $a['time'] === $b['time'] ) {
-                return 0;
-        }
-
-        return $a['time'] > $b['time'];
 }