Time if i udtræk

Og hvori består dit bøvl?  Får du tabelen skrevet ud med det forventede indhold men ikke de forventede farver?  Er alle farverne ens i stedet for at variere mellem #536ca6 og #EEAA0?  I så fald hvilken af farverne får du?  Fortæl hvor det du får er noget andet end det du forventede.

Undskyld ja. jeg kan godt se at jeg slet ikke nævner fejlen. Det er netop som du siger at alle linjerne er i den sidstnævnte farve (else farven) og det burde de jo ikke.
Det er som om jeg har skrevet det første forkert hvilket naturligvis "aktiverer" else call´et.

Magnus

Ja, det ser ud til at variabelen $session->time aldrig bliver større end variabelen $timestamp minus 60*60*24*24 gjort til unix timestamp.  Så er spørgsmålet naturligvis hvad du har stående i de to variabler $session->time og $timestamp.  Du kan prøve at skrive dem ud ved midlertidigt i toppen af kodeafsnittet at skrive:

echo "SESSIONTIME = ";var_dump($session->time);echo "<br>TIMESTAMP = ";var_dump($timestamp);die;

Hvis det viser sig, at $session->time er NULL, så forklarer det naturligvis, hvorfor den aldrig bliver større end en anden variabel.  Og det skulle ikke undre mig hvis $session->time er NULL.  $session er normalt en array, således at hvis der er et time element skal det hentes ved $session['time'].

Og så undrer din brug af strtotime mig, som du selv er inde på.  Ifølge  http://php.net/manual/en/function.strtotime.php forandrer strtotime en engelsk datotid beskrivelse til et Unix timestamp.  Hvis variabelen $timestamp indeholder en unix timestamp (antal sekunder siden 1 januar 1970) og $session['time'] en engelsk datotid beskrivelse, så skal det jo være omvendt, noget i retning af

if(stototime($session['time']) - 60*60*24*7 > $timestamp){ $bgcolor='#536ca6';}

Prøv at kik på det og meld tilbage.

Hej Christian. jeg vender lige tilbage i morgen. jeg prøver frem og tilbage da dette er en del af en længere side. jeg kan ikke blot paste dine "echoes" ind i toppen. Der er $ slet ikke erklæret endnu...
Jeg skriver igen, tak for nu

Magnus

Hej igen.
jeg har prøvet mig lidt frem og afhængig af hvor jeg smider echos´ ind, får jeg forskellige værdier. altså enten en timestamp eller NULL.
jeg prøver her at smide noget mere af koden:

/* Display table contents */
  echo "<table align=\"left\" border=\"1\" cellspacing=\"0\" cellpadding=\"3\">\n";
  echo "<tr><td><b><a href='admin.php?value=1'>Brugernavn</a></b></td>
  <td><b><a href='admin.php?value=2'>Level</a></b></td>
  <td><b><a href='admin.php?value=3'>Email</a></b></td>
  <td><b><a href='admin.php?value=4'>Last Active</a></b></td>
    </tr>\n";
    $bgcolor="";
  for($i=0; $i<$num_rows; $i++){
      $uname  = mysql_result($result,$i,"username");
      $ulevel = mysql_result($result,$i,"userlevel");
      $email  = mysql_result($result,$i,"email");
      $time  = mysql_result($result,$i,"timestamp");
            $time2 = strtotime("-604800", $time);
            if(($session->time) > $time2){ $bgcolor='#536ca6';} else { $bgcolor='#EEAA00';} 
      echo "<tr bgcolor='".$bgcolor."'><td>$uname</td><td>$ulevel</td><td>$email</td><td><span class='dropt'>$time<span style='width:200px;'>$time</span></span></td></tr>\n";
  }
  echo "</table><br>\n";
}

Dette bliver hentet fra følgende:

function displayUsers(){
  global $database;
  $q = "SELECT username,userlevel,email,userid, FROM_UNIXTIME(timestamp) as timestamp FROM ".TBL_USERS ." order by timestamp DESC";
  $result = $database->query($q);
  /* Error occurred, return given name by default */
  $num_rows = mysql_numrows($result);
  if(!$result || ($num_rows < 0)){
      echo "Error displaying info";
      return;
  }

Det lykkedes mig faktisk selv at finde fejlen.
Det viser sig jo at jeg jo hentede timestampen from unixtime hvilket er svært at sammenligne med et reelt timestamp det er jo er forskellige formater. Derfor denne ekstra hent af timestamp uden from_unixtime og så virker det.

...FROM_UNIXTIME(timestamp) as timestamp2, timestamp ...
og derfor:

    $bgcolor="";
  for($i=0; $i<$num_rows; $i++){
      $uname  = mysql_result($result,$i,"username");
      $ulevel = mysql_result($result,$i,"userlevel");
      $email  = mysql_result($result,$i,"email");
      $time  = mysql_result($result,$i,"timestamp2");   
$time2 = time()-(604800);
$time3  = mysql_result($result,$i,"timestamp");   
            if(($time3) > $time2){ $bgcolor='#536ca6';} else { $bgcolor='#EEAA00';} 
      echo "<tr bgcolor='".$bgcolor."'><td>$uname</td><td>$ulevel</td><td>$email</td><td><span class='dropt'>$time<span style='width:200px;'>$time2</span></span></td></tr>\n";
    }


Tak for hjælpen Christian
Du må gerne få pointene hvis du vil?

Magnus

I så fald, svar fra mig.