Is there a plan for a Drupal 7 version of this module?

Comments

tudor.sitaru’s picture

Assigned: Unassigned » tudor.sitaru

Yes, a Drupal 7 version is coming this week.

andyzhang’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new11.24 KB

Updated to support D7. Updated webtrends.js to the version 9.4.0.
As D7 doesn't have hook_footer, replaced it with hook_page_alter, and also added webtrends.install file to remove module variables during uninstall.

jerdavis’s picture

Any word on getting this vetted and added to the d7 branch?

ramlev’s picture

When testing the patch in version 7.x-1.x-dev it cannot patch.

output from terminal
==================
$ git apply webtrends-d7-version-1147900-1.patch
webtrends-d7-version-1147900-1.patch:20: trailing whitespace.
// Version: 9.4.0
webtrends-d7-version-1147900-1.patch:122: trailing whitespace.
try{
webtrends-d7-version-1147900-1.patch:164: trailing whitespace.
}
error: patch failed: webtrends.info:3
error: webtrends.info: patch does not apply
==================

When cloning 7.x-1.x-dev from git.drupal.org then i can path the files, but with some warnings about trailing whitespaces.

output from terminal
==================
$ git apply webtrends-d7-version-1147900-1.patch
webtrends-d7-version-1147900-1.patch:20: trailing whitespace.
// Version: 9.4.0
webtrends-d7-version-1147900-1.patch:122: trailing whitespace.
try{
webtrends-d7-version-1147900-1.patch:164: trailing whitespace.
}
webtrends-d7-version-1147900-1.patch:251: new blank line at EOF.
+
warning: 4 lines add whitespace errors.
==================

mgifford’s picture

There is a dev release, shouldn't this be closed?

ronan’s picture

StatusFileSize
new10.5 KB

This applies cleanly to the 7.x dev branch for me but it still gives the settings a d6 path. This version moves the settings to config > system.