I'm trying to write a teams module and I'm having some trouble.

<?php
/*
* Teams Module for BE Bot <http://bebot.fieses.net>
* Module coded by Craized <http://www.craized.net>

/*
Add a "_" at the beginning of the file (Teams.php) if you do not want it to be loaded.
*/

$teams = new Teams($bot);
$commands["pgmsg"]["teams"] = &$teams;
$commands["pgjoin"][] = &$teams;
$commands["pgleave"][] = &$teams;

/*
The Class itself...
*/
class Teams
{
var $bot;
var $pgroup;



/*
Constructor:
Hands over a referance to the "Bot" class.
*/
function Teams (&$bot)
{
$this -> bot = &$bot;
}


/*
This gets called on a msg in the privgroup with the command
*/
function pgmsg($name, $msg)
{
$msg = explode(" ", $msg);
$red = "<font color=#FF0000>";
if ($msg[1] == "") {
$teams = ceil(count($this -> pgroup)/6);
for($num=1; $num<=$teams; $num++) {
$result .= $red . $num . ": LDR ";
for($spot=1; $spot<=6; $spot++) {
if($this -> team -> $num[$spot]) {
if($spot == 1) {
$result .= $this -> team -> $num[$spot];
} else {
$result .= " - " . $spot . "." . $this -> team -> $num[$spot];
}
}
}
$this -> bot -> send_pgroup($result);
}
}
if($msg[1] == "clear") {
foreach($this -> pgroup as $player) {
$this -> pgroup[$player] = "lft";
unset($this -> team);
}
$this -> bot -> send_pgroup($red . "Teams cleared");
}
if(preg_match("#^[0-9]#", $msg[1], $info)) {
$team = $msg[1];
$spot = count($this -> team)+1;
if ($spot <= 6){
if($msg[2] == "") {
$this -> team -> $team[$spot] = $name;
$this -> bot -> send_pgroup($red . $name . " assigned to team " . $msg[1]);
} else {
$user = ucfirst($msg[2]);
if($this -> pgroup[$user]) {
$this -> team -> $team[$spot] = $user;
$this -> bot -> send_pgroup($red . $user . " assigned to team " . $msg[1] . " in spot " . $spot);
} else {
$this -> bot -> send_pgroup($red . $msg[2] . " not in " . $this -> bot -> botname);
}
}
} else {
$this -> bot -> send_pgroup($red . "Sorry, team " . $team . " is currently full.");
}
}
if($msg[1] == spot) {
$team = $msg[2];
$count = count($this -> team)+1;
$this -> bot -> send_pgroup($count);
}
}


/*
This gets called if someone joins the privgroup
*/
function pgjoin($name)
{
$this -> pgroup[$name] = "lft";
}



/*
This gets called if someone leaves the privgroup
*/
function pgleave($name)
{
if (isset($this -> pgroup[$name])) {
unset($this -> pgroup[$name]);
}
}
}
?>


$this -> pgroup[$name] is not beign set when a person joins the channel as far as I can tell. Also, if someone wants to write the show teams part for me I'm willing to use it. Otherwise, I'll keep working.

Thank you for any help.


EDIT: I added the show teams part of the script, it probably doesn't work, but I have no way of telling without that damn variable getting set.

EDIT: I've updated the code again, this is as far as I can get on my own.

EDIT: Highlighted text, dynamically updated.

Thank you, noted and changed, however it made no difference, just something I would have had to deal with later  :x

I figured its hard enough to read someone else's code, but plain text makes it even worse. I have a highlighted version here. I'll try and keep this updated.

<?php

$teams = new Teams($bot);
$commands["pgmsg"]["teams"] = &$teams;
$commands["pgjoin"][] = &$teams;
$commands["pgleave"][] = &$teams;

class Teams
{
   var $bot;
   var $pgroup;


   function Teams (&$bot)
   {
      $this -> bot = &$bot;
      $this -> pgroup = array();
   }

   function pgmsg($name, $msg)
   {
      if (preg_match("/^" . $this -> bot -> commpre . "teams$/i", $msg))
      {
         $this -> bot -> send_pgroup("<font color=#FF0000>Teams setup for " . $this -> bot -> botname . "\n");

         for($index = 1; $index <= 4; $index++)
         {
            $found = 0;
   
            $result = $index . ": LDR ";

            foreach($this -> pgroup as $player => $team)
            {
               if ($team == $index)
               {
                  $result .= $player . " ";
                  $found += 1;
               }
            }


            if ($found > 0)
               $this -> bot -> send_pgroup($result);
         }

         $found = 0;
         $result = "Unteamed: ";

         foreach($this -> pgroup as $player => $team)
         {
            if ($team == 0)
            {
               $result .= $player . " ";
               $found += 1;
            }
         }

         if ($found > 0)
            $this -> bot -> send_pgroup($result);
      }
      else if (preg_match("/^" . $this -> bot -> commpre . "teams clear$/i", $msg))
      {
         if ($this -> bot -> leader == $name)
         {
            foreach($this -> pgroup as $player => $team)
            {
               $this -> pgroup[$player] = 0;
            }
            $this -> bot -> send_pgroup("Teams cleared");
         }
         else
         {
            $this -> bot -> send_pgroup("You have to be leader to clear team setup.");
         }
      }
      else if (preg_match("/^" . $this -> bot -> commpre . "teams ([0-4])$/i", $msg, $info))
      {
         $found = 0;
   
         $this -> pgroup[$name] = 0;

         foreach($this -> pgroup as $player => $team)
         {
            if ($team == $info[1])
               $found += 1;
         }

         if ($info[1] == 0)
         {
            $this -> bot -> send_pgroup($name . " removed from team setup");
         }
         else if ($found > 5)
         {
            $this -> bot -> send_pgroup($name . " cannot be added to full team " . $info[1]);
         }
         else
         {
            $this -> pgroup[$name] = $info[1];
            $this -> bot -> send_pgroup($name . " assigned to team " . $info[1]);
         }
      }
   }


   function pgjoin($name)
   {
      $this -> pgroup[$name] = 0;
   }

   function pgleave($name)
   {
      if (isset($this -> pgroup[$name]))
      {
         unset($this -> pgroup[$name]);
      }
   }
}

?>

this is based on your work! just modified abit! thanks again