Rename html_linebreak filter

NEW
Unassigned

Status

()

Bugzilla
Bugzilla-General
16 years ago
13 years ago

People

(Reporter: bbaetz, Unassigned)

Tracking

Details

(URL)

(Reporter)

Description

16 years ago
TT's CVS has added a filter called 'html_line_break', which converts:

Foo
Bar

into:

Foo<br />
Bar<br />

(The original suggestion was to use html_linebreak, but the TT maintainter
preferred the extra _)

To avoid confusion, we should probably rename ours. I'm open to suggestions for
a name...

(As a side note, has myk/anyone tried contributing our various OPS patches?
Stuff like contains, and so on?)
attribute_quote (by analogy with html_quote and url_quote)?

Gerv
(Reporter)

Comment 2

16 years ago
It doesn't quote though - it _only_ does line breaks
escape_crlf ? crlf_escape?

Gerv
I haven't contributed anything back to TT.  TT filters tend to have simple,
non-verb names, so my suggestion would be either "line_break" or
"html_attribute" ("html_attr").
(Reporter)

Comment 5

16 years ago
its not strictly a line_breeak filter, though. I'll think about it.

Updated

15 years ago
OS: Linux → All
Hardware: PC → All
Reassigning bugs that I'm not actively working on to the default component owner
in order to try to make some sanity out of my personal buglist.  This doesn't
mean the bug isn't being dealt with, just that I'm not the one doing it.  If you
are dealing with this bug, please assign it to yourself.
Assignee: justdave → general
QA Contact: mattyt-bugzilla → default-qa
You need to log in before you can comment on or make changes to this bug.