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
[Resource] Use typehints introduced by PHP7+ #8439
Conversation
pamil
commented
Aug 23, 2017
Q | A |
---|---|
Bug fix? | no |
New feature? | no |
BC breaks? | yes |
Related tickets | part of #8425 |
License | MIT |
4eb8b41
to
1e4d7b6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Such a big diff should be prohibited by law 😄 🚓
@@ -115,7 +115,7 @@ protected function getModelNamespace() | |||
* | |||
* @throws UnknownDriverException | |||
*/ | |||
protected function getMappingCompilerPassInfo($driverType) | |||
protected function getMappingCompilerPassInfo($driverType): array |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
string $driverType
@@ -32,11 +32,20 @@ public function createRouteCollection(); | |||
* @param array $requirements An array of requirements for parameters (regexes) | |||
* @param array $options An array of options | |||
* @param string $host The host pattern to match | |||
* @param string|array $schemes A required URI scheme or an array of restricted schemes | |||
* @param string|array $methods A required HTTP method or an array of restricted methods | |||
* @param array $schemes A required URI scheme or an array of restricted schemes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would change the comment to A required array of restricted schemes
or remove it at all.
* @param string|array $schemes A required URI scheme or an array of restricted schemes | ||
* @param string|array $methods A required HTTP method or an array of restricted methods | ||
* @param array $schemes A required URI scheme or an array of restricted schemes | ||
* @param array $methods A required HTTP method or an array of restricted methods |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similarly here.
@@ -19,16 +19,18 @@ | |||
interface ToggleableInterface | |||
{ | |||
/** | |||
* Missing scalar typehint because it conflicts with AdvancedUserInterface. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😢
1e4d7b6
to
34323aa
Compare
Great work Kamil! 🌕 |
[Resource] Use typehints introduced by PHP7+
[Resource] Use typehints introduced by PHP7+
[Resource] Use typehints introduced by PHP7+