Closed Bug 13075 Opened 25 years ago Closed 24 years ago

Quip maintenance


(Bugzilla :: Bugzilla-General, enhancement, P2)




Bugzilla 2.12


(Reporter: CodeMachine, Assigned: tara)


There are a few problems I currently see with quips:

(a) There is no duplicate removal, which can lead to easy abuse by some people.
(b) Blanks lines are not removed.
(c) I also think old quips should fall off the end, probably keeping only the
last 100.  I'm not sure if this currently occurs.
Priority: P3 → P2
And of course, HTML tags are allowed when they shouldn't be.
The best solution is escaping them and displaying them rather than stripping

Also we need to make sure there is a limit on the size of each quip to prevent
Probably true, but it's very sad.  There have been some very nice HTML-based
ones in the past.
Whatever you do with HTML, I suggest it should be advertising to the quipper
when entering.

I guess you could allow some HTML.  You could:

Prevent <BLINK>, <SCRIPT>, any unknown tags etc.
<IMG> and <OBJECT> only to trusted sites, eg, to prevent porn.
Allow <B>, <EM>, <FONT> within limits etc. is the new owner of Bugzilla and Bonsai.  (For details,
see my posting in netscape.public.mozilla.webtools,
news:// .)
Assignee: terry → tara
Blank lines are removed now (see bug # 1105).  I can't bring myself to care
about (a) as I've never heard of this being a problem, and I'm not sure why
having a lot of quips (c) is bad.  I'll have to look to see if there's a
performace problem or something with having a super long list....
I consider (a) a problem because people can be encouraged to abuse the system to
make their quip more likely to appear.  You can see this occurs on
if you look at the quip file.  The quips at the top are OK (they've probably
been fixed), but at the bottom there is plenty of abuse.

I'm not sure about how this would be done though, at least using a text file
which it looks like is being used.

Inserting a line into a sorted file could well be an expensive operation
assuming the current operation is a file append.

An alternative is to keep most of the file sorted and add new lines at the
bottom unsorted, periodically sorting the whole thing.  Dupe checking would then
be a binary search on the file possibly followed by a linear scan of the
unsorted options.  This entails a periodic sorting operation though, which
implies locking the quip file for a while.

(c) depends on the opinion of the person running Bugzilla.  Some might want to
keep quips fresh, while some might want to keep a large number.
Blocks: 38852
So here's my take.  The quips are this fun little thing that are eye candy only 
and have no functional value whatsover.  I spent some time reviewing the quips list, and with the exception of about 5 really lame people 
who don't seem to understand the patheticness of putting in their quip 2^N 
times, there still aren't that many dupes.  Given that I have a bazillion other 
bugs, and I'd rather not do this the old terry way (by never closing out bugs I 
don't plan on fixing), I'd like close this out since the part of this bug that 
I think was an actual problem (blank lines) has been taken care of.  IMHO the 
dupes, length of file, number of quips, whatever, can be handled manually, or 
have a cron job deleting the file once a month, or some such.

Giving the reporter a couple of days to scream at me and tell me how 
insensitive I'm being before resolving...
What about the problem of allowing html tags in quips?
Filed bug 41906 (disallow HTML quips) and bug 41972 (Pref to turn quips off).
Well, I *like* being able to do HTML quips.  Like Terry says, there's been some 
cool ones.  I'll look into adding as preferences disallowing HTML or disabling 
quips altogether, tracked by the new bugs.
Closed: 24 years ago
Resolution: --- → FIXED
No longer blocks: 38852
In search of accurate queries....  (sorry for the spam)
Target Milestone: --- → Bugzilla 2.12
VERIFIED, given that this morphed into a not-bug.

Moving closed bugs to Bugzilla product
Component: Bugzilla → Bugzilla-General
Product: Webtools → Bugzilla
QA Contact: matty
Version: other → unspecified
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.