Changeset 7098

Timestamp:

04/12/2018 12:19:34 AM (6 years ago)

Author:

coreymckrill

Message:

WordCamp: Require special caps in order to change a WordCamp's sponsor region

The sponsor region is determined and assigned by a WPCS team member, but
sometimes the camp organizer doesn't know what the field is for and changes it.
This helps prevent unintentional changes.

Location:

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

Files:

• multi-event-sponsors/classes/mes-region.php (modified) (2 diffs)
• multi-event-sponsors/views/template-region-dropdown.php (modified) (1 diff)
• wcpt/wcpt-wordcamp/wordcamp-admin.php (modified) (1 diff)
• wcpt/wcpt-wordcamp/wordcamp-new-site.php (modified) (1 diff)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/multi-event-sponsors/classes/mes-region.php

@@ -146 +146 @@
         $site_id         = get_post_meta( $post->ID, '_site_id', true );
         $cb_push_name    = wcpt_key_to_str( 'push-mes-sponsors', 'wcpt_' );
+        $protected       = WordCamp_Admin::is_protected_field( $key );
 
         require( dirname( __DIR__ ) . '/views/template-region-dropdown.php' );
@@ -162 +163 @@
         }
 
+        if ( WordCamp_Admin::is_protected_field( $key ) ) {
+            return;
+        }
+
         $post_key = wcpt_key_to_str( $key, 'wcpt_' );
         if ( isset( $_POST[ $post_key ] ) ) {

sites/trunk/wordcamp.org/public_html/wp-content/plugins/multi-event-sponsors/views/template-region-dropdown.php

@@ -4 +4 @@
 <?php /** @var $cb_push_name    string */ ?>
 <?php /** @var $site_id         int    */ ?>
+<?php /** @var $protected       bool   */ ?>
 
 <select id="<?php echo esc_attr( $field_name ); ?>" name="<?php echo esc_attr( $field_name ); ?>">
-    <option value="">None</option>
-
-    <?php foreach ( $regions as $region ) : ?>
-
-        <option value="<?php echo esc_attr( $region->term_id ); ?>" <?php selected( $selected_region, $region->term_id ); ?>>
-            <?php echo esc_html( $region->name ); ?>
+    <?php if ( ! $protected ) : ?>
+        <option value="">None</option>
+        <?php foreach ( $regions as $region ) : ?>
+            <option value="<?php echo esc_attr( $region->term_id ); ?>" <?php selected( $selected_region, $region->term_id ); ?>>
+                <?php echo esc_html( $region->name ); ?>
+            </option>
+        <?php endforeach; ?>
+    <?php else : ?>
+        <option value="<?php echo esc_attr( $selected_region ); ?>" selected>
+            <?php echo esc_html( get_term( $selected_region, MES_Region::TAXONOMY_SLUG )->name ); ?>
         </option>
-
-    <?php endforeach; ?>
+    <?php endif; ?>
 </select>
 
-<?php if ( $site_id ) : ?>
+<?php if ( $site_id && ! $protected ) : ?>
     <label>
         <input type="checkbox" name="<?php echo esc_attr( $cb_push_name ); ?>" value="1" />

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

@@ -973 +973 @@
 
         if ( ! current_user_can( 'wordcamp_manage_mentors' ) ) {
-            $protected_fields += array(
+            $protected_fields = array_merge( $protected_fields, array(
                 'Mentor WordPress.org User Name',
                 'Mentor Name',
                 'Mentor E-mail Address',
-            );
+            ) );
+        }
+
+        if ( ! current_user_can( 'wordcamp_wrangle_wordcamps' ) ) {
+            $protected_fields = array_merge( $protected_fields, array(
+                'Multi-Event Sponsor Region',
+            ) );
         }
 

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

@@ -201 +201 @@
 
         if ( empty( $_POST[ wcpt_key_to_str( 'push-mes-sponsors', 'wcpt_' ) ] ) ) {
+            return;
+        }
+
+        if ( WordCamp_Admin::is_protected_field( 'Multi-Event Sponsor Region' ) ) {
             return;
         }