|
wojciech.kapcia@tigase.net opened 7 years ago
|
|
I'm generally against allowing usage of values with different letter cases, but in this case, I agree that we should allow this - at least for compatibility reasons. However, I would still want to clean up documentation and use proper values without a need for case-insensitive lookup of enum values. |
|
|
Andrzej Wójcik wrote:
Totally agree
Daniel - do you think you will be able to update documentation (case of some options)? |
|
After a thorough check of all enums in code from a grep search, I have not found any more instances of enums that are not in line with documentation and code. |
|
|
Thank you. |
| Type |
Bug
|
| Priority |
Normal
|
| Assignee | |
| RedmineID |
6842
|
| Version |
tigase-server-8.0.0
|
| Spent time |
38h 45m
|
Not sure if this is bug per se but currently
DefaultTypesConverteris case-sensitive with enums, which obviously causes issues when configuration is not strict with it (or was converted from previous version) which may cause errors during startup.The proposal is to make it case-insensitive (as the enums cases are currently all over the place) as alternatively adjusting documentation (eg.: http://docs.tigase.org/tigase-server/snapshot/Administration_Guide/html_chunk/watchdog.html) and converter would be… tedious without much benefit.
%kobit %andrzej.wojcik ( %bmalkow ?)