Re: [mlfichadh] mises à jour du 2 mars

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 " " si la cellule est vide car pour certain navigateurs les bordures de cellules sont alors mal rendues.

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 ?