163202 - Rename html_linebreak filter

Status: NEW

(Bugzilla :: Bugzilla-General, defect)

Description

[Bradley Baetz (:bbaetz)]

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?)

Comment 1

[Gervase Markham [:gerv]]

attribute_quote (by analogy with html_quote and url_quote)?

Gerv

Comment 2

[Bradley Baetz (:bbaetz)]

It doesn't quote though - it _only_ does line breaks

Comment 3

[Gervase Markham [:gerv]]

escape_crlf ? crlf_escape?

Gerv

Comment 4

[Myk Melez [:myk] [@mykmelez]]

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").

Comment 5

[Bradley Baetz (:bbaetz)]

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

Comment 6

[Dave Miller [:justdave]]

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.

Comment 7

[Jonathan]

Is this still relevant?