"Я пойду!" 1.0b

хм... у нас будет схожая задачка. но для 5ки. гляну JS

1. t() в меню
в leoevents_menu() заголовок ("title") и описание ("description") в 6-ке не надо прогонять через ф-ию t(). Сразу пиши 'title' => 'Go' (http://api.drupal.org/api/function/hook_menu/6 -- "title": Required. The untranslated title of the menu item)

2. Регистрация тем
Темы в шестерке надо регистрировать через hook_theme() (http://api.drupal.org/api/function/hook_theme/6 -- там пример и все есть). Если при разработке модуля ты в leoevents_theme() чего-то поменял, то необходимо перезагрузить список модулей (обновить), чтобы тема инициализировалась и заработала. И потом использовать:

return theme('leoevents', $nid,$link_text);

3. Алиасы таблиц
Не критично, но вроде как принято использовать алиасы таблиц. Тобиш строчка 101:

$res = db_query('SELECT name, {leoevents}.uid FROM {users}, {leoevents} WHERE {users}.uid = {leoevents}.uid AND nid = %d',$nid);

$res = db_query('SELECT u.name, l.uid FROM {users} u, {leoevents} l WHERE u.uid = l.uid AND l.nid = %d',$nid);

4. Только активные пользователи
Смотри на запрос в пункте 3. Не забывай проверять, чтобы статус юзера был активным, иначе выведутся заблокированные пользователи:

$res = db_query('SELECT u.name, l.uid FROM {users} u, {leoevents} l WHERE u.uid = l.uid AND l.nid = %d AND u.status=1',$nid);

5. При удалении пользователя
При удалении ноды ты удаляешь запись в табличке leoevents. Не забывай удалять записи при удалении пользователя (http://api.drupal.org/api/function/hook_user/6)

6. Вопрос красоты.
Не забывай убирать последнюю запятую в списке пользователей идущих на событие... 103—106 строчки. Можно либо формировать массив с линками на юзеров, а после while-а делать implode(', ', $my_array). Либо после while-a вырезать последнюю запятую из строчки. Первый способ затратней по памяти.

7. Вопрос оптимизации.
Так или иначе, если у тебя будет пятьсот юзеров, идущих на событие, то в функции leoevents_who_go() алиас на юзерские пути запросится 500 раз. Раз уж друпал так тупит с путями, то мне нравится пользовать следующее.
СРАЗУ ГОВОРЮ ФУНКЦИЮ НЕ СИЛЬНО ТЕСТИРОВАЛ!!!! ОНА СЫРАЯ. ДАЖЕ ВОЗМОЖНО НЕРАБОЧАЯ. ВОТ.

/**
 * Вспомогательные константы для удобства передачи в функцию
 * массовой загрузки алиасов nedv_alias_loader().
 * Данные константы передаются первым параметром и служат для указания фунции:
 * осуществлять сбор путей без запроса к БД для получения алиаса (NEDV_COLLECT_PATHS),
 * прекратить сбор и вернуть список алиасов собранных путей (NEDV_RETURN_ALIASES).
 *
 * @see nedv_alias_loader()
 */
define('NEDV_COLLECT_PATHS', TRUE);
define('NEDV_RETURN_ALIASES', FALSE);
 
 
/**
 * Массовая загрузка алиасов.
 *
 * Для сокращения количества запросов к БД для получения алиасов для ссылок,
 * была создана функция массовой загрузки алиасов. Функция работает по следующей схеме:
 * при запуске ф-ии с $collect=TRUE все переданные пути собираются в массив; при
 * передаче параметра $collect=FALSE собранные пути разбиваются на группы по 100 штук.
 * Для каждой такой группы выдается запрос для получения алиасов.
 *
 * Полученные алиасы возвращаются функцией. Если для какого-то пути алиас не найден,
 * возвращается сам путь. Массив путей очищается.
 *
 * Важное замечание. В возращаемом массива алиасов полностью сохраняются
 * Ключи, указанные при
 *
 * @param $collect
 *   Флаг, позволяющий определить собирать ли алиасы или выдавать результаты.
 *   Для удобства определены константы NEDV_COLLECT_PATHS (TRUE), и NEDV_RETURN_ALIASES (FALSE).
 * @param $path
 *   Путь, для которого ищется алиас или массив путей array({ключ} => {путь}, ...)
 * @param $key
 *   Уникальный ключ для текущего пути.
 *
 * @return
 *   При $collect=TRUE функция ничего не возвращает, при $collect=FALSE, функция
 *   вернет массив полученных алиасов.
 *
 * @todo
 *   Возможно было бы полезно сделать некий ключ, по которому можно было бы
 *   собирать пути в несколько групп (как в пейджере).
 */
function nedv_alias_loader($collect, $path = NULL, $key = NULL) {
	// Массив собранных путей.
	static $collected_paths = array();
 
	// Обрабатываем переданные пути.
	if ($path !== NULL) {
		if (is_array($path)) {
			// array_merge позволит сохранить ключи
			$collected_paths = array_merge($collected_paths, $path);
		}
		elseif ($key === NULL) {
			$collected_paths[] = $path;
		}
		else {
			$collected_paths[$key] = $path;
		}
	}
 
	if ($collect == FALSE && (count($collected_paths) > 0)) {
		// Передан запрос на получение алиасов.
		$aliases = array();
		// Если  модуль path установлен, загружаем алиасы для тех путей,
		// для которых они существуют.
		if (module_exists('path')) {
			// Разбиваем запрос на части по 100 алиасов.
			$parts = array_chunk($collected_paths, 100, TRUE);
			foreach ($parts as $destinations_part) {
				$query = db_query("SELECT src, dst FROM {url_alias} WHERE src IN ('". implode("', '", $destinations_part) ."')");
				while ($alias = db_fetch_object($query)) {
					$aliases[$alias->src] = $alias->dst;
				}
			}
		}
		// Перебираем собранные пути, и для каждого пути либо ставим алиас,
		// либо оставляем путь, если алиас для него не найден.
		$paths = array();
		foreach ($collected_paths as $key => $path) {
    	$paths[$key] = (isset($aliases[$path])) ? $aliases[$path] : $path;
		}
		// Очищаем массив собранных путей.
		$collected_paths = array();
 
		return $paths;
	}
}

Как с ней работать.
Смотри на строчки 103 и ниже. Было:

...
 
while($data = db_fetch_object($res)){
  #$out .= $data->name;
  $out .= l($data->name,'user/'.$data->uid).', ';
}
 
...

Стало:

...
 
$anybody_goes = FALSE;
while($data = db_fetch_object($res)){
	// Ставим путь для странички юзера в очередь на получение алиаса пути
	nedv_alias_loader(NEDV_COLLECT_PATHS, 'user' . $data->uid, $data->name);
	$anybody_goes = TRUE;
}
if ($anybody_goes) {
	// Загружаем алиасы для ссылок на странички юзеров
	$user_links = nedv_alias_loader(NEDV_RETURN_ALIASES);
	if (is_array($user_links) && (($user_cnt = count($user_links)) > 0)) {
		$i = 1;
		foreach ($user_links as $user_name => $user_link){
			$out .= l($user_name, $user_link, array('alias' => TRUE));
			$out .= ($i != $user_cnt) ? ', ' : '';
			$i++;
		}
	}
 
}
 
...

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

8. Подгрузка JS
Еще меня смущает leoevents_html()
Я МОГУ ОШИБАТЬСЯ!!!! Вроде как во views можно выводить такую страничку, на которой распечатываются несколько целых нод одновременно (не тизеры, а именно целые ноды).
Так вот, для каждой такой ноды будет вызываться leoevents_html() => drupal_add_js(drupal_get_path('m... будет вызываться столько же раз. И получится в коде странички несколько раз будет инклюдится твой JS-файл.
Как этого избежать:

function leoevents_html($nid, $teaser){
	static $js_connected = FALSE;
	if (!js_connected) {
		drupal_add_js(drupal_get_path('module', 'leoevents') .'/leoevents.js');
		$js_connected = TRUE;
	}
...

Еще раз скажу, могу ошибаться.

писал все на глаз. без тестов. могут быть ошибки

название ф-ии подгрузки алиаса и констант начинается с nedv... это я просто скопирнул из кода своего модуля... меняй как хочешь.

А... еще) называй ф-ии человекопонятными названиями (leoevents_alrd_go()...) ну и по-английски писать правильно бы надо... (вроде пишется "who_goes")... у самого с этим проблемы и поэтому уйма времени уходит на поиск перевода слов в словарике :((

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

По комментарю zhylik:

1,3 - исправил

2 Кроме theme('leoevents', $nid,$link_text) пришлось добавить такой код:

function leoevents_theme() {
  return array(
    'leoevents' => array(
      'arguments' => array('nid', 'link_text'),
    ),
  );
}

6 Сделал так

  if($out){
    return drupal_substr($out, 0,drupal_strlen($out)-2);
  }else{
    return t('None');
  }

8 Вывел в тизер аттачит яву вроде один раз, оставил как было, но заметил глюк, что если щелкнуть на одной ссылке, ява меняла текст для всех элементов на странице. Исправил, теперь и когда много документов все должно быть нормально. Теперь можно выводить в тизерах. Для этого меняем 57 строку.

7. Вопрос оптимизации
Не совсем понял.
Алиасы - это синомы путей, так? Но я же вроде их нигде не использую. Или функция l() перехватывается модулем path, тогда может проще и быстрее ссылку вручную создавать? Это получается самый быстр вариант с точки зрения производительности.

7. Да. Но, тогда пользователь не сможет корректно добавить алиас на такие пути. Т.е. добавить-то он добавит, но при выводе страничек href в твоих ссылках останется исходным. Все модули для друпала must поддерживать алиасы.

9. Потом (вообще на глаз говорю...) отключи JS в броузере и попробуй добавить юзера к событию. Думаю ссылка сработает и юзер добавится, но откроется какая-то бредовая страница... с содержанием вида "{ "link_text": "I will go!", "who_go": " " }"
а правильно было бы делать редирект на ноду обратно и выводить зелененькое сообщение, о том, что вы добавлены.

реализацию подобного можно посмотреть на примере закладок на этом сайте.

спасибо, полезно

а для 5-ой версии работает?

нет, на 5ку не расчитан

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

Для себя я полностью отказался от алиасов, если они нужны правим 123 строку:

function leoevents_ulink($name,$uid){
  return '<a href="/user/'.$uid.'">'.$name.'</a>';//l($name,'user/'.$uid);    
}

При отключении ява скрипт юзер будет редиректится на ноду события.

    drupal_goto('node/'.$nid, NULL, NULL, 302);

Спасибо, пригодилось. Кстати, почему в комплекте нет перевода на русский?

перевод надо будет добавить

а пока добавил кеширование списка идущих на мероприятие

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

Прикрепляю версию без страницы с настройками, но с исправленным .info В нём добавлены зависимость от event и текстовое описание. ru.po прилагается.

UPD variable_get() используется, значит должна быть страница с настройками.

спасибо, variable_get() и leoevents_settings() сделаны с расчетом на будущее, сейчас не нужны

Здорово... У меня возникло пара вопросов как у потенциального пользователя модуля.

1. Тип события event задаётся каким-то другим модулем или его нужно создавать самостоятельно?
2. Не хотите ли Вы зарегистрировать модуль на drupal.org, чтобы предоставить доступ к нему большему числу пользователей и получить советы/помощь в доведении его до стабильной версии?

1. Создаешь самостаятельно или переименовываешь любой существующий или в модуле меняешь так чтоб он к какому-то другому цеплялся.

2. Я думал об этом - можно. Но мне бы хотелось как-нибудь узнать статистику скачек модуля - насколько он популярен

и если стоит выкладывать, то какие есть предложения по названию модуля? как я понял своим именем называть не принято, а что-то короткое, чтобы отражало суть мне пока в голову не приходит (только "go_list" или "participates_list")

1. Тогда имеет смысл добавить в настройки модуля тип контента, к которому он цепляется. Так, ИМХО, удобнее. Или это и так есть в Road map?

2. Я думаю, он будет довольно популярен. Я не проверял, есть ли аналогичные модули в репозитории, но такая фишка, на мой взгляд, может понадобится многим. В качестве названия можно использовать что-нибудь вроде attendance.

3. Я планирую использовать использовать Ваш модуль для события, дата которого будет время от времени меняться. Это будет что-то вроде еженедельного сбора единомышленников, поэтому хочется привязать каждый из этих сборов к одной ноде, а раз в неделю указывать новую дату и время собрания. Можно ли будет добавить сброс числа посетителей после наступления даты и времени события?

могу предложить название "go2event"
А на drupal.org как раз и будет статистика.

тут пришла мысль - "takeapart" или "takepart" помоему лучше отражает суть

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

по первому пункту, конечно, прежде чем на д.о выкладывать сделаю возможность хоть каких-то настроек из админки :)

По названию — уже лучше какой-нибудь «areUgoing» -)

По третьему пункту. ОК. Я думаю, что я либо буду делать это сниппетом, либо попробую «дорасти» до того, чтобы написать и предложить такую функцию в качестве опции к вашему модулю -)

В тему запросов фич. Странно выглядит - событие в архиве, но записаться можно. Можно менять "уже идут" на "туда пошли" когда event (из модуля) становится прошедшим и выключать "я пойду"?

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

но! под конкретные события (клубные вечеринки или музейные выставки) могут требовать еще свои поля. ССК может добавлять поля к типу материала созданного модулем свои поля? просто у меня он не стоит не могу посмотреть. Или может сразу интегрировать его с ССК расчитывая только на совместное использование и соотвествующим полем даты созданным через ССК?

имелся ввиду конкретный  event

fasdalf@fasdalf.ru
На last.fm подобное реализовано таким образом, что на событие в прошлом тоже можно «пойти», но в формате «я ходил». А фраза «туда идут» как раз меняется на «там были».

PS Ещё там есть вариант «я собираюсь пойти»/«я думал сходить» -)

nleo
Да, CCK может добавлять свои поля к материалу, созданному модулем. Во всяком случае, я нигде не читал обратных утверждений. А интегрировать Ваш модуль с CCK мне кажется неразумным — получается, что желание реализовать небольшую и лёгкую функцию будет тянуть зависимость от тяжёлого CCK, что не есть гуд.

«я ходил»
Хорошая мысль. «Всё уже придумано до нас» (с)

будет тянуть зависимость
Ну, есть же if (module_exist('content')) {};
Я в том смысле, что поддержка CCK и зависимость от него - не одно и то же.
Но в этом случае без админской панельки уже не обойтись - и типы материалос искать, и дату окончания из event/cck выбирать. Уже "nice tiny module" вырастет в убер-монстра.

Я же комментировал конкретную фразу: «Или может сразу интегрировать его с ССК расчитывая только на совместное использование» ;)

По-моему, это ещё не монстр получается, зато модуль, которым удобно управлять и настраивать.

Итак релизю версию 1.0b

Добавил настройки в админке: к каким нодам привязывать и где показывать (при просмотре документа или в тизере и при просмотре). Поумолчанию показывается и в тизере и в ноде, но не привязан ни к какому типу документов.

Теперь буду думать над второй версией интегрированной с модулем Event. Идея "Я ходил" мне тоже понравилась.

и сразу версия 1.1b с поддержкой модуля Event

Новое:
В админке можно включить поддержку модуля Event
Переделано кеширование, теперь весь вывод модуля кешируется, а не только список идущих

Проблемы:
1. Никаких проверок на наличие модуля Event
2. Когда время события наступает, кешь не сбрасывается, поэтому можно наблюдать надписи в будущем времени, до тех пор пока кто-нибудь не нажмет "Я пойду" или вручную не будет сброшен кеш.
3. Кажется я забыл прошедшие времена английского, не уверен, что правильно их написал:

Основные изменения в коде:

Файл перевода не дополнял, поскольку возможно исходные строки еще изменятся

спасибо!!! пора и на drupal.org!!!

Отличный и полезный модуль,наши девелоперы радуют

Отлично!

1.

Как я понимаю, нужно просто добавить следующее условие:
if (module_exist('event'))
3.

Мне кажется, правильнее переделать фразы так:

Как я понимаю, в шестом Друпале запросы к базе принято оформлять несколько иначе — вместо «DELETE FROM {leoevents} WHERE uid = %d AND nid = %d» нужно писать «DELETE FROM {leoevents} l WHERE uid = %d AND nid = %d», например.

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

Вот пример из книги Pro Drupal development

у меня:

ничего не понимаю

Я пока не тестил, ибо у меня даже типа содержимого пока для событий не предусмотрено.

Может имеет смысл variable_get() вынести в отдельную переменную и, аналогично примеру, «пойти от противного»?

в общем для версий 1.0b и 1.1b

надо в leoevents_nodeapi() заменить строчку

похоже книге нельзя верить)

пробовал - от перестановки мест слагаемых...

Может быть это из-за того, что книга о пятом Друпале, а это шестой?

если бы, 2th edition о шестом. там даже описана структура массива типов нод для шестого, а код как в книжке про 5. но я правда ее внимательно не читал, я больше по русской книжке про 5й, а с этой сверяюсь когда что-то не получается

Хм... Перевод книжки о пятом, кстати, кривой до безобразия. Я дошёл до 15 главы пока что, так вот, даже не имея никаких знаний о Друпале, можно найти кучу ошибок -(( А книжка про шестой у вас в электронном виде или в печатном, если не секрет?

Хм... А зачем там array_flip?

это более удобный аналог http://drupal.org/project/signup ? Или в чёмто есть принципиальные отличия? Или это вообще для тренировки?

электронный

она меняет местами ключи и значения хэша, зачем? это станет ясно если взглянуть на структуру массива, который возвращает variable_get('leoevents_nodetypes', array())
'page' => 0, 'event' => 'event', 'story' => 0

мне просто понадобились списки идущих и я их написал про signup я не знал. там конечно гораздо больше функций, модуль на 118 килобайт против 4 у меня) (в запакованом виде) Цель потренироваться тоже стояла, но она вторична

Где-то тут уже видел: человек пишет модуль, выкладывает, людям нравится: "да такой модуль нужен!", потом оказывается на д.о уже есть подобный, но с большим функционалом)

для моих целей мой меня пока вполне устраивает, но singup тоже гляну, спасибо, что сказали

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

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

Почему бы тогда просто не подставлять в качестве ключа $node->type и не проверять TRUE он или нет? Я пока плохо знаком с синтаксисом php, но логика такова:
if leoevents_nodetypes[$node->type] != 0 {}

А не могли бы Вы подкинуть ссылку, где можно скачать книжку, или её саму на мыло <мой ник на яндексе>?

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

книжка в пути

;) Будем считать это моим первым вкладом в сообщество разработчиков Друпал ;-)

Я перестраховался с «!= 0» на тот случай, если php делает различие между NULL и 0, потому что тогда if ($node_types[$node->type]) будет давать TRUE для всех типов, вне зависимости от того, прописано для них 0 или 1.

PS Большое спасибо за книжку!

У меня почему-то отпал AJAX при обновлении до 1.1Б. На On Click ничего не висит, JS включен (gmap работает)
У текста "i did go" нет воскл. знака.
настройки по типам материалов модно встраивать в /admin/content/node-type/%name% (но мне удобнее как сейчас)

предлагаю проверить подключается ли JS файл, и сравнить id и class элементов в документе и в JS файле

лаги с кешем обнаружил, откатываю способ кеширования как было в предидущих версиях. обновление после нового года будет

UPD: или не с кешем, что-то непонятное. но это уже не сегодня

Интересная тема, буду следить за развитием, спасибо за модуль.