Forum Replies Created

Viewing 15 replies - 1 through 15 (of 23 total)
  • Thread Starter nwells

    (@nwells)

    I agree, this shouldn’t be manually changed. The trouble is, it is being deployed in your plugin files being set to “dev”:

    return array (
    'name' => 'Ninja Tables',
    'slug' => 'ninja-tables',
    'domain_path' => '/language',
    'text_domain' => 'ninja-tables',
    'hook_prefix' => 'ninjatables',
    'rest_namespace' => 'ninjatables',
    'rest_version' => 'v2',
    'env' => 'dev',
    );

    Then, in WPDBConnection.php, it checks the environment variable and sets to show DB errors if the site is not in the “prod” environment:

    protected function setupWpdbInstance($wpdb)
    {
    $this->wpdb = $wpdb;

    $this->wpdb->show_errors(
    $this->shouldShowErrors()
    );
    }

    /**
    * Determine if database errors should be shown.
    *
    * @return bool
    */
    protected function shouldShowErrors()
    {
    return strpos(App::env(), 'prod') === false;
    }

    Yes, ideally, there wouldn’t be any DB errors but this site has a plugin with a bug in it unfortunately which is causing the error – I’ll be working with them as well.

    Does that help clear up where the issue is in your plugin and how it is causing an issue as it thinks it is in dev mode?

    Thread Starter nwells

    (@nwells)

    Thanks for looking into it.

    Weirdly, it seems to have resolved itself. Maybe there was a bug in WooCommerce itself or another plugin causing the issue? Not sure, happy that it is working though.

    nwells

    (@nwells)

    I’ve just had reports of the same issue. It looks like an issue with simple products. In the frontend asset, woo-ajax-add-to-cart.js a new line has been recently added to append the product ID:

    formData.append('product_id', $(submitter).val());

    The result of this is 2 product IDs are now being submitted with the form as there is still the original product ID in the DOM.

    Plugin Author nwells

    (@nwells)

    The error appears to be related to a refund order which I hadn’t considered before. It is also failing to retrieve the billing country for the order which is normally a required field. Is this field not required on your website during checkout? Or was it a manually created order on your site and not all the data was filled out?

    Plugin Author nwells

    (@nwells)

    Well done getting more orders each day.

    WordPress defaults its query to 10 pages which can be adjusted in your general settings. To save the hassle, I’ve change the query used for the email to always include every order regardless of what that setting is now.

    I’ve just pushed through an update to the plugin so it should be available within the next 24 hours..

    nwells

    (@nwells)

    I’ve just tried it on a site and it worked fine. It is likely the context of what page you’re adding it on i.e. the code above assumes it is on a product single page. If it is is anything else then it likely won’t work due to how the ZIP plugin works out what to show.

    nwells

    (@nwells)

    You’d need to wrap that code in a new shortcode if you don’t already have one and, yes, add it to your functions.php file

    add_shortcode('elementor_zip', function() {
        if ( ! class_exists( 'WC_Zipmoney_Payment_Gateway' ) ) {
             return;
        }
        $zip                                     = new WC_Zipmoney_Payment_Gateway();
        $zip->WC_Zipmoney_Payment_Gateway_Config = new WC_Zipmoney_Payment_Gateway_Config( $zip );
        $zipWidget                               = new WC_Zipmoney_Payment_Gateway_Widget( $zip );
        $zipWidget->render_root_el();
        $zipWidget->render_widget_product();
    });

    You could then add the shortcode [elementor_zip] as a Shortcode Widget in Elementor.

    I’ve not tried out this exact example but it will be something like that.

    I just posted some code to do this here if you’re still after something:
    https://ww.wp.xz.cn/support/topic/how-to-integrate-with-elementor/

    I had the same issue and ended up having to create a shortcode to then place the widget in the Elementor template. The Zip widget hooks in to the normal WooCommerce woocommerce_single_product_summary which of course doesn’t get triggered when creating your own product template in Elementor.

    My shortcode contains this if it is of any use to anyone:

    if ( ! class_exists( 'WC_Zipmoney_Payment_Gateway' ) ) {
                return;
            }
            $zip                                     = new WC_Zipmoney_Payment_Gateway();
            $zip->WC_Zipmoney_Payment_Gateway_Config = new WC_Zipmoney_Payment_Gateway_Config( $zip );
            $zipWidget                               = new WC_Zipmoney_Payment_Gateway_Widget( $zip );
            $zipWidget->render_root_el();
            $zipWidget->render_widget_product();

    Ideally, yes, we’d have a nice Elementor widget which the above could easily be converted in to one.

    nwells

    (@nwells)

    I found a partial fix (v 1.1.17) by editing the free plugin file, woo-variation-swatches.php around line 362.
    There is an IF statement with a comment saying “If defer enable…” but the IF statement is actually just checking if the pro version is active – I suspect this is the bug. My temporary fix was to comment out that whole first block of the IF statement and just use the wp_enqueue_script in the ELSE block.

    Part of the issue is wp_localize_script won’t register the handle woo-variation-swatches as it hasn’t been set yet.

    I still have a minor issue on archive pages which I’ll work through but it at least works on the product pages now. Hopefully they can fix this before their next version otherwise any fix will need to be applied again.

    Plugin Author nwells

    (@nwells)

    Not currently but it could be added.
    You’d expect to just have the same summary but of all orders from the last x number of days?

    Plugin Author nwells

    (@nwells)

    Yes that is something that would be useful. I’ve typically hard coded it in there when testing but I’m the developer and can do that πŸ™‚

    I’ll look to create a button along with a field so you can set the date you want to test it on as well.

    Would that be enough do you think?

    • This reply was modified 6 years, 4 months ago by nwells.
    Plugin Author nwells

    (@nwells)

    @dethfire I can confirm that it works fine on WP5.0.3 and WC3.5.4. I’m yet to test on the very latest but will do so when the next release comes out shortly which will have improved support for translations.

    Yes it is still being supported.

    Plugin Author nwells

    (@nwells)

    Thanks for the rating! I’ve made a note to include this in the next release.

    Plugin Author nwells

    (@nwells)

    Thanks for the request – I’ve added this to the list of updates for the new release. Will let you know once its ready.

Viewing 15 replies - 1 through 15 (of 23 total)