Viewing tag pages, or the main page with a list of articles results in a page with "0 view this month."

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ram4nd’s picture

Component: User interface » Reports module
Status: Active » Needs work

I have the same problem. UTF-8 characters are urlencoded and it doesn't work. I bet it requires a simple url decode.

ram4nd’s picture

FileSize
930 bytes
ram4nd’s picture

Same issue seems to apply on the newer branch.

rongok’s picture

I had the same issue. The patch in #2 fixed the issue.

ram4nd’s picture

Status: Needs work » Reviewed & tested by the community
grendzy’s picture

Status: Reviewed & tested by the community » Closed (duplicate)

I believe this is fixed in #1345542: pull defaultPage from management API, instead of assuming index.html.

The proposed patch won't work, because it will just break it for people with the default "index.html".

ram4nd’s picture

Main fix is the urldecode.

I don't know what you mean by default "index.html".

ram4nd’s picture

FileSize
847 bytes

This will still be valid.

ram4nd’s picture

Status: Closed (duplicate) » Needs review
grendzy’s picture

I will try to clarify... when a pageView is recorded on the home page, Google tracks the URL according to the "defaultPage" setting in your account profile. Early versions of this module had an error and assumed that defaultPage was always "index.html" (which I think is Google's default-default, at least on older accounts).

This patch makes the same error in reverse, by assuming the the defaultPage is always "/". If you "fix" it for one group of users, you will simply break it for the other, which is why the correct solution is in #1345542: pull defaultPage from management API, instead of assuming index.html.

ram4nd’s picture

No I am not talking about front page any more. I am talking about other characters in url, that come in urlencoded shape. For the module to work you need to decode them first for the api.

grendzy’s picture

I see, that makes sense.

  • Plazik committed 2dbca70 on 7.x-3.x authored by ram4nd
    Issue #2241927 by ram4nd: "0 views this month" generated pages
    

  • Plazik committed d8c7d5b on 7.x-1.x authored by ram4nd
    Issue #2241927 by ram4nd: "0 views this month" generated pages
    
Plazik’s picture

Status: Needs review » Fixed

@ram4nd, thanks for patch! Commited to 7.x-1.x and 7.x-3.x.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.