Beefy Boxes and Bandwidth Generously Provided by pair Networks
Perl: the Markov chain saw
 
PerlMonks  

Re^2: 0 (bug)

by tye (Sage)
on Feb 24, 2006 at 08:08 UTC ( [id://532484]=note: print w/replies, xml ) Need Help??


in reply to Re: 0
in thread Chatterbox bug

Yes, you've found a bug... with Recently Active Threads. It doesn't deal with empty titles usefully (they result in a link with no title text, which most browsers simply ignore while some just make them nearly impossible to see or follow).

I find such problems much worse than the many, many places where PM treats '0' that same as ''.

Some of these places I like more than others. For example, we recently added another case of this "bug" so that [0] renders as [0], not a link. While the chatter one is not as easy to like.

But we went years without anyone appearing to even notice this CB 'feature', so the downside is very small. Perhaps we even saved ourselves from some sloppy client code that sent message=0 when no message was desired. (:

The fact is that treating '0' like '' is so much simpler in Perl and often results in code that handles '' and undef values better than is easy to do while allowing '0' through.

So I'm not exactly opposed to 'fixing' the 'bug' that prevents '0' from being chatted. But I'm certainly not in a rush to do it and I won't mind if it is never fixed. And I'm against starting a hunt to search out the tons of similar places and add the extra complexity to the code to try to treat '0' differently while hoping to not break the handling of empty and undefined strings.

Sure, it isn't rocket science, but ignoring '0' is usually less troublesome than getting the other cases wrong. So I mostly look at it as a very Perlish quirk and thus very appropriate on this particular website and I'll likely continue to be fairly cavalier with mistreating '0' this way in PerlMonks code, while reducing code complexity. And continue to be amused and not annoyed when I discover more places here where that happens. :)

- tye        

Replies are listed 'Best First'.
532808
by de-merphq (Beadle) on Feb 25, 2006 at 21:49 UTC

    Ive fixed Recently Active Threads to show the same thing as when id:// linking to a titleless node: [untitled node, ID 532808].

    But while looking into something related to this (what title is displayed for titleless nodes) I noticed that Show Containers appears to be broken nowadays... Anyway, i've also patched the node title rendering to be consistant.

    Dont get me wrong here, im not going to hunt down any more of these, I just figured id handle the easy/obvious ones.

    ---
    alter ego of demerphq

Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://532484]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others wandering the Monastery: (4)
As of 2024-04-24 21:11 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found