Vendor ACL changes automatically!

Start
Prev
1
Next
End

liaskas
Topic Author
Offline

Posts: 205
Thank you received: 12

3 years 11 months ago #318859

-- HikaShop version -- : 4.3.0
-- HikaMarket version -- : 3.0.3
-- Joomla version -- : 3.9.18
-- PHP version -- : 7.2.30

Hello...

We are dealing with a strange situation that may be because of a missconfiguration on our system but we can not figure out.

New Vendor on our site.
ACL of this Vendor is set to Inherit: YES and everything works fine. Screenshot bellow:

Now if the Joomla Super User of the site makes a change to this vendor's joomla group through the CMS's Users / Manage (i.e. add the user to one more Joomla user group), then the ACL of the Vendor in HikaMarket becomes Inherit: No and nothing works for the vendor in the frontend. Screenshot bellow:

Can you please advise?
Thank you in advance.

Attachments:

Last edit: 3 years 11 months ago by liaskas.

Please Log in or Create an account to join the conversation.

Jerome
Away
HikaShop core developer

Posts: 26000
Thank you received: 4004
MODERATOR

3 years 11 months ago #318869

Hello,

I was not able to reproduce your issue in my tests websites but I might have an idea of the source for that issue.
In the HikaMarket vendor class (administrator/components/com_hikamarket/classes/vendor.php) please replace

$vendor_access = implode(',', $accesses) . ',' . $vendor_access;

$vendor_access = implode(',', $accesses) . ',' . trim($vendor_access, ',');

If the problem persists, we will need more details about a value in your database for that vendor, the content of the column "vendor_access".

Regards,

Jerome - Obsidev.com
HikaMarket & HikaSerial developer / HikaShop core dev team.

Also helping the HikaShop support team when having some time or couldn't sleep.
By the way, do not send me private message, use the "contact us" form instead.

Please Log in or Create an account to join the conversation.

liaskas
Topic Author
Offline

Posts: 205
Thank you received: 12

3 years 11 months ago #318880

Seems that the modification did the trick and solved the issue.
Will this modification be included in the next release or we will do it again after upgrade?

Thank you!

Please Log in or Create an account to join the conversation.