Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

EZP-31097: Introduced strict types for URLService #2854

Merged
merged 1 commit into from
Nov 18, 2019

Conversation

adamwojs
Copy link
Member

@adamwojs adamwojs commented Nov 1, 2019

Question Answer
JIRA issue EZP-31097
Bug/Improvement yes
New feature no
Target version master
BC breaks yes
Tests pass yes
Doc needed yes

This PR introduces strict types for the \eZ\Publish\API\Repository\URLService

TODO:

  • Implement feature / fix a bug.
  • Implement tests.
  • Fix new code according to Coding Standards ($ composer fix-cs).
  • Ask for Code Review.

@adamwojs adamwojs self-assigned this Nov 1, 2019
@adamwojs adamwojs force-pushed the url_service_typehint branch 2 times, most recently from cb9dd12 to 37c0f50 Compare November 2, 2019 09:42
eZ/Publish/Core/Repository/URLService.php Outdated Show resolved Hide resolved
eZ/Publish/Core/Repository/URLService.php Outdated Show resolved Hide resolved
@adamwojs adamwojs force-pushed the url_service_typehint branch from e2e786c to 6248854 Compare November 7, 2019 12:04
@adamwojs
Copy link
Member Author

adamwojs commented Nov 7, 2019

PR rebased in order to fix code style issues

eZ/Publish/API/Repository/URLService.php Show resolved Hide resolved
eZ/Publish/API/Repository/URLService.php Outdated Show resolved Hide resolved
@adamwojs
Copy link
Member Author

PR updated according to @mikadamczyk code review suggestions.

@katarzynazawada katarzynazawada self-assigned this Nov 15, 2019
@adamwojs adamwojs force-pushed the url_service_typehint branch from 3ec650a to b38b14e Compare November 15, 2019 13:24
@adamwojs adamwojs force-pushed the url_service_typehint branch from b38b14e to 75ce31e Compare November 18, 2019 10:48
@lserwatka lserwatka merged commit 39f03a1 into master Nov 18, 2019
@lserwatka lserwatka deleted the url_service_typehint branch November 18, 2019 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

6 participants