View Issue Details

IDProjectCategoryView StatusLast Update
0000282vhffs-usersdisplaypublic2016-05-06 22:48
Reportermzf Assigned Togradator  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Summary0000282: Les liens des numéros de page de la partie publique ne fonctionnent pas
DescriptionSur la page des projets de la partie publique, lorsque l'on fait une recherche ou qu'on liste un grand nombre de projet, les liens avec les numéros qui permettent de naviguer de page en page ne mettent pas à jour le contenu de la page.

Version de vhffs : 4.6-dev-a37c9f3359
Navigateur : Firefox 45.0.2 ou Internet Explorer 11
OS : Windows 7 ou Debian Stretch
Steps To ReproduceSur Tuxfamily.org :
* ouvrir la page : https://projects.tuxfamily.org/?do=allgroups;letter=v
beaucoup de projets commencent par la lettre 'v' donc il y a des numéros avec des liens pour passer d'une page à une autre.
* cliquer sur un des liens numérotés en haut de la page : "<< < 1 2 3 4 > >>"
* après une courte animation pour faire patienter la page n'est pas mise à jour.
TagsNo tags attached.

Activities

gradator

2016-04-27 09:33

administrator   ~0000451

En effet. Soucis javascript, cela fonctionne sans.

Merci ;-)

mzf

2016-05-06 21:35

reporter   ~0000452

J'ai essayé de comprendre ce qui se passait et déjà il y a le lien qui est en http standard alors que la page d'origine est en https. Et donc Firefox bloque la requête.
Message dans la console :
Blocage du chargement du contenu mixte actif (mixed active content) « http://projects.tuxfamily.org/?do=allgroups;letter=v;page=2 »

Je ne suis pas vraiment sûr que ce soit l'origine du problème car en autorisant le chargement de contenus mixte, la requête se fait bien mais la méthode est de type OPTIONS alors que l'on pourrait s'attendre à un GET standard.

Mais finalement, en modifiant les liens de http vers https dans l'inspecteur de Firefox le problème semble résolu. Je ne connais pas le code de vhffs mais ça vaut le coup de retester avec les liens qui sont bien en https.
(modification dans vhffs-public/templates/common/pager.tt ?)

En espérant que ça aura aidé à la résolution de ce problème :)

mzf

2016-05-06 22:20

reporter   ~0000453

En analysant la page, les liens généré par pager.tt comportent l'url en dur alors que les autres sont relatifs (comme par exemple les liens sur les lettres des projets).

Donc en supprimant "[% pager.url %]" dans pager.tt cela devrait fonctionner. J'ai généré le diff en pièce jointe, sans pouvoir le tester hélas.

mzf

2016-05-06 22:20

reporter  

patch.diff (1,969 bytes)   
diff --git a/vhffs-public/templates/common/pager.tt b/vhffs-public/templates/common/pager.tt
index 704397c..6ea158b 100644
--- a/vhffs-public/templates/common/pager.tt
+++ b/vhffs-public/templates/common/pager.tt
@@ -2,10 +2,10 @@
 [% IF (pager.size) %]
 <div class="pager">
 [% IF (pager.current_page != 1) %]
-<a class="first-page ajax" href="[% pager.url %]?[% pager.query_string %];page=1">&#60;&#60;</a>
-<a class="previous-page ajax" href="[% pager.url %]?[% pager.query_string %];page=[% pager.current_page - 1 %]">&#60;</a>
+<a class="first-page ajax" href="?[% pager.query_string %];page=1">&#60;&#60;</a>
+<a class="previous-page ajax" href="?[% pager.query_string %];page=[% pager.current_page - 1 %]">&#60;</a>
 [% FOR p IN pager.previous_pages %]
-<a class="previous-page-number ajax" href="[% pager.url %]?[% pager.query_string %];page=[% p %]">[% p %]</a>
+<a class="previous-page-number ajax" href="?[% pager.query_string %];page=[% p %]">[% p %]</a>
 [% END %]
 [% ELSE %]
 <span class="first-page-disabled">&#60;&#60;</span>
@@ -14,10 +14,10 @@
 <span class="current-page">[% pager.current_page %]</span>
 [% IF (pager.current_page != pager.last_page) %]
 [% FOR p IN pager.next_pages %]
-<a class="next-page-number ajax" href="[% pager.url %]?[% pager.query_string %];page=[% p %]">[% p %]</a>
+<a class="next-page-number ajax" href="?[% pager.query_string %];page=[% p %]">[% p %]</a>
 [% END %]
-<a class="next-page ajax" href="[% pager.url %]?[% pager.query_string %];page=[% pager.current_page + 1 %]">&#62;</a>
-<a class="last-page ajax" href="[% pager.url %]?[% pager.query_string %];page=[% pager.last_page %]">&#62;&#62;</a>
+<a class="next-page ajax" href="?[% pager.query_string %];page=[% pager.current_page + 1 %]">&#62;</a>
+<a class="last-page ajax" href="?[% pager.query_string %];page=[% pager.last_page %]">&#62;&#62;</a>
 [% ELSE %]
 <span class="next-page-disabled">&#62;</span>
 <span class="last-page-disabled">&#62;&#62;</span>
patch.diff (1,969 bytes)   

gradator

2016-05-06 22:48

administrator   ~0000454

Exact!. On a fait le même patch en même temps, mais j'ai commité le tiens, il est arrivé 2 minutes avant le mien :-)

https://git.tuxfamily.org/vhffs4/vhffs.git/commit/?id=eac06e85ba62b1af01db7a7a64d2de7b0db242a1

Et j'ai modifié pour que ce problème ne puisse plus arriver facilement:

https://git.tuxfamily.org/vhffs4/vhffs.git/commit/?id=cf45656e4b2df790a81e427a23af3bb5192fb3fb

Merci tout plein pour l'analyse, tu déchires :)

Sylvain

Issue History

Date Modified Username Field Change
2016-04-26 20:01 mzf New Issue
2016-04-27 09:33 gradator Note Added: 0000451
2016-04-27 09:34 gradator Assigned To => gradator
2016-04-27 09:34 gradator Status new => confirmed
2016-04-27 09:34 gradator Assigned To gradator => mzf
2016-04-27 09:34 gradator Status confirmed => assigned
2016-04-27 09:34 gradator Assigned To mzf => gradator
2016-05-06 21:35 mzf Note Added: 0000452
2016-05-06 22:20 mzf Note Added: 0000453
2016-05-06 22:20 mzf File Added: patch.diff
2016-05-06 22:48 gradator Note Added: 0000454
2016-05-06 22:48 gradator Status assigned => closed
2016-05-06 22:48 gradator Resolution open => fixed