Skip to content

Conversation

@mehdibo
Copy link
Contributor

@mehdibo mehdibo commented Dec 19, 2021

Improves #979

@mehdibo
Copy link
Contributor Author

mehdibo commented Dec 19, 2021

Oops, still need more fixing

@mehdibo mehdibo changed the title Changing getter PHPDoc return type on Collection WIP: Changing getter PHPDoc return type on Collection Dec 19, 2021
@mehdibo mehdibo changed the title WIP: Changing getter PHPDoc return type on Collection Changing getter PHPDoc return type on Collection Dec 19, 2021
@mehdibo
Copy link
Contributor Author

mehdibo commented Dec 19, 2021

Let me know if I caused the tests to fail, looks like they are not related to my changes

@allan-simon
Copy link

Is there something that can be done to help this pull request to move forward, this will help making the autogenerated code already compliant with phpstan checks , which would be a blessing :)

@jrushlow jrushlow changed the title Changing getter PHPDoc return type on Collection [make:entity] Chaing getter PHPDoc return type on Collection Feb 15, 2022
Copy link
Collaborator

@jrushlow jrushlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mehdibo for taking care of this! sorry for the delay.

@jrushlow jrushlow added the Status: Reviewed Has been reviewed by a maintainer label Feb 15, 2022
@jrushlow jrushlow mentioned this pull request Feb 15, 2022
@mehdibo mehdibo changed the title [make:entity] Chaing getter PHPDoc return type on Collection [make:entity] Change getter PHPDoc return type on Collection Feb 15, 2022
@weaverryan
Copy link
Member

Awesome! Thank you Mehdi!

@weaverryan weaverryan merged commit 93057dd into symfony:main Feb 16, 2022
weaverryan added a commit that referenced this pull request Feb 16, 2022
This PR was merged into the 1.0-dev branch.

Discussion
----------

[release] prep v1.37.0

# RELEASE

Hi Makers!

This release handles several bug fixes within MakerBundle and adds support for translating ResetPasswordBundle Exception messages, using constants instead of strings with `make:voter`, improved types for `Collection` getter methods in Entities, and support for attributes in `make:registration`!

Diff: v1.36.4...v1.37.0

Happy making!

---

# CHANGELOG

## [v1.37.0](https://github.com/symfony/maker-bundle/releases/tag/v1.37.0)

*February 15th, 2022*

### Feature

- [#1062](#1062) - [MakeRegistration] add support for verify email attributes - *`@jrushlow`*
- [#1059](#1059) - [make:reset-password] Translate exception reasons provided by ResetPasswordBundle - *`@bocharsky`-bw*
- [#1057](#1057) - [Voter] Refactor attributes - *`@mdoutreluingne`*
- [#1040](#1040) - [make:entity] Chaing getter PHPDoc return type on Collection - *`@mehdibo`*

### Bug Fix

- [#1060](#1060) - Add missing Passport use statement - *`@bocharsky`-bw*
- [#1032](#1032) - [reset-password] Coding standards - Twig - *`@seb`-jean*
- [#1031](#1031) - [verify-email] Coding standards - Twig - *`@seb`-jean*
- [#1027](#1027) - Fixing wrong messaging in make:auth about checking password in final steps - *`@weaverryan`*
- [#985](#985) - [make:auth] fix security controller attributes - *`@jrushlow`*

Commits
-------

52063a9 [release] prep v1.37.0
saylor-mik87786 added a commit to saylor-mik87786/maker-bundle that referenced this pull request Jun 3, 2025
This PR was merged into the 1.0-dev branch.

Discussion
----------

[release] prep v1.37.0

# RELEASE

Hi Makers!

This release handles several bug fixes within MakerBundle and adds support for translating ResetPasswordBundle Exception messages, using constants instead of strings with `make:voter`, improved types for `Collection` getter methods in Entities, and support for attributes in `make:registration`!

Diff: symfony/maker-bundle@v1.36.4...v1.37.0

Happy making!

---

# CHANGELOG

## [v1.37.0](https://github.com/symfony/maker-bundle/releases/tag/v1.37.0)

*February 15th, 2022*

### Feature

- [#1062](symfony/maker-bundle#1062) - [MakeRegistration] add support for verify email attributes - *`@jrushlow`*
- [#1059](symfony/maker-bundle#1059) - [make:reset-password] Translate exception reasons provided by ResetPasswordBundle - *`@bocharsky`-bw*
- [#1057](symfony/maker-bundle#1057) - [Voter] Refactor attributes - *`@mdoutreluingne`*
- [#1040](symfony/maker-bundle#1040) - [make:entity] Chaing getter PHPDoc return type on Collection - *`@mehdibo`*

### Bug Fix

- [#1060](symfony/maker-bundle#1060) - Add missing Passport use statement - *`@bocharsky`-bw*
- [#1032](symfony/maker-bundle#1032) - [reset-password] Coding standards - Twig - *`@seb`-jean*
- [#1031](symfony/maker-bundle#1031) - [verify-email] Coding standards - Twig - *`@seb`-jean*
- [#1027](symfony/maker-bundle#1027) - Fixing wrong messaging in make:auth about checking password in final steps - *`@weaverryan`*
- [#985](symfony/maker-bundle#985) - [make:auth] fix security controller attributes - *`@jrushlow`*

Commits
-------

52063a93 [release] prep v1.37.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Reviewed Has been reviewed by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants