bugs ....
Salut,
D=E9sol=E9, la 1.2.3 =E9tait donc incompl=E8te (=E7a faisait les mises =E0 j=
our,
mais pas la bonne cr=E9ation de table =E0 l'installation).
J'ai donc install=E9 3 fichiers corrig=E9s dans la 1.2.03 (qui devient du
coup une 1.2.4):
/ecrire/inc.php3
/ecrire/inc_version.php3
/ecrire/inc_base.php3
(au passage, dans inc_version, j'ai ajout=E9 une variable
"$spip_version.afficher", qui est le nom de la version "en clair",
que j'affiche en bas de chaque page "SPIP 1.2.4 est distribu=E9 sous
GPL" - pour info, le changement de $spip_version provoque une mise =E0
jour de la base, avec proc=E9dure de v=E9rification par FTP, donc on ne
peut pas le modifier =E0 chaque fois).
ARNO*
Coucou,
Heu, c'est juste pour proposer que les corrections de bugs se
concentrent plutôt sur les versions déjà sorties, quand les bugs
ne sont pas liés aux nouvelles fonctionnalités. Vu que la 1.2,
à mon avis, ne sortira pas avant quelques semaines.... J'ai
répercuté la modif du inc_base sur la 1.0.5, en passant.
Sinon, y a deux trois trucs à corriger en priorité :
- les problèmes de sécurité avec le nouveau statut d'admin
de rubrique. Aussi, si le inc_auth pouvait être mis au propre,
ce serait bien.... Une fonction qui s'appelle sous_rubriques
alors qu'elle fait quelque chose de spécifique, c'est à modifier :))
(et la fonction ne doit pas être top rapide si l'arborescence
est touffue, non ?)
Pareil pour l'utilisation à outrance de variables globales
inutiles ($connect_id_rubrique ??). Et le code suivant n'a
l'air de servir à rien (jamais appelé ?) :
if ($perso_activer_messagerie){
$connect_activer_messagerie=$perso_activer_messagerie;
$connect_activer_imessage=$perso_activer_imessage;
}
D'autre part, on peut désactiver la messagerie dans la config
précise, mais ce n'est pas testé dans inc_auth, d'où la mise
à jour systématique des dates de connexion, j'imagine.
Je passe sur les problèmes de typo dans le code (pas d'espaces
à certains endroits, etc. ;-))
En plus, comme c'était dans la 1.1, il aurait été bien de
fignoler cette version, et la sortir, avant de se lancer
dans la 1.2 (enfin bon, je l'ai déjà dit).
- dans uzine, l'inscription des rédacteurs est réglée sur "non"
dans la config. Or, j'imagine qu'on peut toujours s'inscrire
depuis l'espace public, non ?
- je vais oser un troll, mais plutôt que $spip_version_afficher,
ce serait pas mieux $spip_version_affichee ?
a+
Antoine.
Coucou,
Heu, c'est juste pour proposer que les corrections de bugs se
concentrent plut=F4t sur les versions d=E9j=E0 sorties, quand les bugs
ne sont pas li=E9s aux nouvelles fonctionnalit=E9s. Vu que la 1.2,
=E0 mon avis, ne sortira pas avant quelques semaines.... J'ai
r=E9percut=E9 la modif du inc_base sur la 1.0.5, en passant.
Oui oui, t'as bien fait. Je l'avais port=E9 dans la 1.2, parce que je
ne voulais pas en oublier (les nouvelles lignes puis la 1.0.5) et
qu'il y a certainement d=E9j=E0 des testeurs de la 1.2.
Sinon, y a deux trois trucs =E0 corriger en priorit=E9 :
Je m'y mets.
- les probl=E8mes de s=E9curit=E9 avec le nouveau statut d'admin
de rubrique.
Je regarde s'il y a des oublis.
Cela dit, il y a d=E9j=E0:
- le fait qu'un admin restreint ne peut pas modifier les acc=E8s des
admins (ni lui ni les autres), ni cr=E9er de nouvel admin;
- un admin (acc=E8s complet) ne peut pas se passer lui-m=EAme en acc=E8s res=
treint;
- plusieurs v=E9rifications sur les articles, et sur les br=E8ves. Je
vais v=E9rifier de ce c=F4t=E9 s'il reste des oublis.
Aussi, si le inc_auth pouvait =EAtre mis au propre,
ce serait bien....
Une fonction qui s'appelle sous_rubriques
alors qu'elle fait quelque chose de sp=E9cifique, c'est =E0 modifier :))
"recuperer_sous_rubriques" ?
(et la fonction ne doit pas =EAtre top rapide si l'arborescence
est touffue, non ?)
Exact. Mais j'ai pr=E9f=E9r=E9 faire simple et robuste et ne pas me lancer
dans un bidouillage approximatif. L=E0, c'est ultra-basique, mais c'est
tr=E8s clair. J'aurais pu m'inspirer du calcul des rubriques actives,
mais j'ai eu peur de faire des b=EAtises...
Pareil pour l'utilisation =E0 outrance de variables globales
inutiles ($connect_id_rubrique ??).
Ah? $connect_id_rubrique est la liste de toutes les rubriques
accessibles =E0 l'admin =E0 acc=E8s restreint. Je le calcule une fois (dans
inc_auth), ensuite dans les fonctions de v=E9rification, il faut bien
que je l'appelle en variable globale. (Cela =E9videmment dans l'=E9tat
actuel de mes connaissances. 
Et le code suivant n'a
l'air de servir =E0 rien (jamais appel=E9 ?) :
if ($perso_activer_messagerie){
$connect_activer_messagerie=3D$perso_activer_messagerie;
$connect_activer_imessage=3D$perso_activer_imessage;
}
C'est utilis=E9 dans la page d'=E9dition de l'auteur, lorsqu'il modifie
sa propre fiche pour activer ou non sa messagerie interne. Comme
inc_auth est appel=E9 en tout premier et que la modification dans la
base de donn=E9es intervient ensuite, j'ai besoin de forcer
temporairement en fonction de la modif effectu=E9e par l'utilisateur
(sinon le r=E9sultat de ses modifications personnelles sur "Messagerie
interne" interviendraient avec un tour de retard).
D'autre part, on peut d=E9sactiver la messagerie dans la config
pr=E9cise, mais ce n'est pas test=E9 dans inc_auth, d'o=F9 la mise
=E0 jour syst=E9matique des dates de connexion, j'imagine.
C'est vrai. Mais inc_meta est appel=E9 apr=E8s inc_auth, donc je ne peux
pas utiliser cette donn=E9e =E0 ce moment-l=E0.
- dans uzine, l'inscription des r=E9dacteurs est r=E9gl=E9e sur "non"
dans la config. Or, j'imagine qu'on peut toujours s'inscrire
depuis l'espace public, non ?
Je viens d'installer le inc-calcul que j'avais sous la main, et le
fonctionnement est devenu coh=E9rent. Je suppose que je n'avais pas
fait de mise-=E0-jour de inc-calcul depuis la 1.0.4.
- je vais oser un troll, mais plut=F4t que $spip_version_afficher,
ce serait pas mieux $spip_version_affichee ?
Yop. C'est chang=E9... Je t=E9l=E9chargerait =E7a dans spip-dev ASAP.
Amicalement,
ARNO*
Salut,
Je viens de t=E9l=E9charger la version 1.2.5. Puisque d=E9sormais on
affiche directement le num=E9ro dans l'espace priv=E9, j'ai effac=E9 dans
spis-dev le dossier "SPIP-1.2.3", et le dossier de spip-dev pour
cette version redevient plus simplement "SPIP-1.2".
Il s'agit d'une version "de consolidation" (je crois qu'on dit comme
=E7a), c'est-=E0-dire sans nouveaut=E9s, mais avec des corrections.
Essentiellement:
- nettoyage (pour ce que je sais en faire) de inc_auth.php3;
- consolidation des v=E9rifications d'acc=E8s pour les admins =E0 acc=E8s
restreint sur quelques fichiers;
- dans 'messagerie.php3", cosm=E9tique: correction du texte de la
colonne de gauche (le logo "rouge" devient bien "vert) :-))
- dans 'message.php3', un peu de cosm=E9tique et, un poil plus
important, quand on ajoute un auteur (soit par le menu d=E9roulant,
soit par recherche), on ne peut pas ajouter un auteur qui a d=E9sactiv=E9
la messagerie et l'exp=E9diteur ne peut pas s'ajouter comme
destinataire.
Amicalement,
ARNO*
Bugs dans 1.2.5
Quand je valide un pense bete avec affichage dans calendrier:
Warning: 0 is not a MySQL result index in inc.php3 on line 956
Warning: 0 is not a MySQL result index in inc.php3 on line 958
quand je clique dans suivre les forums
Warning: 0 is not a MySQL result index in
/usr/local/monsite.com/httpdocs/ecrire/controle_forum.php3 on line 228
Warning: 0 is not a MySQL result index in
/usr/local/monsite.com/httpdocs/ecrire/controle_forum.php3 on line 124
Warning: 0 is not a MySQL result index in
/usr/local/monsite.com/httpdocs/ecrire/controle_forum.php3 on line 126
quand je cree un article et je valide j'obtiets dans la fenetre auteur ...
Warning: Variable used in global statement already exists in the function
in /usr/local/monsite.com/httpdocs/ecrire/articles.php3 on line 61
et en dessous de l'interface:
Warning: 0 is not a MySQL result index in inc.php3 on line 956
Warning: 0 is not a MySQL result index in inc.php3 on line 958
voila pour l'instant ...
CM
Salut tout le monde,
Je vais faire vite, car il est tard (ou t=F4t). Je rentre d'une soir=E9e
de travail avec Fil et Antoine, j'ai pris des notes et je les
transcris ici comme m=E9mento de ce qui reste =E0 faire avant la
prochaine release officielle (qui sera la 1.2).
=46il et Antoine travaillaient encore quand je les ai quitt=E9s, je
suppose qu'ils livreront d=E9j=E0 une version modifi=E9e dans la matin=E9e,
ils vous diront ce qu'ils ont modifi=E9.
Ce que j'ai not=E9:
- Pour la gestion des admins =E0 acc=E8s restreint, optimiser la fonction
"recuperer_sous_rubriques" dans inc_auth.
- Modifier les relations entre les auteurs et les messages
(messagerie interne). Ajouter une colonne "id_auteur" pour les
messages, sucrer la colonne du type de lien (exp=E9diteur,
destinataire) dans "spip_auteurs_messages".
- Dans la m=EAme id=E9e, moins jouer sur la notion de "exp=E9diteur" et de
"destinataire"; l'exp=E9rience sur uZine montre que, avec les forums
associ=E9s aux messages, il ne s'agit d=E8s lors plus r=E9ellement de
messages de l'un vers l'autre, mais d'un espace de discussion entre
deux personnes, o=F9 la notion d'exp=E9diteur et de destinataire n'est
pas tellement pertinente. Du coup, trouver une phras=E9ologie plus
adapt=E9e dans l'interface.
- Am=E9liorer la suppression de l'exp=E9diteur (puisque d=E9sormais
l'auteur du message sera stock=E9 dans le message lui-m=EAme).
- Placer le lien "Nouveau pense-b=EAte" sur chaque page dans la colonne
de gauche (sous la liste des r=E9dacteurs connect=E9s).
- Utiliser la messagerie pour permettre aux admins de placer des
messages d'accueil sur la page "A suivre" (annonces importantes,
etc.). Voir si le forum interne peut =EAtre remplac=E9 par des messages
associ=E9s =E0 des forums? (pertinence? interface?).
- fixer une dur=E9e de vie plus limit=E9e au cookie des admins.
- dans les squelettes, plus grande souplesse dans l'utilisation des
minuscules et des majuscules.
- dans les forums de l'espace priv=E9, faire passer l'id_message,
id_article... dans les r=E9ponses aux forums (pour l'instant on ne
stocke que l'id_parent, c'est peu pratique =E0 g=E9rer).
- permettre d'associer les mots-cl=E9s aux br=E8ves.
- Crit=E8re d'exclusion dans les s=E9lections (!=3D, !>...)
- ajouter le crit=E8re {meme_mot} dans les boucles articles
(fonctionnement similaire =E0 {meme_auteur}).
- am=E9liorer le fonctionnement du menu d=E9roulant de la liste des
rubriques ("cet article se trouve dans dans la rubrique..."). Sur un
site qui contient de tr=E8s nombreuses rubriques, le menu est
interminable et c'est tr=E8s p=E9nalisant.
Amicalement,
ARNO*
Fil et Antoine travaillaient encore quand je les ai quittés, je
suppose qu'ils livreront déjà une version modifiée dans la matinée,
ils vous diront ce qu'ils ont modifié.
Oui. En fait nous avons :
* fini de transférer les modifs de la 1.0.5 dans la 1.2
* corrigé un bug dans la création automatique de nouveaux rédacteurs
* nettoyé en partie le code
* et SURTOUT ajouté les mots-clés sur les brèves (les squelettes par défaut
comprennent, dans article.html et dans breve.html, un exemple de boucle sur
les brèves/mots-clés. Dans le même mouvement l'affichage des mots-clés a été
un peu simplifié (on ne met plus le nombre d'articles liés à ce mot-clé dans
la partie "articles.php3", c'est peut-être à revoir)
-> le tout forme la version 1.2.6 (disponible dans /devel/ sous le nom 1.2)
- permettre d'associer les mots-clés aux brèves. (FAIT)
- ajouter le critère {meme_mot} dans les boucles articles
(fonctionnement similaire à {meme_auteur}). (JE CROIS QUE çA EXISTE DÉJÀ)
tchüß
-- Fil
@ Fil (fil@rezo.net) :
> - ajouter le critère {meme_mot} dans les boucles articles
> (fonctionnement similaire à {meme_auteur}). (JE CROIS QUE çA EXISTE DÉJÀ)
En fait, désolé, je me suis mélangé les pinceaux, à cause du nom donné à la
boucle ci-dessous ;). Mais voici donc, en attendant, une boucle permettant
d'afficher 10 des brèves associées à un mot-clé :
<BOUCLE_meme_mot_breves(BREVES){id_mot}{doublons}{0,10}{par hasard}>
<BR><IMG SRC="puce.gif"> <I><A HREF="#URL_BREVE">#TITRE</A></I>
</BOUCLE_meme_mot_breves>
-- Fil
Coucou,
Je viens de changer le nom de la version en "1.2 beta 6".
Ce qui est plus logique car quand on sortira cette version,
elle s'appellera 1.2 tout court, non 1.2.15...
a+
Antoine.
bonne idée l'histoire du beta ...
je m'envais tester de ce pas ....
est-ce que 1.2.6b corrige les bugs que j'ai envoyé hier soir ?
Christophe
Salut,
Un changement presque invisible mais important
dans la beta 7 : j'essaie de détecter que PHP
est exécuté en module Apache (inc_version,
variable $php_module). Si oui, alors le .htaccess
n'est pas généré et l'authentification PHP est
utilisée à la place. J'espère que ça améliorera
les problèmes de compatibilité avec certains
hébergeurs. De plus, ça élimine le test spécifique
Altern et Windows.
J'ai testé en local (Windows), et sur Altern, Free,
Multimania, l'Autre Net, Kilio, Rezo.net. Ca marche.
a+
Antoine.
@ Christophe Mannino (christophe.mannino@zenblade.com) :
est-ce que 1.2.6b corrige les bugs que j'ai envoyé hier soir ?
A priori non. Nous n'avons pas retouché les "pense-bête", qui font partie
d'un lot de modifs plus important ... à venir (cf. mail d'ARNO)
-- Fil
on peut avoir une version zip please pour 1.2.7 ?
merci
Christophe