Fork me on GitHub

Rapport de message :*
 

Re: Newbb 5 final

Titre du sujet : Re: Newbb 5 final
par JJDai sur 23/04/2020 21:23:21

Voila mes premières remarques:


_MD_NEWBB_AUTO_CREATE_ABOUT
Cette constante est vide en français et en anglais.
Elle est utilisée dans le fichier "modules\newbb\include\functions.welcome.inc.php" ligne 67

-------------------------------------------------------------------
En debut de chaque fichier de langue on trouve un test avec une définition d'existence du chargement du fichier :
if (defined('NEWBB_ADMIN_DEFINED')) {
     return;
}
define('NEWBB_ADMIN_DEFINED'true);


Il me me semble que ce code est inutile puisque Xoops charge les fichiers avec la commande "include_once" qui garanti que le fichier n'est chargé qu'une seul fois
de plus les constantes en questions sont inutilisées d’après ce que je vois.

-------------------------------------------------------------------
Dans les deux fichiers common.php anglais et français on trouve les constantes définies comme suit:
define('CO_' $moduleDirNameUpper '_GDLIBVERSION''GD Library version: ');


Il vaudrait mieux faire je pense :
define('_CO_NEWBB_GDLIBVERSION''GD Library version: ');


La concaténation est recalculée pour chaque constante, ça me semble inutile ou alors il faudrait le faire dans tous les fichiers de langues, mais ça ne me parait pas une bonne idée.
Vaut mieux déclarer ces constantes comme dans les autres fichiers.
De plus il faudrait préfixer les constantes avec un "_" comme dans les autres fichiers de langues pour rester cohérent.
Ça nécessite de faire ce changement dans tout le code.

---------------------------------
Maintenant je vais faire quelques tests en créant des sujet, forum, ...

PS : Attention les templates du thème xbotstrap pour le module newbb il y a des constantes qui ne sont pas définie. Je pense que c'est juste un problème de préfixe:
Au lieu de _AM_NEWBB_NOTOPIC il faudrait mettre _MI_NEWBB_NOTOPIC par exemple.


Sinon je n'ai pas constaté de correction orthographique a faire, bon boulot M'Dame ...
JJDai
Propulsé avec XOOPS | Graphisme adapté par Tatane, Grosdunord, Montuy337513

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