Не правильно работает код php?

Question

[viktorross]

здравствуйте, подскажите в чем может быть проблема, не могу понять в чем ошибка

я хочу в этом скрипте выполнить 2 запроса сразу

class banners {
	function addHit($id=0) {
		global $db,$config_abs_path;
		if(!$id) $id=$this->id;
		$res_del=$db->query('update '.TABLE_BANNERS.' set `clicks` = `clicks`+1 where `id`="'.$id.'"');

		require_once $config_abs_path."/classes/users.php";
		$email=users::getNEmail($id);
		return 1;
	}
}

при выполнении этого

require_once $config_abs_path."/classes/users.php";
$email=users::getNEmail($id);

в бд записывается постоянно значение из 1 строки бд

class users {
static function getEmail($id='') {

		global $db;
		if(!$id) return;
		$email=$db->fetchRow('select email from '.TABLE_USERS.' where id="'.$id.'"');
		return $email;
	}
function getNEmail($id) {

		global $db;
		$email=users::getEmail($id);
		$db->query("insert into `class_click` set `email` = '$email'");
	}

}

почему так происходит?

получается он не видит авторизации? хотя во втором скрипте в самом файле сессия запущена

Comments

[viktorross]

VicTHOR, она получает email из бд, но не текущего пользователя, а просто первый по id в базе, и записывает уже это значение в другую таблицу, т.е получается это происходит без авторизации пользователя

[viktorross]

VicTHOR, вот эта функция получает email авторизованного пользователя

class users {
static function getEmail($id='') {

    global $db;
    if(!$id) return;
    $email=$db->fetchRow('select email from '.TABLE_USERS.' where id="'.$id.'"');
    return $email;
  }

вот эти 2 функции при вызове должны записать email этого пользователя в другую таблицу

class banners {
  function addHit($id=0) {
    global $db,$config_abs_path;
    if(!$id) $id=$this->id;
    $res_del=$db->query('update '.TABLE_BANNERS.' set `clicks` = `clicks`+1 where `id`="'.$id.'"');

    require_once $config_abs_path."/classes/users.php";
    $email=users::getNEmail($id);
    return 1;
  }
}

function getNEmail($id) {

    global $db;
    $email=users::getEmail($id);
    $db->query("insert into `class_click` set `email` = '$email'");
  }

мне нужно каждый раз записывать email пользователя вызывавшего функцию addHit в таблицу, нужно чтобы смотреть кто и на что нажимает на сайте, суть в том что все это работает, но почему-то записывается не email того кто авторизовался, а просто email первого id в базе

можно авторизоваться этой функцией, но как ни крутил это все, результат тот же

global $is_admin, $logged_in, $crt_usr;
	$auth=new auth();
	$logged_in = $auth->loggedIn();
	$is_admin = $auth->adminLoggedIn();
	if(!$logged_in && !$is_admin)
		$logged_in = $auth->checkCookieLogin();

	// vars for user account navbar
	if($logged_in) $crt_usr = $auth->crtUserId();

}

[viktorross]

VicTHOR, 2 запроса (addHit и getEmail) уже были для других целей, я их просто подключил к новому getNEmail

Answer 1

[AUser0]

Код вполне рабочий. Проверяйте место вызова banners::addHit(), там аргументом явно указан не ID текущего пользователя, а что-то ещё (первый из базы данных).

P.S. Или class_click меняется где-то ещё.

Comments

[viktorross]

AUser0 , вы имеете ввиду там где id=0? А как можно дописать , как вообще это можно оформить по-другому?

[AUser0]

Нет. Там, где делается ВЫЗОВ этой функции, с идентификатором какого-то пользователя.

[AUser0]

Кстати, строка if(!$id) $id=$this->id; в приведённом коде некорректна, потому что переменная $this->id нигде не объявлена и ей не присвоено какое-либо значение.

[viktorross]

AUser0, вот тут присваивается внутри класса banners

public function __construct($id=0)
	{

		global $db;
		$this->path="../images/baners/";
		if($id) {
			$this->id=$id;
			$this->array=array();
			$this->array=$db->fetchAssoc('select * from '.TABLE_BANNERS.' where `id`='.$id);
		}
	}

	function getId() {
		return $this->id;
	}

может быть как раз внутри класса banners нужно авторизоваться перед подключением?

require_once $config_abs_path."/classes/users.php";
    $email=users::getNEmail($id);
    return 1;

хотя в самом /classes/users.php уже есть авторизация, вероятно она не работает когда вызов идет из другого файла, хотя фактически внутри класса banners идет только вызов, а обработка и запись уже внутри класса users.

Хотя обычно у меня авторизация идет перед тем, как выводить данные в шаблон, вот так

global $logged_in, $crt_usr;

$usr = new users();
$user = $usr->getUser($crt_usr);
$smarty->assign("user",$user);

ну или вот так

$auth = new auth();
$last_login=$auth->getLoginBefore($logged_in);
$total_views = listings::getViewed($crt_usr);

я пробовал сделать тоже самое в нужных файлах внутри классов, просто без вывода в шаблон, но не работает..

[AUser0]

Ладно, с $this->id разобрались, он корректен.
И повторно авторизовываться не надо, вы же передаёте $id пользователя, этого достаточно.
Параметры вызова banners::addHit() проверяли? Там этот $id передаётся?

[viktorross]

AUser0, и все-таки вероятно нет авторизации в файле banners, также как и в файле users, авторизация идет при передаче в шаблон, получается у меня в классах идут запросы в базу, а для какого именно пользователя уже передается из других файлов перед выводом в шаблон, я нашел такой код

class banners {

// used in files which don't run the common() function from util.php but need auth information
function checkAuth() {

require_once $config_abs_path."/classes/auth.php");

	global $is_admin, $logged_in, $crt_usr;
	$auth=new auth();
	$logged_in = $auth->loggedIn();
	$is_admin = $auth->adminLoggedIn();
	if(!$logged_in && !$is_admin)
		$logged_in = $auth->checkCookieLogin();

	// vars for user account navbar
	if($logged_in) $crt_usr = $auth->crtUserId();

}

и весь код переложил в один класс banners

function addHit($id=0) {
		global $db,$config_abs_path;
		if(!$id) $id=$this->id;
		$res_del=$db->query('update '.TABLE_BANNERS.' set `clicks` = `clicks`+1 where `id`="'.$id.'"');

		require_once $config_abs_path."/classes/users.php";
		$email=users::getEmail($id);

		$email=$db->query("insert into `class_click` set `email` = '$email'");

		return 1;
	}

}

результат тот же самый, первый email из базы записывает, не подскажете как в addHit подключить авторизацию через функцию checkAuth?

если подключить ее, то должно все заработать, потому что в самом классе banners нет авторизации, так как он предназначен только для вывода баннеров, которым без разницы есть авторизация или нет

[viktorross]

AUser0, кстати, если вот сюда поставить соответствующий id в бд например 2, то в таблицу class_click запишется email из id 2, т.е все будет работать корректно

require_once $config_abs_path."/classes/users.php";
$email=users::getEmail($id='2');

[AUser0]

Ещё раз повторю, может наконец услышите: найдите в исходниках вызов функции addHit, и проверьте параметр, с которым она вызывается.

[viktorross]

AUser0, вот вызов функции addHit

case 'banner':
		require_once $config_abs_path."/classes/banners.php";
		$id = get_numeric_only("id");
		$banner=new banners();
		$banner->addHit($id);
	break;

[viktorross]

AUser0, сейчас посмотрел, действительно подставляется id баннера а не пользователя, как исправить понятия не имею, и там(users) и там(banners) все по id идентифицируется..

[AUser0]

Теперь найдите в других файлах определение ID залогиненного пользователя (видимо checkAuth() из приведённого кода выше), и используйте этот код для вызова addHit($crt_usr).

Либо используйте этот код внутри самой addHit(), если хотите, что бы учёт был только для залогиненного пользователя (т.е. вызов addHit() без указания ID как параметра).