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

Prevent Exception when sonata.page.page.class dont exists #432

Closed
wants to merge 1 commit into from

Conversation

maikelohcfg
Copy link

when i try to easy-extends it throw a reflection error Application\Sonata\PageBundle\Entity\Post don't exist and it denied me to create the extended bundle so it loop in continue error, with this it fix that.

@rande
Copy link
Member

rande commented Dec 11, 2014

Are you sure about the class name: Application\Sonata\PageBundle\Entity\Post ?

There is no such class in the PageBundle.

@maikelohcfg
Copy link
Author

Sorry my mistake this is the error fixed

[ReflectionException]

Class Application\Sonata\PageBundle\Entity\Page does not exist

2014-12-11 5:08 GMT-04:30 Thomas [email protected]:

Are you sure about the class name:
Application\Sonata\PageBundle\Entity\Post ?

There is no such class in the PageBundle.


Reply to this email directly or view it on GitHub
#432 (comment)
.

Lic. Maikel Ortega Hernández

@core23
Copy link
Member

core23 commented Feb 11, 2016

Please rebase and run make cs. @maikelohcfg

@SonataCI
Copy link
Collaborator

Could you please rebase your PR and fix merge conflicts?

@soullivaneuh
Copy link
Member

According to the new Sonata version management and next major release plan, this project has been refactored regarding branching and versioning.

If you see this message, your PR concerns a patch or a minor release and is not targeting the right branch.

So I'm closing this one, but don't see it as a refusal. If you think your work is still relevant and want to continue, feel free to reopen it on the right branch (e.g. the default one).

Regards.

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

Successfully merging this pull request may close these issues.

5 participants