Skip to:
Content

bbPress.org

Changeset 6663


Ignore:
Timestamp:
08/20/2017 09:41:37 PM (2 years ago)
Author:
johnjamesjacoby
Message:

Converter: ensure UI state variables are correctly localized.

This change combines the new UI state variables into an array, so they retain their type-casting through the localization API. It also inverts the logic in 1 comparison to make sure the correct user feedback is displayed in its intended location.

Trunk, for 2.6. Props jrf. Fixes #3146.

Location:
trunk/src/includes/admin
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/includes/admin/assets/js/converter.js

    r6636 r6663  
    8080
    8181        if ( values['_bbp_converter_delay_time'] ) {
    82             BBP_Converter.delay = parseInt( values['_bbp_converter_delay_time'], 10 ) * 1000;
     82            BBP_Converter.state.delay = parseInt( values['_bbp_converter_delay_time'], 10 ) * 1000;
    8383        }
    8484
     
    129129
    130130        // Bail if not running
    131         if ( ! BBP_Converter.running ) {
     131        if ( ! BBP_Converter.state.running ) {
    132132            return;
    133133        }
     
    155155     */
    156156    function bbp_converter_wait() {
    157         clearTimeout( BBP_Converter.running );
     157        clearTimeout( BBP_Converter.state.running );
    158158
    159159        // Bail if not running
    160         if ( ! BBP_Converter.running ) {
     160        if ( ! BBP_Converter.state.running ) {
    161161            return;
    162162        }
    163163
    164164        // Wait, then POST
    165         BBP_Converter.running = setTimeout( function() {
     165        BBP_Converter.state.running = setTimeout( function() {
    166166            bbp_converter_post();
    167         }, parseInt( BBP_Converter.delay, 10 ) );
     167        }, parseInt( BBP_Converter.state.delay, 10 ) );
    168168    }
    169169
     
    176176     */
    177177    function bbp_converter_start() {
    178         clearTimeout( BBP_Converter.running );
    179         clearInterval( BBP_Converter.status );
    180 
    181         BBP_Converter.running = true;
     178        clearTimeout( BBP_Converter.state.running );
     179        clearInterval( BBP_Converter.state.status );
     180
     181        BBP_Converter.state.running = true;
    182182
    183183        var log = BBP_Converter.strings.start_continue;
    184         if ( BBP_Converter.started ) {
     184        if ( false === BBP_Converter.state.started ) {
    185185            log = BBP_Converter.strings.start_start;
    186             BBP_Converter.started = true;
     186            BBP_Converter.state.started = true;
    187187        }
    188188
     
    211211     */
    212212    function bbp_converter_stop( button, log ) {
    213         clearTimeout( BBP_Converter.running );
    214         clearInterval( BBP_Converter.status );
    215 
    216         BBP_Converter.running = false;
    217         BBP_Converter.status  = false;
     213        clearTimeout( BBP_Converter.state.running );
     214        clearInterval( BBP_Converter.state.status );
     215
     216        BBP_Converter.state.running = false;
     217        BBP_Converter.state.status  = false;
    218218
    219219        if ( ! button ) {
     
    262262     */
    263263    function bbp_converter_status( data ) {
    264         var remaining = parseInt( BBP_Converter.delay, 10 ) / 1000;
     264        var remaining = parseInt( BBP_Converter.state.delay, 10 ) / 1000;
    265265
    266266        status.text( BBP_Converter.strings.status_counting.replace( '%s', remaining ) );
    267         clearInterval( BBP_Converter.status );
    268 
    269         BBP_Converter.status = setInterval( function() {
     267        clearInterval( BBP_Converter.state.status );
     268
     269        BBP_Converter.state.status = setInterval( function() {
    270270            remaining--;
    271271            status.text( BBP_Converter.strings.status_counting.replace( '%s', remaining ) );
    272272
    273273            if ( remaining <= 0 ) {
    274                 clearInterval( BBP_Converter.status );
     274                clearInterval( BBP_Converter.state.status );
    275275
    276276                if ( parseInt( data.current_step, 10 ) < parseInt( data.final_step, 10 ) ) {
  • trunk/src/includes/admin/classes/class-bbp-converter.php

    r6661 r6663  
    128128        wp_localize_script( 'bbp-converter', 'BBP_Converter', array(
    129129
    130             // Vars
     130            // Nonce
    131131            'ajax_nonce' => wp_create_nonce( 'bbp_converter_process' ),
    132             'delay'      => (int)  get_option( '_bbp_converter_delay_time', 2 ),
    133             'running'    => false,
    134             'status'     => false,
    135             'started'    => (bool) get_option( '_bbp_converter_step', 0 ),
     132
     133            // UI State
     134            'state' => array(
     135                'delay'   => (int)  get_option( '_bbp_converter_delay_time', 2 ),
     136                'started' => (bool) get_option( '_bbp_converter_step',       0 ),
     137                'running' => false,
     138                'status'  => false
     139            ),
    136140
    137141            // Strings
    138             'strings'    => array(
     142            'strings' => array(
    139143
    140144                // Button text
Note: See TracChangeset for help on using the changeset viewer.