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

Required fields aren't checked when publishing a node #1581

Closed
svenluijten opened this issue Aug 24, 2017 · 2 comments
Closed

Required fields aren't checked when publishing a node #1581

svenluijten opened this issue Aug 24, 2017 · 2 comments

Comments

@svenluijten
Copy link
Contributor

svenluijten commented Aug 24, 2017

When adding a subpage to a node, it creates an unpublished node entity which the user is then redirected to to edit. When I then click on "Publish", the page is published without further questions, while I expected it to check if the required fields are met (@Assert\NotBlank()'s on the entity).

A good solution for this would be to make the "publish" action also "save" the entity, so that the validation fires and the user gets an error message if they haven't filled in all required fields.

@Numkil
Copy link
Contributor

Numkil commented Nov 13, 2017

@svenluijten This is indeed a problem. I have added it to our roadmap to be fixed. I will keep you up to date with progress when we have some time to start working on it.

@Devolicious
Copy link
Contributor

Not really a roadmap kind of issue, it's more bugfix imho.

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

4 participants