Another php code question.

Premium Joined: Jan 26, 2003 Posts: 230 Location: USA

Code:

<?php

/************************************************************************/
/* PHP-NUKE: Advanced Content Management System */
/* ============================================ */
/* */
/* Copyright (c) 2002 by Francisco Burzi */
/* http://phpnuke.org */
/* */
/* This program is free software. You can redistribute it and/or modify */
/* it under the terms of the GNU General Public License as published by */
/* the Free Software Foundation; either version 2 of the License. */
/************************************************************************/

if (eregi("footer.php",$PHP_SELF)) {
Header("Location: index.php");
die();
}

$footer = 1;

function footmsg() {
global $foot1, $foot2, $foot3, $copyright, $totaltime, $starttime;
$mtime = microtime();
$mtime = explode(" ",$mtime);
$mtime = $mtime[1] + $mtime[0];
$endtime = $mtime;
$totaltime = ($endtime - $starttime);
$totaltime = ""._PAGEGENERATION." ".substr($totaltime,0,5)." "._SECONDS."";
echo "<font class=\"footmsg\">\n";
if ($foot1 != "") {
echo "$foot1<br>\n";
}
if ($foot2 != "") {
echo "$foot2<br>\n";
}
if ($foot3 != "") {
echo "$foot3<br>\n";
}
// DO NOT REMOVE THE FOLLOWING COPYRIGHT LINE. YOU'RE NOT ALLOWED TO REMOVE NOR EDIT THIS.
echo "$copyright<br>$totaltime<br>\n</font>\n";
}

function foot() {
global $prefix, $user_prefix, $dbi, $index, $user, $cookie, $storynum, $user, $cookie, $Default_Theme, $foot1, $foot2, $foot3, $foot4, $home, $module, $name;
if ($home == 1) {
blocks(Down);
}
if ($module == 1 AND file_exists("modules/$name/copyright.php")) {
$cpname = ereg_replace("_", " ", $name);
echo "<div align=\"right\"><a href=\"javascript:openwindow()\">$cpname &</a></div>";
}
themefooter();
if (is_user($user) AND !ereg("Private_Messages", $_SERVER[REQUEST_URI])) {
$uname = $cookie[1];
$popres = sql_query("SELECT uid, popmeson FROM ".$user_prefix."_users where uname='$uname'", $dbi);
list($uid, $popmeson) = sql_fetch_row($popres, $dbi);
if ($popmeson == 1) {
   $messres = sql_query("SELECT * FROM ".$prefix."_priv_msgs where to_userid='$uid' and read_msg='0'", $dbi);
   $messnum = sql_num_rows($messres, $dbi);
   if ($messnum == 1) {
   echo "<script language=\"javascript\">{ alert('"._YOUHAVEONEMSG."'); }</script>";
   }
   if ($messnum > 1) {
   echo "<script language=\"javascript\">{ alert('"._YOUHAVE." $messnum "._NEWPMSG."'); }</script>";
   }
}
}

WysiwygJscriptFooter();
echo "</body>\n";
echo "</html>";
die();
}

foot();

?>

What is wrong with this code. I am thinking uhhh the sql table pointing stuff, am I right?

Joined: Jan 31, 2004 Posts: -88

on the $user_prefix."_users line, change uid= to user_id= and uname= to username=

Artificialintel

Premium Joined: Jan 26, 2003 Posts: 230 Location: USA

Code:

$popres = sql_query("SELECT uid, popmeson FROM ".$user_prefix."_users where username='$uname'", $dbi);
list($user_id, $popmeson) = sql_fetch_row($popres, $dbi);

like that

Joined: Jan 31, 2004 Posts: -88

no, like this:

Code:

$popres = sql_query("SELECT user_id, popmeson FROM ".$user_prefix."_users where username='$uname'", $dbi);
list($uid, $popmeson) = sql_fetch_row($popres, $dbi);

Artificialintel

Joined: Jan 31, 2004 Posts: -88

no, like this:

Code:

$popres = sql_query("SELECT user_id, popmeson FROM ".$user_prefix."_users where username='$uname'", $dbi);
list($uid, $popmeson) = sql_fetch_row($popres, $dbi);

Artificialintel