Incorrect field reference

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

We'll be closing our Forum support from 10th June, 2020 and move to Email Support assistance.

  • If you are a WCFM premium add-ons user, contact us- here
  • Want to know more before buying our add-ons? Send Pre sale queries- here
  • If you are a WCFM free user, please open a support ticket at WordPress.org
  • For WCFM App related queries, reach us- here
From now the forum will be read-only!

Multi Vendor Marketplace Plugin | WCFM Marketplace Forums WCFM Incorrect field reference

Viewing 9 reply threads
  • Author
    Posts
    • #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

    • #121901
      Sushobhan
      Keymaster

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

    • #122700
      tgl
      Participant

      It also changed one of the parent menus:

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

      Store Vendors turns into Sold by: Vendors

    • #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!

    • #123407
      tgl
      Participant

      It also changed the name of every widget:

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

    • #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.

    • #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!

    • #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

    • #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.

    • #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!

Viewing 9 reply threads
  • You must be logged in to reply to this topic.