Changeset 2824

Timestamp:

03/28/2016 05:11:35 PM (10 years ago)

Author:

kovshenin

Message:

WordCamp Post Type: Soft-enforce a post status progression for WordCamps via the UI.

Location:

sites/branches/application-tracking/wordcamp.org/public_html/wp-content/plugins/wcpt

Files:

• views/wordcamp/metabox-status.php (modified) (1 diff)
• wcpt-wordcamp/wordcamp-admin.php (modified) (4 diffs)
• wcpt-wordcamp/wordcamp-loader.php (modified) (2 diffs)

sites/branches/application-tracking/wordcamp.org/public_html/wp-content/plugins/wcpt/views/wordcamp/metabox-status.php

@@ -12 +12 @@
                         <span id="post-status-display">
                             <select name="post_status">
+                                <?php $transitions = WordCamp_Loader::get_valid_status_transitions( $post->post_status ); ?>
                                 <?php foreach ( WordCamp_Loader::get_post_statuses() as $key => $label ) : ?>
                                     <?php $status = get_post_status_object( $key ); ?>
-                                    <option value="<?php echo esc_attr( $status->name ); ?>" <?php selected( $post->post_status, $status->name ); ?> >
+                                    <option value="<?php echo esc_attr( $status->name ); ?>" <?php
+                                        if ( $post->post_status == $status->name ) {
+                                            selected( true );
+                                        } elseif ( ! in_array( $status->name, $transitions ) ) {
+                                            echo ' disabled ';
+                                        }
+                                    ?>>
                                         <?php echo esc_html( $status->label ); ?>
                                     </option>

sites/branches/application-tracking/wordcamp.org/public_html/wp-content/plugins/wcpt/wcpt-wordcamp/wordcamp-admin.php

@@ -45 +45 @@
         add_action( 'wcpt_added_to_planning_schedule',                array( $this, 'add_organizer_to_central' ), 10 );
         add_action( 'wcpt_added_to_planning_schedule',                array( $this, 'mark_date_added_to_planning_schedule' ), 10 );
-        add_filter( 'wp_insert_post_data',                            array( $this, 'enforce_post_status_progression' ), 10, 2 );
+        add_filter( 'wp_insert_post_data',                            array( $this, 'enforce_post_status' ), 10, 2 );
         add_filter( 'wp_insert_post_data',                            array( $this, 'require_complete_meta_to_publish_wordcamp' ), 10, 2 );
 
@@ -695 +695 @@
 
     /**
-     * Force WordCamp posts to go through the expected status progression.
+     * Enforce a valid post status for WordCamps.
      *
      * @param array $post_data
@@ -701 +701 @@
      * @return array
      */
-    public function enforce_post_status_progression( $post_data, $post_data_raw ) {
+    public function enforce_post_status( $post_data, $post_data_raw ) {
         if ( $post_data['post_type'] != WCPT_POST_TYPE_ID || empty( $_POST['post_ID'] ) ) {
             return $post_data;
@@ -709 +709 @@
         if ( ! $post ) {
             return $post_data;
-        }
-
-        // @todo: Rewrite all of this according to the new flow.
-        if ( ! empty( $_POST ) ) {
-            $previous_post_status = get_post( absint( $_POST['post_ID'] ) );
-            $previous_post_status = $previous_post_status->post_status;
-
-            if ( 'pending' == $post_data['post_status'] && ! in_array( $previous_post_status, array( 'draft', 'pending', 'publish' ) ) ) {
-                $this->active_admin_notices[] = 2;
-                $post_data['post_status'] = $previous_post_status;
-            }
-
-            if ( 'publish' == $post_data['post_status'] && ! in_array( $previous_post_status, array( 'pending', 'publish' ) ) ) {
-                $this->active_admin_notices[] = 2;
-                $post_data['post_status'] = $previous_post_status;
-            }
         }
 

sites/branches/application-tracking/wordcamp.org/public_html/wp-content/plugins/wcpt/wcpt-wordcamp/wordcamp-loader.php

@@ -117 +117 @@
     }
 
+    /**
+     * Get WordCamp post statuses.
+     *
+     * @return array
+     */
     public static function get_post_statuses() {
         return array(
@@ -137 +142 @@
         );
     }
+
+    /**
+     * Return valid transitions given a post status.
+     *
+     * @param string $status Current status.
+     *
+     * @return array Valid transitions.
+     */
+    public static function get_valid_status_transitions( $status ) {
+        $transitions = array(
+            'wcpt-formal-review'   => array( 'wcpt-inactive', 'wcpt-application-rev' ),
+            'wcpt-inactive'        => array( 'wcpt-formal-review', 'wcpt-application-rev' ),
+            'wcpt-application-rev' => array( 'wcpt-inactive', 'wcpt-scheduling' ),
+            'wcpt-scheduling'      => array( 'wcpt-inactive', 'wcpt-interview-sched', 'wcpt-application-rev' ),
+            'wcpt-interview-sched' => array( 'wcpt-inactive', 'wcpt-needs-site', 'wcpt-scheduling', 'wcpt-rejected' ),
+            'wcpt-needs-site'      => array( 'wcpt-inactive', 'wcpt-agreement-sig', 'wcpt-interview-sched' ),
+            'wcpt-rejected'        => array( 'wcpt-inactive' ),
+            'wcpt-agreement-sig'   => array( 'wcpt-inactive', 'wcpt-site-created', 'wcpt-needs-site' ),
+            'wcpt-site-created'    => array( 'wcpt-inactive', 'wcpt-listing-added', 'wcpt-agreement-sig' ),
+            'wcpt-listing-added'   => array( 'wcpt-inactive', 'wcpt-needs-budget-re', 'wcpt-site-created' ),
+            'wcpt-needs-budget-re' => array( 'wcpt-inactive', 'wcpt-budget-approved', 'wcpt-listing-added' ),
+            'wcpt-budget-approved' => array( 'wcpt-inactive', 'wcpt-venue-signed', 'wcpt-needs-budget-re' ),
+            'wcpt-venue-signed'    => array( 'wcpt-inactive', 'wcpt-scheduled', 'wcpt-budget-approved' ),
+            'wcpt-scheduled'       => array( 'wcpt-inactive', 'wcpt-debrief', 'wcpt-venue-signed' ),
+            'wcpt-debrief'         => array( 'wcpt-inactive', 'wcpt-budget-closed', 'wcpt-scheduled' ),
+            'wcpt-budget-closed'   => array( 'wcpt-inactive', 'wcpt-debrief' ),
+        );
+
+        if ( empty( $transitions[ $status ] ) )
+            return array( 'wcpt-inactive', 'wcpt-formal-review' );
+
+        return $transitions[ $status ];
+    }
 }