Codezwiz >> Forums >> General PHP Nuke Help >> Remove points from SQL when page is viewed.

Well I know the bottom will not work. Just a quick glance of the above without hard study.

I don't think $points can be included as a global. I will have a closer look at it a little later today.

You should remove some of the globals not needed and combine the two you have so that its only one set.

I did the following just kinda stabbing in the dark, but it should work.

$points = $userinfo['points'];
$username = $userinfo['username'];
$db->sql_query("UPDATE ".$prefix."_users SET points = '$points-1000' WHERE username = '$username'");

Here is kinda what I was thining. This is just the php part. Be sure to backup if you have part of it working already.

<?php
/************************************************************************/
/* */
/* Copyright (c) 2001-2002 by CrossWalkCentral */
/* http://www.crosswalkcentral.net */
/* */
/* CrossWalkCentral */
/* You Web Hosting Community! */
/* */
/* Let us customize this script for you. */
/* */
/* Please let us know what you think of the Support Ticket System */
/* at http://www.crosswalkcentral.net/modules.php?name=Forum */
/* */
/* This program is free software. */
/* */
/* DOWNLOAD SUPPORT TICKET SYSTEM PRO FOR 20USD AT */
/* HTTP://WWW.CROSSWALKCENTRAL.NET */
/* */
/************************************************************************/

if (!eregi("modules.php", $PHP_SELF)) {
die ("You can't access this file directly...");
}

include("header.php");
include("config.php");
//CUSTOM ACCESS BY POINTS - MJ HUFFORD - [ Register or login to view links on this board. ]
global $userinfo, $user, $db, $cookie, $prefix, $anonymous, $dbi, $user_prefix, $stop, $module_name, $redirect, $mode, $t, $f, $admin, $nukeurl, $startdate;
customized_access_points($userinfo['points'], 1000); //WHERE 1000 = MIN POINTS TO ACCESS MODULE
$points = $userinfo['points'];
$username = $userinfo['username'];
$db->sql_query("UPDATE ".$prefix."_users SET points = '$points-1000' WHERE username = '$username'");
//$content = ""; <--This should only be used for a block!
getusrinfo($user);
cookiedecode( $user );
$ip = getenv( "REMOTE_ADDR" );
$username = $cookie[1];
if ( !isset( $username ) )
{
$username = "$ip";
$guest = 1;
}

?>

Try something like this:

//ESTABLISH GLOBALS
global $db, $cookie;

//GET USERNAME
$username = $cookie[1];

//UPDATE SQL
$db->sql_query("UPDATE ".$prefix."_users SET points = points - 1000 WHERE username = '".$username."'");

Iv been hacking away at this for ages... somethings not right..

I have my doubts about "$points-1000" and also I dont think $username is declaired.

Also I would think the points should be calculated first in php then updated to sql.

OPPS... Didnt see the post above.

EDIT: SOLVED THANK YOU BOTH VERY MUCH. I hope this also comes in handy for others

Kewl...glad to help!

mjhufford is always stealing my glory.

Good to hear you got it all worked out sniper.

EDIT: what is the final code?

You both did well... All I have to do now is get the submission form to shutdown a server and start it up again

EDIT: Final Code

if (!eregi("modules.php", $PHP_SELF)) {
die ("You can't access this file directly...");
}

include("header.php");
include("config.php");
//CUSTOM ACCESS BY POINTS - MJ HUFFORD - [ Register or login to view links on this board. ]
global $userinfo, $user, $db, $cookie, $prefix, $anonymous, $dbi, $user_prefix, $stop, $module_name, $redirect, $mode, $t, $f, $admin, $nukeurl, $startdate;
customized_access_points($userinfo['points'], 1000); //WHERE 1000 = MIN POINTS TO ACCESS MODULE
$points = $userinfo['points'];
$username = $userinfo['username'];

//ESTABLISH GLOBALS
global $db, $cookie;

//GET USERNAME
$username = $cookie[1];

//UPDATE SQL
$db->sql_query("UPDATE ".$prefix."_users SET points = points - 1000 WHERE username = '".$username."'");

getusrinfo($user);
cookiedecode( $user );
$ip = getenv( "REMOTE_ADDR" );
$username = $cookie[1];
if ( !isset( $username ) )
{
$username = "$ip";
$guest = 1;
}

?>