WPIntell

Source evidence

External images

Peadig's Twitter Feed: Embedded Timeline WordPress Plugin · review · 2012-11-23T08:42:00+00:00

mixedsentiment
mediumseverity
0.8relevance
4replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

3 / 22 rows with source links

13.6% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

19 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

review · 3 stars
Ryan Hellyer unresolved
The 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 🙂

Comments

4 shown
Ryan Hellyer 2012-11-23T09:04:00+00:00

It seems there are some other images which need converted to https too. I’ll have another patch ready soon.

Ryan Hellyer 2012-11-23T09:19:00+00:00

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.

Ryan Hellyer 2012-11-23T09:42:00+00:00

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

Alex Moss 2013-03-22T17:18:00+00:00

Hey Ryan, The PRO version of the plugin now has this option 🙂