Как вызвать функцию через Ajax?

Question

[smoky-jo]

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

function clickBanner(c, b) {
  url_str = b + "/include/get_info.php?type=banner&id=" + c;
  $.ajax({
    url: url_str,
    success: function(d) {}
  })
}

есть вот такой onclick в шаблоне

<div onclick="clickBanner({$v.id},'{$live_site}');">

и код php

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

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

case 'user_q':
    require_once $config_abs_path."/classes/users.php";
    $id = get_numeric_only("id");
    $click_r=new users();
    $click_r->addEmail($id);
  break;

в файле users

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

как правильно сделать вызов функции при нажатии на div чтобы это сработало?

Answer 1

[AUser0]

function clickBanner(c, b) {
  url_str = b + "/include/get_info.php";
  $.ajax({
    method: "GET",
    url: url_str,
    data: { 'type': "banner", 'id': c },
    success: function(d) {}
  })
}

Только у вас всё та же проблема: в get_info.php передаёте ID баннера, а записывать в базу нужно email кликнувшего баннер пользователя (через addHit())...

Comments

[smoky-jo]

AUser0 , с тем вариантом я все еще пытаюсь разобраться, но пока не выходит, тот кто делал намудрил, мне сложно разобраться в этом, думаю создать еще один onclick

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

function clickAddEmail(c, b) {
  url_str = b + "/include/get_info.php";
  $.ajax({
    method: "GET",
    url: url_str,
    data: { 'type': "user_q", 'id': c },
    success: function(d) {}
  })
}

<div onclick="clickAddEmail({$id},'{$live_site}');">

case 'user_q':
    require_once $config_abs_path."/classes/users.php";
    $id = get_numeric_only("id");
    $click_r=new users();
    $click_r->addEmail($id);
  break;

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

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

[AUser0]

Сработает, но добавит email непонятного пользователя. Вам надо вместо кода $id = get_numeric_only("id"); вставлять другой код, который даст вам ID текущего пользователя, а не ID кликнутого баннера.

[smoky-jo]

AUser0 , нет, там же уже подключается не banners.php, а users.php..

смотрите как сделал

добавил функцию получения id пользователя

function userIdget($id) {

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

				}

добавил вывод id текущего пользователя в шаблон

require_once $config_abs_path.'/classes/users.php';
		global $logged_in,$crt_usr;
		$usr = new users();

		$get_id_usr = $usr->userIdget($crt_usr);
		$smarty->assign("get_id_usr", $get_id_usr);

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

<div onclick="clickAddEmail({if $get_id_usr}{$get_id_usr}{/if},'{$live_site}');">

js

<script>
	function clickAddEmail(c, b) {
	  url_str = b + "/include/get_info.php";
	  $.ajax({
	    method: "GET",
	    url: url_str,
	    data: { 'type': "user_q", 'id': c },
	    success: function(d) {}
	  })
	}
	</script>

в users.php записал email

function addEmail($id='') {
			global $db;
				if(!$id) return;
			$email=users::getEmail($id);
			$email=$db->query("insert into `class_click` set `email` = '$email'");
			}

и добавил это после авторизации в get_info

$auth = new auth();
		$user_id = $auth->crtUserId();
		$user = new users;
		$group = $user->getGroup($user_id);

		case 'user_q':
		    require_once $config_abs_path."/classes/users.php";
		    $id = get_numeric_only("id");
		    $click_r=new users();
		    $click_r->addEmail($id);
		  break;

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

спасибо большое за помощь))

[AUser0]

Ой, блииииин, какая же...

Вам надо это переписывать, что бы ID пользователя вообще не передавался в AJAX.
Потому что иначе можно будет заспамить систему простыми запросами:
site.org/get_info.php?type=user_q&id=1
site.org/get_info.php?type=user_q&id=2
..........
site.org/get_info.php?type=user_q&id=9999999998
site.org/get_info.php?type=user_q&id=9999999999

И вы не сможете никак с этим бороться, потому что
просто
тупо
дырища!

[smoky-jo]

AUser0, а как можно в этом Ajax запросе очистить cookies в конце запроса? у меня почему-то пока не очищу cookies в браузере не добавляется вторая запись с одним и тем же email

[AUser0]

Плюс подумайте об не авторизованном пользователе, какой у вас получится сбойный JS-код на баннере?

[smoky-jo]

AUser0, попробовал из браузера запустить site.org/get_info.php?type=user_q&id=9999999998

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

а если не авторизован то просто 0 записать или если без else то новая запись вообще не создастся в бд

{if $get_id_usr}{$get_id_usr}{else}0{/if}

[AUser0]

попробовал из браузера запустить site.org/get_info.php....

Надеюсь вы догадались заменить адрес сайта на свой, и прописать правильный путь к нужному (get_info.php) файлу? Или тупо открывали ссылку так, как её я для примера написал?

А {else} - да, решит проблему.