[spip-dev] Re: Quelques remarques ...

@ Antoine Pitrou (pitrou@free.fr) :

Non, c'était déjà corrigé.

Oui, c'est vrai (contrairement à ma première idée). Mais c'était à mon avis
mal corrigé: en effet, les brèves étaient baladées dans le nouveau secteur.
Désormais on interdit dès le départ de déplacer un secteur contenant des
brèves. Et ça me paraît nettement préférable (car en cas de fausse manip, il
était impossible de retrier à l'envers les brèves qui se seraient
mélangées).

PS: peux-tu expliquer à quoi correspond le premier bug cité dans TODO.txt ?

-- Fil

Fil wrote:

PS: peux-tu expliquer à quoi correspond le premier bug cité dans TODO.txt ?

Faut remonter dans les archives de la liste... La personne qui
l'a rapporté l'a fait à plusieurs reprises, en finissant par
demander si on l'avait bien pris en compte... elle va être
contente ;))

C'est un bug dans l'admin rubriques, donc à corriger par ceux
qui utilisent la fonctionnalité (Arno ?).

a+

Antoine.

Fil wrote:

Oui, c'est vrai (contrairement à ma première idée). Mais c'était à mon avis
mal corrigé: en effet, les brèves étaient baladées dans le nouveau secteur.
Désormais on interdit dès le départ de déplacer un secteur contenant des
brèves. Et ça me paraît nettement préférable (car en cas de fausse manip, il
était impossible de retrier à l'envers les brèves qui se seraient
mélangées).

Beurk :wink: Il vaudrait mieux mettre un simple avertissement non ?
Là c'est chiant, impossible de déplacer le secteur sans se palucher
à la main d'éventuelles dizaines de brèves....

Bon, en attendant, je modifie d'autres fichiers :
spip_image, inc.php3, index.php3, recherche.php3.

Tu pourrais tester la recherche sur le diplo ?

@ Antoine Pitrou (pitrou@free.fr) :

> Oui, c'est vrai (contrairement à ma première idée). Mais c'était à mon avis
> mal corrigé: en effet, les brèves étaient baladées dans le nouveau secteur.
> Désormais on interdit dès le départ de déplacer un secteur contenant des
> brèves. Et ça me paraît nettement préférable (car en cas de fausse manip, il
> était impossible de retrier à l'envers les brèves qui se seraient
> mélangées).

Beurk :wink: Il vaudrait mieux mettre un simple avertissement non ?
Là c'est chiant, impossible de déplacer le secteur sans se palucher
à la main d'éventuelles dizaines de brèves....

Pas évident de trouver la bonne interface, à la fois garde-fou mais
qui laisse cette possibilité. Un bouton rouge :wink:

Tu pourrais tester la recherche sur le diplo ?

La recherche dans l'espace privé ? Pas très rapide, mais fonctionnel. Je
suis sûr qu'on peut faire beaucoup mieux, mais je ne vois pas bien comment.
Mais là je vois que tu as fait sauter la case de recherche ???

Bon, enfin, bref, je crois que j'ai corrigé le bug des admins restreints
(dans rubriques_edit.php3) et sécurisé la chose (dans naviguer.php3).

Bonne nuit

-- Fil

@ Fil (fil@rezo.net) :

Mais là je vois que tu as fait sauter la case de recherche ???

Oups, tu l'as juste changé de place... Mais oublié de remettre la signature
SPIP $version_affichee en aligné à droite.

-- Fil

@ Antoine Pitrou (pitrou@free.fr) :

Tu pourrais tester la recherche sur le diplo ?

J'ai compris ! Ton nouveau module de recherche interne. Euh, c'est
risible... (par ailleurs petit détail la case "recherche" n'apparaît pas
dans la page recherche).

Résultats de la recherche :

tibet

Articles trouvés
LA DEMISSION DES CLERS. LA CRISE DES SCIENCES SOCIALES ET L'OUBLI DE LA
POLITIQUE Pfefferkorn_Roland 1er janvier 1994
« Risibles démocraties » 1er janvier 1994
Interminable attente 8,5 Eric de Decker a été, pendant trois mois,
volontaire de la Croix-Rouge, en charge de la Vojvodine. De sa mission, il a
rapporté des carnets de voyage détaillés, dont voici un court extrait :
1er janvier 1994
Fin de l'immobilisme en Côte-d'Ivoire Galy_Michel 1er janvier 1994
Irréductibles Indiens du Guatemala Lemoine_Maurice 1er février 1994
Impossible réconciliation en Angola et au Mozambique Brittain_Victoria ,
Watkins_Kevin 1er février 1994
Reléguer le bien public sur les bas-côtés Schiller_Herbert I. 1er mars
1994
L'empire Berlusconi transformé en machine de propagande Giordano
Luchini_Eduardo 1er mars 1994
DICTIONNAIRE BIOGRAPHIQUE DU MOUVEMENT OUVRIER FRANCAIS, tome 43
Dreyfus_Michel 1er mars 1994
LES LENDEMAINS QUI DECHANTENT.LE PARTI COMMUNISTE FRANCAIS A LA LIBERATION
Dreyfus_Michel 1er juin 1994

Brèves trouvées
Droits et Libertés face à l'informatisation de la société
Public citizen, Global Trade Watch
Communications for a Sustainable Future
Université de Bir Zeit
Front de Libération du Jammu et Cachemire
Ministère serbe de l'information
Parti libertaire
Association « 17 octobre 1961 contre l'oubli »

-- Fil

Corrections dans ecrire/ :

- inc.php3 pour rétablir un pied de page correct (à droite)

- rubriques_edit.php3 et naviguer.php3 pour finir les choses d'hier soir :

    -> accès restreint : interdire de déplacer une sous-rubrique dont le
       parent n'est pas accessible (on ne peut donc déplacer que des
       sous-rubriques de rubriques accessibles, et ce vers des rubriques
       accessibles, ce qui me paraît être le fonctionnement normal).

       Exemple : on m'affecte deux Rubriques : 1 et 2, avec leurs
       sous-rubriques (respectivement 10,11,12... et 20,21,22....)
       SPIP m'autorise à déplacer les rubriques 10,11, etc.. vers toute
       rubrique de ma hiérarchie, mais m'interdit de déplacer 1 ou 2.

    -> déplacement de secteur avec brèves vers un autre secteur : au lieu
       de l'interdiction "en dur" introduite hier soir, j'ajoute une
       case de confirmation à cocher si l'on veut "vraiment" déplacer
       la rubrique (et mêler les brèves à celles du secteur de destination).

- recherche.php3 : je passe ton code sophistiqué en commentaires et remets le
       mien (moins rapide, mais fonctionnel), en attendant que tu débuggues...

Bonjour,

À (At) 10:02 +0200 09/10/2001, Fil écrivait (wrote) :

- rubriques_edit.php3 et naviguer.php3 pour finir les choses d'hier soir :

    -> accès restreint : interdire de déplacer une sous-rubrique dont le
       parent n'est pas accessible (on ne peut donc déplacer que des
       sous-rubriques de rubriques accessibles, et ce vers des rubriques
       accessibles, ce qui me paraît être le fonctionnement normal).

Il faudrait aussi faire ça pour article_edit.php3 quand
l'administrateur est restreint. J'ai essayé d'insérer (pompé depuis
rubriques_edit.php3) :

  if ($connect_toutes_rubriques) {
    echo "<OPTION".mySel("0",$id_parent).">Racine du site\n";
  } else {
    echo "<OPTION".mySel("0",$id_parent).">Ne pas
d&eacute;placer...\n";
  }
  // si le parent ne fait pas partie des rubriques restreintes,
modif impossible
  if (acces_rubrique($id_parent)) {
    enfant(0);
  }

à la place de :

  enfant(0);

uniquement mais je n'obtiens que « Ne pas déplacer » pour l'instant
et non pas toutes les rubriques auxquelles il a droit. Sinon, l'adm.
restreint n'a qu'un menu local vide dans lversion b14 actuelle (avant
ma modif)...

Sinon, MERCI : l'adm. restreint peut éditer ses rubriques et ne peut
plus les déplacer ailleurs que chez lui.

Merci,

Gilles.

@ Gilles Perez-Lambert (Gilles.Perez@univ-montp3.fr) :

Il faudrait aussi faire ça pour article_edit.php3 quand
l'administrateur est restreint.

Voilà :
articles_edit.php3 (affichage du bon menu pour les admins restreints)
articles.php3 (sécurisation de la gestion de rubrique restreinte)

et, tant que j'y suis :
rubriques_edit.php3 (petit nettoyage du code)
inc_version.php3 (suite à la remarque d'Alain)
index.php3 (accent sur SE DÉCONNECTER)