Skip to content

Support for properties in NEON statement syntax in section services - PART 3 #339

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 19 commits into
base: master
Choose a base branch
from

Conversation

mildabre
Copy link
Contributor

@mildabre mildabre commented Nov 24, 2024

This PR is a part of complex correction of NEON statement syntax with added support for properties.

part 1: nette/di#318
part 2: nette/neon#73

All three PR should be merged together !

The complex solution + tests are published of forum here:

https://forum.nette.org/cs/36724-chyby-v-konfiguraci-syntaxe-property-konstanty-v-sekci-services-aktualizace#p228053

dg and others added 19 commits September 10, 2024 12:09
- inteface Nette\Application\IRouter replaced by Nette\Routing\Router
…e() for persistent parameters. [Closes nette/nette#703][Closes nette/nette#703][Closes nette#69]" (possible BC break)

This reverts commit cda17f4.

See https://forum.nette.org/cs/35528-stejne-pojmenovany-parametr-akce-presenteru-a-persistentni-odlisne-chovani-v-nette-2-0-oproti-aktualnimu#p221742

BC break: Property must be nullable, ie: #[Persistent] public ?int $foo
…rs dynamically (BC break)

TODO: scanDir must be active
# 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.

3 participants