why
    $var->{"previous.url"} = $self->getUrl("func=previousThread") if (defined $previous);
why not
    $var->{"previous.url"} = $previous->getUrl() if (defined $previous);
in Thread.pm?

at some point someone removed "my $previous = $self->getPreviousThread;" in favor of the $self->getUrl("func=previousThread"), but now that It's back, why use that "ugly" and not the direct  url to the next/prev Thread...

this:
https://svn.webgui.org/svnweb/plainblack/diff/WebGUI/lib/WebGUI/Asset/Post/Thread.pm?rev1=1330;rev2=1331 is reversed by this:
https://svn.webgui.org/svnweb/plainblack/diff/WebGUI/lib/WebGUI/Asset/Post/Thread.pm?rev1=4031;rev2=4032

From an OO standpoint, it breaks encapsulation.  But that happens so frequently in wG that it would be hypocritical to call it a bug.

So maybe we need a different category for code improvements? 

I'm of the opinion that it's such a small change that it might be better to just do it.

One side effect of this bug is inaccurate information from macros (using the ^PageTitle(); macro, for example).