reneoelke/php52to53

PHP_CodeSniffer standard for compatibility checking of PHP 5.2 to 5.3 migrations.

php52to53's Issues

Call-Time-Pass by Reference

Call time pass by reference meckert auch bei Zuweisungen mit Referenz. Tatsächlich ist aber nur das übergeben von Variablen an Funktionen per Referenz nicht mehr erlaubt.

$fixture =& new CakeTestFixture($this->db);

Code freigegeben?

Kann das Projekt 'öffentlich' gemacht werden?

Inklusive composer.json (siehe #6) könnte es gleich auf packagist.org registriert werden.

keyword "use"

Der Sniff für die Keywords die nicht mehr erlaubt sind prüft noch nicht ob bei dem gefundenen Stück Code eine Variable benutzt wird. In dem folgenden Fall ist use erlaubt:

function Services_JSON($use = 0)
    {
        $this->use = $use;
    }

Info für 'Deprecated' (E_DEPRECATED)?

Sollte für Features, die in PHP 5.3 zwar vorhanden sind (E_DEPRECATED Notiz verursachen) eine Warnung ausgegeben werden (Log-Level INFO)?

keyword "namespace"

In dem folgenden Code ist namespace erlaubt.

class XmlNode extends Object {
/**
 * Name of node
 *
 * @var string
 * @access public
 */
    var $name = null;
/**
 * Node namespace
 *
 * @var string
 * @access public
 */
    var $namespace = null;

Dokumentation

Es fehlt jegliche Dokumentation, was abgedeckt wurde und was noch fehlt.

Wäre gut (idealerweise auch in Englisch) eine Liste zu haben.

Dient folgenden Zwecken:

• UnitTests erstellen
• Abdeckung gegen das Changelog testen
• Leitfaden für Entwickler
• Werbung für das Projekt (viel guter Text)

Konstanten E_DEPRECATED

Der Test für die Konstanten-Prüfung für z. Bsp. E_DEPRECATED prüft nicht auf eine definierung einer Konstanten sondern auf die Verwendung. E_DEPRECATED ist ja eine neue Konstante.

Kein Test für ze1 Kompatibilität

Das INI-Feld 'zend.ze1_compability_mode' wurde mit 5.3 entfernt.

Cross-repo compatibility and sniff naming conventions

Sniffs are named inconsistently within foobugs-standards repositories and across them.
Especially Deprecated, Generic and PHP namespaces, which are not used for php52to53.

I would opt for them to be applied as unit tests are added to make the user experience consistent across the standards.
Furthermore it might help grouping sniffs of future standards.

I would recommend to replace some sniffs with proper extension of existing CodeSniffer sniffs like Generic.PHP.DeprecatedFunction (there are not that many, but there are not none).

Unit tests erstellen?

Lohnt es sich für das Projekt UnitTests zu erstellen?

Wenn Ihr eine Liste aller Sniffs (Blacklist) hättet, wäre es ziemlich übersichtlich. Ohne ist es etwas müßig, aber wohl lehrreich genug :-)