[Finder] simplified code #15962
[Finder] simplified code #15962
Conversation
| Q | A |
|---|---|
| Bug fix? | no |
| New feature? | no |
| BC breaks? | no |
| Deprecations? | yes |
| Tests pass? | yes |
| Fixed tickets | n/a |
| License | MIT |
| Doc PR | n/a |
| return !preg_match('/[*?[:alnum:] \\\\]/', $start); | ||
| } | ||
|
|
||
| if ($start === '{' && $end === '}') { |
There was a problem hiding this comment.
these are not the only bracket types supported for delimiters btw. Our code is missing support for other ones: http://php.net/manual/en/regexp.reference.delimiters.php
There was a problem hiding this comment.
btw, the logic in the Expression subsystem was supporting () as delimiters, so this is actually a regression
9c0f54d to
3e1e1c7
Compare
|
👍 this complex system is not necessary anymore now that we deprecated the find-based adapters. @fabpot is it possible to add tests for the regex detection here, including the case of all kinds of bracket delimiters ? and you need to mark tests of the |
3e1e1c7 to
4fa3eaf
Compare
|
The tests where already in the iterator test class for |
|
good to me |
This PR was merged into the 2.8 branch. Discussion ---------- [Finder] simplified code | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | yes | Tests pass? | yes | Fixed tickets | n/a | License | MIT | Doc PR | n/a Commits ------- 4fa3eaf [Finder] simplified code