Fork me on GitHub






Newbb 5 final
Admin Frxoops
Inscrit: 16/03/2009 16:40
De Tende (06)
Messages: 3183
Il y a quelques temps j'avais traduit la dernière version de Newbb, Newbb 5 final et je souhaitais des retours avant de la proposer officiellement ici.

Je vous la propose pour l'instant sur le forum et si tout est bon je la proposerai officiellement dans la section téléchargements de modules.

Cette version requiert :

Xoops 2.5.9 minimum (avec module admin 1.2)

Php 5.5 minimum

mysql 5.5 minimum


EDIT: voir messages suivants pour le téléchargement plus récent et corrigé.

Posté le : 23/04 19:15

Open in new window

Xoopseuse Mai 2012 | Responsable équipe internationale | Recherche avancée en bas de page !
Partager Twitter Partagez cette article sur GG+
Re: Newbb 5 final
Team FrXoops
Inscrit: 14/05/2004 22:32
Messages: 2189
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

Posté le : 23/04 21:23

Ne pas perdre patience, même si cela semble impossible, c'est déjà de la patience.
Origami
Pli'Art Origami
[url=http://mfpp-origami.fr]Mouvement Français des Plieurs de Papie...
Partager Twitter Partagez cette article sur GG+
Re: Newbb 5 final
Team FrXoops
Inscrit: 14/05/2004 22:32
Messages: 2189
ha bin si j'ai repéré quelques coquilles:

Les plus récents d'abord : manque un s
define('_MD_NEWBB_NEWEST''Les plus récent d'abord');


Les plus anciens d'abord : manque un s
define('_MD_NEWBB_OLDEST''Les plus ancien d'abord');



Entrez une réponse rapide ici : manque un e

Enregsirez-vous pour participer au forum. : un peu de dyslexie
define('_MD_NEWBB_ANONNOTALLOWED''Les utilisateurs non enregistrés ne sont pas autorisés à poster des messages.<br>Enregsirez-vous pour participer au forum.');



Etes vous : manque un tiret
define('_MD_NEWBB_AREUSUREDEL''Etes vous sûr de bien vouloir supprimer ce message et ses messages enfants');


Message sélectionné suupprimé : peut-être un u de trop
define('_AM_NEWBB_POSTSDELETED''Message sélectionné suupprimé');


JJDai

Posté le : 23/04 21:34

Ne pas perdre patience, même si cela semble impossible, c'est déjà de la patience.
Origami
Pli'Art Origami
[url=http://mfpp-origami.fr]Mouvement Français des Plieurs de Papie...
Partager Twitter Partagez cette article sur GG+
Re: Newbb 5 final
Admin Frxoops
Inscrit: 16/03/2009 16:40
De Tende (06)
Messages: 3183
Merci pour tes retours, je vais corriger ça ;)

Posté le : 23/04 23:50

Open in new window

Xoopseuse Mai 2012 | Responsable équipe internationale | Recherche avancée en bas de page !
Partager Twitter Partagez cette article sur GG+
Re: Newbb 5 final
Admin Frxoops
Inscrit: 16/03/2009 16:40
De Tende (06)
Messages: 3183
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 :

Attacher un fichier:


zip newbb.zip Taille: 849.46 KB; Hits: 8

Posté le : 24/04 01:05

Open in new window

Xoopseuse Mai 2012 | Responsable équipe internationale | Recherche avancée en bas de page !
Partager Twitter Partagez cette article sur GG+
Re: Newbb 5 final
Admin Frxoops
Inscrit: 16/03/2009 16:40
De Tende (06)
Messages: 3183

Posté le : 24/04 02:27

Open in new window

Xoopseuse Mai 2012 | Responsable équipe internationale | Recherche avancée en bas de page !
Partager Twitter Partagez cette article sur GG+
Re: Newbb 5 final
Team FrXoops
Inscrit: 14/05/2004 22:32
Messages: 2189
Merci Cesag

Citation :
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


Tout simplement parce que j'ai pas mal bidouiller mon theme et que je ne veux pas recommencer tout à zéro.
Cependant si je ne prend que le dossier du module ça devrait le faire.
JJDai

Posté le : 24/04 10:30

Ne pas perdre patience, même si cela semble impossible, c'est déjà de la patience.
Origami
Pli'Art Origami
[url=http://mfpp-origami.fr]Mouvement Français des Plieurs de Papie...
Partager Twitter Partagez cette article sur GG+

 Haut   Précédent   Suivant




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

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