Changeset 5575

Timestamp:

06/15/2017 02:17:54 PM (9 years ago)

Author:

coreymckrill

Message:

WordCamp Post Types: Make sponsor invoice province field optional

Some regions don't have a state or province as part of their official
address format. However, some banks always require that field on
invoices, regardless of the country. The solution in this patch is to
make the state/province field optional, but add the string
"Not applicable" if the field is left blank when the form data is
saved.

Props davidmosterd
Fixes #2711

Location:

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wc-post-types

Files:

• views/sponsors/metabox-sponsor-info.php (modified) (2 diffs)
• wc-post-types.php (modified) (3 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wc-post-types/views/sponsors/metabox-sponsor-info.php

@@ -142 +142 @@
 
 <ul class="wcpt-form">
-        <li class="wcpt-form-header">
-                <?php _e( 'Address', 'wordcamporg' ); ?>
-        </li>
-
-        <li>
-                <label for="_wcpt_sponsor_street_address1">
-                        <?php _e( 'Street Address 1:', 'wordcamporg' ); ?>
-                </label>
-
-                <input
-                        type="text"
-                        class="regular-text"
-                        id="_wcpt_sponsor_street_address1"
-                        name="_wcpt_sponsor_street_address1"
-                        value="<?php echo esc_attr( $street_address1 ); ?>"
-                    required
-                />
-
-                <?php $this->render_form_field_required_indicator(); ?>
-        </li>
-
-        <li>
-                <label for="_wcpt_sponsor_street_address2">
-                        <?php _e( 'Street Address 2:', 'wordcamporg' ); ?>
-                </label>
-
-                <input
-                        type="text"
-                        class="regular-text"
-                        id="_wcpt_sponsor_street_address2"
-                        name="_wcpt_sponsor_street_address2"
-                        value="<?php echo esc_attr( $street_address2 ); ?>"
-                />
-        </li>
-
-        <li>
-                <label for="_wcpt_sponsor_city">
-                        <?php _e( 'City:', 'wordcamporg' ); ?>
-                </label>
-
-                <input
-                        type="text"
-                        class="regular-text"
-                        id="_wcpt_sponsor_city"
-                        name="_wcpt_sponsor_city"
-                        value="<?php echo esc_attr( $city ); ?>"
-                    required
-                />
-
-                <?php $this->render_form_field_required_indicator(); ?>
-        </li>
-
-        <li>
-                <label for="_wcpt_sponsor_state">
-                        <?php _e( 'State / Province:', 'wordcamporg' ); ?>
-                </label>
-
-                <input
-                        type="text"
-                        class="regular-text"
-                        id="_wcpt_sponsor_state"
-                        name="_wcpt_sponsor_state"
-                        value="<?php echo esc_attr( $state ); ?>"
-                    required
-                />
-
-                <?php $this->render_form_field_required_indicator(); ?>
-        </li>
-
-        <li>
-                <label for="_wcpt_sponsor_zip_code">
-                        <?php _e( 'ZIP / Postal Code:', 'wordcamporg' ); ?>
-                </label>
-
-                <input
-                        type="text"
-                        class="regular-text"
-                        id="_wcpt_sponsor_zip_code"
-                        name="_wcpt_sponsor_zip_code"
-                        value="<?php echo esc_attr( $zip_code ); ?>"
-                        maxlength="30"
-                    required
-                />
-
-                <?php $this->render_form_field_required_indicator(); ?>
-        </li>
-
         <li>
                 <label for="_wcpt_sponsor_country">
@@ -249 +162 @@
                 <?php $this->render_form_field_required_indicator(); ?>
         </li>
+
+        <li class="wcpt-form-header">
+                <?php _e( 'Address', 'wordcamporg' ); ?>
+        </li>
+
+        <li>
+                <label for="_wcpt_sponsor_street_address1">
+                        <?php _e( 'Street Address 1:', 'wordcamporg' ); ?>
+                </label>
+
+                <input
+                        type="text"
+                        class="regular-text"
+                        id="_wcpt_sponsor_street_address1"
+                        name="_wcpt_sponsor_street_address1"
+                        value="<?php echo esc_attr( $street_address1 ); ?>"
+                    required
+                />
+
+                <?php $this->render_form_field_required_indicator(); ?>
+        </li>
+
+        <li>
+                <label for="_wcpt_sponsor_street_address2">
+                        <?php _e( 'Street Address 2:', 'wordcamporg' ); ?>
+                </label>
+
+                <input
+                        type="text"
+                        class="regular-text"
+                        id="_wcpt_sponsor_street_address2"
+                        name="_wcpt_sponsor_street_address2"
+                        value="<?php echo esc_attr( $street_address2 ); ?>"
+                />
+        </li>
+
+        <li>
+                <label for="_wcpt_sponsor_city">
+                        <?php _e( 'City:', 'wordcamporg' ); ?>
+                </label>
+
+                <input
+                        type="text"
+                        class="regular-text"
+                        id="_wcpt_sponsor_city"
+                        name="_wcpt_sponsor_city"
+                        value="<?php echo esc_attr( $city ); ?>"
+                    required
+                />
+
+                <?php $this->render_form_field_required_indicator(); ?>
+        </li>
+
+        <li>
+                <label for="_wcpt_sponsor_state">
+                        <?php _e( 'State / Province:', 'wordcamporg' ); ?>
+                </label>
+
+                <input
+                        type="text"
+                        class="regular-text"
+                        id="_wcpt_sponsor_state"
+                        name="_wcpt_sponsor_state"
+                        value="<?php echo esc_attr( $state ); ?>"
+                />
+
+                <span class="description">
+                        <?php esc_html_e( 'Only necessary if you want this to be shown on your invoice', 'wordcamporg' ); ?>
+                </span>
+        </li>
+
+        <li>
+                <label for="_wcpt_sponsor_zip_code">
+                        <?php _e( 'ZIP / Postal Code:', 'wordcamporg' ); ?>
+                </label>
+
+                <input
+                        type="text"
+                        class="regular-text"
+                        id="_wcpt_sponsor_zip_code"
+                        name="_wcpt_sponsor_zip_code"
+                        value="<?php echo esc_attr( $zip_code ); ?>"
+                        maxlength="30"
+                    required
+                />
+
+                <?php $this->render_form_field_required_indicator(); ?>
+        </li>
 </ul>
 

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wc-post-types/wc-post-types.php

@@ -1635 +1635 @@
                 $country         = get_post_meta( $sponsor->ID, '_wcpt_sponsor_country',           true );
 
+                if ( $state === $this->get_sponsor_info_state_default_value() ) {
+                        $state = '';
+                }
+
                 $available_countries = array( 'Abkhazia', 'Afghanistan', 'Aland', 'Albania', 'Algeria', 'American Samoa', 'Andorra', 'Angola', 'Anguilla', 'Antigua and Barbuda', 'Argentina', 'Armenia', 'Aruba', 'Ascension', 'Ashmore and Cartier Islands', 'Australia', 'Australian Antarctic Territory', 'Austria', 'Azerbaijan', 'Bahamas, The', 'Bahrain', 'Baker Island', 'Bangladesh', 'Barbados', 'Belarus', 'Belgium', 'Belize', 'Benin', 'Bermuda', 'Bhutan', 'Bolivia', 'Bosnia and Herzegovina', 'Botswana', 'Bouvet Island', 'Brazil', 'British Antarctic Territory', 'British Indian Ocean Territory', 'British Sovereign Base Areas', 'British Virgin Islands', 'Brunei', 'Bulgaria', 'Burkina Faso', 'Burundi', 'Cambodia', 'Cameroon', 'Canada', 'Cape Verde', 'Cayman Islands', 'Central African Republic', 'Chad', 'Chile', "China, People's Republic of", 'China, Republic of (Taiwan)', 'Christmas Island', 'Clipperton Island', 'Cocos (Keeling) Islands', 'Colombia', 'Comoros', 'Congo, (Congo – Brazzaville)', 'Congo, (Congo – Kinshasa)', 'Cook Islands', 'Coral Sea Islands', 'Costa Rica', "Cote d'Ivoire (Ivory Coast)", 'Croatia', 'Cuba', 'Cyprus', 'Czech Republic', 'Denmark', 'Djibouti', 'Dominica', 'Dominican Republic', 'Ecuador', 'Egypt', 'El Salvador', 'Equatorial Guinea', 'Eritrea', 'Estonia', 'Ethiopia', 'Falkland Islands (Islas Malvinas)', 'Faroe Islands', 'Fiji', 'Finland', 'France', 'French Guiana', 'French Polynesia', 'French Southern and Antarctic Lands', 'Gabon', 'Gambia, The', 'Georgia', 'Germany', 'Ghana', 'Gibraltar', 'Greece', 'Greenland', 'Grenada', 'Guadeloupe', 'Guam', 'Guatemala', 'Guernsey', 'Guinea', 'Guinea-Bissau', 'Guyana', 'Haiti', 'Heard Island and McDonald Islands', 'Honduras', 'Hong Kong', 'Howland Island', 'Hungary', 'Iceland', 'India', 'Indonesia', 'Iran', 'Iraq', 'Ireland', 'Isle of Man', 'Israel', 'Italy', 'Jamaica', 'Japan', 'Jarvis Island', 'Jersey', 'Johnston Atoll', 'Jordan', 'Kazakhstan', 'Kenya', 'Kingman Reef', 'Kiribati', 'Korea, North', 'Korea, South', 'Kuwait', 'Kyrgyzstan', 'Laos', 'Latvia', 'Lebanon', 'Lesotho', 'Liberia', 'Libya', 'Liechtenstein', 'Lithuania', 'Luxembourg', 'Macau', 'Macedonia', 'Madagascar', 'Malawi', 'Malaysia', 'Maldives', 'Mali', 'Malta', 'Marshall Islands', 'Martinique', 'Mauritania', 'Mauritius', 'Mayotte', 'Mexico', 'Micronesia', 'Midway Islands', 'Moldova', 'Monaco', 'Mongolia', 'Montenegro', 'Montserrat', 'Morocco', 'Mozambique', 'Myanmar (Burma)', 'Nagorno-Karabakh', 'Namibia', 'Nauru', 'Navassa Island', 'Nepal', 'Netherlands', 'Netherlands Antilles', 'New Caledonia', 'New Zealand', 'Nicaragua', 'Niger', 'Nigeria', 'Niue', 'Norfolk Island', 'Northern Cyprus', 'Northern Mariana Islands', 'Norway', 'Oman', 'Pakistan', 'Palau', 'Palmyra Atoll', 'Panama', 'Papua New Guinea', 'Paraguay', 'Peru', 'Peter I Island', 'Philippines', 'Pitcairn Islands', 'Poland', 'Portugal', 'Pridnestrovie (Transnistria)', 'Puerto Rico', 'Qatar', 'Queen Maud Land', 'Reunion', 'Romania', 'Ross Dependency', 'Russia', 'Rwanda', 'Saint Barthelemy', 'Saint Helena', 'Saint Kitts and Nevis', 'Saint Lucia', 'Saint Martin', 'Saint Pierre and Miquelon', 'Saint Vincent and the Grenadines', 'Samoa', 'San Marino', 'Sao Tome and Principe', 'Saudi Arabia', 'Senegal', 'Serbia', 'Seychelles', 'Sierra Leone', 'Singapore', 'Slovakia', 'Slovenia', 'Solomon Islands', 'Somalia', 'Somaliland', 'South Africa', 'South Georgia & South Sandwich Islands', 'South Ossetia', 'Spain', 'Sri Lanka', 'Sudan', 'Suriname', 'Svalbard', 'Swaziland', 'Sweden', 'Switzerland', 'Syria', 'Tajikistan', 'Tanzania', 'Thailand', 'Timor-Leste (East Timor)', 'Togo', 'Tokelau', 'Tonga', 'Trinidad and Tobago', 'Tristan da Cunha', 'Tunisia', 'Turkey', 'Turkmenistan', 'Turks and Caicos Islands', 'Tuvalu', 'U.S. Virgin Islands', 'Uganda', 'Ukraine', 'United Arab Emirates', 'United Kingdom', 'United States', 'Uruguay', 'Uzbekistan', 'Vanuatu', 'Vatican City', 'Venezuela', 'Vietnam', 'Wake Island', 'Wallis and Futuna', 'Yemen', 'Zambia', 'Zimbabwe' );
                 // todo use WordCamp_Budgets::get_valid_countries_iso3166() instead. need to switch multi-event sponsors at same time.
@@ -1641 +1645 @@
 
                 require_once( __DIR__ . '/views/sponsors/metabox-sponsor-info.php' );
+        }
+
+        /**
+         * Returns the default value for the state input when it's empty
+         *
+         * @return string
+         */
+        protected function get_sponsor_info_state_default_value() {
+                return 'Not Applicable';
         }
 
@@ -1835 +1848 @@
                         foreach ( $text_values as $id ) {
                                 $values[ $id ] = sanitize_text_field( filter_input( INPUT_POST, '_wcpt_sponsor_' . $id ) );
+                        }
+
+                        if ( empty( $values['state'] ) ) {
+                                $values['state'] = $this->get_sponsor_info_state_default_value();
                         }