[mlfichadh] mises à jour du 10 mars

[Antoine Viel <antoine.viel@xxxxxxxxxxxxxxxx>]

Bonjour,

Merci à Jean et à Louis pour vos modifs du jour, le projet est vraiment 
sur la bonne voie.

Dès que j'aurai un peu de temps je testerai le contenu de la branche 
"login",
l'objectif étant de fusionner la branche dans le tronc pour le weeek-end 
prochain.
En parallèle, je vais commencer à documenter la procédure d'installation 
à partir
du contenu de la branche, et j'écrirai des scripts shell (Gnu bash) pour 
faciliter l'installation si nécessaire.

Au passage, qu'en est-il de la version Windows ? Louis : continues-tu à 
développer sur cette plateforme ?
Si oui, pourrais-tu tester si la version actuelle (ainsi que la branche 
login) tourne toujours sans problème sur cette plateforme ?

Un dernier point : comme on en a discuté au téléphone avec Jean, 
l'introduction de la nouvelle table "login"
a conduit a effectuer une requête compliquée (comportant un sous-requête 
SQL compatible avec une version de MySQL supérieure à 4.1) pour 
l'extraction dans "LstAdresses.php".
Deux questions se posent alors :
- à l'origine, je visais la portabilité maximale (toute plateforme et 
versions de MySQL et PHP 4+), faut-il choisir d'imposer aussi une 
version minimale de MySQL ? (Au passage, il faudra mettre à jour le 
serveur national)
- d'autre part, il n'est peut-être pas nécessaire de sélectionner le 
login et le mot de passe dans la page des extractions "LstAdresses.php", 
qui relève davantage de l'exploitation de la base de données que de son 
administration. C'est pourquoi je proposerais plutôt de scinder en deux 
les requêtes d'extraction. D'un côté la requête d'exploitation limitée 
aux champs utilisateurs (nom, prénom, coordonnées, adhésion, etc), et de 
l'autre une requête d'extraction "administrateur" listant les logins et 
les mots de passe. Cette dernière solution permettrait aussi de se 
passer de la sous-requête SQL et ainsi de garder la compatibilité avec 
MySQL 3.x et 4.0. Qu'en pensez-vous ?

A bientôt,

Antoine


Louis MATRINGE wrote:
> Bonjour,
> Pas de commentaire encore sur tes réponses mais juste pour signaler 
> que je viens de faire le "commit" que j'avais tenté hier mais j'avais 
> des pb de droits d'accès...
>
> Pour le bug d'affichage du bouton, c'était un problème de 
> double-quotes manquantes après la parenthèses qui suit $Adh->NUMADH") 
> et avant le >.
>       echo "<button type='button' name='BtAdhes'".
>                " 
> onClick=\"javascript:Aff_Adhesions(".$Adh->NUMADH.")*\"*>Autres 
> adhésions".
>                " <IMG src='images/button_ok.png'></button>\n";
c'est aussi le petit bug que j'avais corrigé en révision 29 ce midi ....


> Jean Saquet a écrit :
> > Bonjour,
> >
> > J'ai envoyé ma version avec séparation du login dans la branche "login".
> > Je n'y ai pas recopié les modifs faites par Louis indiquées 
> > ci-dessous, mais j'y réponds dans le texte.
> >
> > Le 10 mars 08 à 00:41, Louis MATRINGE a écrit :
> >
> > > Bonjour,
> > > Je viens de tester le "trunk" de ce jour (rev=26) après les mises à 
> > > jour du 2 mars d'Antoine.
> > > Pas de gros soucis a priori, je me suis quand même permis d'apporter 
> > > quelques bricoles de modifications :
> > >
> > > Module des extractions :(ListAdresses.php)
> > > -Application des styles sur les cases à cocher de sélection des champs.
> > > -Dans le cas de mise en forme en tableau, application d'un espace 
> > > insécable "&nbsp;" si la cellule est vide car pour certain 
> > > navigateurs les bordures de cellules sont alors mal rendues.
> > OK
> > > Module d'initialisation des tables : (initbase.php)
> > > - J'ai déplacé les lignes de code d'initialisation de la  première 
> > > fiche dans la partie de création des tables sans erreur, sinon on 
> > > pouvait passer dans la création de la fiche même si les tables 
> > > n'aient pas été créées correctement.
> > Je n'avais pas dû regarder ceci d'assez près. Merci pour la correction
> > > - J'ai également ajouté le nom de login comme nom d'adhérent (champ 
> > > NOM) sinon la fiche n'était pas accessible via la liste déroulante 
> > > de gauche. (ou plutôt on ne pouvait rien voir car le nom était vide)
> > En fait, j'avais corrigé ceci, mais Antoine ne l'avait pas 
> > complètement recopié. Je pensais l'avoir re-corrigé. de toutes 
> > façons, le pb est résolu autrement dans la branche "login" (une ligne 
> > est ajoutée dans la table login, mais ce n'est pas lié à une fiche 
> > adhérent)
> > > - J'ai du également modifier le code de la fonction Connexion() de 
> > > l'objet "AccesBase" car quand on voulat ouvrir un session 
> > > d'administration avec une base c'était "pas marche!" alors j'ai fait 
> > > une modif qui crée automatiquement la base vide afin de pouvoir 
> > > ensuite initialiser les tables. Et pourquoi ne pas enchaîner 
> > > automatiquement ces deux traitements ?
> > C'est vrai que je n'ai pas essayé d'ouvrir une base qui n'existait 
> > pas. Mais la création de la base est plutôt - à mon avis - de niveau 
> > "installation". Par exemple, si on prend un espace chez un 
> > fournisseur, il risque de nous octroyer UNE base et un accès associé..
> > > Module d'affiche de la liste des adhérents (fiche_adh.php)
> > > - J'ai supprimé le bouton "Go" a face de la liste déroulante  de 
> > > choix de la liset d'affichage car il n'apporte rien.
> > Tout à fait d'accord. Moi, j'ai modifié la liste d'affichage 
> > pour qu'elle ne soit pas trop haute si peu d'adhérents (penser à Ste 
> > Austreberthe :-)
> > J'ai également modifié l'appel du script pour les villes et 
> > profession en remplaçant le "on change" par "on click". En effet, 
> > pour changer, il faut qu'il y ait plus d'une ligne. Or, dans ma 
> > version, après initialisation des tables il n'y a pas d'adhérent, 
> > donc pas de ville ni de profession.
> > J'ai bien entendu aussi modifié le script pour que la saisie du 
> > login/ mot de passe ne soit pas obligatoire.
> >
> > Il reste des petits bugs dans le script au niveau du code postal 
> > (déjà signalé) et de la saisie d'une nouvelle valeur pour ville ou 
> > profession, la ligne "Autre valeur" étant dupliquée. Mais je ne suis 
> > pas spécialiste du javascript, alors, je n'ai pas essayé d'y toucher.
> > > - Corection également d'un petit bug d'affichage du bouton "Autres 
> > > adhésions" (correct avec FireFox, mais invisible sous Konqueror )
> > Ainsi que sous Safari. C'était quoi le bug ? J'ai cherché mais n'ai 
> > pas trouvé.
> > > Module indexadmin.html :
> > > - Ajout d'un ien pour revenir au menu de l'application.
> > > -Ne faut il pas également prévoir un lien depuis le menu 
> > > d'application  (index.php) pour appeler le menu d'administration ?
> >
> > Oui, on peut. Personnellement, pour les tests, j'ouvre deux fenêtres, 
> > une en admin, l'autre en utilisateur lambda. C'est pourquoi je ne les 
> > avais pas mis.
> >
> > Reste peut être à expliquer ce que j'ai fait dans la branche login ?
> > Je vais faire ça pour la documentation. D'ailleurs, je pense qu'il ne 
> > faut pas se polariser sur la doc de l'installation et de la 
> > configuration, vu que :
> > - j'ai fait une grande partie de la configuration
> > - vous allez faire l'installation !!!
> >
> >
> > à +
> > Au fait, et la mailing-list ? a-t-on des explications de ses 
> > évanouissements ?



---