Manage Products Listing Issue with latest update

  • Resolved ouija

    (@ouija)


    5 years, 3 months ago

    Hi there,

    The latest update (6.2.6) seems to have some minor issues listing products under each ‘page status’ filter.

    I am using this the WC-Vendors plugin, along with this filter here: https://www.ads-software.com/support/topic/published-draft-pending-counts-are-incorrect-based-on-product-status/#post-11178875

    And while the filter is working and returning the correct number of products based on the publish status, the actual number of products being listed in the manage page itself is not correct (for example, one vendor has 11 products with a publish status of “draft”, and the count is returning/displaying correctly as 11 (based off that filter) but there are actually only 6 products being returned in the page (and I’m not sure why the other 5 aren’t listed)

    Update: I figured out the issue. It’s only returning the first 25 results, and not all results. I also have a filter in this page to only display the posts with that product status (so that if you click on “Archived”, then only the “Archived” posts are shown (by default, the plugin seems to not do this properly — maybe this is a bug or oversight when it comes to WC-Vendors?).

    Anyways, I haven’t fixed it yet — but at least I know what is causing it.

    • This topic was modified 5 years, 3 months ago by ouija.
    • This topic was modified 5 years, 3 months ago by ouija.
Viewing 4 replies - 1 through 4 (of 4 total)
  • Thread Starter ouija

    (@ouija)

    The problem is (at least with WC Vendors), underthe Manage Products page, when I click on one of these ‘status filter’ links (that this function updates the counts for: https://www.ads-software.com/support/topic/published-draft-pending-counts-are-incorrect-based-on-product-status/#post-11178875), it’s NOT returning only the products with that particular status, but rather every product regardless of the status.

    So, for example, if I click “Drafts”, it’s still returning published products, pending products, drafts and this newly added archived products.

    It should only return the products with that particular status, and nothing else.

    Thread Starter ouija

    (@ouija)

    It seems like it’s ./controllers/products/wcfm-controller-products.php that isn’t restricting the returned products based on the post_status, but I can’t see why… It looks like it should be getting the product_status post variable.

    For now, I’ve added a break in the ‘wcfm_products_array’ foreach loop at line 242 of this file, ie:

    if( isset($_POST['product_status']) && !empty($_POST['product_status']) && ( $_POST['product_status'] != 'any' ) && sanitize_text_field( $_POST['product_status'] ) != $wcfm_products_single->post_status ) continue;

    Which prevents different product statuses from returing based on the post variable, but there is still an issue with the number of results and counts, because the intitial query isn’t filtering by status properly…

    • This reply was modified 5 years, 3 months ago by ouija.
    Thread Starter ouija

    (@ouija)

    I’ve opted to remove/hide pagnation for now and just return all results per ‘status filter’ until this issue is resolved; I’m not quite sure how to fix this to utilize the post_status properly, but this is at least a resolution for the time being.

    Updated that same ./controllers/products/wcfm-controller-products.php page and changed:

    'posts_per_page'   => -1,
//'offset'           => $offset,

    And added css rules to hide pagnation elements and a filter to change the ‘info’ text setting for wc-frontend-manager.

    • This reply was modified 5 years, 3 months ago by ouija.
    Plugin Author WC Lovers

    (@wclovers)

    Hi,

    Product list page and all WCFM Dashboard list pages has pagination and it shows 25 items per page. – https://ibb.co/hyDJwhz

    This default count (25) can be change.

    Thank You

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Manage Products Listing Issue with latest update’ is closed to new replies.