View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000282 | vhffs-users | display | public | 2016-04-26 20:01 | 2016-05-06 22:48 |
Reporter | mzf | Assigned To | gradator | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Summary | 0000282: Les liens des numéros de page de la partie publique ne fonctionnent pas | ||||
Description | Sur 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 Reproduce | Sur 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. | ||||
Tags | No tags attached. | ||||
|
En effet. Soucis javascript, cela fonctionne sans. Merci ;-) |
|
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 :) |
|
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. |
|
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"><<</a> -<a class="previous-page ajax" href="[% pager.url %]?[% pager.query_string %];page=[% pager.current_page - 1 %]"><</a> +<a class="first-page ajax" href="?[% pager.query_string %];page=1"><<</a> +<a class="previous-page ajax" href="?[% pager.query_string %];page=[% pager.current_page - 1 %]"><</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"><<</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 %]">></a> -<a class="last-page ajax" href="[% pager.url %]?[% pager.query_string %];page=[% pager.last_page %]">>></a> +<a class="next-page ajax" href="?[% pager.query_string %];page=[% pager.current_page + 1 %]">></a> +<a class="last-page ajax" href="?[% pager.query_string %];page=[% pager.last_page %]">>></a> [% ELSE %] <span class="next-page-disabled">></span> <span class="last-page-disabled">>></span> |
|
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 |
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 |