- Statut : non résolu
- Ce sujet contient 20 réponses, 6 participants et a été mis à jour pour la dernière fois par
jeffrey-omega, le il y a 13 années et 8 mois.
-
AuteurMessages
-
31 mai 2011 à 16 h 56 min #773369
Arf comme par hasard ! En fait, non c’est bon il n’y avait pas d’erreur dans le code, c’est juste que normalement quand on déclare une constante on l’écrit en majuscule, en l’occurrence ça donnerait :
define(‘FUNCTIONS’, TEMPLATEPATH . ‘/functions’);
if (is_file(FUNCTIONS . ‘/theme-options.php’)) {
require_once(FUNCTIONS . ‘/theme-options.php’);
}Mais bon le is_file est facultatif…
EDIT: Par contre je trouve ça hyper dégueu de mettre une constante dans une autre (TEMPLATEPATH est déjà une constante). A la rigueur un:
$functions = TEMPLATEPATH . ‘/functions’;
if (is_file($functions . ‘/theme-options.php’)) {
require_once($functions . ‘/theme-options.php’);
}Me paraît plus propre…
31 mai 2011 à 16 h 58 min #773370c’est OK, c’était un define. Je m’en doutais un peu, j’aime bien les mettre en majuscules justement pour éviter des confusions, mais chacun fait comme il veut.
31 mai 2011 à 17 h 12 min #773371shamateur wrote:Arf comme par hasard ! En fait, non c’est bon il n’y avait pas d’erreur dans le code, c’est juste que normalement quand on déclare une constante on l’écrit en majuscule, en l’occurrence ça donnerait :define(‘FUNCTIONS’, TEMPLATEPATH . ‘/functions’);
if (is_file(FUNCTIONS . ‘/theme-options.php’)) {
require_once(FUNCTIONS . ‘/theme-options.php’);
}Mais bon le is_file est facultatif…
EDIT: Par contre je trouve ça hyper dégueu de mettre une constante dans une autre (TEMPLATEPATH est déjà une constante). A la rigueur un:
$functions = TEMPLATEPATH . ‘/functions’;
if (is_file($functions . ‘/theme-options.php’)) {
require_once($functions . ‘/theme-options.php’);
}Me paraît plus propre…
ben c’est pas moi le créateur du thème lol
31 mai 2011 à 17 h 21 min #773372Bon alors ce que je dois faire c’est utiliser un des 2 codes que me propose l’ami « shamateur » ?
31 mai 2011 à 18 h 34 min #773373Tu n’as pas besoin de changer ton thème.
Le plugin t’avertit sur UN RISQUE lié à l’inclusion d’un fichier. C’est la syntaxe « require_once’ qui l’inquiète
Donc tu vas (ou pas) vérifier que le fichier en question est correct, et ensuite tu cliques sur ce n’est pas un virus.31 mai 2011 à 20 h 26 min #773374Lumière de Lune wrote:Tu n’as pas besoin de changer ton thème.Le plugin t’avertit sur UN RISQUE lié à l’inclusion d’un fichier. C’est la syntaxe « require_once’ qui l’inquiète
Donc tu vas (ou pas) vérifier que le fichier en question est correct, et ensuite tu cliques sur ce n’est pas un virus.mais pourtant ils sont tous là depuis le début je penses que je vous ai un peu fait perdre votre temps, « theme-options.php » ainsi que tout les autres sont là sinon mon thème ne pourrais pas fonctionnais si j’enlève un seul des fichier la page refuse de s’afficher (Tester par le créateur du thème). Je me souviens qu’il m’avait dit que lorseque je devais modifier certains choses il ne fallait pas remplacer certains fichier ou que si vraiment il fallait changer un fichier je devais passer par lui ou demander de l’aide à des gens qui s’y connaissent mais là le problème c’est que je m’inquiétait parce que l’antivirus me disais que le fichier été dangereux et je ne savais pas pourquoi.
Je vous pries de m’excuser vous avez déjà donner la réponse à ma question, problème résolu
Merci pour votre aide 🙂
-
AuteurMessages
- Vous devez être connecté pour répondre à ce sujet.