Vérifier la validité des fichiers de configuration XML de Magento

Pour rappel, Magento s'appuie sur des fichiers XML pour configurer ses modules (entre autre). Le soucis avec l’utilisation qu'en fait Magento, c'est qu'il ne vérifie pas l'intégrité des fichiers XML.

En effet, si vous faite une faute de frappe, qu'une balise est mal fermé ou tout être erreur dans la structure XML de votre fichier de configuration, Magento levera une erreur (ou plutôt un warning) du type :

Warning: simplexml_load_string() [function.simplexml-load-string]: Entity: line 19: parser error : expected '>' in magento\lib\Varien\Simplexml\Config.php on line 510

Notes :

Vous me direz, ça n'est pas bloquant : un coup d'oeil dans le fichier exception.log pour déterminer l'origine du problème et l'affaire est réglée ! Sauf que non, ce type d'erreur n'est pas "protégée" par un try/catch... De ce fait, mise à part le fait de savoir qu'un fichier XML de configuration est en cause, vous n'êtes pas très avancé.

L'idée, pour se simplifier, est de connaitre le non (où plutôt le chemin) du fichier en cause et la ligne qui pose problème.

Pour cela, je vous propose un overlap de classe Varien en charge des fichiers de configuration XML : Varien_Simplexml_Config. Le code est simple et mériterait d'être retravaillé, mais il fonctionne bien en l'état.

app/code/local/Varien/Simplexml/Config.php :

class Varien_Simplexml_Config {
    [...]
    /**
     * Imports XML file
     *
     * @param string $filePath
     * @return boolean
     */
    public function loadFile($filePath)
    {
        if (!is_readable($filePath)) {
            //throw new Exception('Can not read xml file '.$filePath);
            return false;
        }

        $fileData = file_get_contents($filePath);
        $fileData = $this->processFileData($fileData);
        /**
         * Custom code: $filePath added
         */
        // original
        //return $this->loadString($fileData, $this->_elementClass);
        // new
        return $this->loadString($fileData, $this->_elementClass, $filePath);
    }

    /**
     * Imports XML string
     *
     * @param  string $string
     * @return boolean
     */
    public function loadString($string)
    {
        if (is_string($string)) {
            /**
             * Custom code: dealing with config file XML errors
             */
            libxml_use_internal_errors(true);
            $xmlcheck = simplexml_load_string($string);
            if (!is_object($xmlcheck)) {
                $argslist = func_get_args();
                $filePath = $argslist[2];
                foreach(libxml_get_errors() as $error) {
                    $error = array(
                        'message' => trim($error->message),
                        'code'    => 'Code: ' . $error->code,
                        'file'    => 'File: ' . ($error->file != '' ? $error->file : $filePath),
                        'line'    => 'Line: ' . $error->line,
                        'column'  => 'Column: ' . $error->column,
                    );
                }
                libxml_clear_errors();
                // log
                Mage::log("\n" . implode("\n", $error), Zend_Log::ERR, 'exception_config.log');
                // throw in developer mode
                if (Mage::getIsDeveloperMode()) {
                    Mage::printException(new Exception(implode("\n", $error)), 'CONFIG FILE ERROR');
                }
            }
            // /Custom code
            $xml = simplexml_load_string($string, $this->_elementClass);

            if ($xml instanceof Varien_Simplexml_Element) {
                $this->_xml = $xml;
                return true;
            }
        } else {
            Mage::logException(new Exception('"$string" parameter for simplexml_load_string is not a string'));
        }
        return false;
    }
    [...]
}

Le principe est très simple : je me contente d'utiliser l'outillage founi par PHP et libxml via libxml_use_internal_errors() :

Voilà, c'est tout simple et le résultat est tout de même plus explicite :

CONFIG FILE ERROR
expected '>'
Code: 73
File: magento\app\code\local\Narno\MonModule\etc\config.xml
Line: 20
Column: 1

Il semblerait que j'ai mal fermé une balise à la ligne 20 du fichier de configuration de MonModule ! :-)

Encore une fois j'insiste sur le fait qu'il s'agit d'un overlap simple et perfectible (et surtout bourrin). Si vous avez des idées d'amélioration, n'hésitez pas ! :-)

Pour aller plus loin dans la validation des fichiers XML de Magento, il serait nécessaire de valider leur schéma via une DTD (via l'option _LIBXMLDTDVALID de l'objet SimpleXMLElement), mais le chantier s'annonce long et pénible...

Initialement publié sur Narno.com.

icon github icon linkedin icon twitter