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

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' ),
                         ),