Prepare for symfony 4 #SymfonyConHackday2017

[Jan0707]

Minor edits to enable installation with symfony 4

[ghost]

@Jan0707 Please also update the service definition to use the FQCN's. :grin:

[ireznik]

@Jan0707 any plans on fixing the mentioned issues in the near future?

[Jan0707]

Sorry folks, I was travelling. I renamed the service, but added a legacy alias, so that we do not break all the installations out there ;)

[ghost]

Please create a new release when this is fixed @Jan0707.

[Jan0707]

Not really sure what to do about the memory issue here ...

[adiq]

Hey, I have added some tweaks to this changes: https://github.com/Jan0707/ExcelBundle/pull/1 https://travis-ci.org/adiq/ExcelBundle/builds/324158521

This fixes:

• memory issue by injecting php ini file with memory_limit=-1
• some build issues by correcting symfony versions
• readme tweaks to use the new service name

Now, it all comes down to what versions of symfony/php would you want to support. From what I discovered symfony versions <3.2 have an issue when running tests due to var-dumper package not been loaded (see https://github.com/symfony/symfony/issues/20201 and https://github.com/symfony/symfony/pull/21491).

Maybe some compromise would be just to release v3 version as the users of older symfony versions would not benefit from this changes IMHO and it would be easier to maintain ;-)

[Jan0707]

Could we at least manage to support 3.4 as LTS ?

[adiq]

That's the question to the maintainer. Newest 3.X version is included in tests and is passing successfully

[Jan0707]

@adiq I just merged your pr into my pr.

[Tomsgu]

@liuggio can you please merge this and tag a new release? Or is there any issue with the pr?

[mrljaime]

Hello people. I taked a look into travis test and the test doesn't finish well becuase composer can't be dowloaded. I newbie at this time contribuying to some repository but I want to make use of the bundle over symfony's 4 version.

Can I do something to run travis test again to gain that someone aprove this PR?

Cheers. Jaime.

[adiq]

@mrljaime as PhpExcel lib in this bundle is deprecated and maintainer doesn't want to support this, it's recommended to use https://github.com/roromix/SpreadsheetBundle instead. It's using new, maintained lib, is compatible with newer versions of Symfony and is similar in use to this bundle ;-)

[mrljaime]

Thank's a lot @adiq

I will use it right now.

[ghost]

@liuggio Are you planning on checking out this PR or is it time that we use a different bundle?