6
Vote

MultiTenancy: separating entries in Settings.txt (UrlHosts and Themes)

description

For the consistency of the syntax and better readability, I suggest the following changes in the structure of Settings.txt files:

RequestUrlHosts: currently the entries are separated with comma -> make it possible to parse it with or without space after the comma (currently the host is passed to generate a URL without calling Trim() on it and it breaks the display of the links of host names that are separated with comma+space, tiny fix)

Themes: currently entries are separated with semicolon -> should be generated with using comma+space after selecting themes in the MultiTenancy Admin UI (but make it possible to parse it the old way), and themes are not selected correctly if the entries are separated with semicolon+space

This would result in a better readability and easier syntax with backwards compatibility.

comments