fairrandir: Комментарии

Главные вкладки

6 сентября 2018 в 16:48

Проверка isset($stars) бессмысленна, ибо значение вы получаете через variable_get, у variable_get второй аргумент - значение по умолчанию, в вашем случае - 5. То есть $stars всегда будет или значением variable, или 5.

5 сентября 2018 в 15:49

merlin wrote:

Лучше удалить

Как влияет на производительность - лучше проверить. Это раз.
Поля, которые не используются действительно лучше удалять, дабы не было путаницы. Это два.
Зачатую во вьюхе лучше выводить не поля, а отрендеренную ноду в определенном вьюмоде, ибо если надо будет добавить вывод во вьюху это надо будет сделать только во вьюмоде, а не во всех вьюхах.

5 сентября 2018 в 15:22
1

Смысл - давать доступ к файлам только при соблюдении определенных условий, например - только собственные файлы пользователя, скачивание файла только после оплаты доступа или заполнения формы, ещё что-то. Как-то так.

4 сентября 2018 в 9:08

kolotunbobo wrote:
Написано в теме [Решено]. А ведь с ошибками проблему товарищ не решил.

Приведенный вами фрагмент кода тоже не решает проблемы с ошибками. Зачем там два define?

27 августа 2018 в 16:00

Тут вообще весь код непонятно как должен работать. Переменная $city_result не используется и затирается новым значением сразу после создания, а в settings каждый проход значение перезаписывается.

23 августа 2018 в 15:49

Аналогия:
Я использую на проекте гайки с левой резьбой. Везде всегда правая, а мне вот удобнее с левой. Часто ли вы встречаете, что на проект приходят люди, у которых все инструменты и документация на гайки с правой?

Правильный ответ - нечасто, но достаточно одного раза, чтобы перестать так делать.

23 августа 2018 в 14:44

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

23 августа 2018 в 13:47

VasyOK wrote:

Предполагается, что на сайте разработчики русско-понимающие.

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

22 августа 2018 в 15:36

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

22 августа 2018 в 13:18

Правильно ли я понял, что при появлении поста в инстаграме нужно создавать ноду на сайте? Тогда скорее всего понадобится самописный модулёк, который тупо по крону будет дёргать API инстаграма и создавать ноду, если появились новые посты.

17 августа 2018 в 15:14
1

Очень удобно.

Чтобы пользователь сайта мог узнать об изменении правил ему всего-то надо, что зайти на гитхаб, найти открытый issue про Тёмную материю, в котором пункт 7 гласит, что "В правила «Модерация и ответственность» вносим соответствующие корректировки (будет отдельный issue)." С галочкой. И непонятно, когда эта галочка появилась.

17 августа 2018 в 10:52

Ммм. ИМХО, не совсем корректно уведомлять об изменениях в правилах на стороннем сайте. К тому же не помню я там упоминаний о внесении изменений в правила.

14 августа 2018 в 14:41
1

У хука form_alter есть второй аргумент - $form_state. В данной переменной в $form_state['term'] лежит заглушка под термин, если форма для добавления, или же существующий термин, если форма для редактирования. В $form['#term'] какие-то значения лежат только в случае формы для редактирования.

Короче - проверяйте $form_state['term'], а не $form['#term']