Skip to content

Infinite Scroll: Apply post_gallery action with integer as third parameter#8015

Merged
dereksmart merged 1 commit intomasterfrom
add/third-parameter-to-do-action-post-gallery
Oct 19, 2017
Merged

Infinite Scroll: Apply post_gallery action with integer as third parameter#8015
dereksmart merged 1 commit intomasterfrom
add/third-parameter-to-do-action-post-gallery

Conversation

@oskosk
Copy link
Copy Markdown
Contributor

@oskosk oskosk commented Oct 18, 2017

This parameter has been present since 4.2 and by applying the filter here, we are currently breaking all callbacks that expect the $instance as third param.

Fixes #7939

Changes proposed in this Pull Request:

  • Makes the application of the post_gallery filter include a third parameter to not make PHP throw when calling callbacks that expect it.

This parameter has been present since 4.2 and by applying the filter here, we are currently breaking all callbacks that expect the $instance
@oskosk oskosk requested a review from a team as a code owner October 18, 2017 17:48
@oskosk oskosk changed the title Infinite Scroll: Do post_gallery action with integer as third parameter Infinite Scroll: Apply post_gallery action with integer as third parameter Oct 18, 2017
@jeherve jeherve added the Bug When a feature is broken and / or not performing as intended label Oct 19, 2017
Copy link
Copy Markdown
Member

@jeherve jeherve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jeherve jeherve added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Oct 19, 2017
@dereksmart dereksmart merged commit e81a75f into master Oct 19, 2017
@dereksmart dereksmart deleted the add/third-parameter-to-do-action-post-gallery branch October 19, 2017 14:23
@dereksmart dereksmart removed the [Status] Ready to Merge Go ahead, you can push that green button! label Oct 19, 2017
jeherve added a commit that referenced this pull request Oct 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug When a feature is broken and / or not performing as intended [Feature] Infinite Scroll

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants