ExampleFeedburner broken #1367

Closed
robocoder opened this Issue May 20, 2010 · 5 comments

2 participants

@robocoder

1) the service is now only available via https:

2) the "dates" parameter needs to be timezone adjusted because days begin/end at 12am CDT (GMT-5); depending on the user's timezone, GetFeedData may only return a single day of stats, resulting in:

Error fetching the Feedburner stats. Expected XML, Got:
@robocoder

(In [2199]) refs #1367 - use https://

@halfdan
Piwik Open Source Analytics member

vipsoft: Regarding the timezone, shouldn't be enough to add the following lines?

$timezone = date_default_timezone_get(); // Save current timezone
date_default_timezone_set('CDT');
$yesterday = date('Y-m-d',mktime(0, 0, 0, date("m"), date("d")-1,   date("Y")));
$beforeYesterday = date('Y-m-d',mktime(0, 0, 0, date("m"), date("d")-2,   date("Y")));
date_default_timezone_set($timezone); // Restore timezone

CDT works, although it is not officialy supported according to the list of supported timezones (http://au2.php.net/manual/en/timezones.php). Alternatively we could use 'America/Chicago'.

@robocoder

Should. Since this is an example, we should probably use this opportunity to use Piwik_Date, making any necessary changes along the way.

@robocoder

(In [2205]) fixes #1367

@robocoder

(In [2206]) fixes #1367 - contrary to the Awareness API docs, the feed data wasn't updated until 12 am PDT

@robocoder robocoder added this to the Piwik 0.6.2 milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment