Issue #3182166: [D7] preg_split in _filter_url breaks for long html tags

added 1 commit

• 0581c377 - Backporting codes

Compare with previous version

added 1 commit

• 19182483 - Updates following D10's final commit.

Compare with previous version

added 45 commits

• 19182483...b5879a0e - 42 commits from branch project:7.x
• f0ab4a00 - Tests
• 106483c0 - Backporting codes
• 9ef0f40f - Updates following D10's final commit.

Compare with previous version

added 1 commit

• b9ff989f - Update method's docblock

Compare with previous version

added 1 commit

• aac169da - Fixing whitespace, commenting and using array() syntax

Compare with previous version

added 1 commit

• 0bfdde0e - Missed the single quotes (i.e. an empty string as the only array element)

Compare with previous version

resolved all threads

added 10 commits

• 0bfdde0e...925b99cd - 4 commits from branch project:7.x
• 71383e80 - Tests
• ef637c01 - Backporting codes
• 1576cf87 - Updates following D10's final commit.
• 5f6a3063 - Update method's docblock
• 481b7321 - Fixing whitespace, commenting and using array() syntax
• 89ac244b - Missed the single quotes (i.e. an empty string as the only array element)

Compare with previous version

added 1 commit

• e7f8209a - Restricting the effect the pcre.backtrack_limit changes

Compare with previous version

added 54 commits

• e7f8209a...f7d135ce - 47 commits from branch project:7.x
• 0be62e3e - Tests
• 68f4ba9f - Backporting codes
• 3d51c487 - Updates following D10's final commit.
• 768ddb44 - Update method's docblock
• 32cc18a6 - Fixing whitespace, commenting and using array() syntax
• 95096a59 - Missed the single quotes (i.e. an empty string as the only array element)
• d7a3c8fa - Restricting the effect the pcre.backtrack_limit changes

Compare with previous version

added 8 commits

• 63434249 - 1 commit from branch project:7.x
• ff2d58d5 - Tests
• 0231c713 - Backporting codes
• da6a9b73 - Updates following D10's final commit.
• 80224c3b - Update method's docblock
• 21c8151b - Fixing whitespace, commenting and using array() syntax
• 8a2a5aea - Missed the single quotes (i.e. an empty string as the only array element)
• 06413fce - Restricting the effect the pcre.backtrack_limit changes

Compare with previous version

added 1 commit

• 3bcdd644 - Use preg_last_error() for BC with PHP < 7.3

Compare with previous version

added 59 commits

• 3bcdd644...3616b351 - 58 commits from branch project:7.x
• 121db247 - Merge branch '7.x' into 3182166-d7-pregsplit-in

Compare with previous version

I think this change is an accident, when switching from if ($chunks !== FALSE) { ... } to if (preg_last_error()) { $text = $saved_text; continue; }.

Move the implode() line back, outside the for loop that processes the chunks. It will give the same answer, but the current version does a lot of unnecessary implode()s.

good catch, fixed!

changed this line in version 18 of the diff

added 2 commits

• 6a7e3790 - Cover the preg_replace_callback call too
• 35c3a164 - Optional refactor that adds coverage for all preg_* functions inside _filter_url()

Compare with previous version

added 1 commit

• a10fd0f8 - Fix accidental misplacement of call to implode()

Compare with previous version

added 1 commit

• ab18f190 - Polishing

Compare with previous version

added 1 commit

• 73d20f0b - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added 1 commit

• ed53b172 - Use `continue DEPTH` syntax instead of Exceptions

Compare with previous version

added 2 commits

• f12843ad - More polishing
• ab2bc882 - Reduce changeset (we don't need this change either)

Compare with previous version

added 1 commit

• 425aa016 - Whitespacing fixes

Compare with previous version

added 6 commits

• 425aa016...c3db7a94 - 5 commits from branch project:7.x
• c389e681 - Merge remote-tracking branch 'origin/7.x' into 3182166-d7-pregsplit-in

Compare with previous version