Простая ошибка в php?

Question

[Artemed]

<?php 
$query = array();
if (isset($_GET['utm_source'])) {
  $query['subid1']  = $_GET['utm_source'];
}
if (isset($_GET['utm_campaign'])) {
  $query['subid2']  = $_GET['utm_campaign'];
}
if (isset($_GET['utm_content'])) {
  $query['subid3']  = $_GET['utm_content'];
}
if (isset($_GET['utm_term'])) {
  $query['subid4']  = $_GET['utm_term'];
}
?>
$data = drupal_http_build_query($query);
$params = !empty($data) ? '?' . $data : '';
}
<iframe id="ifr" src="https://yandex.ru/<?php print $query; ?>" width="100%" height=100% scrolling="yes" align="left" frameborder=0>
</iframe>
?>
</html>

Comments

[Максим Федоров]

?> не там закрывает код и после хтмл кода еще один стоит зачем-то
} стоит лишний

[Максим Федоров]

<?php 
    $query = array();
    if (isset($_GET['utm_source'])) {
        $query['subid1']  = $_GET['utm_source'];
    }
    if (isset($_GET['utm_campaign'])) {
        $query['subid2']  = $_GET['utm_campaign'];
    }
    if (isset($_GET['utm_content'])) {
        $query['subid3']  = $_GET['utm_content'];
    }
    if (isset($_GET['utm_term'])) {
        $query['subid4']  = $_GET['utm_term'];
    }
 
    $data = drupal_http_build_query($query);
    $params = !empty($data) ? '?' . $data : '';

?>



<iframe id="ifr" src="https://yandex.ru/<?php print $query; ?>" width="100%" height=100% scrolling="yes" align="left" frameborder=0>
</iframe>
</html>

[Artemed]

Максим Федоров, помогите, пожалуйста, я заплачУ!

У меня не друпал. Нашел этот код в интернете. Соответственно эта строчка не актуальна.
$data = drupal_http_build_query($query);
$params = !empty($data) ? '?' . $data : '';

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

Схема:
Яндекс.директ -----> Витрина ------> партнерский магазин

[Максим Тимофеев]

Artemed, Вы взяли кусок коды, вырвали из проекта и удивляетесь что оно не работает? А Вас не смущает, что если вытянуть батарейки из машинки они перестают ездить?
Как минимум стоит прочитать для чего существует этот кусок и что он делает:
https://api.drupal.org/api/drupal/includes%21commo...
Ведь там же простой php, бери и копируй.

Answer 1

[fans74]

здесь вы пытаетесь вывести строку:
<?php print $query; ?>

но $query у вас не строка, а массив:
$query = array();

Answer 2

[Fwwll]

а лишние
?>
не мешают?

Answer 3

[Денис Евра]

Учти ответы других ораторов, про лишние ?> и }

А так у тебя, по коду вижу, проблема в том, что ты $query скармливаешь функции drupal_http_build_query() и потом формируешь $params, однако в iframe инклюдишь уже неактуальный $query. У тебя там иклюдится должно вот так: src="https://yandex.ru/<?=$params?>"

Comments

[Денис Евра]

А, у вас не drupal? Тогда тип того:

<?php
$query = array();
if (isset($_GET['utm_source'])) {
  $query['subid1']  = $_GET['utm_source'];
}
if (isset($_GET['utm_campaign'])) {
  $query['subid2']  = $_GET['utm_campaign'];
}
if (isset($_GET['utm_content'])) {
  $query['subid3']  = $_GET['utm_content'];
}
if (isset($_GET['utm_term'])) {
  $query['subid4']  = $_GET['utm_term'];
}

$params = '?';
foreach ($query as $key => $param) {
    $params .= $key.'='.$param.'&';
}
$params = substr($params, 0, -1);

?>

<iframe id="ifr" src="https://yandex.ru/<?=$params?>" width="100%" height=100% scrolling="yes" align="left" frameborder=0>
</iframe>

</html>