| Re: Newbb 5 final |
Titre du sujet : Re: Newbb 5 final
par Cesagonchu sur 24/04/2020 01:05:03
1) Citation :JJDai a écrit :
_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 effet il n'y a rien mais vu que les développeurs n'ont rien mis pour l'instant je ne peux pas modifier cela.
Je vais tout de même leur signaler.
Si tu veux y mettre quelque chose, il suffit que tu ajoutes About au fichier anglais et  propos au fichier français.
2) Citation :JJDai a écrit :
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.
3)Citation :JJDai a écrit :
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.
Là pareil, je ne peux pas modifier le module, c'est aux développeurs de voir ça. Je vais leur signaler aussi.
4) Citation :JJDai a écrit :
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.
J'avais signalé ce soucis ici:
https://github.com/XoopsModules25x/newbb/issues/70
et c'est corrigé dans la version traduite en français que j'ai mis à télécharger plus haut donc je ne comprends pas pourquoi tu as le define _AM_NEWBB_NOTOPIC dans le fichier main.php ligne 490 alors qu'il y a _MD_NEWBB_NOTOPIC
EDIT : Ah, peut-être que tu n'as pas copié le dossier EXTRA de Newbb vers le dossier themes de Xoops comme précisé ici :
https://www.frxoops.org/modules/newbb/ ... id=241047#forumpost241047
Citation :Cesagonchu a écrit :
Si tu actives le thème xbootstrap, le module va activer automatiquement son thème correspondant. Tu n'as pas besoin de bouger les fichiers du thème de Xoops, ils sont au bon endroit.
Parfois certains modules contiennent aussi un dossier spécial "bootstrap" et là par contre il faut copier ce qui est dans ce dossier pour l'envoyer dans le thème bootstrap.
Par exemple le module Newbb contient un dossier nommé "EXTRA". Il te faudra donc copier ce que contient EXTRA/themes/xbootstrap/modules/newbb/ vers ton site Xoops themes/xbootstrap/modules/newbb
Concernant tes autres signalements de fautes de frappes, j'ai tout rectifié, voici la version corrigée du module Newbb 5 final en français ci-dessous :
EDIT: voir messages suivants pour le téléchargement plus récent et corrigé.
|
|
