@fdellwing opened this Issue on May 11th 2018 Contributor

https://matomo.org/docs/tracking-api/#use-case-tracking-data-using-the-php-client: This page has a directlink to download the MatomoTracker.php -> http://demo.matomo.org/index.php?module=SitesManager&action=downloadPiwikTracker&idSite={$IDSITE}&piwikUrl=http://piwik.example.org/

But the file it downloads is not very helpful.

<?php
/**
 * Piwik - free/libre analytics platform
 *
 * <a class='mention' href='https://github.com/link'>@link</a> http://piwik.org
 * <a class='mention' href='https://github.com/license'>@license</a> http://www.gnu.org/licenses/gpl-3.0.html GPL v3 or later
 *
 */

// PiwikTracker.php is now managed by composer. To prevent breaking existing
// code, this file has been left as a redirect to its new location in the
// vendor directory.
if (!class_exists('PiwikTracker')) {
    require_once __DIR__ . '/../../vendor/piwik/piwik-php-tracker/PiwikTracker.php';
}

if (PiwikTracker::VERSION !== 1) {
    throw new Exception("Expected PiwikTracker in libs/PiwikTracker/PiwikTracker.php to be version 1 for keeping backward compatibility.");
}

Instead it should link to https://raw.githubusercontent.com/matomo-org/matomo-php-tracker/master/PiwikTracker.php or https://github.com/matomo-org/matomo-php-tracker/releases/latest

(Or you could rework the Action if you really want to)

@Findus23 commented on May 11th 2018 Member

@fdellwing Many thanks for the info.

As a quick fix I replaced the URL with https://raw.githubusercontent.com/matomo-org/matomo-php-tracker/master/PiwikTracker.php, but I think the php-tracker doc may need to be rewritten and moved to https://developer.matomo.org/api-reference/PHP-Piwik-Tracker as currently this refers to the README which refers back to the dev-docs.
I'm not familiar with the PHP client so I can't really help

@tsteur commented on May 13th 2018 Member

@fdellwing I think I fixed this, can you double check please?

@fdellwing commented on May 14th 2018 Contributor

Well there is the link to the github repo for some days now, so it seems fixxed for now?

@tsteur commented on May 14th 2018 Member

thx

This Issue was closed on May 14th 2018
Powered by GitHub Issue Mirror