Вопрос по Drupal Code Standarts

Аватар пользователя Niklan Niklan 15 декабря 2012 в 12:30

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

Собираюсь подать заявку на перевод моего первого модуля на друпал орге в полный проект. Вроде все сделал согласно условиям, но порывшись в других заявках на рассмотрение, всем тычут пальцем на код. Вот и возник такой вопрос, они проверяют все через 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 15 декабря 2012 в 13:41
"Niklan" wrote:

они проверяют все через PAreview

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

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

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

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

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

Аватар пользователя Niklan Niklan 15 декабря 2012 в 14:40
"Softovick" wrote:

http://drupal.org/coding-standards[/quote]
По данному материалу все ок. Ему все соответствует.

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

"<a href="mailto:volocuga@drupal.org">volocuga@drupal.org</a>" wrote:

да уж...

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

Аватар пользователя volocuga@drupal.org volocuga@drupal.org 15 декабря 2012 в 15:00
"Niklan" wrote:

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

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

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

Аватар пользователя Niklan Niklan 15 декабря 2012 в 15:08
"<a href="mailto:volocuga@drupal.org">volocuga@drupal.org</a>" wrote:

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

Нет, Ubuntu, IDE - NetBeans

"<a href="mailto:volocuga@drupal.org">volocuga@drupal.org</a>" wrote:

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

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

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

Аватар пользователя Niklan Niklan 15 декабря 2012 в 15:51
"<a href="mailto:volocuga@drupal.org">volocuga@drupal.org</a>" wrote:

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

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

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

Аватар пользователя volocuga@drupal.org volocuga@drupal.org 15 декабря 2012 в 16:14
"Niklan" wrote:

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

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

"Niklan" wrote:

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

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

Аватар пользователя Niklan Niklan 16 декабря 2012 в 0:38

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

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

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

Аватар пользователя Softovick Softovick 16 декабря 2012 в 13:55
Niklan wrote:

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

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

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

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

Аватар пользователя Niklan Niklan 16 декабря 2012 в 15:24
"Softovick" wrote:

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

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

"Softovick" wrote:

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

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