Changeset 4256

Timestamp:

10/18/2016 12:22:09 AM (9 years ago)

Author:

iandunn

Message:

Multi Event Sponsors: Hide Push sponsors checkbox if the site doesn't exist

The sponsors can't be pushed until the site exists, so having it visible can be confusing.

Location:

sites/trunk/wordcamp.org/public_html/wp-content/plugins/multi-event-sponsors

Files:

• classes/mes-region.php (modified) (1 diff)
• views/taxonomy-meta-region.php (modified) (1 diff)
• views/template-region-dropdown.php (modified) (2 diffs)

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

@@ -144 +144 @@
         $regions         = get_terms( self::TAXONOMY_SLUG, array( 'hide_empty' => false ) );
         $selected_region = get_post_meta( $post->ID, $key, true );
+        $site_id         = get_post_meta( $post->ID, '_site_id', true );
         $cb_push_name    = wcpt_key_to_str( 'push-mes-sponsors', 'wcpt_' );
 

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

@@ -5 +5 @@
 
 if ( 'edit' == $wp_list_table->current_action() ) : ?>
+    <?php // todo fatal - call to member function on non-object ?>
 
     <?php wp_nonce_field( "mes_edit_region_{$region_id}_meta", 'mes_edit_region_meta_nonce' ); ?>

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

@@ -3 +3 @@
 <?php /** @var $selected_region int    */ ?>
 <?php /** @var $cb_push_name    string */ ?>
+<?php /** @var $site_id         int    */ ?>
 
 <select id="<?php echo esc_attr( $field_name ); ?>" name="<?php echo esc_attr( $field_name ); ?>">
@@ -16 +17 @@
 </select>
 
-<label>
-    <input type="checkbox" name="<?php echo esc_attr( $cb_push_name ); ?>" value="1" />
-    Push sponsors to site
-</label>
+<?php if ( $site_id ) : ?>
+    <label>
+        <input type="checkbox" name="<?php echo esc_attr( $cb_push_name ); ?>" value="1" />
+        Push sponsors to site
+    </label>
+<?php endif; ?>