We have a bogus call to pager_query() that actually executes an SQL query or two for no purpose whatsoever. Instead, we should just write a helper function since this is missing from core.

CommentFileSizeAuthor
#5 apachesolr-667110-D5.patch4.36 KBclaudiu.cristea
#2 bogus-pager-667110-2.patch2.38 KBpwolanin
#1 bogus-pager-667110-1.patch3.27 KBpwolanin

Comments

pwolanin’s picture

pwolanin commented
Title: Replace bogus use of page_query() » Replace bogus use of pager_query()
Status: Active » Needs review
StatusFileSize
new bogus-pager-667110-1.patch3.27 KB
pwolanin’s picture

pwolanin commented
StatusFileSize
new bogus-pager-667110-2.patch2.38 KB

oops part of another patch in there

pwolanin’s picture

pwolanin commented
Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Needs review » Patch (to be ported)

committed to 6.x-1.x.

Core issue for D7: #667112: There is no way to initialize a pager without running an SQL query

robertdouglass’s picture

robertdouglass commented
Version: 6.x-2.x-dev » 5.x-2.x-dev

#667110 by pwolanin: Fixed Replace bogus use of pager_query().

Good idea. Committed to 6.2

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
commented
Status: Patch (to be ported) » Fixed
StatusFileSize
new apachesolr-667110-D5.patch4.36 KB

Committed to DRUPAL-5--2, in #318126.

Patch attached.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

jpmckinney’s picture

jpmckinney commented

Marked #368227: Search Results Paging Won't Show duplicate.