Comments 29
Отличный пример написания модуля для ССК. Спасибо.
Изобретаете SWF Tools, или я что-то не так понял?
Далее будет много кода, мало картинок и вообще всё скучно и уныло...
Ну почему скучно, вон какие буквы разноцветные, красиво.
У меня используется rand(1, 10000). Если есть более адекватный способ, буду рад услышать его в комментариях.
Более адекватный — rand(1, 99999999)
Тогда уж
Но можно и что-нибудь более экзотическое сделать, например
mt_rand(1, PHP_MAX_INT)Но можно и что-нибудь более экзотическое сделать, например
substr(md5(uniqid('', true)), 0, 10); =)
static $swfobject_id = 1;
И вообще надо юзать SWFObject API (лінк на drupal.org/project/swfobject_api )
6.x-1.0-beta1
Кто вам сказал, что надо его использовать?
Кто вам сказал, что надо его использовать?
И что с того, что beta — многие хорошие модули доступны в dev только, но работают на ура. А надо — мы ж не велосипеды пишем. Есть модуль для вывода swfobject кода. Надо его юзать. Идеология CMS + унификация кода. Вы же не оспариваете CCK :)
Ну и акцент был на static переменную, а то rand(0, 1000) — это тот еще изврат.
Ну и акцент был на static переменную, а то rand(0, 1000) — это тот еще изврат.
Рисунок к посту сделан кровью? )))
The Best CMS in the world.
слишком много писать, поэтому просто дам ссылку
drupal.org/coding-standards
drupal.org/coding-standards
Да, да, да…
Есть еще вот такие стандарты. Не придирайтесь.
Есть еще вот такие стандарты. Не придирайтесь.
А еще, можно переложить вывод поля на шаблон еще проще будет.
можно еще TextField+TinyMCE c кнопкой Embedd Media
Зачем нужна проверка на включенный модуль filefield? Он у вас в зависимостях, и должен быть включен при нормальной работе системы.
К тому же не забываем про хук api.drupal.ru/api/function/hook_requirements/6 — он как раз нужен для того, для чего вы использовали hook_init.
Оформите пожалуйста код в соответствии с соглашениями принятыми в Drupal.
К тому же не забываем про хук api.drupal.ru/api/function/hook_requirements/6 — он как раз нужен для того, для чего вы использовали hook_init.
Оформите пожалуйста код в соответствии с соглашениями принятыми в Drupal.
Да вообще то, учитывая стандарты написания zend у него все нормально. А у друпы есть ошибки кстати. Когда копировали они стандарты к себе, то оцепятались, сравни-увидишь.
Читаем цели создания стандартов zend (http://zendframework.com/manual/ru/coding-standard.overview.html):
Я бы не назвал это целями, ну да ладно. Суть такова — у большого проекта должен быть стандарт. Соглашение от zend хорошо, но сообщество друпал выработало свой стандарт и уж если пишешь под друпал, то надо его соблюдать.
Хороший стандарт кодирования важен в любом проекте, и особенно там, где множество разработчиков работают над одним проектом. Наличие стандарта кодирования помогает гарантировать, что код высокого качества, с меньшим количеством ошибок, и легко поддерживается.
Я бы не назвал это целями, ну да ладно. Суть такова — у большого проекта должен быть стандарт. Соглашение от zend хорошо, но сообщество друпал выработало свой стандарт и уж если пишешь под друпал, то надо его соблюдать.
да нет своего стандарта. есть стандарт только разработчика языка. А то что сообщество требует придерживаться этого стандарта- нет друпал стандарта есть только зенда-- то это ж хорошо.
Да нет никакого «стандарта разработчика языка». Есть соглашения внутри проекта.
Если один разработчик предпочитает табы, другой — пробелы, то внешний вид кода этих двоих оставляет желать лучшего.
Если один разработчик предпочитает табы, другой — пробелы, то внешний вид кода этих двоих оставляет желать лучшего.
zendframework.com/manual/ru/coding-standard.overview.html я и не знал что в переводе это означает соглашение. Кстати не ты ли в свое время писал на друп ру про то как кто то в друпе использует на каждом шагу лишние?
Да, проверка в модуле на включенный модуль filefield уже лишняя, раз filefield описан в зависимостях в info файле.
function theme_swffield_flash(){
//…
}
я бы вынес генерацию html в темплейт. думаю, так правильнее будет.
//…
}
я бы вынес генерацию html в темплейт. думаю, так правильнее будет.
Sign up to leave a comment.
Работа с CCK Filefield. Вставка и отображение Flash