SQL syntax error when registering

Hey guys, I'm just wondering why do I get this error when I register an account on my retro? I'm currently on localhost.

"You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'rank, look, gender, seckey, ip_last, ip_reg, account_created, last_online, auth_' at line 1"

Can you post an Image of your error please

Quote:

---

Originally Posted by Glee

Can you post an Image of your error please

---

I just copied and pasted what is said on the page after I register. Nothing else. Just that on a blank page.

What CMS are you using?? Can you check your table structure "seckey"

Quote:

---

Originally Posted by Glee

What CMS are you using?? Can you check your table structure "seckey"

---

Revcms, and I already imported the seckey query. Still nothing.

What are you hosting your CMS on? Because I don't think RevCMS is compatible with PHP 7

hosting on vps and its on php 5. something, pm me if you have teamviewer

maybe paste your line 1

Quote:

---

Originally Posted by Aamiainen

maybe paste your line 1

---

Where can i find that?

Quote:

---

Originally Posted by shaban1

Where can i find that?

---

mm i mean this line mentioned on your error "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'rank, look, gender, seckey, ip_last, ip_reg, account_created, last_online, auth_' at line 1"

I think that your error should mention exactly where you can find it, but i'd guess it's in /app/class/users.class.php or template.class.php but i'm not sure about it, you just have to find it or it's quite hard helping out without seeing the code.

Please post the code that the error is referring to.

Quote:

---

Originally Posted by Aamiainen

mm i mean this line mentioned on your error "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'rank, look, gender, seckey, ip_last, ip_reg, account_created, last_online, auth_' at line 1"

I think that your error should mention exactly where you can find it, but i'd guess it's in /app/class/users.class.php or template.class.php but i'm not sure about it, you just have to find it or it's quite hard helping out without seeing the code.

---

Does that mean my MYSQL server doesnt work for those DB querys or something?

- - - Updated - - -

Quote:

---

Originally Posted by Zoxq

Please post the code that the error is referring to.

---

If I knew where it was referring to then I'd be able to, but I have no idea.

Quote:

---

Originally Posted by shaban1

If I knew where it was referring to then I'd be able to, but I have no idea.

---

This happens when you're registering? And what content management system are you using? RevCMS?

Quote:

---

Originally Posted by Zoxq

This happens when you're registering? And what content management system are you using? RevCMS?

---

Yes for both of your questions.

- - - Updated - - -

Quote:

---

Originally Posted by Zoxq

Please post the code that the error is referring to.

---

Class.users.php:

Code:

---

<?php

namespace Revolution;
if(!defined('IN_INDEX')) { die('Sorry, you cannot access this file.'); }
class users implements iUsers
{
       
        /*-------------------------------Authenticate-------------------------------------*/
       
        final public function isLogged()
        {
                if(isset($_SESSION['user']['id']))
                {
                        return true;
                }
               
                return false;
        }
       
        /*-------------------------------Checking of submitted data-------------------------------------*/
       
        final public function validName($username)       
        {
                if(strlen($username) <= 25 && ctype_alnum($username))               
                {                       
                        return true;               
                }                                 
               
                return false;       
        }                 
               
        final public function validEmail($email)       
        {               
                return preg_match("/^[a-z0-9_\.-]+@([a-z0-9]+([\-]+[a-z0-9]+)*\.)+[a-z]{2,7}$/i", $email);       
        }                 
       
        final public function validSecKey($seckey)
        {
                if(is_numeric($seckey) && strlen($seckey) == 4)
                {
                        return true;
                }
               
                return false;
        }
       
        final public function nameTaken($username)       
        {               
                global $engine;               
               
                if($engine->num_rows("SELECT * FROM users WHERE username = '" . $username . "' LIMIT 1") > 0)
                {
                        return true;
                }       
               
                return false;
        }
       
        final public function emailTaken($email)
        {
                global $engine;
               
                if($engine->num_rows("SELECT * FROM users WHERE mail = '" . $email . "' LIMIT 1") > 0)
                {
                        return true;
                }
               
                return false;
        }
               
        final public function userValidation($username, $password)
        {               
                global $engine;
                if($engine->num_rows("SELECT * FROM users WHERE username = '" . $username . "' AND password = '" . $password . "' LIMIT 1") > 0)
                {
                        return true;
                }       
               
                return false;
        }                 
       
        /*-------------------------------Stuff related to bans-------------------------------------*/
       
        final public function isBanned($value)
        {
                global $engine;
                if($engine->num_rows("SELECT * FROM bans WHERE value = '" . $value . "' LIMIT 1") > 0)
                {
                        return true;
                }
                       
                return false;
        }
       
        final public function getReason($value)
        {
                global $engine;
                return $engine->result("SELECT reason FROM bans WHERE value = '" . $value . "' LIMIT 1");
        }
       
        final public function hasClones($ip)
        {
                global $engine;
                if($engine->num_rows("SELECT * FROM users WHERE ip_reg = '" . $_SERVER['REMOTE_ADDR'] . "'") == 1)
                {
                        return true;
                }
               
                return false;
        }
       
        /*-------------------------------Login or Register user-------------------------------------*/
       
        final public function register()
        {
                global $core, $template, $_CONFIG;
               
                if(isset($_POST['register']))
                {
                        unset($template->form->error);
                       
                        $template->form->setData();
                               
                        if($this->validName($template->form->reg_username))
                        {
                                if(!$this->nameTaken($template->form->reg_username))
                                {
                                        if($this->validEmail($template->form->reg_email))
                                        {
                                                if(!$this->emailTaken($template->form->reg_email))
                                                {
                                                        if(strlen($template->form->reg_password) > 6)
                                                        {
                                                                if($template->form->reg_password == $template->form->reg_rep_password)
                                                                {
                                                                        if(isset($template->form->reg_seckey))
                                                                        {
                                                                                if($this->validSecKey($template->form->reg_seckey))
                                                                                {
                                                                                        //Continue
                                                                                }
                                                                                else
                                                                                {
                                                                                        $template->form->error = 'Secret key must only have 4 numbers';
                                                                                        return;
                                                                                }
                                                                        }
                                                                        if($this->isBanned($_SERVER['REMOTE_ADDR']) == false)
                                                                        {
                                                                                if(!$this->hasClones($_SERVER['REMOTE_ADDR']))
                                                                                {
                                                                                        if(!isset($template->form->reg_gender)) { $template->form->reg_gender = 'M'; }
                                                                                        if(!isset($template->form->reg_figure)) { $template->form->reg_figure = $_CONFIG['hotel']['figure']; }
                                                                               
                                                                                        $this->addUser($template->form->reg_username, $core->hashed($template->form->reg_password), $template->form->reg_email, $_CONFIG['hotel']['motto'], $_CONFIG['hotel']['credits'], $_CONFIG['hotel']['pixels'], 1, $template->form->reg_figure, $template->form->reg_gender, $core->hashed($template->form->reg_key));
                                                       
                                                                                        $this->turnOn($template->form->reg_username);
                                                                       
                                                                                        header('Location: ' . $_CONFIG['hotel']['url'] . '/me');
                                                                                        exit;
                                                                                }
                                                                                else
                                                                                {
                                                                                        $template->form->error = 'Sorry, but you cannot register twice';
                                                                                }
                                                                        }
                                                                        else
                                                                        {
                                                                                $template->form->error = 'Sorry, it appears you are IP banned.<br />';
                                                                                $template->form->error .= 'Reason: ' . $this->getReason($_SERVER['REMOTE_ADDR']);
                                                                                return;
                                                                        }
                                                                }
                                                                else       
                                                                {
                                                                        $template->form->error = 'Password does not match repeated password';
                                                                        return;
                                                                }


                                                        }
                                                        else
                                                        {
                                                                $template->form->error = 'Password must have more than 6 characters';
                                                                return;
                                                        }
                                                }
                                                else
                                                {
                                                        $template->form->error = 'Email: <b>' . $template->form->reg_email . '</b> is already registered';
                                                        return;
                                                }
                                        }
                                        else
                                        {
                                                $template->form->error = 'Email is not valid';
                                                return;
                                        }
                                }
                                else
                                {
                                        $template->form->error = 'Username is already registered';
                                        return;
                                }
                        }
                        else
                        {
                                $template->form->error = 'Username is invalid';
                                return;
                        }
                }
        }       
       
        final public function login()
        {
                global $template, $_CONFIG, $core;
               
                if(isset($_POST['login']))
                {
                        $template->form->setData();
                        unset($template->form->error);
                       
                        if($this->nameTaken($template->form->log_username))
                        {
                                if($this->isBanned($template->form->log_username) == false || $this->isBanned($_SERVER['REMOTE_ADDR']) == false)
                                {
                                        if($this->userValidation($template->form->log_username, $core->hashed($template->form->log_password)))
                                        {
                                                $this->turnOn($template->form->log_username);
                                                $this->updateUser($_SESSION['user']['id'], 'ip_last', $_SERVER['REMOTE_ADDR']);
                                                $template->form->unsetData();
                                                header('Location: ' . $_CONFIG['hotel']['url'] . '/me');
                                                exit;
                                        }
                                        else
                                        {
                                                $template->form->error = 'Details do not match';
                                                return;
                                        }
                                }
                                else
                                {
                                        $template->form->error = 'Sorry, it appears this user is banned<br />';
                                        $template->form->error .= 'Reason: ' . $this->getReason($template->form->log_username);
                                        return;
                                }
                        }
                        else
                        {
                                $template->form->error = 'Username does not exist';
                                return;
                        }
                }
        }
       
        final public function loginHK()
        {
                global $template, $_CONFIG, $core;
               
                if(isset($_POST['login']))
                {       
                        $template->form->setData();
                        unset($template->form->error);
                       
                        if(isset($template->form->username) && isset($template->form->password))
                        {
                                if($this->nameTaken($template->form->username))
                                {       
                                        if($this->userValidation($template->form->username, $core->hashed($template->form->password)))
                                        {
                                                if(($this->getInfo($_SESSION['user']['id'], 'rank')) >= 4)
                                                {
                                                        $_SESSION["in_hk"] = true;
                                                        header("Location:".$_CONFIG['hotel']['url']."/ase/dash");
                                                        exit;
                                                }
                                                else
                                                {
                                                        $template->form->error = 'Incorrect access level.';
                                                        return;
                                                }
                                        }
                                        else
                                        {
                                                $template->form->error = 'Incorrect password.';
                                                return;
                                        }               
                                }
                                else
                                {
                                        $template->form->error = 'User does not exist.';
                                        return;
                                }
                        }
       
                        $template->form->unsetData();
                }
        }       
       
        final public function help()
        {
                global $template, $_CONFIG;
                $template->form->setData();
               
                if(isset($template->form->help))
                {
                        $to = $_CONFIG['hotel']['email'];
                        $subject = "Help from RevCMS user - " . $this->getInfo($_SESSION['user']['id'], 'username');
                        $body = $template->form->question;
                               
                        if (mail($to, $subject, $body))
                        {
                                $template->form->error = 'Message successfully sent! We will answer you shortly!';
                        }
                        else
                        {
                                  $template->form->error = 'Message delivery failed.';
                        }
                }
        }


        /*-------------------------------Account settings-------------------------------------*/
       
        final public function updateAccount()
        {
                global $template, $_CONFIG, $core, $engine;
               
                if(isset($_POST['account']))
                {
               
                        if(isset($_POST['acc_motto']) && strlen($_POST['acc_motto']) < 30 && $_POST['acc_motto'] != $this->getInfo($_SESSION['user']['id'], 'motto'))
                        {
                                $this->updateUser($_SESSION['user']['id'], 'motto', $engine->secure($_POST['acc_motto']));
                                header('Location: '.$_CONFIG['hotel']['url'].'/account');
                                exit;
                        }
                        else
                        {
                                $template->form->error = 'Motto is invalid.';
                        }
                       
                        if(isset($_POST['acc_email']) && $_POST['acc_email'] != $this->getInfo($_SESSION['user']['id'], 'mail'))
                        {
                                if($this->validEmail($_POST['acc_email']))
                                {
                                        $this->updateUser($_SESSION['user']['id'], 'mail', $engine->secure($_POST['acc_email']));
                                        header('Location: '.$_CONFIG['hotel']['url'].'/account');
                                        exit;
                                }
                                else
                                {
                                        $template->form->error = 'Email is not valid';
                                        return;
                                }
                        }
                       
                        if(!empty($_POST['acc_old_password']) && !empty($_POST['acc_new_password']))
                        {
                                if($this->userValidation($this->getInfo($_SESSION['user']['id'], 'username'), $core->hashed($_POST['acc_old_password'])))
                                {
                                        if(strlen($_POST['acc_new_password']) >= 8)
                                        {
                                                $this->updateUser($_SESSION['user']['id'], 'password', $core->hashed($_POST['acc_new_password']));
                                                header('Location: '.$_CONFIG['hotel']['url'].'/me');
                                                exit;
                                        }
                                        else
                                        {
                                                $template->form->error = 'New password is too short';
                                                return;
                                        }
                                }
                                else
                                {
                                        $template->form->error = 'Current password is wrong';
                                        return;
                                }
                        }
                }               
        }
               
               
        final public function turnOn($k)
        {       
                $j = $this->getID($k);
                $this->createSSO($j);
                $_SESSION['user']['id'] = $j;       
                $this->cacheUser($j);       
                unset($j);
        }
       
        /*-------------------------------Loggin forgotten-------------------------------------*/       
       
        final public function forgotten()
        {
                global $template, $_CONFIG, $core;
               
                if(isset($_POST['forgot']))
                {
               
                        $template->form->setData();
                        unset($template->form->error);
                       
                        if($this->nameTaken($template->form->for_username))
                        {
                                if(strlen($template->form->for_password) > 6)
                                {
                                        if($this->getInfo($this->getID($template->form->for_username), 'seckey') == $core->hashed($template->form->for_key))
                                        {
                                                $this->updateUser($this->getID($template->form->for_username), 'password', $core->hashed($template->form->for_password));
                                                $template->form->error = 'Account recovered! Go <b><a href="index">here</a></b> to login!';
                                                return;
                                        }
                                        else
                                        {
                                                $template->form->error = 'Secret key is incorrect';
                                                return;
                                        }
                                }
                                else
                                {
                                        $template->form->error = 'Password must have more than 6 characters.';
                                        return;
                                }
                        }
                        else
                        {
                                $template->form->error = 'Username does not exist';
                                return;
                        }
                }
        }
       
        /*-------------------------------Create SSO auth_ticket-------------------------------------*/
       
        final public function createSSO($k)       
        {                 
                $sessionKey = 'RevCMS-'.rand(9,999).'/'.substr(sha1(time()).'/'.rand(9,9999999).'/'.rand(9,9999999).'/'.rand(9,9999999),0,33);
               
                $this->updateUser($k, 'auth_ticket', $sessionKey);
               
                unset($sessionKey);
        }         
               
        /*-------------------------------Adding/Updating/Deleting users-------------------------------------*/
       
        final public function addUser($username, $password, $email, $motto, $credits, $pixels, $rank, $figure, $gender, $seckey)       
        {               
                global $engine;                                                                   
                $sessionKey = 'RevCMS-'.rand(9,999).'/'.substr(sha1(time()).'/'.rand(9,9999999).'/'.rand(9,9999999).'/'.rand(9,9999999),0,33);
                $engine->query("INSERT INTO users (username, password, mail, motto, credits, activity_points, rank, look, gender, seckey, ip_last, ip_reg, account_created, last_online, auth_ticket) VALUES('" . $username . "', '" . $password . "', '" . $email . "', '" . $motto . "', '" . $credits . "', '" . $pixels . "', '" . $rank . "', '" . $figure . "', '" . $gender . "', '" . $seckey . "', '" . $_SERVER['REMOTE_ADDR'] . "', '" . $_SERVER['REMOTE_ADDR'] . "', '" . time() . "', '" . time() . "', '" . $sessionKey . "')");       
                unset($sessionKey);       
                                         
        }                                 
               
        final public function deleteUser($k)       
        {               
                global $engine;                                 
                $engine->query("DELETE FROM users WHERE id = '" . $k . "' LIMIT 1");               
                $engine->query("DELETE FROM items WHERE userid = '" . $k . "' LIMIT 1");               
                $engine->query("DELETE FROM rooms WHERE ownerid = '" . $k . "' LIMIT 1");       
        }       
                 
        final public function updateUser($k, $key, $value)       
        {               
                global $engine;                                 
                $engine->query("UPDATE users SET " . $key . " = '" . $engine->secure($value) . "' WHERE id = '" . $k . "' LIMIT 1");
                $_SESSION['user'][$key] = $engine->secure($value);               
        }
       
        /*-------------------------------Handling user information-------------------------------------*/         
       
        final public function cacheUser($k)
        {
                global $engine;                         
                $userInfo = $engine->fetch_assoc("SELECT username, rank, motto, mail, credits, activity_points, look, auth_ticket, ip_last FROM users WHERE id = '" . $k . "' LIMIT 1");
               
                foreach($userInfo as $key => $value)
                {
                        $this->setInfo($key, $value);
                }
        }       
       
        final public function setInfo($key, $value)
        {
                global $engine;
                $_SESSION['user'][$key] = $engine->secure($value);
        }


        final public function getInfo($k, $key)
        {
                global $engine;
                if(!isset($_SESSION['user'][$key]))
                {
                        $value = $engine->result("SELECT $key FROM users WHERE id = '" . $engine->secure($k) . "' LIMIT 1");
                        if($value != null)
                        {                       
                                $this->setInfo($key, $value);
                        }
                }
                       
                return $_SESSION['user'][$key];
        }
       
       
       
        /*-------------------------------Get user ID or Username-------------------------------------*/
       
        final public function getID($k)       
        {               
                global $engine;               
                return $engine->result("SELECT id FROM users WHERE username = '" . $engine->secure($k) . "' LIMIT 1");       
        }               
       
        final public function getUsername($k)
        {
                global $engine;
                return $this->getInfo($_SESSION['user']['id'], 'username');
        }
       
}
?>

---

Class.template.php

Code:

---

<?php

namespace Revolution;
if(!defined('IN_INDEX')) { die('Sorry, you cannot access this file.'); }
class template implements iTemplate
{


        public $tpl;
       
        private $params = array();


        final public function Initiate()
        {
                global $_CONFIG, $users, $engine, $core, $template;
                $this->setParams('hotelName', $_CONFIG['hotel']['name']);
                $this->setParams('hotelDesc', $_CONFIG['hotel']['desc']);
                $this->setParams('url', $_CONFIG['hotel']['url']);
                $this->setParams('online', $core->getOnline());
                $this->setParams('status', $core->getStatus());
                $this->setParams('web_build', $_CONFIG['hotel']['web_build']);
                $this->setParams('external_vars', $_CONFIG['hotel']['external_vars']);
                $this->setParams('external_texts', $_CONFIG['hotel']['external_texts']);
                $this->setParams('swf_folder', $_CONFIG['hotel']['swf_folder']);
                $this->setParams('furni_data', $_CONFIG['hotel']['furni_data']);
                $this->SetParams('product_data', $_CONFIG['hotel']['product_data']);
                $this->setParams('server_ip', $_CONFIG['hotel']['server_ip']);
               
                $this->setParams('mysql_host', $_CONFIG['mysql']['hostname']);
                $this->setParams('mysql_port', $_CONFIG['mysql']['port']);
               
                $this->setParams('skin', $_CONFIG['template']['style']);
               
                if($users->isLogged())
                {       
                        $this->setParams('username', $users->getInfo($_SESSION['user']['id'], 'username'));
                        $this->setParams('rank', $users->getInfo($_SESSION['user']['id'], 'rank'));
                        $this->setParams('motto', $users->getInfo($_SESSION['user']['id'], 'motto'));
                        $this->setParams('email', $users->getInfo($_SESSION['user']['id'], 'mail'));
                        $this->setParams('coins', $users->getInfo($_SESSION['user']['id'] ,'credits'));
                        $this->setParams('activitypoints', $users->getInfo($_SESSION['user']['id'], 'activity_points'));
                        $this->setParams('figure', $users->getInfo($_SESSION['user']['id'], 'look'));
                        $this->setParams('ip_last', $users->getInfo($_SESSION['user']['id'], 'ip_last'));
                       
                        if($this->params['rank'] > 3)
                        {
                                $this->setParams('housekeeping', '<li><a href="ase/">Housekeeping</a></li>');
                        }
                        else
                        {
                                $this->setParams('housekeeping', '');
                        }
                       
                        if($_GET['url'] == 'me' || $_GET['url'] == 'account' || $_GET['url'] == 'home' || $_GET['url'] == 'settings' || $_GET['url'] == 'community')
                        {
                                $template->form->getPageHome();                               
                        }
                       
                        if($_GET['url'] == 'news' || $_GET['url'] == 'articles')
                        {
                                $template->form->getPageNews();
                        }               
                }
               
        }
       
        final public function setParams($key, $value)
        {       
                $this->params[$key] .= $value;
        }
       
        final public function filterParams($str)
    {
        foreach($this->params as $key => $value)
        {
            $str = str_ireplace('{' . $key . '}', $value, $str);
        }


        return $str;
        }
         
        final public function write($str)
        {
                $this->tpl .= $str;
        }
       
        final public function outputTPL()
        {
                echo $this->filterParams($this->tpl);
                unset($this->tpl);
        }
}
?>

---

The errors occurs in the addUser function.