Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update webtrends to 9.4.0 #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

kasperg
Copy link
Contributor

@kasperg kasperg commented Jul 20, 2011

Updated codebase to latest based on what has been sent to Helsingør Libraries from KPI index.

@kasperg
Copy link
Contributor Author

kasperg commented Jul 21, 2011

When we update we should tag the version 1.2.0.

A few notes regarding the markup added to the footer:

According to the code sent to Helsingør libraries it should be:

<img alt="DCSIMG" id="DCSIMG" width="1" height="1" src="//visionize10.visionize.dk/dcsc6jmhht3uwyqgqs2ngnzic_2h8s/njs.gif?dcsuri=/nojavascript&amp;WT.js=No&amp;WT.tv=9.4.0&amp;dcssip=www.helsbib.dk"/>

There seems to be a few errors here which I have corrected:

  • The src does not contain a protocol. I have added http:
  • the dcssip argument is no longer prefixed with WT. I have readded this prefix.

The result:

<img alt="DCSIMG" id="DCSIMG" width="1" height="1" src="http://visionize10.visionize.dk/$dcs_id/njs.gif?dcsuri=/nojavascript&amp;WT.js=No&amp;WT.tv=9.4.0&amp;WT.dcssip=$domain"/>

Someone which more experience with WebTrends feel free to correct me.

@cableman
Copy link
Member

I think that the src should be // (without prefix) as that will default back to the protocol used by the page. In this case it will allow both https and http.

@kasperg
Copy link
Contributor Author

kasperg commented Aug 16, 2011

I did not know that.

In that case I'm more hesitant regarding the other change as that might be intentional as well. @cableman can you test on one of your installations?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants