J'ouvre ce post pour lister les bugs et autres remarque
J'avertirai dans un nouveau post des nouveaux uploads correspondant aux correctifs et/ou remarques divers en indiquant un numero de version.
L'installation est super simple :
- supprimer le répertoire module/news
- uploader le contenu du pack
- A la connexion de l'administration du module, la mise à jour s'effectuera tout seul !!!
Cordialement,
Shiryu.
Dernière édition par shiryu69 le Jeu Jan 31, 2008 4:36 pm; édité 1 fois
Ton install auto, pourquoi pas (Quoique ça consomme une requête SQL à chaque lecture de index.php) mais j'aurai préféré l'installation classique de Max.
De plus elle est mal placée.
Ce qui fait que si tu le poses tel quel, ben ça plante car l'install se trouve dans l'admin et bien évidemment la première chose que regarderont les users ne sera pas forcément l'admin du module mais certainement leur site.
Tes tests sur les constantes, pige pas trop à quoi ça peut servir.
Les "die ('Access denied constants MAXIMUS directly...')"", j'aime pas trop non plus. Préfère les redirections sur l'index du module ou du site ou autre.
Du fait de sa hiérarchie, ce module devient incompatible avec certains modules et blocks, comme le Top10 ou Topics et Stories_Archives (Pas des moindres, les deux derniers.).
Les redirect_url, je les évite. Perso, rien à faire que Sentinel les analyse.
Déjà qu'il pompe à donf, ... .
(Je vote Header, mais chacun fait ce qu'il veut.).
Sinon, il y a du boulot, c'est clair, voire même de l'inspiration. :mgreen:
Une autre remarque ?
Code:
if ($numrows = $GLOBALS['db']->sql_numrows($result) != 1) {
redirect_url("index.php");
die();
}
Tu te fais des lignes de codes pour rien, je trouve.
Par contre comme ceci :
Code:
if ($numrows = $GLOBALS['db']->sql_numrows($result) != 1) die(header("location: index.php");
tu te retrouves avec une seule ligne.
Bien évidemment, c'est essentiellement ma façon de voir le code, tu en fais ce que tu veux.
Aller, je vais tester plus en avant ce module.
P.S. : Attention !
Surtout ne supprimez pas le répertoire News 1.1.6 sur votre FTP si vous désirez tester ce module sur votre site.
Renommez-le, c'est un conseil qui vous économisera des transferts inutiles au cas où vous vouliez revenir en arrière.
Si vous le testez, désactivez l'Url Rewriting, complétement incompatible avec cette version.
Si vous utilisez la version 2.0.0 du module Content, inutile d'uploader le répertoire root/kernel/fpdf.
_________________
shiryu69 Citoyen Max Canada
Inscrit le: Jan 05, 2007
Messages: 10
Localisation: fr
Posté le:
Jeu Jan 31, 2008 2:41 am
Citation:
install auto :
Au départ j'étais parti pour un script d'install à part. Mais étant donné que je suis encore en phase de test, j'automatise tout. C'est une étape de moins pour les testeurs. Il est convenu que je revienne à l'upgrade manuel une fois la version finie!
Citation:
Tes tests sur les constantes, pige pas trop à quoi ça peut servir.
Peux tu préciser stp ?
Citation:
Les "die ('Access denied constants MAXIMUS directly...')"", j'aime pas trop non plus. Préfère les redirections sur l'index du module ou du site ou autre.
C'est sur que c'est un peu brutal, j'opterai pour ta solution.
Citation:
Du fait de sa hiérarchie, ce module devient incompatible avec certains modules et blocks, comme le Top10 ou Topics et Stories_Archives (Pas des moindres, les deux derniers.).
Effectivement ! La question avait été soulevé pour la première BETA.
Il me semble difficile de prendre en compte ceci sans revenir en arrière au niveau du module.
Je ne vois pas d'autre solution de modifier le module.php (ligne ~96):
Code:
$modpath1 = $modpath."modules/".MODULE_NAME."/".$file.".php";
$modpath2 = $modpath."modules/".MODULE_NAME."/includes/".$file.".php";
if (file_exists($modpath1) or file_exists($modpath2)) {
if ( ($raw->view == 0) || ($raw->view == 1 AND (defined('USER_ALLOW') OR defined('ADMIN_ALLOW'))) || ($raw->view == 2 AND defined('ADMIN_ALLOW')) || ($raw->view == 3) || ($raw->view > 3 AND in_groups($raw->groups)) ) {
if (is_file("$modpath1")) { include_once("$modpath1"); }
elseif (is_file("$modpath2")) { include_once("$modpath2"); }
} else {
restrited_module(""._RESTRICTEDAREA."");
}
} else {
restrited_module(""._FILENOTFOUND."");
}
Citation:
Les redirect_url, je les évite. Perso, rien à faire que Sentinel les analyse.
(Je vote Header, mais chacun fait ce qu'il veut.).
Ok, si je peux éviter d'alourdir inutilement le module, je suis preneur !
Citation:
Aller, je vais tester plus en avant ce module. Wink
je te remercie de prendre un peu de temps pour tester le module.
shiryu69 Citoyen Max Canada
Inscrit le: Jan 05, 2007
Messages: 10
Localisation: fr
Posté le:
Jeu Jan 31, 2008 4:57 pm
correctif v1
* remplacement des "redirect_url" par la fonction "header"
* correction de la prévisualisation d'article
* Ajout d'un système de chargement de modèles !
système de modèles :
Possibilités de charger un modèle pour le compléter ensuite. Les modèles sont chargés à partir du dossier /modules/News/modeles
pour créer un modèle :
- créer un répertoire correspondant au nom du modèle avec 2 fichiers (home.tpl et body.tpl)
- les 2 fichiers sont de simples templates en HTML.
Le contenu de chaque fichier sera pré-chargé respectivement dans le chapeau et le corps de l'article !
Pour supprimer un modèle :
- supprimer simplement le répertoire correspondant au modèle !
Modifier un modèle :
- éditer les fichiers home.tpl et body.tpl
Vous trouverez un exemple concret dans l'archive. Il s'agit d'une fiche de présentation d'individu...
shiryu69 Citoyen Max Canada
Inscrit le: Jan 05, 2007
Messages: 10
Localisation: fr
Voir le sujet suivant Voir le sujet précédent
Vous ne pouvez pas poster de nouveaux sujets dans ce forum Vous ne pouvez pas répondre aux sujets dans ce forum Vous ne pouvez pas éditer vos messages dans ce forum Vous ne pouvez pas supprimer vos messages dans ce forum Vous ne pouvez pas voter dans les sondages de ce forum
!!!
