Advertisement Management - Рекламный менеджер

[Alecto]

есть кто живой?

[Anvar]

Решение с группами.
Удалено! Код испорчен при отправке (Quick Reply Reloaded глючит).

Отправлено спустя 40 минут 11 секунд:
Проблема с модулем, необходимо заменить изменённые файлы из https://github.com/bb3mobi/ads , папки acp и migration, после чего выбрать модуль в админке.

Архив в первом сообщении заменю позже, может быть ещё поработаю над расширением.

[Alecto]

нет, не работает.
пользователь в группе НОВЫЕ по умолчанию, стоит не показывать новым - он не видит, меня группу по умолчанию на Зарегистрированные - он по прежнему не видит блок.
чертовщина стала твориться.
пока не снял все исключение - площадка вообще не отображалась.

Отправлено спустя 4 минуты 32 секунды:

необходимо заменить изменённые файлы

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

[Anvar]

Можно не выключать, потом кэш почистить.

[Alecto]

выключил, заменил две папки, включил, обновил кэш.
при попытке зайти в адмке в менеджер рекламы:

Код: Выделить всё

Общая ошибка
Не найден модуль ./../includes/acp/\bb3mobi\ads\acp\acp_ads.php

BACKTRACE

FILE: (not given by php)
LINE: (not given by php)
CALL: msg_handler()

FILE: [ROOT]/includes/functions_module.php
LINE: 571
CALL: trigger_error()

FILE: [ROOT]/adm/index.php
LINE: 81
CALL: p_master->load_active()

Отправлено спустя 3 минуты 6 секунд:
пролечилось через систему--управление модулями, выбрал модуль.

[Anvar]

Да там старый был прописан в базе, по этому его нужно было обновить средствами ACP.

[Alecto]

с кодом для групп что-то получится?

и еще, сделал себе, возможно идея понравится...
стандартные площадки решил не трогать, может пригодятся, но у меня добавляется 8 площадок: 4 вверху и 4 внизу.
соответственно и без того не маленькая таблица становится еще больше и до рекламных блоков прокручиваться становится не удобно.
т.е. имеем вверху страницы площадки, которые настраиваются единожды и забываются, а рекламные блоки, с которыми приходится достаточно часто работать - задвинуты вниз.
нашел следующий выход... сделал слайдер для этой таблицы:

найти

Код: Выделить всё

<!-- IF S_POSITION_LIST -->

добавить после

Код: Выделить всё

	<a id="ads_positions_Link" onclick="toggleTable();" href="#">Показать/Скрыть таблицу положений</a>

	<script>
  function toggleTable()
    {

    var status = document.getElementById("ads_positions").style.display;

    if (status == 'block') {
      document.getElementById("ads_positions").style.display="none";
    } else {
      document.getElementById("ads_positions").style.display="block";
    }
  }
	</script>

<div id="ads_positions" style="display: none;">

найти

Код: Выделить всё

			<td colspan="5" class="row3">{L_NO_POSITIONS_CREATED}</td>
		</tr>
	<!-- END positions -->
	</table>

добавить в новую строку после

Код: Выделить всё

</div>

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

в общем случае, можно добавить языковую переменную, или что-то интернациональное типа POS+/-, но в частном - не стал этого делать.
еще как вариант - можно попробовать поменять местами рекламные позиции и баннеры, но при свертке страница все таки намного компактней.
идеальным решением было бы подключение jquery-слайдера для той же цели.

[Anvar]

Можно будет дополнить, действительно не удобно там сделано. Хотя наверное {ADS_1} и тд, может быть заменить на название событий получится.

С группами пока нет времени тестировать, чуть позже посмотрю. По умолчанию перебираются все группы пользователей в которых он состоит.

[Alecto]

логично, но там где-то ошибка.
по крайней мере, я исключил из показа АДМИНИСТРАТОРЫ, СУПЕРМОДЕРАТОРЫ, НОВЫЕ ПОЛЬЗОВАТЕЛИ.
пользователь состоит в группе НОВЫЕ - поумолчанию и ЗАРЕГИСТРИРОВАННЫЕ.
так вот, он вообще перестал видеть рекламный блок, по я не убрал все исключения.
а надо, чтобы пользователь не видел, только пока НОВЫЕ по умолчанию, т.е. ЗАРЕГИСТРИРОВАННЫЕ становится поумолчанию - он должен увидеть рекламу.

может быть пример не совсем удачный, поскольку группа НОВЫЕ удаляется у пользователя, но есть множество других аналогичных случаев, где доп.группы расширяют права пользователей, а при существующем подходе управление рекламой для групп невозможно.

Отправлено спустя 2 минуты 19 секунд:

Хотя наверное {ADS_1} и тд, может быть заменить на название событий получится.

не понял, зачем? есть Имя области показа, есть Код области показа.
Имя используется человеком для выбора положения рекламы, а код - для идентификации в шаблоне. Все отлично, удобно и замечательно.
Я полностью переписал шаблоны под свои площадки, а неиспользуемые - удалил, т.е. мне без отображения {ADS_ в админке - никак нельзя.
те же, кто будут использовать вариант "из коробки" - им должно быть это безразлично, а захотят полезть в шаблоны - опять же, пригодится.

[Anvar]

Я думал сделать так: Создаёшь категорию и в новом поле вписываешь событие шаблона, после этого код будет отображаться в том месте где это событие расположено. Лезть уже самому в шаблоны и добавлять код не потребуется.

Хотя как по мне, так чем меньше работы - тем лучше))) И вообще по умолчанию наверное можно исключить сайдбары и тд., которые просили добавить.