Skip to content

Post Images: avoid errors when is_module_active doesn't exist#3116

Merged
samhotchkiss merged 1 commit intomasterfrom
fix-3114
Dec 10, 2015
Merged

Post Images: avoid errors when is_module_active doesn't exist#3116
samhotchkiss merged 1 commit intomasterfrom
fix-3114

Conversation

@jeherve
Copy link
Copy Markdown
Member

@jeherve jeherve commented Dec 9, 2015

Like on WordPress.com.
Error introduced in #3114. Sorry, I didn't think about that use case earlier.

Like on WordPress.com.
Error introduced in #3114
@jeherve jeherve added [Feature] Sharing Post sharing, sharing buttons [Feature] Publicize Now Jetpack Social, auto-sharing [Status] Needs Review This PR is ready for review. labels Dec 9, 2015
@jeherve jeherve added this to the 3.8.2 milestone Dec 9, 2015
@jeherve jeherve added the Bug When a feature is broken and / or not performing as intended label Dec 9, 2015
@dereksmart dereksmart 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 Dec 9, 2015
samhotchkiss added a commit that referenced this pull request Dec 10, 2015
Post Images: avoid errors when is_module_active doesn't exist
@samhotchkiss samhotchkiss merged commit 95878b5 into master Dec 10, 2015
@samhotchkiss samhotchkiss removed the [Status] Ready to Merge Go ahead, you can push that green button! label Dec 10, 2015
@samhotchkiss samhotchkiss deleted the fix-3114 branch December 10, 2015 17:00
georgestephanis pushed a commit that referenced this pull request Apr 12, 2017
When Photon is active, it resizes very large (`$too_big`) images to be 1200x1200px.
It also adjusts `og:image:width` and `og:image:height` to use these new values.
When Photon isn't active on the other hand, wp_get_attachment_image_src returns the original image. `og:image:width` and
`og:image:height` shouldn't be updated to 1200x1200.

Merges #3114 and #3116

Merges r131877-wpcom.
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] Publicize Now Jetpack Social, auto-sharing [Feature] Sharing Post sharing, sharing buttons Touches WP.com Files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants