LiipMagentoBundle icon indicating copy to clipboard operation
LiipMagentoBundle copied to clipboard

Bundle is incompatible with symfony 2.1.x

Open sybarite opened this issue 12 years ago • 7 comments

Are you guys going to update this project, if not I have already made changes for it to be compatible with symfony 2.1.x. Should I submit the patches?

sybarite avatar Jan 09 '13 13:01 sybarite

Liip currently does not have any active projects using the Bundle, but if there are issues we are more than happy to merge fixes. Ideally the Bundle should remain compatible with all 2.x versions in the master branch.

lsmith77 avatar Jan 09 '13 13:01 lsmith77

can you submit a PR to fix any 2.1/2.2 issues?

lsmith77 avatar Mar 21 '13 13:03 lsmith77

Will this bundle be upgraded to latest version of SF2? I've looked into bundle and see that security factory definition in security.yml is not compatible with latest version. @sybarite Did you fix this bundle already? If so would you be so kind and submit patches?

tworzenieweb avatar Mar 25 '13 12:03 tworzenieweb

@lsmith77 @tworzenieweb Hey sorry for the late reply. Yes I have fixed the issues. But its been a long time since I worked on my symfony + magento project. I have in fact completely rewritten the library to customize for my project requirements. Give me some a day or two to get the patches in sync with this project.

sybarite avatar Mar 25 '13 14:03 sybarite

No problem at all. I've decided to rely on magento soap / rest api. It's not a perfect solution in some cases but it's easier for me since i don't know magento core.

Also i needed to make integration with Doctrine Entites so i have only magento_id inside local database product and rest is loaded using post load method of entity. That's why i made some magento client class for working with soap and other class for overloading CRUD operations for entity.

tworzenieweb avatar Mar 25 '13 14:03 tworzenieweb

It would be nice to have an updated version of that bundle.

WaZz2 avatar Sep 05 '14 14:09 WaZz2

as we are not using the Bundle, we kind of are waiting for a PR

lsmith77 avatar Sep 05 '14 14:09 lsmith77