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

Symfony integration #1714

Closed
5 of 9 tasks
WouterSioen opened this issue Aug 31, 2016 · 6 comments
Closed
5 of 9 tasks

Symfony integration #1714

WouterSioen opened this issue Aug 31, 2016 · 6 comments

Comments

@WouterSioen
Copy link
Member

WouterSioen commented Aug 31, 2016

Replace SpoonCookie/SpoonSession with Request/Response object #399

We need to pass a response object to every action/widget/... that can manipulate the sessions and cookies.

Refactor Fork hooks using EventDispatcher #405

step 1: delete all triggerEvents + deprecate the triggerEvent method
step 2: add events for everything

Refactor locale module using Translation #408

Refactor Fork and Pages with Routing component #409

We already use the router for our applications. We can probably use the ChainedRouter to do the further routing in the frontend based on our Pages module.

Replace SpoonThumbnail/SpoonFile #708

There's probably a package for this.

Refactor out all exit statements #1105

Replace them with good clear exceptions. This will also help us test more stuff.

Done in #1797

Refactor out constants #1106

Container parameters are the way to go

Remove API #1142

The API needs to be deprecated and altered by an easy way to add new API endpoints. This will also fix the usage of SpoonTemplate.

We can use a combination of bundles to achieve these results (https://github.com/nelmio/NelmioApiDocBundle, https://github.com/FriendsOfSymfony/FOSRestBundle, ...)

Refactor out SpoonFilter::getGetValue and SpoonFilter::getPostValue #1347

We should use the request object for this.

carakas added a commit to justcarakas/forkcms that referenced this issue Oct 28, 2016
part of forkcms#1714
you can use a combination of bundles to replace it if you want to (https://github.com/nelmio/NelmioApiDocBundle, https://github.com/FriendsOfSymfony/FOSRestBundle, ...)
carakas added a commit to justcarakas/forkcms that referenced this issue Nov 14, 2016
part of forkcms#1714
you can use a combination of bundles to replace it if you want to (https://github.com/nelmio/NelmioApiDocBundle, https://github.com/FriendsOfSymfony/FOSRestBundle, ...)
carakas added a commit to justcarakas/forkcms that referenced this issue Nov 14, 2016
part of forkcms#1714
you can use a combination of bundles to replace it if you want to (https://github.com/nelmio/NelmioApiDocBundle, https://github.com/FriendsOfSymfony/FOSRestBundle, ...)
@carakas carakas added this to the 5.0.0 milestone Jan 13, 2017
@myspulin
Copy link

Hi, I do some development on the last version API and I wonder why is class BaseApi deprecated. In the comments there was mentioned bundle NelmioApiDocBundle or FOSRestBundle is it already decided which one will be used? Is there any estimation when will be implemented? Thanks for info.

@jessedobbelaere
Copy link
Member

@myspulin, currently there's no plan to integrate FOSRestBundle already. (NelmioApiDocBundle is a bundle to generate documentation pages for your api based on Swagger).

The focus right now is more on delivering Fork CMS 5 and then working up to version 6, trying to move out the legacy code (Spoon Library etc) and go full-stack Symfony with modules as bundles etc, migrating modules to Doctrine, ...

It shouldn't be very hard though to add FOSRestBundle as a lot of Fork CMS is Symfony based already. Recently someone on our Slack chat was implementing FOSRestBundle and trying it out, you can maybe ask on Slack how he's progressing and if it's easy to do.

@myspulin
Copy link

@jessedobbelaere, thanks for update, it make sense to remove legacy code first I cant wait for new version. Regarding FOSRestbundle I will try the Slack and ask the others. Thank you!

@carakas
Copy link
Member

carakas commented Feb 28, 2017

@myspulin you can find a demo implementation of the FOSRestBundle here master...justcarakas:rest-api-challange

@jeroendesloovere
Copy link
Member

Interesting article about using "keys" for "translations".
http://obtao.com/blog/2013/06/how-to-organize-your-translations-in-symfony/

@carakas carakas modified the milestones: 6.0.0, 5.0.0 Aug 1, 2017
@carakas carakas removed this from the 6.0.0 milestone Mar 17, 2021
@carakas
Copy link
Member

carakas commented Mar 17, 2021

No longer needed to keep a separate issue for this as we do it when and where we can

@carakas carakas closed this as completed Mar 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants