Skip to content

Fix argument type passed to generateMenuUrl()#6011

Merged
greg0ire merged 1 commit intosonata-project:3.xfrom
phansys:absolute_url
Apr 1, 2020
Merged

Fix argument type passed to generateMenuUrl()#6011
greg0ire merged 1 commit intosonata-project:3.xfrom
phansys:absolute_url

Conversation

@phansys
Copy link
Member

@phansys phansys commented Mar 31, 2020

Subject

Fix type of argument 3 passed to UrlGeneratorInterface::generateMenuUrl().

I am targeting this branch, because this change respects BC.

Changelog

### Fixed
- Type of argument 3 passed to `UrlGeneratorInterface::generateMenuUrl()`.

To do

  • Add tests.

@phansys phansys requested a review from a team March 31, 2020 04:34
@phansys phansys added patch and removed pedantic labels Mar 31, 2020
core23
core23 previously approved these changes Mar 31, 2020
@phansys phansys marked this pull request as ready for review March 31, 2020 07:23
@phansys phansys requested a review from core23 March 31, 2020 07:23
core23
core23 previously approved these changes Mar 31, 2020
@phansys phansys changed the title Fix argument type passed to generateMenuUrl() [WIP] Fix argument type passed to generateMenuUrl() Mar 31, 2020
@phansys phansys force-pushed the absolute_url branch 3 times, most recently from 9e6145c to cc1468a Compare March 31, 2020 15:52
@phansys phansys changed the title [WIP] Fix argument type passed to generateMenuUrl() Fix argument type passed to generateMenuUrl() Mar 31, 2020
@phansys phansys requested a review from core23 March 31, 2020 15:55
@phansys
Copy link
Member Author

phansys commented Mar 31, 2020

I think it's finally ready to merge. DefaultRouteGenerator was modified too since there were more inconsistencies.

@phansys phansys requested a review from a team March 31, 2020 17:43
@greg0ire greg0ire merged commit 05b7556 into sonata-project:3.x Apr 1, 2020
@greg0ire
Copy link
Contributor

greg0ire commented Apr 1, 2020

Thanks @phansys !

@phansys phansys deleted the absolute_url branch April 1, 2020 12:04
@phansys phansys mentioned this pull request Apr 1, 2020
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants