Actions
Bug #20133
closedTechnique editor allows id with only case difference, do an error, but still save technique
Status:
Released
Priority:
N/A
Assignee:
Category:
Web - Technique editor
Target version:
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:
Description
So, I have a technique starting by an upper case. It's id also has an upper case (should not ?).
If I clone it, change first letter to lower case, try to save, I have an error which is totally not actionnable and should be change (something like "an error occured server side, try latter").
Then, in logs I get:
[2021-10-15 10:06:49+0000] ERROR com.normation.rudder.rest.RestUtils - "Could not create ncf technique <- Technique name and ID must be unique. 'technique with blocks' already used"
As I'm still on the technique, I try to delete it.
There, I get a screen with 0 techniques remaining (I had only the other one, so perhaps it removed both based on the id, case insensitive?).
A bit of stress, then I click back on "techniques" in the main menu, and I get not only two techniques... But also two drafts ???
And yes, in the file system, contrary to what rudder and log told, there is a bunch of technique:
- id must be case insensitive
- error must be actionnable. What fail? How can I fix it?
- if log say it was an error, there should not be files in the fs
- draft must be deleted once technique is saved without error (what happened for the first one). I don't want draft for saved techniques.
Files
Actions