Requires hook_features_pipe_COMPONENT_alter().
Can be added without use of hook_features_api() to "views_load_more.features.inc" since it is auto-detected.

Patch to follow.
Will need backport to D6.

CommentFileSizeAuthor
#7 1787260-views_load_more-features-support-7.patch1.93 KBm4olivei
#2 1787260-views_load_more-features-support-1.patch2.13 KBwjaspers
#1 1787260-views_load_more-features-support-1.patch2.14 KBwjaspers

Comments

wjaspers’s picture

wjaspers commented
StatusFileSize
new 1787260-views_load_more-features-support-1.patch2.14 KB

A D7 patch.

wjaspers’s picture

wjaspers commented
StatusFileSize
new 1787260-views_load_more-features-support-1.patch2.13 KB

D7 patch.
Untested, but similar code has worked just fine for other custom pagers.

wjaspers’s picture

wjaspers commented
Status: Active » Needs review

Looking at D6 information, this patch looks like it can be safely backported as well.

wjaspers’s picture

wjaspers commented
Title: Add support for features detection of views » Add support for features detection of views_load_more pager

Tested this patch against D6 as well.
Unfortunately, drupal_alter doesn't handle references very well, so this solution doesn't really work. The code runs, but will never add our dependency as expected.

brettbirschbach’s picture

brettbirschbach commented

For anyone trying to use this patch - need to change views_load_more_pager_features_pipe_views_view_alter() to views_load_more_features_pipe_views_view_alter() for the hook to be picked up.

m4olivei’s picture

m4olivei
he/him
Primary language English
Location Grimsby, ON
commented
Assigned: Unassigned » m4olivei
Issue summary: View changes

Assigning to myself for review.

m4olivei’s picture

m4olivei
he/him
Primary language English
Location Grimsby, ON
commented
StatusFileSize
new 1787260-views_load_more-features-support-7.patch1.93 KB

Here's a revised patch. I corrected the hook name, moved the hook into the module file, so it's always picked up, and took out the check for existing views_load_more dependency. We don't save much and returning at that point, missed jQuery Waypoints if it's needed.

m4olivei’s picture

m4olivei
he/him
Primary language English
Location Grimsby, ON
commented
Status: Needs review » Closed (fixed)

Committed to 7.x-1.x. Thanks!