Fix 200107 1240

[kazimentou]

Utilisation constante de class pour plxMotor
On vérifie s'il existe un dossier "data/template" avant d'appeler scandir
Tout le monde ne nomme son dossier de données "data" !!!!
C'est quoi l'intérêt du dossier data/template ?

Prise en compte de la langue du navigateur pour install.php et update/index.php
révision grid dans update/index.php

[ensag-dev]

Très bonne idée de récupérer la langue du navigateur pour l'installation.
Cependant je ne vois pas où vous faites une comparaison entre les langues de l'entête accepted_language et les langues disponibles sur l'installation de PluXml.

[bazooka07]

Il y a quelques lignes de code plus loin pour contrôle :

if(!array_key_exists($lang, plxUtils::getLangs())) {
	$lang = DEFAULT_LANG;
}

En passant la fonction pxUtils::getLangs me parait bien verbeuse.

[haruka-7]

Oui, très bonne idée pour la langue.

Concernant le fix sur le dossier "data/templates" je préfère faire le test is_dir() dans la fonction getTemplates, comme ceci :

public function getTemplates($templateFolder) {
	if(is_dir($templateFolder)) {
		$files = array_diff(scandir($templateFolder), array('..', '.'));
		if (!empty($files)) {
			foreach ($files as $file) {
				$this->aTemplates[$file] = new PlxTemplate($templateFolder, $file);
			}
		}
	}
}

Il peut servir, par exemple, à créer des templates d'e-mails, comme ceux utilisés pour le mail de mot de passe oublié dans "core/templates".

[haruka-7]

Mergé sur develop (5.8.1). Merci pour la PR.

[ensag-dev]

D'accord merci pour les explications.
En parlant de langues, on pourrait en profiter pour corriger un bug qui existe sur les plugins :
si la langue choisi par l'admin n'est pas dispo sur le plugin alors on aura droit à des erreurs au lieu des chaînes de caractères. Il faudrait parcourir les langues de l'administrateur et s'il n'y a pas de correspondance créer un contournement pour afficher une des langues dispo.

[bazooka07]

Effectivement on ne fait aucun test pour savoir si le fichier de langue existe dans le plugin dans la méthode plxPlugins::getInstance().

Il faut parcourir le dossier lang du plugin et faire un choix par défaut.
Je serai tenté de proposer l'anglais sauf qu'en général les auteurs sont francophones et même que cela.

[haruka-7]

J'ai créé une issue pour traiter ce problème.