LiipTranslationBundle icon indicating copy to clipboard operation
LiipTranslationBundle copied to clipboard

Support for doctrine storage

Open yellow1912 opened this issue 10 years ago • 15 comments

Hello,

Given that Doctrine is the primary lib used in SF2, is there any plan to implement support for it soon?

yellow1912 avatar Feb 09 '15 14:02 yellow1912

afaik nobody at liip uses doctrine to store translation at the moment so we don't have plans to do that. if you want to contribute an implementation however, please do.

dbu avatar Feb 09 '15 14:02 dbu

This should be pretty easy, you just have to implement the following interface: https://github.com/liip/LiipTranslationBundle/blob/master/Persistence/PersistenceInterface.php

jeanmonod avatar Feb 09 '15 15:02 jeanmonod

I will try that tomorrow and see how it goes :). I may not have time to add unit tests for it yet tho.

yellow1912 avatar Feb 09 '15 15:02 yellow1912

Cool, just be aware of this before starting: PersistenceInterface.php#L11

jeanmonod avatar Feb 10 '15 08:02 jeanmonod

@lsmith77 @thormeier As you are doing maintenance on that bundle, I think this is the most important story to implement if we want to give visibility on the bundle...

jeanmonod avatar Jun 17 '15 05:06 jeanmonod

Has anybody succeeded with adding Doctrine to the bundle?

maZahaca avatar Sep 03 '15 13:09 maZahaca

While I no longer use this bundle, we have a private bundle that is inspired with liip theme's code and we are supporting doctrine based twig templates as well as css/javascripts.

I will do a write up on this topic hopefully in the next week to help others on this part.

yellow1912 avatar Sep 03 '15 14:09 yellow1912

@yellow1912 ok, maybe you can share your bundle to public?

maZahaca avatar Sep 03 '15 14:09 maZahaca

@maZahaca not a big problem, only that we don't have enough tests for it yet so we do not want to publish something untested until we are more ready

yellow1912 avatar Sep 03 '15 15:09 yellow1912

@yellow1912 I guess this is normal to have unstable branches on the way to tags and stable releases. Anyway, when you will ready let me know :boar: For now I'll try with this bundle.

maZahaca avatar Sep 03 '15 17:09 maZahaca

Hello @yellow1912 , how works your private bundle and can you share it with us? Thanks Marcel

marcelglaeser avatar Jun 28 '16 11:06 marcelglaeser

The crowd wants to be shared an amazing code!

maZahaca avatar Jun 28 '16 11:06 maZahaca

And an own fork for us? :)

marcelglaeser avatar Jun 28 '16 11:06 marcelglaeser

fork to everyone :) just to improve this code a bit more

maZahaca avatar Jun 28 '16 11:06 maZahaca

I'm sorry, I didnt notice all these notification just now. I have ported Liip theme to use Doctrine Storage, however I haven't done so for Translation. I will have to do so very soon though, since our project will need that within a month. I will be sure to open source it since I may not have time for unit testing and will welcome all unit tests. I'm thinking if I need to use LiipTranslation however. Our project is pretty custom and we may not need something as complicated as this bundle. In anycase I will try to make it as re-useable as possible while not trying to impose lots of constraints and assumptions on it.

yellow1912 avatar Jun 28 '16 13:06 yellow1912