Conversation
review · 3 starsThe WP Twitter Feed widget plugin loads images from an external source. They need to be hosted locally, both for performance reasons and to ensure they don’t break when using https. Here is a Diff. which fixes the problem. http://stuff.ryanhellyer.net/images-fix.diff EDIT: I wrote a plugin to correct for these problems … http://geek.ryanhellyer.net/products/wp-twitter-feed-fix/
It seems there are some other images which need converted to https too. I’ll have another patch ready soon.
There appears to be a dead function called twitterintentjs() in the plugin too. It may pay to remove that. It loads JS from Twitter anyway, and it would be nice to avoid having to do that.
I also noticed that it seems to be using output buffering, but I don’t think the buffer is ever actually used. I removed it and it seems to be working just fine without it. Here’s a new diff. http://stuff.ryanhellyer.net/images-fix2.diff
Hey Ryan, The PRO version of the plugin now has this option 🙂
It seems there are some other images which need converted to https too. I’ll have another patch ready soon.
There appears to be a dead function called twitterintentjs() in the plugin too. It may pay to remove that. It loads JS from Twitter anyway, and it would be nice to avoid having to do that.
I also noticed that it seems to be using output buffering, but I don’t think the buffer is ever actually used. I removed it and it seems to be working just fine without it. Here’s a new diff. http://stuff.ryanhellyer.net/images-fix2.diff
Hey Ryan, The PRO version of the plugin now has this option 🙂