Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Go into SF 4 #403

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

[WIP] Go into SF 4 #403

wants to merge 24 commits into from

Conversation

ElectricMaxxx
Copy link
Member

@ElectricMaxxx ElectricMaxxx commented Feb 23, 2018

With this PR i'd like to use SF4 only together with dev-master of all our bundles and component. I also changed the directory structure.

Todo:

  • use SF 4 Dependencies
  • re-add or evaluate dependencies, that i could load together with SF 4
    • symfony/distribution-bundle
    • Parameters thingy bundle
    • Eko???
    • SonataTranslation (currently has a problem to install with phpcr-bundle
  • remove forked repositories by getting their PRs merged
  • Cleanup readme file to match the new setup
  • Introduce CkEditor by FOS

Copy link
Member

@dbu dbu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks a lot for getting this started!

@@ -0,0 +1,10 @@
# This file is a "template" of which env vars need to be defined for your application
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this file be committed? i think only the .env.dist should be, and .env should be in .gitignore

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are completely right

@ElectricMaxxx
Copy link
Member Author

@dbu i think i need your help now. For some reasons the switcher from lunetics local seems to ask the wrong router and commes with an exeption:

[2018-03-08 06:13:51] request.CRITICAL: Uncaught PHP Exception Twig_Error_Runtime: "An exception has been thrown during the rendering of a template ("Warning: Illegal offset type in isset or empty")." at /home/maximilian/OpenSource/Cmf/cmf-sandbox/templates/base.html.twig line 58 {"exception":"[object] (Twig_Error_Runtime(code: 0): An exception has been thrown during the rendering of a template (\"Warning: Illegal offset type in isset or empty\"). at /home/maximilian/OpenSource/Cmf/cmf-sandbox/templates/base.html.twig:58, ErrorException(code: 0): Warning: Illegal offset type in isset or empty at /home/maximilian/OpenSource/Cmf/cmf-sandbox/var/cache/dev/srcDevDebugProjectContainerUrlGenerator.php:148)"} []

really hidden in dev.log. The console says:

127.0.0.1:60380 [500]: /en - Uncaught Symfony\Component\Debug\Exception\FatalThrowableError: Type error: Too few arguments to function FOS\RestBundle\Controller\ExceptionController::__construct(), 0 passed in /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/symfony/http-kernel/Controller/ControllerResolver.php on line 111 and exactly 3 expected in /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/friendsofsymfony/rest-bundle/Controller/ExceptionController.php:43
Stack trace:
#0 /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/symfony/http-kernel/Controller/ControllerResolver.php(111): FOS\RestBundle\Controller\ExceptionController->__construct()
#1 /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/symfony/http-kernel/Controller/ContainerControllerResolver.php(100): Symfony\Component\HttpKernel\Controller\ControllerResolver->instantiateController('FOS\\RestBundle\\...')
#2 /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/symfony/framework-bundle/Controller/ControllerResolver.php(57): Symfony\Component\HttpKernel\Controller\Containe in /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/friendsofsymfony/rest-bundle/Controller/ExceptionController.php on line 43

but this is an other issue. But debugging from /home/maximilian/OpenSource/Cmf/cmf-sandbox/vendor/lunetics/locale-bundle/Lunetics/LocaleBundle/Twig/Extension/LocaleSwitcherExtension.php:73 into the geration of the routes, i saw that the exception comes from the static router wich tries to use our documents as key in an array. You know what is going on here?

@ElectricMaxxx
Copy link
Member Author

@ElectricMaxxx
Copy link
Member Author

Yes, got the first working sandbox now. had to do some changes in vendor/ folder (temporary and PRs are open now on third party), but it works locally now.

@dbu
Copy link
Member

dbu commented Mar 13, 2018

great! sorry i am not being more responsive.

what do you think about the state of the phpcr bundle? should i do an alpha release or are there any problems still open?

@ElectricMaxxx
Copy link
Member Author

@dbu from my POV these packages are ready:

  • i was able to load them throug packagist for this branch
  • i was able to load fixtures even for documents in Document folder of an non bundle
  • i was able to run the sandbox (maybe we should have a look in the current tests)

@ElectricMaxxx
Copy link
Member Author

@dbu i think i can get almost all test issues here, but can you give me a hand with the local switcher thing? I commented out the complete switcher in template as they seem use the wrong router, a router which is not able to use an cmf document. To reproduce simply de-comment the line and see the error

@dbu
Copy link
Member

dbu commented Aug 28, 2018

do you get 2.6.2 or the latest master? you fixed something in the bundle about the router: lunetics-org/LocaleBundle@2.6.2...master (or maybe broke, i don't know ;-) )

good idea if you squash the commits.

@ElectricMaxxx
Copy link
Member Author

Ahh, you are right .. i did something there. It took quite while

@dbu
Copy link
Member

dbu commented Aug 28, 2018

can you please rebase on develop? i cleaned up the file header in styleci. and then apply the styleci fixes that remain on this branch?

@ElectricMaxxx
Copy link
Member Author

you where right, the master of lunetics/locale-bundle works fine

@ElectricMaxxx
Copy link
Member Author

@dbu you did the style changes on master? a rebase with so many file movement will be awesome now ...

@ElectricMaxxx
Copy link
Member Author

oh, was easier then i thought.

@ElectricMaxxx
Copy link
Member Author

@lunetics can you create a new release with the router change on master pls?

@dbu
Copy link
Member

dbu commented Sep 13, 2018

i now tagged 2.6.3 of the luneticsbundle with the latest changes.

@ElectricMaxxx
Copy link
Member Author

@dbu i think i am very close to done here. i simply fixed everything to "cmf 2.1" (what ever that is g) and SF 4.1. Doing so we should be able to release sandbox in 2.1 now for the released cmf versions. And i would do same for 2.2 then, that with SF 4.2 and CMF 2.2 (the things i currently prepare).
But i still have a problem to find the correct `liip/functional-test-bundl" version, can you give a hand here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants