Skip to content

Improvement of chapter "Výrazové prostředky" - updated version #1065

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 119 commits into
base: master
Choose a base branch
from

Conversation

mildabre
Copy link
Contributor

@mildabre mildabre commented Nov 4, 2024

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

@dg dg force-pushed the master branch 13 times, most recently from 57b2d7a to 08462dc Compare May 27, 2025 17:42
@dg dg force-pushed the master branch 14 times, most recently from 6a70853 to 98c72c7 Compare June 6, 2025 00:00
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.