WordPress.org
Get WordPress

Making WordPress.org

Changeset 2682


Ignore:
Timestamp:
03/03/2016 07:33:42 PM (11 years ago)
Author:
iandunn
Message:

WordCamp Budgets: Make Vendor Payment and Payment Method fields required.

Location:
sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments
Files:
16 edited

Legend:

Unmodified
Added
Removed

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/css/wordcamp-budgets.css

    r2633 r2682  
    5454        color: #dc3232;
    5555}
     56
     57/* Make room for the required field indicator (*) */
     58body.post-type-wcp_payment_request input.large-text,
     59body.post-type-wcp_payment_request input.regular-text,
     60body.post-type-wcp_payment_request textarea.large-text,
     61body.post-type-wcp_payment_request .meta-box-sortables select {
     62        width: 95%;
     63}
     64
     65@media screen and ( max-width: 782px ) {
     66        body.post-type-wcp_payment_request .form-table td textarea {
     67                display: inline;
     68                width: 95%;
     69        }
     70}
     71
    5672
    5773/* margins from above selector aren't applied */

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/includes/payment-request.php

    r2676 r2682  
    180180                        plugins_url( 'javascript/payment-requests.js', __DIR__ ),
    181181                        array( 'wordcamp-budgets', 'wcb-attached-files', 'jquery' ),
    182                         2,
     182                        3,
    183183                        true
    184184                );
    … …  
    339339         * @param string $name
    340340         * @param string $description
    341          */
    342         protected function render_textarea_input( $post, $label, $name, $description = '' ) {
     341         * @param bool   $required
     342         */
     343        protected function render_textarea_input( $post, $label, $name, $description = '', $required = true ) {
    343344                $text = $this->get_field_value( $name, $post );
    344345
    … …  
    352353         * @param string $label
    353354         * @param string $name
    354          */
    355         protected function render_select_input( $post, $label, $name ) {
     355         * @param bool   $required
     356         */
     357        protected function render_select_input( $post, $label, $name, $required = true ) {
    356358                $selected = get_post_meta( $post->ID, '_camppayments_' . $name, true );
    357359                $options  = $this->get_field_value( $name, $post );
    … …  
    360362        }
    361363
    362         protected function render_country_input( $post, $label, $name ) {
     364        /**
     365         * Render a select dropdown for countries
     366         *
     367         * @param WP_Post $post
     368         * @param string  $label
     369         * @param string  $name
     370         * @param bool    $required
     371         */
     372        protected function render_country_input( $post, $label, $name, $required = true ) {
    363373                $selected = $this->get_field_value( $name, $post );
    364374                $options = WordCamp_Budgets::get_valid_countries_iso3166();
    … …  
    373383         * @param string $label
    374384         * @param string $name
    375          */
    376         protected function render_radio_input( $post, $label, $name ) {
     385         * @param bool   $required
     386         */
     387        protected function render_radio_input( $post, $label, $name, $required = true ) {
    377388                $selected = get_post_meta( $post->ID, "_{$this->meta_key_prefix}_" . $name, true );
    378389                $options  = $this->get_field_value( $name, $post );
    … …  
    387398         * @param string $label
    388399         * @param string $name
    389          */
    390         protected function render_checkbox_input( $post, $label, $name, $description = '' ) {
     400         * @param bool   $required
     401         */
     402        protected function render_checkbox_input( $post, $label, $name, $description = '', $required = true ) {
    391403                $value = $this->get_field_value( $name, $post );
    392404
    … …  
    400412         * @param string $label
    401413         * @param string $name
    402          */
    403         protected function render_text_input( $post, $label, $name, $description = '', $variant = 'text', $row_classes = array(), $readonly = false ) {
     414         * @param string $description
     415         * @param string $variant
     416         * @param array  $row_classes
     417         * @param bool   $readonly
     418         * @param bool   $required
     419         */
     420        protected function render_text_input( $post, $label, $name, $description = '', $variant = 'text', $row_classes = array(), $readonly = false, $required = true ) {
    404421                $value = $this->get_field_value( $name, $post );
    405422                array_walk( $row_classes, 'sanitize_html_class' );

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/includes/wordcamp-budgets.php

    r2677 r2682  
    129129                        plugins_url( 'javascript/wordcamp-budgets.js', __DIR__ ),
    130130                        array( 'jquery', 'jquery-ui-datepicker', 'media-upload', 'media-views' ),
    131                         1,
     131                        2,
    132132                        true
    133133                );
    … …  
    161161                        plugins_url( 'css/wordcamp-budgets.css', __DIR__ ),
    162162                        $soft_deps,
    163                         2
     163                        3
    164164                );
    165165        }

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/javascript/payment-requests.js

    r2633 r2682  
    2424                registerEventHandlers : function() {
    2525                        var currency        = $( '#currency' );
    26                         var paymentCategory = $( '#payment_category' );
     26                        var paymentCategory = $( '#payment_category' ),
     27                                paymentDetails  = $( '#wcp_payment_details' );
    2728
    28                         $( '#wcp_payment_details' ).find( 'input[name=payment_method]' ).change( wcb.togglePaymentMethodFields );
     29                        paymentDetails.find( 'input[name=payment_method]' ).change( wcb.togglePaymentMethodFields );
     30                        paymentDetails.find( 'input[name=payment_method]:checked' ).trigger( 'change' ); // Set the initial state
     31
    2932                        paymentCategory.change( app.toggleOtherCategoryDescription );
    3033                        paymentCategory.trigger( 'change' );   // Set the initial state
     34
    3135                        currency.change( wcb.setDefaultPaymentMethod );
    3236                        currency.trigger( 'change' );   // Set the initial state
     37
    3338                        $( '#row-files' ).find( 'a.wcb-insert-media' ).click( wcb.showUploadModal );
    3439
    … …  
    5358                                if ( 'other' == $( this ).find( 'option:selected' ).val() ) {
    5459                                        $( otherCategoryDescription ).removeClass( 'hidden' );
     60                                        $( otherCategoryDescription ).find( ':input' ).prop( 'required', true );
    5561                                } else {
    5662                                        $( otherCategoryDescription ).addClass( 'hidden' );
     63                                        $( otherCategoryDescription ).find( ':input' ).prop( 'required', false );
    5764                                }
    5865

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/javascript/reimbursement-requests.js

    r2646 r2682  
    3131                registerEventHandlers : function() {
    3232                        var reason   = $( '#_wcbrr_reason'   ),
    33                                 currency = $( '#_wcbrr_currency' );
     33                                currency = $( '#_wcbrr_currency' ),
     34                                paymentMethod = $( '#row-payment-method' );
    3435
    3536                        reason.change( app.toggleOtherReasonDescription );
    … …  
    3940                        currency.trigger( 'change' );   // Set the initial state
    4041
    41                         $( '#row-payment-method' ).find( 'input[name=payment_method]' ).change( wcb.togglePaymentMethodFields );
     42                        paymentMethod.find( 'input[name=payment_method]' ).change( wcb.togglePaymentMethodFields );
     43                        paymentMethod.find( 'input[name=payment_method]:checked' ).trigger( 'change' ); // Set the initial state
     44
    4245                        $( '#wcbrr_general_information' ).find( 'a.wcb-insert-media' ).click(   wcb.showUploadModal           );
    4346                        $( '#wcbrr-add-another-expense' ).click(                                app.addNewExpense             );

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/javascript/wordcamp-budgets.js

    r2677 r2682  
    3131                togglePaymentMethodFields : function( event ) {
    3232                        var active_fields_container = '#' + $( this ).attr( 'id' ) + '_fields';
    33                         var payment_method_fields   = '.payment_method_fields';
     33                        var payment_method_fields   = '.payment_method_fields',
     34                                optionalFields          = [ 'needs_intermediary_bank' ];
    3435
    3536                        $( payment_method_fields   ).removeClass( 'active' );
    3637                        $( payment_method_fields   ).addClass( 'hidden' );
     38
     39                        $( payment_method_fields ).each( function( containerIndex, fieldContainer ) {
     40                                $( fieldContainer ).find( ':input' ).each( function( fieldIndex, inputField ) {
     41                                        $( inputField ).prop( 'required', false );
     42                                } );
     43                        } );
     44
    3745                        $( active_fields_container ).removeClass( 'hidden' );
    3846                        $( active_fields_container ).addClass( 'active' );
     47
     48                        $( active_fields_container ).find( ':input' ).each( function( index, field ) {
     49                                if ( $.inArray( $( field ).prop( 'id' ), optionalFields ) > -1 ) {
     50                                        return;
     51                                }
     52
     53                                $( field ).prop( 'required', true );
     54                        } );
     55
     56                        /*
     57                         * Also toggle intermediary bank fields, because otherwise switching to Wire from another method
     58                         * wouldn't remove `required` attributes.
     59                         */
     60                        app.toggleIntermediaryBankFields( event );
    3961
    4062                        // todo make the transition smoother
    … …  
    5577                                        if ( isWirePayment && needsIntermediaryBank ) {
    5678                                                $( row ).removeClass( 'hidden' );
     79                                                $( row ).find( ':input' ).prop( 'required', true );
    5780                                        } else {
    5881                                                $( row ).addClass( 'hidden' );
     82                                                $( row ).find( ':input' ).prop( 'required', false );
    5983                                        }
    6084                                } );

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/views/payment-request/input-country.php

    r2494 r2682  
    55                </label>
    66        </th>
     7
    78        <td>
    8                 <select id="<?php echo esc_attr( $name ); ?>" name="<?php echo esc_attr( $name ); ?>">
     9                <select
     10                        id="<?php echo esc_attr( $name ); ?>"
     11                        name="<?php echo esc_attr( $name ); ?>"
     12                        <?php __checked_selected_helper( $required, true, true, 'required' ); ?>
     13                >
    914                        <option value="">(None)</option>
    1015                        <?php foreach ( $options as $value => $option_label ) : ?>
    … …  
    1419                        <?php endforeach; ?>
    1520                </select>
     21
     22                <?php
     23                        if ( $required ) {
     24                                WordCamp_Budgets::render_form_field_required_indicator();
     25                        }
     26                ?>
    1627        </td>
    1728</tr>

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/views/payment-request/input-radio.php

    r995 r2682  
    1414                                value="<?php echo esc_attr( $option ); ?>"
    1515                                <?php checked( $option, $selected ); ?>
     16                                <?php __checked_selected_helper( $required, true, true, 'required' ); ?>
    1617                                />
    1718
    … …  
    2021                        </label>
    2122                <?php endforeach; ?>
     23
     24                <?php
     25                        if ( $required ) {
     26                                WordCamp_Budgets::render_form_field_required_indicator();
     27                        }
     28                ?>
    2229        </td>
    2330</tr>

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/views/payment-request/input-select.php

    r995 r2682  
    77
    88        <td>
    9                 <select id="<?php echo esc_attr( $name ); ?>" name="<?php echo esc_attr( $name ); ?>">
     9                <select
     10                        id="<?php echo esc_attr( $name ); ?>"
     11                        name="<?php echo esc_attr( $name ); ?>"
     12                    <?php __checked_selected_helper( $required, true, true, 'required' ); ?>
     13                >
    1014                        <option value="null-select-one">
    1115                                <?php printf( __( '-- Select a %s --', 'wordcamporg' ), $label ); ?>
    … …  
    1923                        <?php endforeach; ?>
    2024                </select>
     25
     26                <?php
     27                        if ( $required ) {
     28                                WordCamp_Budgets::render_form_field_required_indicator();
     29                        }
     30                ?>
    2131        </td>
    2232</tr>

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/views/payment-request/input-text.php

    r2060 r2682  
    1313                        value="<?php echo esc_attr( $value ); ?>"
    1414                        <?php if ( $readonly ) { echo 'readonly="readonly"'; } ?>
     15                        <?php __checked_selected_helper( $required, true, true, 'required' ); ?>
    1516                        class="regular-text"
    1617                        />
     18
     19                <?php
     20                        if ( $required ) {
     21                                WordCamp_Budgets::render_form_field_required_indicator();
     22                        }
     23                ?>
    1724
    1825                <?php if ( ! empty( $description ) ) : ?>

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/views/payment-request/input-textarea.php

    r2416 r2682  
    77
    88        <td>
    9                 <textarea id="<?php echo esc_attr( $name ); ?>" name="<?php echo esc_attr( $name ); ?>" class="large-text"><?php
     9                <textarea
     10                        id="<?php echo esc_attr( $name ); ?>"
     11                        name="<?php echo esc_attr( $name ); ?>"
     12                        class="large-text"
     13                    <?php __checked_selected_helper( $required, true, true, 'required' ); ?>
     14                ><?php
    1015                        echo esc_html( $text );
    1116                ?></textarea>
     17
     18                <?php
     19                        if ( $required ) {
     20                                WordCamp_Budgets::render_form_field_required_indicator();
     21                        }
     22                ?>
    1223
    1324                <?php if ( ! empty( $description ) ) : ?>

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/views/payment-request/metabox-general.php

    r2676 r2682  
    2424        <?php $this->render_textarea_input( $post, 'Notes', 'general_notes', 'Any other details you want to share.' ); ?>
    2525</table>
     26
     27<p class="wcb-form-required">
     28        <?php _e( '* required', 'wordcamporg' ); ?>
     29</p>

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/views/payment-request/metabox-payment.php

    r2677 r2682  
    99        <table class="form-table">
    1010                <?php if ( $box['args']['show_vendor_requested_payment_method'] ) : ?>
    11                         <?php $this->render_textarea_input( $post, 'Did the vendor request a specific type of payment?', 'vendor_requested_payment_method', 'Add any relevant details' ); ?>
     11                        <?php $this->render_textarea_input( $post, 'Did the vendor request a specific type of payment?', 'vendor_requested_payment_method', 'Add any relevant details', false ); ?>
    1212                <?php endif;?>
    1313
    … …  
    7777                ); ?>
    7878
    79                 <table>
     79                <table id="intermediary_bank_fields">
    8080                        <?php $this->render_text_input( $post, 'Intermediary Bank Name',              'interm_bank_name' ); ?>
    8181                        <?php $this->render_text_input( $post, 'Intermediary Bank Street Address',    'interm_bank_street_address' ); ?>
    … …  
    104104        </div>
    105105</fieldset>
     106
     107<p class="wcb-form-required">
     108        <?php _e( '* required', 'wordcamporg' ); ?>
     109</p>

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/views/payment-request/metabox-status.php

    r2643 r2682  
    2929                        <?php if ( $post->post_status != 'auto-draft' ) : ?>
    3030                        <div class="misc-pub-section">
    31                                 <?php $this->render_text_input( $post, 'Date Vendor was Paid', 'date_vendor_paid', '', 'date', array(), $date_vendor_paid_readonly ); ?>
     31                                <?php $this->render_text_input( $post, 'Date Vendor was Paid', 'date_vendor_paid', '', 'date', array(), $date_vendor_paid_readonly, false ); ?>
    3232                        </div>
    3333

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/views/payment-request/metabox-vendor.php

    r2494 r2682  
    1313        ?>
    1414</table>
     15
     16<p class="wcb-form-required">
     17        <?php _e( '* required', 'wordcamporg' ); ?>
     18</p>

  • sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/views/wordcamp-budgets/form-field-required-indicator.php

    r2605 r2682  
    11
    2 <span class="wcb-form-required" aria-hidden="true">
    3         <?php // translators: The symbol to indicate the form field is required ?>
    4         <?php _e( '*', 'wordcamporg' ); ?>
     2<span class="wcb-form-required">
     3        <span aria-hidden="true">
     4                <?php // translators: The symbol to indicate the form field is required ?>
     5                <?php _e( '*', 'wordcamporg' ); ?>
     6        </span>
     7
     8        <span class="screen-reader-text">
     9                <?php _e( 'required field', 'wordcamporg' ); ?>
     10        </span>
    511</span>
    6 
    7 <span class="screen-reader-text">
    8         <?php _e( 'required field', 'wordcamporg' ); ?>
    9 </span>
Note: See TracChangeset for help on using the changeset viewer.