Incorrect field reference

This topic has 9 replies, 2 voices, and was last updated 4 years, 6 months ago by Sushobhan.

Viewing 9 reply threads

Author
Posts
- April 22, 2020 at 3:34 pm #121745
  tgl
  Participant
  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
- April 22, 2020 at 8:50 pm #121901
  Sushobhan
  Keymaster
  Hi,
  Thanks for reporting this. We will fix that in our next update.
- April 25, 2020 at 12:56 am #122700
  tgl
  Participant
  It also changed one of the parent menus:
  https://www.screencast.com/t/tnBkXNKZ5R
  Store Vendors turns into Sold by: Vendors
- April 25, 2020 at 1:31 pm #122844
  Sushobhan
  Keymaster
  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!
- April 26, 2020 at 11:43 pm #123407
  tgl
  Participant
  It also changed the name of every widget:
  https://www.screencast.com/t/1vivHFImN
- April 27, 2020 at 4:19 pm #123653
  Sushobhan
  Keymaster
  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.
- April 27, 2020 at 10:21 pm #123817
  tgl
  Participant
  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!
- April 29, 2020 at 11:12 pm #124773
  tgl
  Participant
  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
- April 29, 2020 at 11:29 pm #124781
  tgl
  Participant
  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.
- April 30, 2020 at 10:53 pm #125172
  Sushobhan
  Keymaster
  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

Viewing 9 reply threads

You must be logged in to reply to this topic.

Incorrect field reference

We're shifting our Forum based support to a more dedicated support system!