BUG templateCss

danielsandanielsan Member
septembre 2012 modifié dans Bogues
Bonjour,


je ne sais pas si ça a déjà été évoqué,
mais il y a un petit bug dans la fonction templateCss


La méthode actuelle:
	/**
	 * Méthode qui ajoute, s'il existe, le fichier css associé à un template
	 *
	 * @param	css_dir     répertoire de stockage des fichiers css (avec un / à la fin)
	 * @return	stdout
	 * @scope	global
	 * @author	Stephane F
	 **/
	public function templateCss($css_dir='') {
		# Hook Plugins
		if(eval($this->plxMotor->plxPlugins->callHook('plxShowTemplateCss'))) return;

		$theme = $this->plxMotor->aConf['racine_themes'].$this->plxMotor->style.'/';
		$css = str_replace('php','css',$this->plxMotor->template);
		if(is_file($theme.$css))
			echo "\t".'<link rel="stylesheet" type="text/css" href="'.$theme.$css_dir.$css.'" media="screen" />'."\n";
	}
la condition vérifie si un fichier existe dans $theme.$css
alors qu'elle devrait le vérifier dans $theme.$css_dir.$css ?


La nouvelle méthode serait donc:
	/**
	 * Méthode qui ajoute, s'il existe, le fichier css associé à un template
	 *
	 * @param	css_dir     répertoire de stockage des fichiers css (avec un / à la fin)
	 * @return	stdout
	 * @scope	global
	 * @author	Stephane F
	 **/
	public function templateCss($css_dir='') {
		# Hook Plugins
		if(eval($this->plxMotor->plxPlugins->callHook('plxShowTemplateCss'))) return;

		$theme = $this->plxMotor->aConf['racine_themes'].$this->plxMotor->style.'/';
		$css = str_replace('php','css',$this->plxMotor->template);
		if(is_file($theme.$css_dir.$css))
			echo "\t".'<link rel="stylesheet" type="text/css" href="'.$theme.$css_dir.$css.'" media="screen" />'."\n";
	}


Bien cordialement,

Réponses

  • StéphaneStéphane Member, Former PluXml Project Manager
    Sounds good to me !
    Bug pris en compte.
    Merci

    Consultant PluXml

    Ancien responsable et développeur de PluXml (2010 à 2018)

Connectez-vous ou Inscrivez-vous pour répondre.