PORTAIL WEB, CMS OU SITE PARAMETRABLE

Commentaire de mfaraday le 17/10/2007 14:09:08

Salut

Je n'ai pas eu le temps de tout regarder mais le principe j'apprécie le principe.

Dans un premier temps, en regardant les fichiers, j'ai trouvé appréciable le petit nomnbre de fichiers... mais en regardant "administration.php" je mets tout de suite un bémol... dans un  même fichier php tu mélanges php js et (x)html...

Petits conseils qui n'engagent que moi :
- Mets ton javascript dans un .js
- Sépare (x)html et php
- Mets en place css... (table border="0" => table { border:0; })
- Evite les mises en page par tableau... (cf. http://openweb.eu.org/mise_en_page/ le lien est un peu ancien mais encore d'actualité)

Je vais jeter un regard plus avant ce soir...

Flo

PS : Utilise les objets... par exemple la base de données...

Commentaire de p3x le 17/10/2007 16:53:22

Salut tout le monde,

merci pour tes conseils MFARADAY je les prends en compte et les exploiterais surement dans mes prochains scripts.

BLUEDEMON quel est ton problème ? Une foi l'install terminé le script ne marche pas, c'est ça ?
Car normalement il devrait marcher correctement si tu as entré les bonnes propriétés de connexion à la base de données, je l'ai tester plusieurs fois sur plusieurs machines et sur plusieurs serveurs...

Merci de bien vouloir le noter,
ce serait simpa ;-)

Commentaire de mfaraday le 18/10/2007 09:34:46

Salut

Alors j'ai installé le script. Le principe me semble très bien et c'est simple d'utilisation.

Quelques petites questions...
- Pourquoi ne pas faire un fichier variables.php et le remplir par le script installation.php... Ainsi pas besoin de passer par config pour remplir les variables de BDD.
- Dans la partir admin... quand tu permets aux gens de changer le thème du site, mettre plutôt une liste déroulante avec la liste des thèmes dispo.
- Toujours dans la partie admin... ta navigation est par défaut horizontale mais quand on arrive sur la zone c'est à blanc ou horizontal ou vertical (ça ce n'est qu'un détail)

Sinon j'ai pas regardé plus avant mais quand je rajoute une news ou une page, il me l'insère en double... ouch

Mais je pense que c'est vraiment pas mal dans le principe

Flo

Commentaire de barakacom le 18/10/2007 17:03:07

J'ai cette erreur :

Notice: Undefined variable: insc in C:\Documents and Settings\Administrateur\Mes documents\Ambulances site\site_parametrable\config.php on line 16

Notice: Undefined variable: HTTP_COOKIE_VARS in C:\Documents and Settings\Administrateur\Mes documents\Ambulances site\site_parametrable\config.php on line 19

Commentaire de coucou747 le 18/10/2007 20:02:52

"il faut que tu configure ton serveur apache en mettant la variable "register_globals" à "On"."
=> ou comment faire un truc crade... tant que t'y est, t'as qu'a mettre le error_reporting a 0...

stp, lis mes commentaires, ils t'aideront a faire un code qui passe en strict sur tout serveur qui a php5 peu importe la config...

Commentaire de coucou747 le 18/10/2007 22:54:56

c'etait pas agressif, c'etait juste pour t'expliquer tes erreurs, si tu ne les corrige pas, alors personellement, je ne comprend pas pourquoi tu les as postees... (si tu les laisses, alors ta source n'apprend rien aux debutants, bien au contraire, et comme ta source ne marche sur quasiment aucun serveur, et ne marchera pas sur les servs php6, peu importe leur conf... bah finalement, elle ne sert vraiement a rien dans l'etat actuel des choses...)

Commentaire de p3x le 18/10/2007 23:27:40

mais ca veut pas dire que jte donne tor =)

Commentaire de pdl le 19/10/2007 09:55:36

Bonjour,

J'ai aussi télécharger le script et l'ai installé et j'ai eu quelques soucis, justement à cause de ce fameux "global_register" qui est à off ! Comme je ne veux pas changer se paramètre et que je n'ai pas envie de changer touts le code, ben je créer les variable à la volé, ce qui n'est pas plus propre que le "globel_register", mais ce qui permet de tester le code avant de le modifier.

Suffit de mettre le bout de code suivant :
[code]
while (list($fKey, $fVal) = each($_POST)) {
$$fKey = $fVal;
}
while (list($fKey, $fVal) = each($_GET)) {
$$fKey = $fVal;
}
[/code]
Je répète, c'est pas la bonne solution, mais cela peut débloquer certaines situations.

J'ai aussi eu des problèmes car je suis sous PHP5 et que dans ta source il y a encore beaucoup de <? ou lieu de <?php, faudrait voir à changer cela, car c'est bloquant sous PHP5.

Si non, j'ai aussi fait quelques petits changement, comme ne pas afficher les menu si on n'a pas accès (quand on n'est pas connecté), c'est très simple à ajouter et cela évite de cliquer pour rien !
Petit exemple extrait du fichier index2.php
[code]
<?php if($connec==1) { ?>
<tr>
<td height="22" align="left">
<font size="2">&nbsp;<img border="0" src="images/<?php echo $theme; ?>/fleche.gif"> </font>
<b>
<font size="2" color="#0F3A7F">
<a href="?action=Forum">
<font color="#0F3A7F">
<span style="text-decoration: none">Forum</span></font></a></font></b></td>
</tr>
<tr>
<td height="22" align="left">
<font size="2">&nbsp;<img border="0" src="images/<?php echo $theme; ?>/fleche.gif"> </font>
<b>
<font size="2" color="#0F3A7F">
<a href="?action=Galleries/getcategorie">
<font color="#0F3A7F">
<span style="text-decoration: none">Galleries</span></font></a></font></b></td>
</tr>
<tr>
<td height="22" align="left">
<font size="2">&nbsp;<img border="0" src="images/<?php echo $theme; ?>/fleche.gif"> </font>
<font size="2" color="#0F3A7F">
<b>
<a href="?action=Chat">
<font color="#0F3A7F">
<span style="text-decoration: none">Chat</span></font></a></b></font></td>
</tr>
<?php } ?>
<tr>
<td height="22" align="left">
<font size="2">&nbsp;<img border="0" src="images/<?php echo $theme; ?>/fleche.gif"> </font>
<font size="2" color="#0F3A7F">
<b>
<a href="?action=Inscription">
<font color="#0F3A7F">
<span style="text-decoration: none">Inscription</span></font></a></b></font></td>
</tr>
<?php  
if($admin==1) {
?>
<tr>
<td height="22" align="left">

<font size="2">&nbsp;<img border="0" src="images/<?php echo $theme; ?>/fleche.gif"> </font>
<font size="2" color="#0F3A7F">
<b>
<a href="?action=Administration">
<font color="#0F3A7F">
<span style="text-decoration: none">Administration</span></font></a></b></font>
</td>
</tr>
<?php } ?>
[/code]

Pour le reste, c'est très bien, mais je vais encore modifier tes scripts, pour pouvoir ajouter des pages dynamiquement (j'ai cela sur un dev que j'ai fait) et voir pour séparer les css du code html, comme on l'a déjà dit et aussi certains champs sont trop petits en DB et les textarea aussi (celui surtout pour mettre le code pour créer une page).

Note que si tu as l'intention de la faire, fais-le nous savoir, histoire que l'on ne perde pas tous notre temps à faire 2x la même chose.

Continue, et merci

Pierre

Commentaire de malalam le 19/10/2007 18:30:29 administrateur CS

Hello,

@P3X => Coucou a raison sur toute la ligne. Tu devrais sérieusement penser à changer tes habitudes de programmation, ET à modifier ce code. Effectivement, les serveurs web sont de plus en plus en register_globals à Off, et non pas à On. Il y a une raison à cela. Idem pour short_open_tags. Ce n'est surtout pas la configuration des serveurs qu'il faut modifier dans ce cas, mais ton code. Ca ne me parait pas compliqué de changer ses habitudes, surtout quand elles sont mauvaises. Autant en prendre de bonnes, non ?