Fork me on GitHub

Rapport de message :*
 

Re: xoopsfaq

Titre du sujet : Re: xoopsfaq
par JJDai sur 11/03/2016 11:09:15

Citation :
si pour le groupe webmaster, on ne coche pas "imprimer" on verra quand même l'imprimante.

est-ce que par hasard le groupe webmaster ne serait pas le groupe admin renommé?
ou est-ce que l'utilisateur n'est pas aussi membre du groupe admin.
Pour info le groupe admin a tous les droits même si les options ne sont pas cochées.
C'est un choix que j'ai fait pour faciliter le dev mais il est possible de changer ce comportement.

afin d'exploiter les nouvelle possibilités de php et pour m'y accoutumer, j'envisage d'encapsuler les fonctions du module dans un namespace.
conséquences:
- les fonction seront appeler avec le namespace
- suppression du préfixe des fonctions ce qui facilitera la lecture
- surtout ça permet d'éviter des conflits de nom avec d'autre module surtout quand on récupère une fonction déjà développée pour l'intégrer dans un autre module


exemple:
déclaration d'une fonction
namespace nsXfaq;
defined'XOOPS_ROOT_PATH' ) or die( 'Restricted access' );
include_once 
'constantes.php';


/**
 * CleanVars()
 *
 * @return
 */
function CleanVars( &$global$key$default ''$type 'int' ) {
    }

au lieu de :
defined'XOOPS_ROOT_PATH' ) or die( 'Restricted access' );
include_once 
'constantes.php';


/**
 * xoopsFaq_CleanVars()
 *
 * @return
 */
function xoopsFaq_CleanVars( &$global$key$default ''$type 'int' ) {
    }

JJDai

utilisation de la fonction:
$op nsXfaq\CleanVars$_REQUEST'op''default''string' );

au lieu de :
$op xoopsFaq_CleanVars$_REQUEST'op''default''string' );


j'ai utilisé nsXfaq plutôt que xoopsFaq pour faciliter le remplacement et la recherche dans le code.

j'ai fait un test, ça fonctionne très bien, c'est juste une habitude de codage à changer
qu'en penses-tu ?


Propulsé avec XOOPS | Graphisme adapté par Tatane, Grosdunord, Montuy337513

137 Personne(s) en ligne (118 Personne(s) connectée(s) sur Forum) | Utilisateur(s): 0 | Invité(s): 137 | Plus ...