Project

General

Profile

Anomalie #2560

SVP et url non fonctionnelle

Added by cam.lafit - over 7 years ago. Updated over 7 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Category:
plugin
Target version:
Start date:
02/29/2012
Due date:
% Done:

0%

Resolution:
fixed
Navigateur:

Description

Yop

Si on désactive le javascript les url dans les titres des plugins ne sont pas fonctionnelles. L'url retournée est le ecrire/numerodeversionduplugin

Par exemple si on clique sur dump, l'url associée est /ecrire/1.6.3

svp.diff View (3.96 KB) b b, 03/16/2012 01:36 PM

svp.diff View - Complément pour fonctionner en mode non js (4.87 KB) cam.lafit -, 03/19/2012 11:19 AM

History

#1 Updated by b b over 7 years ago

Si je ne me trompe pas, il est acté que l'espace privé nécessite le javascript, du coup je me demande si ce bug est "valide".

Les liens dont tu parles servent normalement à déclencher le dépliage du bloc d'info de chaque plugin, on pourrait peut être les rendre fonctionnels sans js mais je crains que ça ne soit pas le seul problème qu'on rencontre sur cette page sans js (par exemple la boite de confirmation des actions de la page admin_plugin est gérée par mediabox).

#2 Updated by cam.lafit - over 7 years ago

Ciao

Il est possible en effet qu'il soit dit que le js soit obligatoire
dans l'espace privé. Pourquoi pas mais c'est dommage si ce n'est pas
obligatoire.
Dans le cas présent avoir une url fonctionnelle ne me semble pas idiot
js ou pas js.

#3 Updated by b b over 7 years ago

  • Assignee changed from Eric Lupinacci to cam.lafit -

Vu qu'on a dit que le privé nécessite javasacript, cela me semble être une perte de temps de gérer le coup de ces blocs dépliables sans js alors que pas mal de choses ne fonctionnent pas dans ce cas. À mon avis on devrait fermer ce ticket et le marquer comme invalide. Je te laisse décider de la suite à donner à l'affaire ;) (et je t'assigne le ticket au passage).

#4 Updated by b b over 7 years ago

Je viens de commiter une modif sur la page admin_plugin qui fait qu'on charge le bloc de détails d'un plugin en ajax natif du core :

http://zone.spip.org/trac/spip-zone/changeset/59036

Du coup cela devrait bien se dégrader sans js, tu peux me dire ce que ça donne chez toi ?

++

#5 Updated by cam.lafit - over 7 years ago

Yop

Ici ça se dégrade si bien que ça ne fait plus qu'un lien en #
Peut être pas le résultat escompté :)

Je regarde un peu mieux pour dire.

#6 Updated by cedric - over 7 years ago

  • Target version set to 3.0

oui on pourrait faire un peu mieux que ce js : il suffit de mettre nom/version/etat et image dans l'inclusion, et que le lien soit un lien ajax vers #SELF avec id_paquet=xx : ainsi avec js le bloc ajax se rechargera, sans js c'est toute la page qui se recharge, mais avec le bloc ouvert

#7 Updated by b b over 7 years ago

Exactement, j'avais aussi pensé à une solution de secours de ce style, mais je ne l'ai pas appliqué car toutes ces inclusions font partie du formulaire CVT admin_plugin. Du coup, il faudrait rajouter un argument id_paquet à celui-ci pour permettre le passage de cette variable aux inclusions utilisées par le formulaire.

Vu que Camille semble chaud sur la question, je le laisse nous faire ça ;)

#8 Updated by b b over 7 years ago

Voilà qui devrait régler le problème : http://zone.spip.org/trac/spip-zone/changeset/59354

Sans js, un clic sur le titre redirige sur la même page avec le bloc du plugin déplié. Un clic sur le titre du plugin déplié permet de le replier, et donc de virer l'id_paquet de l'url. C'est pas le top, mais ça fonctionne.

Je report ça sur ?exec=charger_plugin et on devrait pouvoir fermer le ticket.

#9 Updated by b b over 7 years ago

Je viens de tenter de reporter les commits en question sur la page de recherche des plugins, mais je ne comprends pas pourquoi lors d'un clic sur un lien qui doit normalement déclencher le chargement d'un des blocs ajax ciblé c'est tout le formulaire qui est rechargé...

Voici un patch pour tester par chez vous si vous le voulez.

#10 Updated by cam.lafit - over 7 years ago

Yop

Bon j'ai réussi à dépatouiller pour qu'en mode non js cela fonctionne.
Il manquait dans le code : * la transmission de l'ensemble des paramètres de recherche du formulaire dans les url des plugins, cela passe donc en GET * la fonction charger ne récupérait pas la variable id_paquet pour le contexte

Le problème réside encore cote js, le bloc est rechargé mais avec un contenu vide.

#11 Updated by cam.lafit - over 7 years ago

Complément d'information

La page plante sur le rechargement js si aucun bloc descriptif n'est déplié.
Si on charge une page avec le détail d'un des plugins présents alors le js fait ensuite son office.

#12 Updated by cedric - over 7 years ago

  • Status changed from Nouveau to Fermé
  • Resolution set to fixed

Also available in: Atom PDF