Opened 12 years ago

Closed 10 years ago

#19429 closed Bug (fixed)

Admin inlines readonly fields ignore newlines

Reported by: anonymous Owned by: nobody
Component: contrib.admin Version: 1.8
Severity: Normal Keywords:
Cc: Marc Aymerich Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

like bug #19226 but with inlines.

The problem is readonly fields in inlines are rendered inside a <p></p> without <br> tags for each line.

Attachments (1)

ticket_19429.diff (1.8 KB ) - added by Marc Aymerich 12 years ago.

Download all attachments as: .zip

Change History (7)

by Marc Aymerich, 12 years ago

Attachment: ticket_19429.diff added

comment:1 by Marc Aymerich, 12 years ago

Cc: Marc Aymerich added
Component: Uncategorizedcontrib.admin
Easy pickings: set
Has patch: set
Triage Stage: UnreviewedReady for checkin
Type: UncategorizedBug

comment:2 by Marc Aymerich, 12 years ago

The attached patch fixes the problems and also has tests.

comment:3 by Claude Paroz <claude@…>, 12 years ago

Resolution: fixed
Status: newclosed

In 553838a285d6ef4005d7a97fc07aebfb0dcdb921:

Fixed #19429 -- Applied linebreaksbr to read-only fields in inlines

Applied to inlines what ec9d6b1122d did for main fieldsets.

comment:4 by Claude Paroz <claude@…>, 12 years ago

In 57a41d985acabc7fc25f08081e58f0978cc8c1dc:

[1.5.x] Fixed #19429 -- Applied linebreaksbr to read-only fields in inlines

Applied to inlines what ec9d6b1122d did for main fieldsets.
Backport of 553838a28 from master.

comment:5 by mthvth, 10 years ago

Resolution: fixed
Status: closednew
Version: 1.41.8

I had the same issue in 1.8 . A quick inspection of the source code revealed that the problem is exactly the same. The original patch is applicable to the version 1.8

Last edited 10 years ago by mthvth (previous) (diff)

comment:6 by Tim Graham, 10 years ago

Resolution: fixed
Status: newclosed

In 1.8, the implementation has changed, see c94bff2b26f8844fe15d68a79511ff3a6853dc61. If you think that patch is incorrect, please open a new ticket. Thanks.

Note: See TracTickets for help on using tickets.
Back to Top