1. Главная
  2. Блоги
  3. Блог chewie

Помогите с стандартами кодирования...

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

Аватар пользователя chewie chewie 26 сентября 2009 в 14:12

Здравствуйте!
Загрузил свой модуль. Жду импрува. Однако постоянно не прохожу модерацию по стандартам кодирования.
Сделал модуль для оплаты через платежную систему liqpay для уберкарта. Все работает без проблем.
Однако постоянно не могу пройти проверку стандартов кодирования.
Вот модуль на drupal.org (смотрите последний аплоэд)
некоторые участки кода довольно спорные. Например как правильно должен выглядеть этот участок кода, учитывая то, что ключ должен быть строкой:

<?php  if($un[(string)$merchant_id] != $respons['amount']){
    $res = FALSE;
  }?>

Комментарии

Аватар пользователя chewie chewie 27 сентября 2009 в 16:59

MDinc wrote:
<?php
$lp = db_fetch_object(db_query('SELECT * FROM {uc_liqpay_rates} WHERE currency="%s"',$_SESSION['LPCURRENCY']));
?>

тож убило

<?php
$lp = db_fetch_object(db_query('SELECT * FROM {uc_liqpay_rates} WHERE currency=%s',$_SESSION['LPCURRENCY']));
?>

авто подстановщик и так все сделает
Да и потом * дурной тон (надо указывать все желаемые поля)


Спасибо за совет. Немного был сбит с толку модератором

Аватар пользователя chewie chewie 27 сентября 2009 в 17:08

WiseMan wrote:
Поставьте модуль coder - он анализирует на соответствие стандартам кодирования.

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