[Résolu] functions.php dangereux ? (Créer un compte)

  • Statut : non résolu
6 sujets de 16 à 21 (sur un total de 21)
  • Auteur
    Messages
  • #773369
    shamateur
    Membre
    Padawan WordPress
    82 contributions

    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…

    #773370
    Guy
    Participant
    Maître WordPress
    14817 contributions

    c’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.

    #773371
    jeffrey-omega
    Membre
    Initié WordPress
    23 contributions
    shamateur 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

    #773372
    jeffrey-omega
    Membre
    Initié WordPress
    23 contributions

    Bon alors ce que je dois faire c’est utiliser un des 2 codes que me propose l’ami « shamateur » ?

    #773373
    Lumiere de Lune
    Participant
    Maître WordPress
    20531 contributions

    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.

    #773374
    jeffrey-omega
    Membre
    Initié WordPress
    23 contributions
    Lumiè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 🙂

6 sujets de 16 à 21 (sur un total de 21)
  • Vous devez être connecté pour répondre à ce sujet.