Line Break Converter filter should ignore line breaks like the following:

<a href="/" title="This link points to...
Drupal homepage">Drupal homepage</a>

The above produces...

Drupal homepage">Drupal homepage

CommentFileSizeAuthor
#16 filter.module-16.patch2.6 KBtadityar
#14 filter.module-14.patch2.6 KBtadityar
#8 filter.module-8.patch2.79 KBpuradata
#7 filter.module_7.patch699 bytespuradata
#7 filter.test-7.patch465 bytespuradata
#7 filter.module-autop-7.patch2.45 KBpuradata
#5 filter.module.patch627 bytesstevenpatz

Comments

magico’s picture

magico commented

Still happening.

hingo’s picture

hingo commented
Version: x.y.z » 6.0-beta1

This seems to have gotten no attention since Version field was left unspecified.

stevenpatz’s picture

stevenpatz
he/him
commented
Version: 6.0-beta1 » 7.x-dev
stevenpatz’s picture

stevenpatz
he/him
commented
Status: Active » Needs review
Issue tags: +Quick fix
StatusFileSize
new filter.module.patch627 bytes

Something like this?

Status: Needs review » Needs work

The last submitted patch failed testing.

puradata’s picture

puradata commented
StatusFileSize
new filter.module-autop-7.patch2.45 KB
new filter.test-7.patch465 bytes
new filter.module_7.patch699 bytes

Left the status "work to be done" for the following reasons.

1. I rerolled the patch that failed testing, but could not verify that it fixes anything.

2. Submitted a different patch that fixes the sample problem , but it is not an elegant solution.

 <a href="/" title="This link points to...
Drupal homepage">Drupal homepage</a>

3 . Added a test in filter.test to check that we don’t add any brreak tags to an attribute value.

puradata’s picture

puradata commented
StatusFileSize
new filter.module-8.patch2.79 KB

Merged 3 patches from comment 7 to 1.

kenorb’s picture

kenorb commented

I've got similar problem: #506112: wrong line wrapping on drupal_wrap_mail() in mail.inc

tadityar’s picture

tadityar commented
Issue summary: View changes
Status: Needs work » Needs review

setting for review

The last submitted patch, 7: filter.module-autop-7.patch, failed testing.

The last submitted patch, 7: filter.module_7.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 8: filter.module-8.patch, failed testing.

tadityar’s picture

tadityar commented
Status: Needs work » Needs review
StatusFileSize
new filter.module-14.patch2.6 KB

re-rolled patch in 8

Status: Needs review » Needs work

The last submitted patch, 14: filter.module-14.patch, failed testing.

tadityar’s picture

tadityar commented
Status: Needs work » Needs review
StatusFileSize
new filter.module-16.patch2.6 KB

Trying again!

Status: Needs review » Needs work

The last submitted patch, 16: filter.module-16.patch, failed testing.

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.