-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Remove template selection for simplified interface profiles #21909
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
base: 11.0/bugfixes
Are you sure you want to change the base?
Remove template selection for simplified interface profiles #21909
Conversation
orthagh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok for that, it'll be much clearer
|
Honestly, I am still confused about the place templates have now in GLPI 11. Even if it was from a PR I did (3 and a half years ago at the start of GLPI 11 dev for a client-sponsored development), the fact that templates get applied when a ticket is created and stay associated with that template makes for some odd behaviors and I don't know what should be considered intended or not. |
AdrienClairembault
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The form feature call CommonITILObject::getITILTemplateToUse().
If you can double check to make sure this method do not rely on the setting you deleted then the change is OK for me.
(But if you deleted it from the UI, maybe there is also something to clean up in the database?)
How does it use that information? Templates are/were just a UI feature and it seems like Forms completely replace their functionality in this context.
I'd rather not delete data from the DB at all unless it is invalid and would cause bugs. When other fields that exist only for standard interface profiles are set, they are not deleted when switching it to simplified interface. |


Checklist before requesting a review
Description
Fixes #21378
Since Forms have completely replaced direct tickets for simplified interface users, there seems to be no need to have the default template option in the profile form.