Old content with different line endings can break diff entirely

Issue created by @acbramley
Comment about 1 year ago →
🇦🇺Australia mstrelan
This is introduced since DiffEngine is deprecated and replaced with sebastianbergmann/diff. Specifically the issue is that \Drupal\Component\Diff\DiffOpOutputBuilder::hunkOp does not handle \SebastianBergmann\Diff\Differ::DIFF_LINE_END_WARNING.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7
last update about 1 year ago
41 pass
@mstrelan opened merge request.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7
last update about 1 year ago
41 pass
Open in Jenkins → Open on Drupal.org →
Core: 10.1.x + Environment: PHP 8.1 & MySQL 8
last update about 1 year ago
30 pass, 4 fail
Status changed to Postponed about 1 year ago5:50am 26 September 2023
Comment about 1 year ago →
🇦🇺Australia mstrelan
Opened 🐛 DiffOpOutputBuilder::hunkOp does not handle Differ::DIFF_LINE_END_WARNING Active . Postponing on that issue in case there's further work to be done here.
Comment about 1 year ago →
🇺🇸United States micahw156
Confirming the patch for #3389715 resolved my use case for diff module errors.
Comment 4 months ago →
🇩🇪Germany stolzenhain
Thanks for this issue and the linking of #3389715 – patch works fine.

Comments & Activities