Skip to content
Snippets Groups Projects

Issue #3239686: Undefined offset 1 in DeprecationAnalyzer twig analysis

Issue #3239686: Undefined offset 1 in DeprecationAnalyzer twig analysis

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
25 *
26 * @param \Drupal\Core\Extension\Extension $extension
27 * The extension to be analyzed.
28 *
29 * @return \Drupal\upgrade_status\DeprecationMessage[]
30 */
31 public function analyze(Extension $extension): array {
32 return array_map(static function (string $deprecation) {
33 $file_matches = [];
34 $line_matches = [];
35 // @todo uncomment, this is the fix.
36 // preg_match('/([a-zA-Z0-9\_\-\/]+.html\.twig)/', $deprecation, $file_matches);
37 // preg_match('/(\d+).?$/', $deprecation, $line_matches);
38 // @todo remove, proving needs a fix.
39 preg_match('/\s([a-zA-Z0-9\_\-\/]+.html\.twig)\s/', $deprecation, $file_matches);
40 preg_match('/\s(\d+).?$/', $deprecation, $line_matches);
  • Matt Glaman added 1 commit

    added 1 commit

    • 207cbd8b - Ensure two different assertions based on D8 or D9

    Compare with previous version

  • Matt Glaman added 1 commit

    added 1 commit

    Compare with previous version

  • Matt Glaman added 1 commit

    added 1 commit

    • 013c4da1 - Avoid checking messages based on key/return order

    Compare with previous version

  • Matt Glaman added 2 commits

    added 2 commits

    Compare with previous version

  • Matt Glaman added 1 commit

    added 1 commit

    • f09496ee - Split out error checks from D8/D9 to be more explicit

    Compare with previous version

  • Matt Glaman added 1 commit

    added 1 commit

    Compare with previous version

  • closed

  • Please register or sign in to reply
    Loading