Правильный Sql Запрос!

[Dementor]

Сообщение от Krasavella

Это пример другого запроса, для проверки существует ли логин

не понял совсем, вы проверяете существует ли логин, подсчитывая его кол-во? Вы меня окончательно запутали.

Добавлено через 4 минуты
Я не вижу Вашего полного класса, поэтому опять уточняю:

$db->loadResult() - это я так понимаю mysql_result()

Тогда, что у Вас mysql_fetch_assoc() ? Он мне нужен для доработки Вашего скрипта.

[gorodetskiy]

http://forum.zone-game.info/showthread.php?t=26651

Кто то не туда написал ответ на эту тему

[Dementor]

Вот короче, должно работать:

 
//// Count Accounts in DB ////
$query = 'SELECT login, user_id FROM #__accounts WHERE `login`= '.$db->Quote($Login);
$db->setQuery($query);
$useLogin = $db->loadAssocList();
 
$query = "SELECT COUNT(login) FROM #__accounts WHERE user_id={$useLogin['user_id']}";
$db->setQuery($query);
$countLogin = $db->loadResult();
 
if (!$useLogin['login'] || $countLogin >= 5) {
	JError::raiseNotice( '', 'REGISTRATION_COUNT_5' );
	return false; /// STOP if Login >= 5 ///
}
 
Code: PHP

Добавлено через 31 секунду

Сообщение от gorodetskiy

Кто то не туда написал ответ на эту тему

хахаха, уже сам заметил

[Krasavella]

Сообщение от РадужныйПони

должно работать:

Parse error: syntax error, unexpected T_CONSTANT_ENCAPSED_STRING in /storage/home/srv44892/inetpub/components/register.php on line 92

Полностью код:

Полностью код:

Свернуть ↑

PHP код:

<?php
// no direct access
defined('_JEXEC') or die('Restricted access');

jimport('joomla.application.component.model');
JTable::addIncludePath( JPATH_COMPONENT . DS . 'tables' );

class JusersModelRegister extends JModel {
    
//    Constructor
    function __construct() {
        parent::__construct();
    }

//
    public function save($post) {
    global $mainframe;        
    $table    = &$this->getTable();
//// Input Data //// 
    $jLogin        =    $post ['login'];
    $jPassword    =    $post ['password'];
    $jPassword1    =    $post['password1'];
    $jEmail        =    $post ['email'];
    $jComments    =    $post['comments'];
    $jPattern    =    '[^a-z,^A-Z,^0-9]';
    $jDB        =    &JFactory::getJDbo();
    $db        =    &JFactory::getDBO();
//// Check Empty Input Data //// 
    //// Check Empty Login //// 
    if (empty($jLogin)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_LOGIN' ); 
        return false;        
    }
    //// Check Empty Password //// 
    if (empty($jPassword)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_PASS' ); 
        return false;
    }
    //// Check Empty Password //// 
    if (empty($jPassword1)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_PASS1' ); 
        return false;
    }    
    /// Check reEnter Password ///
    if ($jPassword <> $jPassword1) {
        JError::raiseNotice( '', 'REGISTRATION_NOT_PASS2' ); 
        return false;
    }
//// Check Login in DB //// 
    if ($jLogin) {
        //// Server DB //// 
        $querys = 'SELECT count(`login`) FROM #__accounts WHERE login = '.$jDB->Quote($jLogin);                
        $jDB->setQuery($querys);
        if ($jDB->loadResult()) {
            JError::raiseNotice( '', 'REGISTRATION_ALREADY_REGISTRED_ON_SERVER' );
            return false; /// STOP if Login in use ///
        } else {
        //// Site DB ////
            $query = 'SELECT count(`login`) FROM #__accounts WHERE `login`= '.$db->Quote($jLogin);
            $db->setQuery($query);
            if ($db->loadResult()) {
                JError::raiseNotice( '', 'REGISTRATION_ALREADY_REGISTRED' );
                return false; /// STOP if Login in use ///
            } else {
                    //// Count Accounts > 5 Site DB ////
            $query = 'SELECT login, user_id FROM #__accounts WHERE `login`= '.$db->Quote($jLogin);
            $db->setQuery($query);
            $user_id_login = $db->loadAssocList();
            $query2 = 'SELECT count(`login`) FROM #__accounts WHERE `login`= '.$db->Quote($jLogin)' AND `user_id`='.($user_id_login['user_id']);
            $db->setQuery($query2);
            if ($db->loadResult() >= 5) {
                JError::raiseNotice( '', 'REGISTRATION_ALREADY_REGISTRED' );
                return false; /// STOP if Login in use ///
            }
            /// IF LOGIN NOT IN USE in Site DB and Server DB ///
            /// WRITE DATA in Server DB ///
                $jPassCode    = base64_encode(pack('H*', sha1($jPassword)));
                $query = 'INSERT INTO accounts (login, password, access_level) 
                    VALUES ('.$jDB->Quote($post ['login']).', '.$jDB->Quote($jPassCode).', 0)';
                $jDB->setQuery($query);
                if (!$jDB->query()) { /// STOP if error ///
                    JError::raiseError( 500, $jDB->stderr() );
                    return false;
                }
        }
    }
// привязываем поля формы к таблице
        if (!$table->bind($post)) {
            $this->setError($table->getError());
            return false;
        }
// проверяем данные
        if ($table->check($post)) {
            if (!$table->store($post)) {
                $this->setError($table->getError());
                return false;
            }
        } else { 
            $this->setError($table->getError());
            return false;
        }
        return true;
    } 
}
?>

Свернуть ↑Развернуть ↓

В чем ошибка?

[x3k]

PHP код:

$query2 = 'SELECT count(`login`) FROM #__accounts WHERE `login`= '.$db->Quote($jLogin)' AND `user_id`='.($user_id_login['user_id']); 


а зачем вам тут нужен WHERE login, если у вас массив из этого запроса состоит? да и вообще если логин, я так понимаю, уникальный, то тем более, а если так, то ваш запрос вернет 1.

[Dementor]

Сообщение от Krasavella

В чем ошибка?

Вы перехамичили там так... Вы пропустили скобку '}'

Это уже сами смотрите где что было пропущено.

Добавлено через 2 минуты
Ну вот как я поправил. поняв код:

<?php
// no direct access
defined('_JEXEC') or die('Restricted access');
 
jimport('joomla.application.component.model');
JTable::addIncludePath( JPATH_COMPONENT . DS . 'tables' );
 
class JusersModelRegister extends JModel {
 
//    Constructor
    function __construct() {
        parent::__construct();
    }
 
//
    public function save($post) {
    global $mainframe;        
    $table    = &$this->getTable();
//// Input Data //// 
    $jLogin        =    $post ['login'];
    $jPassword    =    $post ['password'];
    $jPassword1    =    $post['password1'];
    $jEmail        =    $post ['email'];
    $jComments    =    $post['comments'];
    $jPattern    =    '[^a-z,^A-Z,^0-9]';
    $jDB        =    &JFactory::getJDbo();
    $db        =    &JFactory::getDBO();
//// Check Empty Input Data //// 
    //// Check Empty Login //// 
    if (empty($jLogin)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_LOGIN' ); 
        return false;        
    }
    //// Check Empty Password //// 
    if (empty($jPassword)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_PASS' ); 
        return false;
    }
    //// Check Empty Password //// 
    if (empty($jPassword1)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_PASS1' ); 
        return false;
    }    
    /// Check reEnter Password ///
    if ($jPassword <> $jPassword1) {
        JError::raiseNotice( '', 'REGISTRATION_NOT_PASS2' ); 
        return false;
    }
//// Check Login in DB //// 
    if ($jLogin) {
        //// Server DB //// 
        $querys = 'SELECT count(`login`) FROM #__accounts WHERE login = '.$jDB->Quote($jLogin);                
        $jDB->setQuery($querys);
        if ($jDB->loadResult()) {
            JError::raiseNotice( '', 'REGISTRATION_ALREADY_REGISTRED_ON_SERVER' );
            return false; /// STOP if Login in use ///
        } 
		else {
        //// Site DB ////
            $query = 'SELECT count(`login`) FROM #__accounts WHERE `login`= '.$db->Quote($jLogin);
            $db->setQuery($query);
            if ($db->loadResult()) {
                JError::raiseNotice( '', 'REGISTRATION_ALREADY_REGISTRED' );
                return false; /// STOP if Login in use ///
            } 
			else {
				//// Count Accounts > 5 Site DB ////
				$query = 'SELECT login, user_id FROM #__accounts WHERE `login`= '.$db->Quote($jLogin);
				$db->setQuery($query);
				$user_id_login = $db->loadAssocList();
				$query2 = 'SELECT count(`login`) FROM #__accounts WHERE `login`= '.$db->Quote($jLogin)' AND `user_id`='.($user_id_login['user_id']);
				$db->setQuery($query2);
				if ($db->loadResult() >= 5) {
					JError::raiseNotice( '', 'REGISTRATION_ALREADY_REGISTRED' );
					return false; /// STOP if Login in use ///
				}
				/// IF LOGIN NOT IN USE in Site DB and Server DB ///
				/// WRITE DATA in Server DB ///
				$jPassCode    = base64_encode(pack('H*', sha1($jPassword)));
				$query = 'INSERT INTO accounts (login, password, access_level) 
					VALUES ('.$jDB->Quote($post ['login']).', '.$jDB->Quote($jPassCode).', 0)';
				$jDB->setQuery($query);
				if (!$jDB->query()) { /// STOP if error ///
					JError::raiseError( 500, $jDB->stderr() );
					return false;
				}
			}
        }
    }
// привязываем поля формы к таблице
        if (!$table->bind($post)) {
            $this->setError($table->getError());
            return false;
        }
// проверяем данные
        if ($table->check($post)) {
            if (!$table->store($post)) {
                $this->setError($table->getError());
                return false;
            }
        } else { 
            $this->setError($table->getError());
            return false;
        }
        return true;
    } 
}
?>
Code: PHP

[Krasavella]

Ошибок больше нет, но теперь появилась другая ошибка.

Fatal error: Call to a member function save() on a non-object in /storage/home/srv44892/inetpub/components/controller.php on line 85

Решила долго не думая посмотреть в другое место, может подскажите как из этого извлечь полезное?

Что то полезное:

Свернуть ↑

PHP код:

<?php
// no direct access
defined('_JEXEC') or die('Restricted access');

jimport('joomla.application.component.model');
JTable::addIncludePath( JPATH_COMPONENT . DS . 'tables' );

class JusersModelJusers extends JModel {
    private $_UserID, $_jAccounts;        
    
//    Constructor
    function __construct() {
        parent::__construct();
    }
// Get User ID
    public function _UserID() {
    $user =& JFactory::getUser();
//  // if User not Guest
    if (!$user->guest) { 
            $this->_UserID = $user->id;
        return $this->_UserID;
//    // if User Guest
        } else {
        return false;
        }
    }
// Get accounts
    public function getjAccounts() {
        if (empty($this->_jAccounts))
        {
            $db = &$this->getDBO();
             $UserID = $this->_UserID();
            $query = 'SELECT * FROM '.$db->nameQuote('#__accounts').' WHERE `user_id`= '.$UserID;
            $db->setQuery($query);
            $this->_jAccounts = $db->loadObjectList();

        return $this->_jAccounts;
        }        
    }    
// Get count user acc    
    public function getjUserCountAcc() {
        if (empty($this->_jUserCountAcc))
        {
            $db = &$this->getDBO();
             $UserID = $this->_UserID();
            $query = 'SELECT COUNT(*) FROM '.$db->nameQuote('#__accounts').' WHERE `user_id`= '.$UserID;
            $db->setQuery($query);
            $this->_jUserCountAcc = $db->loadResult();

        return $this->_jUserCountAcc;
        }        
    }
// EOF    
}
?>

Свернуть ↑Развернуть ↓

И второй кусок:

Что то полезное 2:

Свернуть ↑

PHP код:

<?php
// no direct access
defined('_JEXEC') or die('Restricted access');

jimport( 'joomla.application.component.view');
jimport ('joomla.html.parameter');

class JusersViewJusers extends JView {
    function display($tpl = null) {
//    Get data from model
        $jAccData        = &$this->get('jAccounts');
        $rows             = &$this->get('UserID');
        $jUserCountAcc    = &$this->get('jUserCountAcc');

        $component = JComponentHelper::getComponent('com_jusers');
        $params = new JParameter( $component->params );
        $jLimitAcc = $params->get('jlimitacc');
        $jAccess = $jLimitAcc - $jUserCountAcc;
//    Send data to TMPL
        $this->assignRef('rows', $rows);
        $this->assignRef('jAccData', $jAccData);
        $this->assignRef('jAccess', $jAccess);
        $this->assignRef('jUserCountAcc', $jUserCountAcc);
// Initialization TMPL        
        parent::display($tpl);
    }
}
?>

Свернуть ↑Развернуть ↓

И третий кусочек:

PHP код:

    <div id="jUsersBlock1">
    <?php if ($this->jAccess >= 1 ) { ?>
        <div class="jBlock1"><a href="/register">
          <div id="jAdd"></div></a>
        </div>

Из этого всего нужно добавить тот же <?php if ($this->jAccess >= 1 ) { ?> в

Финальный вариант:

Свернуть ↑

PHP код:

<?php
// no direct access
defined('_JEXEC') or die('Restricted access');

jimport('joomla.application.component.model');
//jimport('jwebcom.jconn');
JTable::addIncludePath( JPATH_COMPONENT . DS . 'tables' );

class JusersModelJregister extends JModel {
    
//    Constructor
    function __construct() {
        parent::__construct();
    }

//
    public function save($post) {
    global $mainframe;        
    $table    = &$this->getTable();
//// Input Data //// 
    $jLogin        =    $post ['login'];
    $jPassword    =    $post ['password'];
    $jPassword1    =    $post['password1'];
    $jEmail        =    $post ['email'];
    $jComments    =    $post['comments'];
    $jPattern    =    '[^a-z,^A-Z,^0-9]';
    $jDB        =    &JFactory::getJDbo();
    $db        =    &JFactory::getDBO();
//// Check Empty Input Data //// 
    //// Check Empty Login //// 
    if (empty($jLogin)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_LOGIN' ); 
        return false;        
    }
    //// Check Empty Password //// 
    if (empty($jPassword)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_PASS' ); 
        return false;
    }
    //// Check Empty Password //// 
    if (empty($jPassword1)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_PASS1' ); 
        return false;
    }    
    /// Check reEnter Password ///
    if ($jPassword <> $jPassword1) {
        JError::raiseNotice( '', 'REGISTRATION_NOT_PASS2' ); 
        return false;
    }
//// Check Login in DB //// 
    if ($jLogin) {
        //// Server DB //// 
        $querys = 'SELECT count(`login`) FROM #__accounts WHERE login = '.$jDB->Quote($jLogin);                
        $jDB->setQuery($querys);
        if ($jDB->loadResult()) {
            JError::raiseNotice( '', 'REGISTRATION_ALREADY_REGISTRED_ON_SERVER' );
            return false; /// STOP if Login in use ///
        } else {
        //// Joomla DB ////
            $query = 'SELECT count(`login`) FROM #__jaccounts WHERE `login`= '.$db->Quote($jLogin);
            $db->setQuery($query);
            if ($db->loadResult()) {
                JError::raiseNotice( '', 'REGISTRATION_ALREADY_REGISTRED' );
                return false; /// STOP if Login in use ///
            }
            /// IF LOGIN NOT IN USE in Joomla DB and Server DB ///
            /// WRITE DATA in Server DB ///
                $jPassCode    = base64_encode(pack('H*', sha1($jPassword)));
                $query = 'INSERT INTO accounts (login, password, access_level) 
                    VALUES ('.$jDB->Quote($post ['login']).', '.$jDB->Quote($jPassCode).', 0)';
                $jDB->setQuery($query);
                if (!$jDB->query()) { /// STOP if error ///
                    JError::raiseError( 500, $jDB->stderr() );
                    return false;
                }
        }
    }
// привязываем поля формы к таблице
        if (!$table->bind($post)) {
            $this->setError($table->getError());
            return false;
        }
// проверяем данные
        if ($table->check($post)) {
            if (!$table->store($post)) {
                $this->setError($table->getError());
                return false;
            }
        } else { 
            $this->setError($table->getError());
            return false;
        }
        return true;
    } 
}
?>

Свернуть ↑Развернуть ↓

Суть такова: если на мастер аккаунте больше 5 игровых аккаунтов, возвращаем ошибку.

Не понимаю

[Dementor]

ой... что-то вы жутко перехимичили. Давайте вы выложите оригинальный код register.php А я вставлю аккуратно, то что Вам нужно.

[Krasavella]

Сообщение от РадужныйПони

А я вставлю аккуратно

Вот оригинальный register.php


Из этого всего нужно добавить тот же <?php if ($this->jAccess >= 1 ) { ?> в

Финальный вариант:

Свернуть ↑

PHP код:

<?php
// no direct access
defined('_JEXEC') or die('Restricted access');

jimport('joomla.application.component.model');
//jimport('jwebcom.jconn');
JTable::addIncludePath( JPATH_COMPONENT . DS . 'tables' );

class JusersModelJregister extends JModel {
    
//    Constructor
    function __construct() {
        parent::__construct();
    }

//
    public function save($post) {
    global $mainframe;        
    $table    = &$this->getTable();
//// Input Data //// 
    $jLogin        =    $post ['login'];
    $jPassword    =    $post ['password'];
    $jPassword1    =    $post['password1'];
    $jEmail        =    $post ['email'];
    $jComments    =    $post['comments'];
    $jPattern    =    '[^a-z,^A-Z,^0-9]';
    $jDB        =    &JFactory::getJDbo();
    $db        =    &JFactory::getDBO();
//// Check Empty Input Data //// 
    //// Check Empty Login //// 
    if (empty($jLogin)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_LOGIN' ); 
        return false;        
    }
    //// Check Empty Password //// 
    if (empty($jPassword)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_PASS' ); 
        return false;
    }
    //// Check Empty Password //// 
    if (empty($jPassword1)) {
        JError::raiseNotice( '', 'REGISTRATION_EMPTY_PASS1' ); 
        return false;
    }    
    /// Check reEnter Password ///
    if ($jPassword <> $jPassword1) {
        JError::raiseNotice( '', 'REGISTRATION_NOT_PASS2' ); 
        return false;
    }
//// Check Login in DB //// 
    if ($jLogin) {
        //// Server DB //// 
        $querys = 'SELECT count(`login`) FROM #__accounts WHERE login = '.$jDB->Quote($jLogin);                
        $jDB->setQuery($querys);
        if ($jDB->loadResult()) {
            JError::raiseNotice( '', 'REGISTRATION_ALREADY_REGISTRED_ON_SERVER' );
            return false; /// STOP if Login in use ///
        } else {
        //// Joomla DB ////
            $query = 'SELECT count(`login`) FROM #__jaccounts WHERE `login`= '.$db->Quote($jLogin);
            $db->setQuery($query);
            if ($db->loadResult()) {
                JError::raiseNotice( '', 'REGISTRATION_ALREADY_REGISTRED' );
                return false; /// STOP if Login in use ///
            }
            /// IF LOGIN NOT IN USE in Joomla DB and Server DB ///
            /// WRITE DATA in Server DB ///
                $jPassCode    = base64_encode(pack('H*', sha1($jPassword)));
                $query = 'INSERT INTO accounts (login, password, access_level) 
                    VALUES ('.$jDB->Quote($post ['login']).', '.$jDB->Quote($jPassCode).', 0)';
                $jDB->setQuery($query);
                if (!$jDB->query()) { /// STOP if error ///
                    JError::raiseError( 500, $jDB->stderr() );
                    return false;
                }
        }
    }
// привязываем поля формы к таблице
        if (!$table->bind($post)) {
            $this->setError($table->getError());
            return false;
        }
// проверяем данные
        if ($table->check($post)) {
            if (!$table->store($post)) {
                $this->setError($table->getError());
                return false;
            }
        } else { 
            $this->setError($table->getError());
            return false;
        }
        return true;
    } 
}
?>

Свернуть ↑Развернуть ↓

Суть такова: если на мастер аккаунте больше 5 игровых аккаунтов, возвращаем ошибку.

Вот сюда

[Dementor]

Сообщение от Krasavella

if ($this->jAccess >= 1 ) {

а эт что такое? куда его добавить?