Skip to content
This repository has been archived by the owner on Dec 5, 2017. It is now read-only.

[WIP] Liip create bundle #20

Closed
wants to merge 2 commits into from
Closed

[WIP] Liip create bundle #20

wants to merge 2 commits into from

Conversation

dbu
Copy link
Member

@dbu dbu commented Aug 6, 2012

switch to using the createphp library instead of hardcoding rdf into the template and in the model classes. i want to rename this bundle to LiipCreateBundle once we have this functionality.

probably we should first merge #19 and then maybe fix this PR.

$type = $this->typeFactory->getType(get_class($model));
$result = $this->restHandler->run($request->request->all(), $type, null, RestService::HTTP_PUT);

// TODO: what about the filter, if its there?
Copy link
Member Author

Choose a reason for hiding this comment

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

i think this is the most important problem atm. we probably either need to push filtering and validating into the rest controller of createphp or overwrite that controller. ideas and PR welcome.

Copy link
Contributor

Choose a reason for hiding this comment

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

i think we should leave out the filter stuff for now ..

Copy link

Choose a reason for hiding this comment

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

filtering could be configured with the normal createphp type setup and then passed along as config options to the rdfMapper, so transporting this information via createphp wouldn't require any serious code changes. For validation, see my other comment. This needs some sort of support from create.js, since it should normally be handled by the client side, with server validation as fallback only (and of course ideally powered by the same configuration)

@lsmith77
Copy link
Contributor

the new Bundle should be SymfonyCmfCreateBundle

@lsmith77
Copy link
Contributor

@lsmith77
Copy link
Contributor

deprecated in favor of SymfonyCmfCreateBundle

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

Successfully merging this pull request may close these issues.

4 participants