Прошелся инспектором по коду

[DeusModus]

Не люблю нотисы и неиспользуемый код, посему прошелся по фреймворку инспектором PHPstorm`а. Не везде он был прав(а в некоторых местах сошел с ума), но кое-что выловить удалось.

Вызовы динамических методов как статических:

Код: Выделить всё

Name: CTestCase.php
Location: file CTestCase.php
Problem synopsis: Dynamic method PHPUnit_Runner_Version::id() called as static
 

Код: Выделить всё

Name: shell.php
Location: file shell.php
Problem synopsis: Dynamic method Text_Diff::_getTempDir() called as static
 

Проблемы с вызовами методов:

Код: Выделить всё

Source: framework\vendors\htmlpurifier\standalone\HTMLPurifier\ConfigSchema\InterchangeBuilder.php
Void from: test\CTestCase.php (tests for demo app)
Code: <...> if(version_compare(PHPUnit_Runner_Version::id(), '3.5.0RC1')<0) <...>
Problem synopsis: Method id of PHPUnit_Runner_Version class required one argument - protected function id($id) { ... }
 

Неиспользуемые локальные переменные:

Код: Выделить всё

Name: Printer.php
Location: file Printer.php
Method: prepareGenerator
Problem synopsis: Unused local variable $all

Код: Выделить всё

Name: CGettextMoFile.php
Location: file CGettextMoFile.php
Method: load
Problem synopsis: Unused local variable $array

Код: Выделить всё

Name: markdown.php
Location: file markdown.php
Method: setup
Problem synopsis: Unused local variable $in_anchor
Comment: The author of the code is confused. He uses an undeclarated field $this->in_anchor and local var $in_anchor

Код: Выделить всё

Name: CCaptcha.php
Location: file CCaptcha.php
Method: renderImage
Problem synopsis: Unused local variable $id

Код: Выделить всё

Name: Array.php
Location: file Array.php
Method: acceptToken
Problem synopsis: Unused local variable $theClass

Код: Выделить всё

Name: Console.php
Location: file Console.php
Method: finalize
Problem synopsis: Unused local variables $len and $i

[samdark]

CTestCase.php
PHPUnit_Runner_Version:

Код: Выделить всё

public static function id()
 

shell.php, Printer.php, markdown.php, Array.php, Console.php
Сторонний код. Лезть туда не очень хочется. По идее можно попробовать зарепортить авторам этого кода.

Проблемы с вызовами методов
Не понял, какой именно метод и где вызывается. Если

Код: Выделить всё

PHPUnit_Runner_Version::id() 

то никакого параметра id в методе PHPUnit нет.

CGettextMoFile.php
Без переменной разве будет как надо работать current?

CCaptcha.php
Поправил.

[DeusModus]

Не понял, какой именно метод и где вызывается. Если

Код: Выделить всё

PHPUnit_Runner_Version::id() 

то никакого параметра id в методе PHPUnit нет.

framework\vendors\htmlpurifier\standalone\HTMLPurifier\ConfigSchema\InterchangeBuilder.php

Код: Выделить всё

    protected function id($id) {
        return HTMLPurifier_ConfigSchema_Interchange_Id::make($id);
    }
 

Метод вызывается как статический, хотя таковым не является. Получаем notice и возможный warning в будущих версиях интерпритатора.

[samdark]

Ну, это HTML Purifier http://htmlpurifier.org/, а не Yii. Можно им закинуть. Если есть время и желание — займитесь.