Changeset 3159

Timestamp:

05/17/2016 07:52:21 PM (10 years ago)

Author:

iandunn

Message:

WordCamp Post Type: Require E-mail Address for needs-site, not pre-planning.

Since the extra statuses were added in r2898, we can be more specific about when the address is actually needed. Requiring it before then is inconvenient.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wcpt/wcpt-wordcamp/wordcamp-admin.php (modified) (5 diffs)

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

@@ -764 +764 @@
         $min_site_id = apply_filters( 'wcpt_require_complete_meta_min_site_id', '2416297' );
 
-        $required_pre_planning_fields = $this->get_required_fields( 'pre-planning' );
+        $required_needs_site_fields = $this->get_required_fields( 'needs-site' );
         $required_scheduled_fields    = $this->get_required_fields( 'scheduled' );
+        // todo realign
 
         // Check pending posts
-        if ( 'wcpt-approved-pre-pl' == $post_data['post_status'] && absint( $_POST['post_ID'] ) > $min_site_id ) {
-            foreach( $required_pre_planning_fields as $field ) {
+        if ( 'wcpt-needs-site' == $post_data['post_status'] && absint( $_POST['post_ID'] ) > $min_site_id ) {
+            foreach( $required_needs_site_fields as $field ) {
                 $value = $_POST[ wcpt_key_to_str( $field, 'wcpt_' ) ];
 
                 if ( empty( $value ) || 'null' == $value ) {
-                    $post_data['post_status']     = 'wcpt-interview-sched';
+                    $post_data['post_status']     = 'wcpt-needs-email';
                     $this->active_admin_notices[] = 1;
                     break;
@@ -799 +800 @@
      * Get a list of fields required to move to a certain post status
      *
-     * @param string $status 'pre-planning' | 'scheduled' | 'any'
+     * @param string $status 'needs-site' | 'scheduled' | 'any'
      *
      * @return array
      */
     public static function get_required_fields( $status ) {
-        $pre_planning = array( 'E-mail Address' );
+        $needs_site = array( 'E-mail Address' );
 
         $scheduled = array(
@@ -828 +829 @@
 
         switch ( $status ) {
-            case 'pre-planning':
-                $required_fields = $pre_planning;
+            case 'needs-site':
+                $required_fields = $needs_site;
                 break;
 
@@ -838 +839 @@
             case 'any':
             default:
-                $required_fields = array_merge( $pre_planning, $scheduled );
+                $required_fields = array_merge( $needs_site, $scheduled );
                 break;
         }
@@ -888 +889 @@
             1 => array(
                 'type'   => 'error',
-                'notice' => __( 'This WordCamp cannot be approved for pre-planning until all of its required metadata is filled in.', 'wordcamporg' ),
+                'notice' => __( 'This WordCamp cannot be moved to Needs Site until all of its required metadata is filled in.', 'wordcamporg' ),
             ),