You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is about src/m2ee/config.py, which didn't get it fair share of spring cleaning yet in the last years. The problem is not that there are many bugs. The problem is that the code is a mess and hard to work with and change, which is not nice for the developer, and makes it easier to introduce new bugs.
When removing Mendix 2.5 compatibility most of the hacks already got removed, now it the time to clean up the rest.
So:
Have a proper in-memory model of settings, not just a yolo style dictionary.
Add input validation so invalid settings are detected early without causing unexpected program behavior.
Make a clear distinction between just reading the config file and assembling derived config. Currently a lot of things are already done immediately, like setup_classpath (which should be done during start instead).
Move config handling that happens outside config.py into it (e.g. conversion of format of params in send_runtime_config in core.py)
Transform the get_blah functions into properties. Put the old functions into a corner as deprecated functions (warn when using them, remove in v8).
The text was updated successfully, but these errors were encountered:
This is about src/m2ee/config.py, which didn't get it fair share of spring cleaning yet in the last years. The problem is not that there are many bugs. The problem is that the code is a mess and hard to work with and change, which is not nice for the developer, and makes it easier to introduce new bugs.
When removing Mendix 2.5 compatibility most of the hacks already got removed, now it the time to clean up the rest.
So:
The text was updated successfully, but these errors were encountered: