Skip to content

Commit

Permalink
Merge branch '4.4' into 5.2
Browse files Browse the repository at this point in the history
* 4.4:
  [Serializer][Validator] Update some phpDoc relative to "getters"
  Update README.md
  [SecurityBundle] Empty line starting with dash under "access_control" causes all rules to be skipped
  [Cache] Apply NullAdapter as Null Object
  • Loading branch information
derrabus committed Apr 13, 2021
2 parents c7ce645 + 9db284c commit 3698d26
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Normalizer/GetSetMethodNormalizer.php
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ public function hasCacheableSupportsMethod(): bool
}

/**
* Checks if the given class has any get{Property} method.
* Checks if the given class has any getter method.
*/
private function supports(string $class): bool
{
Expand All @@ -77,7 +77,7 @@ private function supports(string $class): bool
}

/**
* Checks if a method's name is get.* or is.*, and can be called without parameters.
* Checks if a method's name matches /^(get|is|has).+$/ and can be called non-statically without parameters.
*/
private function isGetMethod(\ReflectionMethod $method): bool
{
Expand Down

0 comments on commit 3698d26

Please sign in to comment.