Home Forums WC Vendors Pro Support JS error on Order Tools

NOTICE: We've Moved to a Ticket System for Support

As of August 31, 2017 (12am EST) our support forums will be retired (read-only), and we will be moving to a support ticket system.  This will allow us to better organize and answer support requests, and provide a more personalized experience as we assist our customers.

For the time being, we will leave our forums open for reading and learning while we work on creating a more robust Knowledge Base for everyone to use.

If you are a WC Vendors Pro customer please open a support ticket here. 

If you are a WC Vendors user please open a support ticket on the Wordpress.org forums.

The information on this forum is outdated and in most instances no longer relevant. Please be sure to check our documentation for the most up to date information.

https://docs.wcvendors.com/

Thank you to all of our customers!

 

Viewing 10 posts - 1 through 10 (of 10 total)
  • Author
    Posts
  • #36535
    kaelobenn
    Participant

    Good day Ben and Jamie

    Sorry to bother you, i am having issues with the order tools namely;

    a. Shipping Label
    b. Order Note
    c. Tracking Number
    d. Mark Shipped

    If i click on any of these it simply redirects and opens up the orders page again, with the exception of Order Note and Tracking Number which displays the customer has been notified message. However on inspection of the emails none is being sent out.

    On the admin woocommmerce screen, the “X” does not turn to a tick to indicate vendor has shipped out.

    also furthermore, upon careful inspection, i have noticed the following javascript errors (unreachable code after return statement), see below

    javascript

    Please do assist,

    thank you

    regards

    Chris

    #36588
    Jamie
    Keymaster

    Hello,

    We don’t define addthis anywhere. If other plugins have broken javascript, that also breaks ours. Are you able to see if any of your plugins are causing an issue? What happens when you run the bare minimu. wp+woo+wcv free + wcv pro ?

    cheers,

    Jamie.

    #36598
    kaelobenn
    Participant

    Hello Jamie thanks for getting back to me,

    After playing around with plugin optimizer (everyone should use this), i discovered that the Woocommerce Sequential Orders Pro plugin is the culprit. This is a plugin i cannot do without as we have already been using if for so long.

    the problem seems that the order tools on the vendor dashboard, use query strings such as this

    /?wcv_mark_shipped=29720

    so if you are using sequential order pro, this breaks all the tools from working as now the order number is no longer the default 29720 but SKY10888. For example as the query strings are now using the custom order numbers like

    /?wcv_mark_shipped=SKY10888

    this results in a redirect to the order screen than performing the desired operation.

    Please kindly provide full support for this plugin and similar plugins that use custom order numbers.

    thank you

    regards

    Christopher

    PS: i still have an outstanding issue that i long put up. When someone clicks on order details, the order number is #0 instead of the SKY10888

    order

    #38046
    kaelobenn
    Participant

    Hello

    Any updates?

    thank you

    #38175
    Jamie
    Keymaster

    Hello,

    The outstanding issue was fixed in version 1.3.3 or below. If you haven’t updated your template then this would not have been fixed. Check your template versions. You will see that we call $order->get_order_number(). Which is how all order numbers should be displayed. We are using the built in internal woocommerce functions to display and retrieve order numbers. This is also the suggested way to be compatible with the plugin mentioned above, so according to them, we are already compatible.

    Also the error you’re getting is a javascript error and has nothing to do with the underlying order id. That error is saying that some function doesn’t exist called ‘addthis’ which we don’t define. So something is calling that when it shouldn’t be.

    cheers,

    Jamie.

    #38346
    kaelobenn
    Participant

    Good day,

    All my template files are up to date though, the main problem is the vendor order tools (order notes, tracking, mark shipped) not working as described due to the sequential order number pro (SONP) changing the url structure.

    URL with SONP off tools work fine
    /?wcv_mark_shipped=29720

    URL with SONP on tools do not work
    /?wcv_mark_shipped=SKY10888

    please try and install SONP and configure on demo site to replicate issue to see what i am describing.

    thank you

    regards

    #43142
    Garry Mitchell
    Participant

    I too am using Sequential Order Numbers (albeit the free version, not the Pro version).

    The issue is that within WCVendors, the URL is using the “pretty” order number, not the underlying order(post) ID.

    The URL call should be using the order ID, not the order number, as the order is subsequently loaded (to perform the update of shipping status etc) via the order (post) ID.

    To fix this (manually, and this WILL break the next time WCVendors is updated), in: wc-vendors-pro/public/partials/order/class-wc-vendors-pro-order-controller.php

    on line 477, change:

    'url' => '?wcv_mark_shipped='.$order->get_order_number()

    to:

    'url' => '?wcv_mark_shipped='.$order->id

    and the same anywhere else a URL is created which is then subsequently used.

    So – when displaying an order to the user/vendor/admin should always call get_order_number() but the underlying methods must use the order (post) ID.

    I hope the above helps you understand why we are experiencing this issue, and I look forward to having this issue resolved!

    Regards,

    Garry

    #43953
    kaelobenn
    Participant

    Good day Garry

    I checked out what you have suggested with the temp fix, it seems to only work with the mark shipped tool. I have tried to replicate this fix with the other tools but it still gives me the same error.

    I wish @Ben and @Jamie would take a quick look at this issue and put it up in the next release of WCVPro.

    thanks

    kaelobenn

    #64991
    kaelobenn
    Participant

    Hello @ben and @jamie please let us know if this issue will ever get resolved. with each update the issue remains.

    #65041
    Anna
    Member

    Hello @kaelobenn
    As far as I can see, we call the order number in the proper fashion for WooCommerce.
    What is it that you believe needs to be fixed?

Viewing 10 posts - 1 through 10 (of 10 total)
  • The forum ‘WC Vendors Pro Support’ is closed to new topics and replies.