LINUX.ORG.RU

История изменений

Исправление webmak, (текущая версия) :

там не только в скобках требования, их валидатор выкатил изменений на 10 страниц 10 шрифтом.

вот к примеру

The use of $_POST is forbidden; use Tools::getValue() instead
вроде бы и безобидное решение, только тут проблема, ихняя хрень берет и GET и POST и прочую хренотеть, а по моей логике этого не льзя так как опять ихняя же функция при валидации создает переменные post в самом скрипте, и для безопастности я принял решение, что нужно брать переменные из post, хоть это и не сильно обезопасит, но сузит возможность совпадения в названия у других модулей.

Также я использовал namespase - у них это табу.

Logical operator "and" is prohibited; use "&&" instead

в своем стиле все функции которые работают с БД я использую названия, а не && || и т.д., мне это удобно и я быстро нахожу сущность функции, но это мой бзик, понимаю, что у других его нет.

потом идет еще

1 space expected after "if"; 0 found
я не делаю оступы после if, а сразу пишу скобки
TRUE, FALSE and NULL must be lowercase
я их всегда пишу с большими, так как это константы
Variable "$nonSortCategories" have not right syntax. Should be : "$non_sort_categories"
я использую camelCase, а вот так non_sort_categories пишу только те переменные которые идут в sql запрос, чтоб уделить им больше внимания
Protected method name "AdminPoductXMLExportController::_initForms" is not in camel caps
format
я всегда все приватные и закрытые члены класса начинаю с _ мне это удобно и понятно. У них это табу и там мнооооогоо чего - перепись моего кода займет больше чем его создание.

Исправление webmak, :

там не только в скобках требования, их валидатор выкатил изменений на 10 страниц 10 шрифтом.

вот к примеру

The use of $_POST is forbidden; use Tools::getValue() instead
вроде бы и безобидное решение, только тут проблема, ихняя хрень берет и GET и POST и прочую хренотеть, а по моей логике этого не льзя так как опять ихняя же функция при валидации создает переменные post в самом скрипте, и для безопастности я принял решение, что нужно брать переменные из post, хоть это и не сильно обезопасит, но сузит возможность совпадения в названия у других модулей.

Также я использовал namespase - у них это табу.

Logical operator "and" is prohibited; use "&&" instead

в своем стиле все функции которые работают с БД я использую названия, а не && || и т.д., мне это удобно и я быстро нахожу сущность функции, но это мой бзик, понимаю, что у других его нет.

потом идет еще

1 space expected after "if"; 0 found
я неделаю оступы после if, а сразу пишу скобки
TRUE, FALSE and NULL must be lowercase
я их всегда пишу с большими, так как это константы
Variable "$nonSortCategories" have not right syntax. Should be : "$non_sort_categories"
я использую camelCase, а вот так non_sort_categories пишу только те переменные которые идут в sql запрос, чтоб уделить им больше внимания
Protected method name "AdminPoductXMLExportController::_initForms" is not in camel caps
format
я всегда все приватные и закрытые члены класса начинаю с _ мне это удобно и понятно. У них это табу и там мнооооогоо чего - перепись моего кода займет больше чем его создание.

Исправление webmak, :

там не только в скобках требования, их валидатор выкатил изменений на 10 страниц 10 шрифтом.

вот к примеру

The use of $_POST is forbidden; use Tools::getValue() instead
вроде бы и безобидное решение, только тут проблема, ихняя хрень берет и GET и POST и прочую хренотеть, а по моей логике этого не льзя так как опять ихняя же функция при валидации создает переменные post в самом скрипте, и для безопастности я принял решение, что нужно брать переменные из post.

Также я использовал namespase - у них это табу.

Logical operator "and" is prohibited; use "&&" instead

в своем стиле все функции которые работают с БД я использую названия, а не && || и т.д., мне это удобно и я быстро нахожу сущность функции, но это мой бзик, понимаю, что у других его нет.

потом идет еще

1 space expected after "if"; 0 found
я неделаю оступы после if, а сразу пишу скобки
TRUE, FALSE and NULL must be lowercase
я их всегда пишу с большими, так как это константы
Variable "$nonSortCategories" have not right syntax. Should be : "$non_sort_categories"
я использую camelCase, а вот так non_sort_categories пишу только те переменные которые идут в sql запрос, чтоб уделить им больше внимания
Protected method name "AdminPoductXMLExportController::_initForms" is not in camel caps
format
я всегда все приватные и закрытые члены класса начинаю с _ мне это удобно и понятно. У них это табу и там мнооооогоо чего - перепись моего кода займет больше чем его создание.

Исправление webmak, :

там не только в скобках требования, их валидатор выкатил изменений на 10 страниц 10 шрифтом.

вот к примеру

The use of $_POST is forbidden; use Tools::getValue() instead
вроде бы и безобидное решение, только тут проблема, ихняя хрень берет и GET и POST и прочую хренотеть, а по моей логике этого не льзя так как опять ихняя же функция при валидации создает переменные post в самом скрипте, и для безопастности я принял решение, что нужно брать переменные из post.

Также я использовал namespase - у них это табу.

Logical operator "and" is prohibited; use "&&" instead

в своем стиле все функции которые работают с БД я использую названия, а && || и т.д., мне это удобно и я быстро нахожу сущность функции, но это мой бзик, понимаю, что у других его нет.

потом идет еще

1 space expected after "if"; 0 found
я неделаю оступы после if, а сразу пишу скобки
TRUE, FALSE and NULL must be lowercase
я их всегда пишу с большими, так как это константы
Variable "$nonSortCategories" have not right syntax. Should be : "$non_sort_categories"
я использую camelCase, а вот так non_sort_categories пишу только те переменные которые идут в sql запрос, чтоб уделить им больше внимания
Protected method name "AdminPoductXMLExportController::_initForms" is not in camel caps
format
я всегда все приватные и закрытые члены класса начинаю с _ мне это удобно и понятно. У них это табу и там мнооооогоо чего - перепись моего кода займет больше чем его создание.

Исходная версия webmak, :

там не только в скобках требования, их валидатор выкатил изменений на 10 страниц 10 шрифтом.

вот к примеру

The use of $_POST is forbidden; use Tools::getValue() instead

вроде бы и безобидное решение, только тут проблема, ихняя хрень берет и GET и POST и прочую хренотеть, а по моей логике этого не льзя
так как опять ихняя же функция при валидации создает переменные post в самом скрипте, и для безопастности я принял решение, что нужно брать переменные из post.

Также я использовал namespase - у них это табу.
Logical operator "and" is prohibited; use "&&" instead

в своем стиле все функции которые работают с БД я использую названия, а && || и т.д., мне это удобно и я быстро нахожу сущность функции, но это мой бзик, понимаю, что у других его нет.

потом идет еще

1 space expected after "if"; 0 found
я неделаю оступы после if, а сразу пишу скобки
TRUE, FALSE and NULL must be lowercase
я их всегда пишу с большими, так как это константы
Variable "$nonSortCategories" have not right syntax. Should be : "$non_sort_categories"
я использую camelCase, а вот так non_sort_categories пишу только те переменные которые идут в sql запрос, чтоб уделить им больше внимания
Protected method name "AdminPoductXMLExportController::_initForms" is not in camel caps
format
я всегда все приватные и закрытые члены класса начинаю с _ мне это удобно и понятно. У них это табу и там мнооооогоо чего - перепись моего кода займет больше чем его создание.