@sgiehl opened this Issue on July 17th 2011 Member

The mobile image search of google is not detected at all:

http://www.google.com/m/imgres?gl=fr&source=mog&client=ms-android-google&q=piwik&hl=fr&start=0&site=images&imgid=ANd9GcQxUscuUydTOpon0arCcnJsjK4HF4CHQe1xB72QGlnC4GOYD3duGPPHtqFU&imgsz=139568&imgurl=http://piwik.org/wp-content/uploads/2008/11/piwik-e280ba-web-analytics-reports7.png&imgrefurl=http://fr.piwik.org/&ht=367&wd=622&tbnh=80&tbnw=136&ei=bgUiTvikOtPo8QPlgruUAw&ved=0CAkQ9QEwAA

Problem seems to be the special handling for google image search in core/Common.php:

if($searchEngineName === 'Google Images'
    || ($searchEngineName === 'Google' && strpos($referrerUrl, '/imgres') !== false) )
{
    $query = urldecode(trim(self::getParameterFromQueryString($query, 'prev')));
    $query = str_replace('&', '&', strstr($query, '?'));
    $searchEngineName = 'Google Images';
}

/imageres is in the url, but there is no prev parameter...

@anonymous-piwik-user commented on August 7th 2011

Attachment: Associated Patch
gi.diff

@anonymous-piwik-user commented on August 7th 2011

Here is a patch I'm using to identify these searches as Google Images. Replace 1385-1391 block in Common.php (from trunk)

        if($searchEngineName === 'Google Images'
    || ($searchEngineName === 'Google' && strpos($referrerUrl, '/imgres') !== false) )
        {
            if(strpos($referrerUrl, '&prev=') !== false)
            {
                $query = urldecode(trim(self::getParameterFromQueryString($query, 'prev')));
                $query = str_replace('&', '&', strstr($query, '?'));
            }
        $searchEngineName = 'Google Images';
    }
@anonymous-piwik-user commented on August 16th 2011

The unit test patch for this is on the forum:
http://forum.piwik.org/read.php?2,80107

@robocoder commented on September 10th 2011 Contributor

(In [5143]) fixes #2569, fixes #2616, refs #2652, adds Google Video

This Issue was closed on September 10th 2011
Powered by GitHub Issue Mirror