Changeset 7635

Timestamp:

08/27/2018 03:25:40 AM (6 years ago)

Author:

vedjain

Message:

WordCamp: Use taxonomy in meetup applications.

Earlier we were adding predefined tags in meetup applications. Switching to taxonomy will let us add new tags without any code change.

Location:

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wcpt

Files:

• wcpt-functions.php (modified) (2 diffs)
• wcpt-meetup/class-meetup-admin.php (modified) (7 diffs)
• wcpt-meetup/meetup-loader.php (modified) (3 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wcpt/wcpt-functions.php

@@ -85 +85 @@
     $private_notes  = get_post_meta( $event_id, '_note'          );
     $status_changes = get_post_meta( $event_id, '_status_change' );
+    $tag_changes    = get_post_meta( $event_id, '_tags_log' );
 
-    foreach ( array( 'note' => $private_notes, 'status_change' => $status_changes ) as $entry_type => $raw_entries ) {
+    foreach ( array( 'note' => $private_notes, 'status_change' => $status_changes, 'tag_change' => $tag_changes ) as $entry_type => $raw_entries ) {
         $entries = array_merge( process_raw_entries( $raw_entries, $entry_type ), $entries );
     }
@@ -120 +121 @@
     }
 
-    usort( $entries, 'wcpt_sort_log_entries' );
-
     return $entries;
 }

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wcpt/wcpt-meetup/class-meetup-admin.php

@@ -22 +22 @@
         public function __construct() {
             parent::__construct();
-
-            add_action( 'wcpt_metabox_save_done', array( $this, 'save_tag_checkboxes' ) );
         }
 
@@ -82 +80 @@
         public function column_headers( $columns ) {
             $columns = array(
-                'cb'             => '<input type="checkbox" />',
-                'title'          => __( 'Title', 'wcpt' ),
-                'tags'           => __( 'Tags', 'wcpt' ),
-                'organizer'      => __( 'Organizer', 'wcpt' ),
-                'date'           => __( 'Date', 'wcpt' ),
-                'meetup.com_url' => __( 'Meetup URL', 'wcpt' ),
-                'helpscout_url'  => __( 'HelpScout Link', 'wcpt' ),
+                'cb'                   => '<input type="checkbox" />',
+                'title'                => __( 'Title', 'wcpt' ),
+                'taxonomy-meetup_tags' => __( 'Meetup Tags', 'wcpt' ),
+                'organizer'            => __( 'Organizer', 'wcpt' ),
+                'date'                 => __( 'Date', 'wcpt' ),
+                'meetup.com_url'       => __( 'Meetup URL', 'wcpt' ),
+                'helpscout_url'        => __( 'HelpScout Link', 'wcpt' ),
             );
 
@@ -191 +189 @@
             );
 
-            add_meta_box(
-                'wcpt_show_tag_logs',
-                'Tag logs',
-                array( $this, 'wcpt_tag_log_metabox' ),
-                $this->get_event_type(),
-                'advanced',
-                'low'
-            );
-
         }
 
@@ -259 +248 @@
 
             self::display_meta_boxes( array(), $meta_keys, array(), $post_id, array() );
-        }
-
-        /**
-         * Renders tags logs
-         */
-        public function wcpt_tag_log_metabox() {
-            global $post_id;
-            $entries = get_post_meta( $post_id, '_tags_log' );
-            $entries = process_raw_entries( $entries, 'Tag changed' );
-            require( WCPT_DIR . '/views/common/metabox-log.php' );
         }
 
@@ -297 +276 @@
                 'Joined chapter date'        => 'date',
                 'Joined chapter by'          => 'text',
-                'More Information requested' => 'checkbox',
-                'Changes Requested'          => 'checkbox',
-                'Needs Meeting'              => 'checkbox'
             );
 
@@ -316 +292 @@
 
             $swag_keys = array(
-                'Needs swag' => 'checkbox',
-                'Swag notes' => 'text',
+                'Swag notes' => 'textarea',
             );
 
@@ -345 +320 @@
         }
 
-        /**
-         * Toggle tags that are rendered as checkboxes
-         *
-         * @param int $post_id
-         */
-        public function save_tag_checkboxes( $post_id ) {
-
-            $post_tags = wp_get_post_tags( $post_id, array( 'fields' => 'names' ) );
-
-            $post_tags = $this->add_or_remove_tag( $post_id, $post_tags, 'wcpt_needs_swag', 'Needs Swag', $_POST['wcpt_swag_notes'] );
-
-            $post_tags = $this->add_or_remove_tag( $post_id, $post_tags, 'wcpt_more_information_requested', 'More Information requested' );
-
-            $post_tags = $this->add_or_remove_tag( $post_id, $post_tags, 'wcpt_changes_requested', 'Changes Requested' );
-
-            $post_tags = $this->add_or_remove_tag( $post_id, $post_tags, 'wcpt_needs_meeting', 'Needs Meeting' );
-
-            if ( $post_tags != wp_get_post_tags( $post_id ) ) {
-                wp_set_post_tags( $post_id, $post_tags );
-            }
-        }
-
-        /**
-         * Add or remove tag. Also logs and make a note when required
-         *
-         * @param $post_id
-         * @param $tag_array
-         * @param $input_name
-         * @param $label
-         * @param string $note_message
-         *
-         * @return array
-         */
-        public function add_or_remove_tag( $post_id, $tag_array, $input_name, $label, $note_message = '' ) {
-
-            if ( $note_message !== '' ) {
-                $note_message = ' Note: ' . $note_message;
-            }
-
-            if ( isset( $_POST[ $input_name] ) && $_POST[ $input_name ] && ( ! in_array( $label, $tag_array ) ) ) {
-
-                $tag_array[] = $label;
-
-                add_post_meta(
-                    $post_id, '_tags_log', array(
-                        'timestamp' => time(),
-                        'user_id'   => get_current_user_id(),
-                        'message'   => 'Tag ' . $label . ' added.' . $note_message,
-                    )
-                );
-
-            } elseif ( ! isset( $_POST[ $input_name] ) && in_array( $label, $tag_array ) ) {
-                $tag_array = array_diff( $tag_array, array( $label ) );
-
-                add_post_meta(
-                    $post_id, '_tags_log', array(
-                        'timestamp' => time(),
-                        'user_id'   => get_current_user_id(),
-                        'message'   => 'Tag ' . $label . ' removed.' . $note_message,
-                    )
-                );
-            }
-
-            return $tag_array;
-        }
+
     }
 

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wcpt/wcpt-meetup/meetup-loader.php

@@ -8 +8 @@
 use WordPress_Community\Applications\Meetup_Application;
 define( 'WCPT_MEETUP_SLUG', 'wp_meetup' );
+define( 'WCPT_MEETUP_TAG_SLUG', 'meetup_tags' );
 
 
@@ -17 +18 @@
     class Meetup_Loader extends Event_Loader {
 
+        /**
+         * Meetup_Loader constructor.
+         */
+        public function __construct() {
+            parent::__construct();
+            add_action( 'init', array( $this, 'register_meetup_taxonomy' ) );
+            add_action( 'set_object_terms', array( $this, 'log_meetup_tags' ), 10, 6 );
+        }
+
+        /**
+         * Log when a tag is added or removed in a meetup event
+         *
+         * @param int    $event_id   Meetup post ID.
+         * @param array  $terms      An array of tags.
+         * @param array  $tt_ids     An array of term taxonomy IDs.
+         * @param string $taxonomy   Taxonomy slug.
+         * @param bool   $append     Whether to append new terms to the old terms.
+         * @param array  $old_tt_ids Old array of term taxonomy IDs.
+         */
+        public function log_meetup_tags( $event_id, $terms, $tt_ids, $taxonomy, $append, $old_tt_ids ) {
+
+            // bail if taxonomy is not meetup tags.
+            if ( WCPT_MEETUP_TAG_SLUG !== $taxonomy ) {
+                return;
+            }
+
+            $tt_added_ids = array_diff( $tt_ids, $old_tt_ids );
+            $tt_removed_ids = array_diff( $old_tt_ids, $tt_ids );
+
+            if ( count( $tt_added_ids ) > 0 ) {
+                $added_terms_query = new WP_Term_Query( array(
+                        'taxonomy'   => WCPT_MEETUP_TAG_SLUG,
+                        'object_ids' => $event_id,
+                        'term_taxonomy_id' => $tt_added_ids,
+                        'fields' => 'names'
+                    )
+                );
+
+                $added_terms = $added_terms_query->get_terms();
+
+                add_post_meta( $event_id, '_tags_log', array(
+                    'timestamp' => time(),
+                    'user_id'   => get_current_user_id(),
+                    'message'   => 'Tags added: ' . join( ', ', $added_terms ),
+                ) );
+            }
+
+            if ( count( $tt_removed_ids ) > 0 ) {
+                $removed_terms = ( new WP_Term_Query( array(
+                        'taxonomy'         => WCPT_MEETUP_TAG_SLUG,
+                        'term_taxonomy_id' => $tt_removed_ids,
+                        'fields'           => 'names',
+                        'hide_empty'       => false,
+                    )
+                ) )->get_terms();
+
+                add_post_meta( $event_id, '_tags_log', array(
+                    'timestamp' => time(),
+                    'user_id'   => get_current_user_id(),
+                    'message'   => 'Tags removed: ' . join( ', ', $removed_terms ),
+                ) );
+            }
+
+        }
+
+        /**
+         * Register custom tags for meetup events.
+         */
+        public function register_meetup_taxonomy() {
+            register_taxonomy(
+                'meetup_tags',
+                Meetup_Application::POST_TYPE,
+                array(
+                    'hierarchical' => false,
+                    'labels'       => array(
+                        'name'          => __( 'Meetup Tags' ),
+                        'singular_name' => __( 'Meetup Tag' ),
+                    ),
+                    'show_admin_column' => true,
+                )
+            );
+        }
 
         /**
@@ -54 +137 @@
             $wcpt_supports = array(
                 'title',
-                'editor',
-                'thumbnail',
                 'revisions',
                 'author',