Вопрос по Drupal Code Standarts

Аватар пользователя Niklan

Всем привет, вопрос скорее всего к тем, кто уже публиковал модули на друплорге.

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

Вы можете посмотреть отчет здесь: http://ventral.org/pareview/httpgitdrupalorgsandboxniklan1785956git

В частности интересует:

  1. Bad line endings were found, always use unix style terminators. - что это может значить. Нету в конце пустой строки - ругается, есть - ругается. Как правильно делается последняя строка файла? Я просто жал энтер и сохранял.
  2. На взгляд около 90+% ошибок это "expected 6 spaces but found 8". Какие пробелы? Я понять вообще не могу. Стоят пробелы только где надо, никаких "левых" не присутствует. Как это исправлять я так и не понял.
  3. Parameter comment indentation must be 2 additional spaces at position 1 - как это понимать?
  4. Function comment short description must end with a full stop - и это

Вообще ошибки какие-то не понятные и странные. Методом проб ничего не помогает, как были так и остались. Прошу помощи.

Прикрепил весь модуль, чтобы могли сами посмотреть код.

P.s. мало ли. Система Ubuntu, IDE: NetBeans. Скорее где-то тут проблема пробелов и переносов.

ВложениеРазмер
Иконка пакета geo-block-master_1.zip7.88 КБ
Тип материала:
0 Thanks

Комментарии

Аватар пользователя volocuga@drupal.org
volocuga@drupal.org 5 лет назад
Niklan написал:
они проверяют все через PAreview

Сейчас единого стандарта нет, есть два модуля - PAreview и Coder. Я проверял в кодере, но PAreview походу имеет своё мнение. Чувак, который раздаёт права на фуллпроджект, некто klausi, как я понял сам автор этой проверялки.

Система заявок на полный проект на друпал.орг - ебанутая и вкратце сводится к следующему: ДОКАЖИ, что ты заслуживаешь право БЕСПЛАТНО разместить проект на д.орг.

При этом ебут мозги вот по какой причине: бонус - ревью. Т.е, когда ты размещаешь свой проект в очереди на рассмотрении, его сначала теребит "мелкая рыбёшка", многие из которых просто хотят получить галочку, что они сделали ревью твоего модуля. Далее, когда вроде всё ОК, приходит юзер klausi и говорит:
"Йо, я сейчас занят и рассматриваю только тех, у кого есть бонус-ревью"

Бонус - ревью - это когда ты сам сделал как минимум 3 ревью чужих модулей, линки на которые ты должен предоставить :) Просле 3-х ревью, klausi соизволит взглянуть на твой модуль. ЕСЛИ он найдёт ошибки, твой бонус аннулируется и ты должен снова сделать минимум 3 ревью, чтобы klausi соизволил снова снизойти до тебя

Бюракратизм прибъёт Друпал, а не количество запросов к БД :)

Аватар пользователя Niklan
Niklan 5 лет назад
Softovick написал:
http://drupal.org/coding-standards


По данному материалу все ок. Ему все соответствует.

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

volocuga@drupal.org написал:

да уж...

Я так понял, если хотя бы 1 из моих модулей пройдет проверку, дальше я смогу публиковать без неё? А то ведь это полнейший маразм.

Аватар пользователя volocuga@drupal.org
volocuga@drupal.org 5 лет назад
Niklan написал:
Я так понял, если хотя бы 1 из моих модулей пройдет проверку, дальше я смогу публиковать без неё? А то ведь это полнейший маразм.

Да, только одна проверка.
У вас разработка в Windows? Возможно в этом проблема. Drupal - это оплот злобных линуксоидов :)

Кстати, прикол. Довольно много "солидных" модулей не проходят полную проверку на соотвествие стандартам, так что не унывай :)

Аватар пользователя Niklan
Niklan 5 лет назад
volocuga@drupal.org написал:
У вас разработка в Windows? Возможно в этом проблема. Drupal - это оплот злобных линуксоидов

Нет, Ubuntu, IDE - NetBeans

volocuga@drupal.org написал:
Кстати, прикол. Довольно много "солидных" модулей не проходят полную проверку на соотвествие стандартам, так что не унывай :)

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

Запостил, посмотрим что будет.

Аватар пользователя volocuga@drupal.org
volocuga@drupal.org 5 лет назад
Niklan написал:
Это полнейший маразм, я так считаю

Так и есть

Niklan написал:
Нет, Ubuntu, IDE - NetBeans

Тогда хз. Походу PAreview кривой, так что я правильно возмущался (моя апликация)

Аватар пользователя Niklan
Niklan 5 лет назад
volocuga@drupal.org написал:
Тогда хз. Походу PAreview кривой, так что я правильно возмущался (моя апликация)

Смотрю спустя месяц вас так и не пропустили. Мда...

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

Аватар пользователя volocuga@drupal.org
volocuga@drupal.org 5 лет назад
Niklan написал:
Смотрю спустя месяц вас так и не пропустили. Мда...

Да не, я забил просто, нет времени на эти пробелы-точки

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

Как бы на это есть sandbox по идее - дозреть до кондиции. С другой стороны, из sandbox народ берёт неохотно, типа "несерьёзно", т.е у тебя нет фидбека просто :) Так что замкнутый круг. Раньше было намного попроще, но теперь зажрались :)

Аватар пользователя Niklan
Niklan 5 лет назад

Пофиксил вообще все ошибки с этого ненормального сайта. Коммитов из-за него ппц наплодилось, что совсем не нравится.

Посмотрим что отпишут и как скоро.

P.s. для избежания 90% ошибок нужно было настроить NetBeans спецом для друпала, а также установить 1 модуль для Unix окончаний "\n" (на Ubuntu, Лол)

Аватар пользователя Softovick
Softovick 5 лет назад
Niklan написал:
Пофиксил вообще все ошибки с этого ненормального сайта. Коммитов из-за него ппц наплодилось, что совсем не нравится.

Посмотрим что отпишут и как скоро.

P.s. для избежания 90% ошибок нужно было настроить NetBeans спецом для друпала, а также установить 1 модуль для Unix окончаний "\n" (на Ubuntu, Лол)

На drupal.org специально для этого инструкция есть, как настроить. Т.е. все таки по стандартам код был оформлен?
А окончания в NetBeans воспроизводились те, которые изначально в файле были - если там был Windows-like, то он во всем проекте так и воспроизводит по умолчанию.

Аватар пользователя Niklan
Niklan 5 лет назад
Softovick написал:
На drupal.org специально для этого инструкция есть, как настроить. Т.е. все таки по стандартам код был оформлен?

Оформлен был по стандартам, только табы заменялись 4 пробелами, вместо 2ух,в этом и был косяк этих ошибок. Стоило только активировать "Показать непечатаемые символы" как сразу понял где проблема зарыта.

Softovick написал:
А окончания в NetBeans воспроизводились те, которые изначально в файле были - если там был Windows-like, то он во всем проекте так и воспроизводит по умолчанию.

Не понимаю откуда вообще "Windows-Like" появился, если вся разработка начиналась, создавались файлы на Ubuntu. Файлы даже на винде не разу небыли. Может какой-то баг NetBeans, но я решил его установкой модуля: Show and change line endings