Improvement of chapter "Výrazové prostředky" - updated version#1065
Improvement of chapter "Výrazové prostředky" - updated version#1065mildabre wants to merge 150 commits into
Conversation
e0bc7f4 to
3c92c82
Compare
|
Thanks — there's a genuinely valuable, non-obvious point buried in here, and I've documented it (622dbe5ce, both languages). Incorporated: the property-vs-constant distinction. I verified it in the DI source ( Not taken, and why:
Closing since the branch can't be merged directly. Really appreciate you flagging the confusing property/constant syntax. |
The term 'výrazové prostředky' is correct, but common developer understands the word "syntaktické prvky" better. I also suggest reordering the items + complete missing description about public properties and explicite note the slight syntax difference between constant and property.
I was very confused by that fact, that approx. 3 years ago the syntax for public properties @myservice::publicProperty was not supported and the only supported syntax at that time was funny: @myservice::$publicProperty() so I have it in my services.neon. Another source of confusion is neon plugin of @mesour which suggests public properties with $ on the beginning.
When I tested yesterday the syntax @myservice::publicProperty I made first letter uppercase @myservice::PublicProperty and got exception that constant not exist. I see that the difference in syntax between constant and property is very slight - first letter uppercase/lowercase. I think its goodidea, but I see important point out this slight difference in documentation as well.
The syntax of public properties is also discussed here on forum: https://forum.nette.org/cs/36714-rfc-upravit-matouci-syntaxi-public-property-sluzby-v-services-neon