BabDev / Tweet-Display-Back

Tweet Display Back is a module for Joomla! that allows users to display a user or list feed from Twitter on their site.
http://www.babdev.com/extensions/tweet-display-back
7 stars 7 forks source link

Double Tweets - 3.1.0 Beta #91

Open dirtyxa opened 9 years ago

dirtyxa commented 9 years ago

I have installed the beta because I could not get the 3.0.3 working without doubles, or to connect (with modified helper.php. I have set up the keys and all is working, except I still have double tweets. I have also tried using the latest helper.php from https://github.com/BabDev/Tweet-Display-Back/blob/master/helper.php and still have double tweets

mbabker commented 9 years ago

Can you try applying this patch? https://github.com/BabDev/Tweet-Display-Back/commit/6bf0178c374478336dde2ca30593d5ede9443d69

dirtyxa commented 9 years ago

I am already running the patch per your recommendation in another issue. Unfortunately I am still encountering the issue of double tweets. The site is 109hair.com

From: Michael Babker [mailto:notifications@github.com] Sent: Sunday, October 05, 2014 4:53 AM To: BabDev/Tweet-Display-Back Cc: Jeff Carlin Subject: Re: [Tweet-Display-Back] Double Tweets - 3.1.0 Beta (#91)

Can you try applying this patch? 6bf0178https://github.com/BabDev/Tweet-Display-Back/commit/6bf0178c374478336dde2ca30593d5ede9443d69

— Reply to this email directly or view it on GitHubhttps://github.com/BabDev/Tweet-Display-Back/issues/91#issuecomment-57931468.

mbabker commented 9 years ago

If you have caching enabled, can you post the .json file somewhere so I could use it for testing? The file will be in your cache folder, it'll have an ID that corresponds to the module's ID.

On Sun, Oct 5, 2014 at 8:40 AM, dirtyxa notifications@github.com wrote:

I am already running the patch per your recommendation in another issue. Unfortunately I am still encountering the issue of double tweets. The site is 109hair.com

From: Michael Babker [mailto:notifications@github.com] Sent: Sunday, October 05, 2014 4:53 AM To: BabDev/Tweet-Display-Back Cc: Jeff Carlin Subject: Re: [Tweet-Display-Back] Double Tweets - 3.1.0 Beta (#91)

Can you try applying this patch? 6bf0178< https://github.com/BabDev/Tweet-Display-Back/commit/6bf0178c374478336dde2ca30593d5ede9443d69>

— Reply to this email directly or view it on GitHub< https://github.com/BabDev/Tweet-Display-Back/issues/91#issuecomment-57931468>.

— Reply to this email directly or view it on GitHub https://github.com/BabDev/Tweet-Display-Back/issues/91#issuecomment-57934794 .

dirtyxa commented 9 years ago

http://109hair.com/tmp/tweetdisplayback_tweets-103.json

From: Michael Babker [mailto:notifications@github.com] Sent: Sunday, October 05, 2014 8:55 AM To: BabDev/Tweet-Display-Back Cc: Jeff Carlin Subject: Re: [Tweet-Display-Back] Double Tweets - 3.1.0 Beta (#91)

If you have caching enabled, can you post the .json file somewhere so I could use it for testing? The file will be in your cache folder, it'll have an ID that corresponds to the module's ID.

On Sun, Oct 5, 2014 at 8:40 AM, dirtyxa notifications@github.com<mailto:notifications@github.com> wrote:

I am already running the patch per your recommendation in another issue. Unfortunately I am still encountering the issue of double tweets. The site is 109hair.com

From: Michael Babker [mailto:notifications@github.com] Sent: Sunday, October 05, 2014 4:53 AM To: BabDev/Tweet-Display-Back Cc: Jeff Carlin Subject: Re: [Tweet-Display-Back] Double Tweets - 3.1.0 Beta (#91)

Can you try applying this patch? 6bf0178< https://github.com/BabDev/Tweet-Display-Back/commit/6bf0178c374478336dde2ca30593d5ede9443d69>

— Reply to this email directly or view it on GitHub< https://github.com/BabDev/Tweet-Display-Back/issues/91#issuecomment-57931468>.

— Reply to this email directly or view it on GitHub https://github.com/BabDev/Tweet-Display-Back/issues/91#issuecomment-57934794 .

— Reply to this email directly or view it on GitHubhttps://github.com/BabDev/Tweet-Display-Back/issues/91#issuecomment-57936736.

mbabker commented 9 years ago

Thanks, I'll take a look at it in the next day or two.

On Sun, Oct 5, 2014 at 10:06 AM, dirtyxa notifications@github.com wrote:

http://109hair.com/tmp/tweetdisplayback_tweets-103.json

From: Michael Babker [mailto:notifications@github.com] Sent: Sunday, October 05, 2014 8:55 AM To: BabDev/Tweet-Display-Back Cc: Jeff Carlin Subject: Re: [Tweet-Display-Back] Double Tweets - 3.1.0 Beta (#91)

If you have caching enabled, can you post the .json file somewhere so I could use it for testing? The file will be in your cache folder, it'll have an ID that corresponds to the module's ID.

On Sun, Oct 5, 2014 at 8:40 AM, dirtyxa <notifications@github.com<mailto: notifications@github.com>> wrote:

I am already running the patch per your recommendation in another issue. Unfortunately I am still encountering the issue of double tweets. The site is 109hair.com

From: Michael Babker [mailto:notifications@github.com] Sent: Sunday, October 05, 2014 4:53 AM To: BabDev/Tweet-Display-Back Cc: Jeff Carlin Subject: Re: [Tweet-Display-Back] Double Tweets - 3.1.0 Beta (#91)

Can you try applying this patch? 6bf0178<

https://github.com/BabDev/Tweet-Display-Back/commit/6bf0178c374478336dde2ca30593d5ede9443d69>

— Reply to this email directly or view it on GitHub<

https://github.com/BabDev/Tweet-Display-Back/issues/91#issuecomment-57931468>.

— Reply to this email directly or view it on GitHub < https://github.com/BabDev/Tweet-Display-Back/issues/91#issuecomment-57934794>

.

— Reply to this email directly or view it on GitHub< https://github.com/BabDev/Tweet-Display-Back/issues/91#issuecomment-57936736>.

— Reply to this email directly or view it on GitHub https://github.com/BabDev/Tweet-Display-Back/issues/91#issuecomment-57937044 .

dirtyxa commented 9 years ago

Any progress on this yet? Thanks

mbabker commented 9 years ago

Unfortunately no, I've had a busy month with a lot of Joomla related travel and tight work deadlines. Hopefully I'll get some time in the next couple of weeks to sit down and take a look since things are starting to slow down for me.

dchowins commented 8 years ago

Hello, I am having this problem with double tweets as well, and I am using version 3.1 which is not beta as far as I can tell. I have it set to pull and display the last 3 tweets. However, it displays only the last two tweets twice each for a total of four. So instead displaying Tweet 1, Tweet 2 & Tweet 3, the following is displayed: Tweet 1, Tweet 1, Tweet 2, & Tweet 2

Here is the system info

PHP Built On Linux potus 3.2.61-grsec-modsign #1 SMP Tue Aug 12 09:58:26 UTC 2014 x86_64 Database Version 5.6.25-log Database Collation utf8_general_ci Database Connection Collation utf8mb4_general_ci PHP Version 5.6.19 Web Server Apache WebServer to PHP Interface cgi-fcgi Joomla! Version Joomla! 3.5.0 Stable [ Unicorn ] 21-March-2016 22:00 GMT Joomla! Platform Version Joomla Platform 13.1.0 Stable [ Curiosity ] 24-Apr-2013 00:00 GMT User Agent Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.110 Safari/537.36

I looked at the cache files and there are all three tweets there and no duplicates in the cache file.

But for this problem, I like this extension a lot. I appreciate any help anyone can offer.

mbabker commented 8 years ago

I've fixed one duplication problem since the last release, replace the helper.php file with the one at https://raw.githubusercontent.com/BabDev/Tweet-Display-Back/5f115559004ed455fdbbeda7547d50fb09ea8c6e/helper.php

I'm working on a cleanup/update dropping the compatibility for Joomla versions earlier than 3.5 so until that's ready I don't see another release being tagged right now.