Работа с CCK Filefield. Вставка и отображение Flash

[stalkers]

Отличный пример написания модуля для ССК. Спасибо.

[krig]

Изобретаете SWF Tools, или я что-то не так понял?

[Anonym]

SWFtools мне показался монстроподобным. У меня не было цели написать конкретно модуль вставки флеша, просто как пример он отлично подошел. Я описывал общий алгоритм добавления своих форматтеров.

[spmbt]

Далее будет много кода, мало картинок и вообще всё скучно и уныло...

Ну почему скучно, вон какие буквы разноцветные, красиво.

У меня используется rand(1, 10000). Если есть более адекватный способ, буду рад услышать его в комментариях.

Более адекватный — rand(1, 99999999)

[krig]

Тогда уж mt_rand(1, PHP_MAX_INT)

Но можно и что-нибудь более экзотическое сделать, например substr(md5(uniqid('', true)), 0, 10); =)

[Anonym]

Такая экзотика только процессор нагружает. Сгенерировать уникальный ID можно и просто рандомом, к тому же, я не думаю, что на странице будет настолько много полей одного типа, чтобы rand(1, 10000) сгенерировал 2 одинаковых значения.

[krig]

Можно встроенными срудствами друпала — user_password(10)

[RedHead]

static $swfobject_id = 1;


И вообще надо юзать SWFObject API (лінк на drupal.org/project/swfobject_api )

[Anonym]

6.x-1.0-beta1
Кто вам сказал, что надо его использовать?

[RedHead]

И что с того, что beta — многие хорошие модули доступны в dev только, но работают на ура. А надо — мы ж не велосипеды пишем. Есть модуль для вывода swfobject кода. Надо его юзать. Идеология CMS + унификация кода. Вы же не оспариваете CCK :)

Ну и акцент был на static переменную, а то rand(0, 1000) — это тот еще изврат.

[KEP]

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

[ferrari]

The Best CMS in the world.

[krig]

All CMS suck, Drupal just happens to suck less

[ferrari]

черт. прошу прощения, забыл:
— троллим под этой чертой ----------------

[cyberpank]

слишком много писать, поэтому просто дам ссылку
drupal.org/coding-standards

[Anonym]

Да, да, да…
Есть еще вот такие стандарты. Не придирайтесь.

[lasc]

А еще, можно переложить вывод поля на шаблон еще проще будет.

[Anonym]

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

[anreyyyy]

можно еще TextField+TinyMCE c кнопкой Embedd Media

[Anonym]

Я TinyMCE не пользуюсь.

[danSamara]

Зачем нужна проверка на включенный модуль filefield? Он у вас в зависимостях, и должен быть включен при нормальной работе системы.
К тому же не забываем про хук api.drupal.ru/api/function/hook_requirements/6 — он как раз нужен для того, для чего вы использовали hook_init.

Оформите пожалуйста код в соответствии с соглашениями принятыми в Drupal.

[shamaner]

Да вообще то, учитывая стандарты написания zend у него все нормально. А у друпы есть ошибки кстати. Когда копировали они стандарты к себе, то оцепятались, сравни-увидишь.

[danSamara]

Читаем цели создания стандартов zend (http://zendframework.com/manual/ru/coding-standard.overview.html):

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

Я бы не назвал это целями, ну да ладно. Суть такова — у большого проекта должен быть стандарт. Соглашение от zend хорошо, но сообщество друпал выработало свой стандарт и уж если пишешь под друпал, то надо его соблюдать.

[shamaner]

да нет своего стандарта. есть стандарт только разработчика языка. А то что сообщество требует придерживаться этого стандарта- нет друпал стандарта есть только зенда-- то это ж хорошо.

[danSamara]

Да нет никакого «стандарта разработчика языка». Есть соглашения внутри проекта.
Если один разработчик предпочитает табы, другой — пробелы, то внешний вид кода этих двоих оставляет желать лучшего.

[shamaner]

zendframework.com/manual/ru/coding-standard.overview.html я и не знал что в переводе это означает соглашение. Кстати не ты ли в свое время писал на друп ру про то как кто то в друпе использует на каждом шагу лишние?

[shamaner]

лишних php дескрипторов

[Xenviper]

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

[brmn]

function theme_swffield_flash(){
//…
}
я бы вынес генерацию html в темплейт. думаю, так правильнее будет.