Skip to content

[10.x] Fixes GeneratorCommand not able to prevent uppercase reserved name such as __CLASS__ - #48667

Merged
taylorotwell merged 1 commit into
10.xfrom
generator-patch-reserve-name
Oct 9, 2023
Merged

taylorotwell merged 1 commit into
10.xfrom
generator-patch-reserve-name

Conversation

@crynobone

Copy link
Copy Markdown
Member

No description provided.

name such as  `__CLASS__`

Signed-off-by: Mior Muhammad Zaki <crynobone@gmail.com>
@crynobone
crynobone marked this pull request as ready for review October 8, 2023 12:11
@taylorotwell
taylorotwell merged commit 4742c90 into 10.x Oct 9, 2023
@taylorotwell
taylorotwell deleted the generator-patch-reserve-name branch October 9, 2023 14:21
timacdonald pushed a commit to timacdonald/framework that referenced this pull request Oct 24, 2023
laravel#48667)

name such as  `__CLASS__`

Signed-off-by: Mior Muhammad Zaki <crynobone@gmail.com>
binaryfire added a commit to hypervel/components-backup that referenced this pull request Sep 10, 2026
Restore current Laravel tests for terminal PHP extension removal, view-name directory conversion, embedded extension preservation and case-insensitive reserved class names. Keep the separate Hypervel unit tests for generator behavior.

Copy from Laravel 13.x at 01d008c9b5f32cb7c5e50a9a22273113d810b2a2 and adapt namespaces, Testbench, strict/native types and provider documentation. Preserve every upstream case and assertion. Additionally assert that reserved names create no class file, registering only that exact path for existing published-file cleanup.

Relevant upstream history: laravel/framework#48667, laravel/framework#51842, laravel/framework#51847 and laravel/framework#51924.

Verified the new test, combined generator and queue-route coverage, console integration tests and formatting. Generated files use the disposable Testbench application and existing cleanup lifecycle.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants