Incorrect field reference

[tgl]

When you change the “sold by label” from the default of “store” (in my case I changed it to “Sold by:”):

https://www.screencast.com/t/Uk5ugCw22SK

It also changes the word “Store” in three menu titles from “Store” to “Sold by:”:

https://www.screencast.com/t/TxhHh6ppziNJ

Store Hours turns into Sold by: Hours
Store Badges turns into Sold by: Badges
Store Verification turns into Sold by: Verification

[Sushobhan]

Hi,
Thanks for reporting this. We will fix that in our next update.

[tgl]

It also changed one of the parent menus:

https://www.screencast.com/t/tnBkXNKZ5R

Store Vendors turns into Sold by: Vendors

[Sushobhan]

Hi,
Kindly use the following snippet-

add_filter( 'wcfmmp_sold_by_label', function($sold_by_label, $vendor_id) {
    if(is_wcfm_page()) {
        return __('Store', 'wc-multivendor-marketplace');
    }
    return $sold_by_label;
}, 10, 2);

Add this code to your child theme’s functions.php
In case you do not have child theme then add code using this plugin – https://wordpress.org/plugins/code-snippets/
Let me know how this goes.
Thanks!

[tgl]

It also changed the name of every widget:

https://www.screencast.com/t/1vivHFImN

[Sushobhan]

Hi,
The image you provided doesn’t show that. Every widget title seems fine there. Are you sure you sent me the right snapshot?
Looking forward to helping you.

[tgl]

You are 100% correct, I was just looking at the inline colon after looking at the others in the admin where I had “Sold by:”.

Sorry about that one!

[tgl]

However, I did just notice it changes the wording in the outgoing emails:

I assume this email is supposed to read “Notification – New Store” and not “Notification – New Sold by:”

https://www.screencast.com/t/XzhDehFJewN

[tgl]

Wouldn’t it make much more sense to create a field that is just for use in the “Sold by” widget?

https://www.screencast.com/t/U2KmgRnYstQY

You list it in the admin as changing the wording in the widget yet clearly you attached it to a widely used WC field that is utilized in many locations.

[Sushobhan]

Hello,
I’ll pass this to my dev team to add a separate filter just to show the ‘Sold by’ label in the frontend. But, till then you can use this updated snippet. I believe this will not give you any unpleasant surprises.

add_filter( 'wcfmmp_sold_by_label', function($sold_by_label, $vendor_id) {
    if ( ( ! is_admin() || defined( 'DOING_AJAX' ) ) && ! defined( 'DOING_CRON' ) && ! is_wcfm_page() ) {
        return $sold_by_label;
    }
    return __( 'Store', 'wc-multivendor-marketplace' );
}, 10, 2 );

Please remove the previous code and replace with this new one.
Thank You!

[Author]

Posts