[RELEASE vB2.0] quick 'who posted'

This is a fairly simple hack, but my users wanted it, so here it is.



What it does is to add a link to the number of replies for a thread, which allows users to quickly see who has posted in a thread without opening the showthread.php page. The results are shown in a small popup window.

Any suggestions or comments are welcome.

[edit: this hack is now included in vBulletin 2.0.3 and above]

To do that, simply add this to your whoposted template:

<a href="javascript:opener.location=('showthread.php?s=$session[sessionhash]&threadid=$threadid');">Go to this thread</a>

you can add self.close() to that javascript link, and the small window will close while the thread loads in the big window

thanks fortunately all my members are visible - since i removed the option of being invisible on my forums

Thanks!

Good idea bira For those who are not too familiar with javascript syntax, that would make the link look like this:

<a href="javascript:opener.location=('showthread.php?s=$session[sessionhash]&threadid=$threadid'); self.close();">Show thread and close this window</a>

Just I am too dumb to know which one, please enlighten me...

Originally posted by freddie


Bira there is no "strict php coding" rulebook that says one must enclose if/then segments withing curly braces.
i.e.

if ($x)
  if ($y)
    if ($z)
      echo "YES";

is the same as and as valid as

if ($x) {
  if ($y) {
    if ($z) {
      echo "YES";
    }
  }
}

Not to mention the first makes your code shorter. Not to argue, bira, but I always use if ($whatever): instead of if ($whatever) { because I am forever forgetting to close the curly braces. So maybe I'm just lazy but I prefer to not use the braces.

good hack!

congrats

Originally posted by tubedogg


Not to mention the first makes your code shorter. Not to argue, bira, but I always use if ($whatever): instead of if ($whatever) { because I am forever forgetting to close the curly braces. So maybe I'm just lazy but I prefer to not use the braces.

Well, you do have to enclose it in brackets when an if block contains multiple statements...

offtopic:

I don't get why many people code like this:

if($this) {
  doThis();
  doThat();
  if($that) {
    bla()
  }
}
else {
  doThose();
  doThese();
}

This way you easily forget those brackets to close, especially with nesting if's and else's...

It's far better to code like this:

if($this)
{	doThis();
  	doThat();
	if($that)
	{	bla();
	}
}
else
{	doThose();
  	doThese();
}

When I go to the vb-code I often can't see what belongs to where...

Originally posted by Mas*Mind

Well, you do have to enclose it in brackets when an if block contains multiple statements...

No, not in the alternative method he's talking about.

if you do if(whaterver): then you can have as many statements as you want, up until you write endif;

BUT, that method as well as my previous complaint just makes the code even more incomprehensible.