Closed (fixed)
Project:
Paging
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Sep 2007 at 19:22 UTC
Updated:
15 Aug 2009 at 06:24 UTC
Jump to comment: Most recent file
Comments
Comment #1
dsp1 commentedoops, did not think it needed the code tag.
all nodes listed as a teaser or full page shows
<!--paging_filter-->right before<p>the nodes do not have a
<!--pagebreak-->even if the node type is not checked in admin > site conf > paging they still show.
does not make sense why
<!--paging_filter-->would be output.unchecking paging in admin > site conf > input format >Filtered HTML makes the
<!--paging_filter-->go away.of course, that turns off the paging feature.
Comment #2
Gurpartap Singh commented<!--paging_filter-->is just a helper for paging functionality.<!--pagebreak-->is different from it, and is supposed to only indicate separation points, not to display in final content.Comment #3
dnewkerk commentedFor other people's reference... I believe what was causing dsp1's issue was the order of the filters in his Input format. I was able to repeat the same results if the Paging filter did not come after the contributed HTML Corrector filter (it may also occur in relation to other filters as well, but this is the one I had personally that I identified as the clear culprit). However, HTML Corrector is in Drupal 6 core now as well, so this may be a more common occurrence once people begin using Drupal 6.
If the mysterious
<!--paging_filter-->comes up printed into your page's output (which it could in the case of disabling and re-enabling other unrelated filters, since doing so can change the order that they previously were in), fortunately it's easy to fix. For the Filtered HTML input format (other input format paths will be a little different) go to: admin/settings/filters/1/order (the Rearrange tab) and set Paging to a weight of 10 (or any weight that ensures it remains below HTML Corrector (or whichever filters are causing the issue).Note that I've added
<!--paging_filter-->to this issue's title so that people with the same problem will be able to find it more easily.Hope this helps.
Comment #4
dsp1 commentedKeyz, thanks for the suggestion.
I set Smileys filter and Gallery2 filter to 9 and Paging to 10.
unfortunately
<!--paging_filter-->is still being output to the final content.<!--paging_filter-->repeats on the front page for ever node listed.I am not using the HTML Corrector filter.
drupal 5.3, gallery2 dev-2.x, Smiley 1.0
Comment #5
dnewkerk commenteddsp1... Can you attempt to disable all other non-core filters to see if any particular one of them is to blame? Also, please clear Drupal's cache... if you have Devel module you can do that easily through one of its menu items, or you can manually empty the appropriate mysql cache table if you prefer. I noticed last night that changes to input filters don't always "catch on" until I've cleared the cache.
Comment #6
dsp1 commentedKeyz, I unchecked all filters including HTML filter except Paging filter.
and still
<!--paging_filter-->repeats on the front page for every node.Comment #7
dnewkerk commentedHmm dsp1... not sure what to tell you. I tested to be sure in both 5.x-1.0 and 5.x-1.x-dev and didn't get the bug other than when I had the input formats in the wrong order (e.g. HTML filter or HTML Corrector after Paging in the Rearrange tab... if HTML filter was after Paging all I got was the effect of no paging - no extra code on the page... the code only appears for me when: HTML filter -> Paging -> HTML Corrector).
I would suggest testing, in this order:
<!--paging_filter-->code has not become embedded in your node's body field (not sure if this can happen, but if it were me I'd check for it, so just mentioning it). Save the node and see if anything changes.Hope this helps.
Comment #8
sunIf
<!--paging_filter-->is displayed, then this issue is a duplicate of #97182: <!--break --> is transformed into html code with lt and gt.If it just appears in the resulting HTML markup (as a comment), but isn't visible in the browser, then this issue is valid.
Paging module strips all instances of
<!--pagebreak-->, but does not remove its helper tag<!--paging_filter-->.Comment #9
sunOption 2) is true. Attached patch removes the comment from output.
Comment #10
Gurpartap Singh commentedFixed in HEAD.
http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/paging/pagi...
Comment #11
Gurpartap Singh commentedNeeds to be comitted to Drupal 5 branch.
Comment #12
sunUsage of global variables is not recommended. If there is really no other way, then the global needs to have a leading underscore and also start with the module's name to avoid module namespace issues.
Comment #13
BradM commentedJust FYI, in my setup, I had to have it filter for
<!--paging_filter-->(as in changing the first<to<). I'm using the line break converter and html corrector, and paging is at the bottom.Comment #14
Gurpartap Singh commented<!--paging_filter-->is now stripped from the output in 5.x-1.x.http://drupal.org/cvs?commit=166579
Thanks!
Comment #16
alexkb commentedIn the D6 version, I still seem to be getting this random '<!--paging_filter-->' html being inserted into the start of items when displaying them views (without any body text thats more then the pagining limit).
Has anyone else experienced this issue?
Comment #17
alexkb commentedJust to add to the above comment, I managed to get around this issue by having a field specific template file for the fields field:
views-view-field--body.tpl.php:
<?php print str_replace('<!--paging_filter-->', '', $output); ?>Hope that can help someone :)
Comment #18
mauryg commentedI posted the same issue as alexkb (#16 above) in node/433604 and then I found this thread. I followed the suggestion from Keyz (#7 above). I set the paging filter weight to 10 and all others above it and cleared the db cache tables using phpMyadmin. That got rid of the random
<!--paging_filter-->instances and the paging is working. But now I am seeing<!--break-->in all the places where I put a teaser break. Curioser and curiouser.MOVED TO #433604