- π§πͺBelgium Wim Leers Ghent π§πͺπͺπΊ
Is this really a problem? π€ Tempted to close this π
The member variables of ConfigEntity's map 1-to-1 to the configuration keys in the corresponding config YAML files. Coding standards dictate lowerCamelCase for member variables in OOP code and underscore_separated for config keys. We need to decide what we want here. Most ConfigEntity's are not bitten by this, because they only contain one-word keys, in which case lowerCamelCase and underscore_separated fall together. :-)
We already have an inconsistency in core with the View entity (View::base_table) and the Breakpoint and BreakpointGroup entities.
-
Decide on one of the following:
underscore_separated
.lowerCamelCase
.lowerCamelCase
for all configuration keys.lowerCamelCase
for non-object-oriented code. (This in a way applies to 2. as well.)underscore_separated
configuration keys to lowerCamelCase
class members.-
-
Active
11.0 π₯
It involves compliance with, or the content of coding standards. Requires broad community agreement.
Enhances developer experience.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Is this really a problem? π€ Tempted to close this π