-
-
Notifications
You must be signed in to change notification settings - Fork 604
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
Recipe with a single 'space' as the title returns "404 page not found" - reproduced on the demo site #3505
Comments
Able to delete from the data management admin account. Regular user can't delete. |
Solution here is to stop a recipe being able to be created when the title (removing whitespace) is null |
The route seems to call
It seems like the correct thing might be just to change that to:
Later code handles deduping if there are previous "New Recipe". Though the proposed changes don't handle leading/trailing spaces so it will have to be a little more complicated than that. |
We should have a "clean string" method somewhere that handles things like that, probably with the migration code. Although honestly we should probably handle that when creating the slug since slugs already clean strings anyway |
First Check
What is the issue you are experiencing?
Create a manual recipe on the demo site by creating a recipe titled with just a single space creates a 404 error and no way to delete or edit the issue
Steps to Reproduce
It creates a recipe without any option to delete from the GUI. Clicking on the edit option creates the 404 error
Note: If you do this twice, the second recipe created with automatically be named "(1)" and it does allow editing and deleting.
Best regards and thanks for a wonderful app.
Please provide relevant logs
No need. It happens on every version I tried. Can even be reproduced on your demo site. Would love to be able to correct my database that has one of these 'no-name can't delete' recipes
Mealie Version
No response
Deployment
Docker (Linux)
Additional Deployment Details
Running on current CasaOS
The text was updated successfully, but these errors were encountered: