Feature: Suppliers (extra fields / translations)

How to configure and adapt Aimeos based shops as developer
Forum rules
Always add your Aimeos and PHP version as well as your environment (Linux/Mac/Win)
Yvler
Posts: 33
Joined: 04 Jun 2015, 10:15

Re: Feature: Suppliers (extra fields / translations)

Post by Yvler » 24 Aug 2015, 15:10

I changed the supplier default.php manager, but still not working..
And I've committed this to core again

User avatar
aimeos
Administrator
Posts: 3421
Joined: 01 Jan 1970, 00:00

Re: Feature: Suppliers (extra fields / translations)

Post by aimeos » 24 Aug 2015, 15:13

Yvler wrote:I changed the supplier default.php manager, but still not working..
And I've committed this to core again
Travis reported an error: https://travis-ci.org/aimeos/aimeos-core/jobs/76996159
The $map variable isn't defined. You should have a look into the customer manager for reference.

Yvler
Posts: 33
Joined: 04 Jun 2015, 10:15

Re: Feature: Suppliers (extra fields / translations)

Post by Yvler » 25 Aug 2015, 09:11

After a lot of debugging, found the problem. I forgot to update the Supplier Item default interface with the listitem & refitem arrays :)
added these changes to the git

also updated the searchitems function in core again to fix the Travis error & use the builditems functions.

Weird enough, I thought I fixed these things last week.. maybe forgot to commit :)

User avatar
aimeos
Administrator
Posts: 3421
Joined: 01 Jan 1970, 00:00

Re: Feature: Suppliers (extra fields / translations)

Post by aimeos » 25 Aug 2015, 09:22

Yvler wrote:After a lot of debugging, found the problem. I forgot to update the Supplier Item default interface with the listitem & refitem arrays :)
added these changes to the git

also updated the searchitems function in core again to fix the Travis error & use the builditems functions.

Weird enough, I thought I fixed these things last week.. maybe forgot to commit :)
You forgot to commit at least the first part again ;-)
https://github.com/aimeos/aimeos-core/pull/29/files

Post Reply