Upd: Adding Symfony 4 Compatibility

[ilohpro]

#SymfonyConHackday2017

[ilohpro]

Unit test has to be updated to manage the new version of Propel.
That's why Travis fails

[gilles-g]

@macroprog
Did you try to not change propel dependencies?
Tests are green on master branch, this is important :)

[ilohpro]

@Spike31 Yes, i start my tests by keeping the propel version that is in the master branch, but i get a lot of other errors as I have do be on PHP 7, and propel 1 is not compatible with it.
I will try to go more in details with it as soon as i can

[Bukashk0zzz]

Any news?

[ilohpro]

Nope, still stuck with the Propel problem, can you help please ?

[Bukashk0zzz]

If have no ideas I will try on weekend.

[eugenekurasov]

Hi,
You change propel/propel-bundle from ~1.5 to ~3.0@dev.
propel/propel-bundle ~1.5 using https:/propelorm/Propel
propel/propel-bundle ~3.0@dev using https:/propelorm/Propel2
Classes from https:/propelorm/Propel also used in other places (For example).

[ilohpro]

I understand that, but the bigest problem i have for now is to replace PropelQuickBuilder::buildSchema.
I can't find a solution to replace it. Do you have an idea ?

[eugenekurasov]

I will try upgrade project to Propel2 at this weekend.

[ilohpro]

Hi, Could you upgrade the project to Propel2 as expected ? Thanks

[eugenekurasov]

Hi, I was at vacation, I will create PR at this week.

[eugenekurasov]

Hi,
I am done PR for migrate to Propel 2. I waiting merge.

[Bukashk0zzz]

@macroprog can you marge sf4 support with @eugenekurasov PR. Than we can set in composer your brunch and update our project to SF4

[gilles-g]

@eugenekurasov PR merged 👍

[eugenekurasov]

@macroprog can you continue your PR? Problems with propel is should be solved

[ilohpro]

I'll do the merge on next week, i can't before

[amitlimbadia]

gr8 so when we will have sf4 LexikTranslationBundle support?

[ilohpro]

Propel bundle has been released for SF3.* but not yet for SF4 (no branch dedicated to this version).
Travis still brake down because of it on the SF4 test and the Mongo Extension.
I'm still trying to find a solution. I'll follow in the coming week.

[amitlimbadia]

Any updates regarding symfony 4 compatibility?

[eugenekurasov]

@macroprog PR (propelorm/PropelBundle#483) - has been merge. Can you continue this PR?

[ilohpro]

As i said in my last post, the propel bundle has been released for Symfony 3.4, but not Symfony 4 (problem with Mongo exntesion => https://travis-ci.org/lexik/LexikTranslationBundle/jobs/334241571 ).
Can you manage it on the propel bundle repo to merge it here ?

[eugenekurasov]

Propel-bundle 3.0 will not support SF4. Propel-bundle already have branch 4.0 with support SF4.
I asked to create new tag in issue (propelorm/PropelBundle#484)

[eugenekurasov]

@macroprog
Has been created new PR with SF4 #305

[ilohpro]

Okay, thank you for this information. I close this one as the other one is in green. Thank you for your help.

[gilles-g]

Thx @macroprog for your time and contribution