Forum Replies Created
-
Posts
-
What do you mean by « everything works as expected« ?
What is expected ?
Migration of all old orders ?
New orders in both good and old way ?Is there a way to clean all old orders from posts tables ?
Does the migration is only based on woocommerce stras are migration ?
Does mvx make specific changes durinh migration ?
Hi Sagita,
We tried updating the MVX plugin to the latest version, but we still noticed this performance issue in the dashboard. When we made the same changes, the problem persisted. Do you have any solutions or an estimate for when this issue will be resolved?
Additionally, we also noticed a memory leak at another point, specifically during order creation. Please see the screenshot.
I noticed that the function $commission->get_posts() retrieves all posts of the commission type, which is causing a memory leak during checkout. To resolve this, I had to add some code to the filter mvx_before_get_commissions_total_data_commission_posts.add_filter(
‘mvx_before_get_commissions_total_data_commission_posts’,
function ( $commissions, $vendor_id, $args ) {
return array();
},
10,
3
);Attachments:
Hi,
Really the links working right now. Can you give more information about the fix?
What’s happened to crash the wp-admin?
I need more details to documentation in my end how the bug were fixed
Hi Sangita,
Yes you can disable theme and plugins, but I need to say to you be careful, cause the server shared with you is also our development environments. So, if disable any plugin or theme, re-active again please.
In that way we can work together
Seem that Pattchstack already send you information regading the Vulnerability. See hereafter.
Patchstack contact email is: sander.jurgens@patchstack.intercom-mail.com
————————————————
Sander from Patchstack <sander.jurgens@patchstack.intercom-mail.com>
Hello!
They have all the vulnerability details sent to them already (we contact vendors multiple times with details to fix it).
If they fail to find the emails, they can contact the support here (this is our only official support place)I am not sure where they have turned to before..
Sander
PatchstackHi Sangita,
Please contact Patchstack for more information about the vulnerability. The information is only disclosed to those who have permission to know, in this case, you.
Hello Sangita.
I’m spending a lot of time trying to help, even though I’ve already resolved it on my end. I won’t continue here after this comment…
I’m also a developer and, regardless of the scenario and the conditions to be met, you need to check if the field is set to a value to check “is_vendor_dashboard_page”, the rest of the conditions are not important.It’s simple: is it the vendor’s dashboard page? If it is not defined in the MVX settings, it is not. And your code is returning as if it were.
Hello Sangita.
Ok, a more detailed flow:
We have our custom dashboard that replaces wp-admin for vendors and vendor staffs. However, they still have access to the front of our website when logged in. When they go to the front, all the JS files registered by our theme disappear because the MVX filter ‘mvx_frontend_enqueue_scripts’ is removing them.What is the function of this filter?
It checks if the user is located in the vendor dashboard and, if yes, it removes all added scripts by theme. Ok, makes sense.Why does the filter remove them even when the user is not in the vendor’s dashboard?
When MVX checks if the user is on the vendor’s dashboard, it returns true because the “Vendor Dashboard Field” in the MVX settings is empty or has been set to a page that no longer exists. It should return false.Solution
When checking, the code must consider whether the page actually exists or whether the field is defined in the MVX Settings.
That’s what I suggested in the last messages.Test it:
Delete the Vendor’s Dashboard page completely and go to the front logged as a Vendor.Hello Sangita. Our license has not expired. We placed the order on September 30, 2023.
About our flow: We have a custom dashboard for vendors, so we don’t use yours. Thus, when checking if the user is on the vendor’s dashboard page and this page does not exist (or is not defined in the MVX settings), the function incorrectly returns true, as if it were the vendor’s dashboard page. I added a condition to check if it isn’t empty (see image).
In my case, as this function returned true, MVX removed all scripts inserted by theme from the queue and should not behave this way.
Hi Sangita, I don’t want to assign a product to multiple vendors. What I mean is that to “replace” the currently assigned vendor, I need to click the “Unassign” button, the page reloads and only then I can assign a new vendor. If I were making a modification to the product, I would need to stop to wait for this reassignment.
When a vendor is assigned, all the others vendors are hidden from <select>. Why not simply be able to change the selected option in “vendor selection”, instead of going through this flow, without having to click the unassign button? Is it not possible due to some permission issue?
