Needs work
Project:
Omniture Integration
Version:
6.x-1.0
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
6 Oct 2008 at 21:35 UTC
Updated:
14 Dec 2012 at 02:57 UTC
Jump to comment: Most recent file
Comments
Comment #1
gregglesMaybe this could be handled by #182201: replace omniture.inc system with hook_omniture_variabless and document it?
If not, perhaps the proposed hook needs to be enhanced.
I'm not sure a themeable function is the right way to go for this, but maybe so.
Comment #2
ultimateboy commentedOne quick fix until the hook system is implemented would be to remove all variables from omniture.module and move them all to the omniture.inc file.
However, I think that a theme function or even a tpl file might be the better way to go. This module is creating code to be included in the page and in my mind, this output should be themable. The problem with this method is that it makes creating a hook system harder. I am not completely set that the hook system is the best option, but even as I write this post I am leaning more and more in that direction.
Comment #3
cyberswat commentedhttp://drupal.org/node/182201#comment-1046683
Comment #4
gregglesYeah, we could do both.Provide the hook first and then hand off those variables to a theme function which is responsible for outputting the actual JS...
I guess for now let's leave this as a duplicate, but if your use case makes a theme function seem valuable then certainly we can re-open it.
Comment #5
Dean Reilly commentedI'm reopening this ticket as the linked to ticket seems to mainly focusing on implementing hook_omniture_variables() which, while definitely helpful, doesn't provide enough control over the output of this module.
I have:
- rewritten the footer hook to use a theme function instead;
- removed the header and footer variables from hook_omniture_variables(); and
- had the variables use just their names rather than javascript fragments.
The main point of this was because I use this module in sites that run versions G and H of omniture and I don't want to have to maintain multiple versions of modules which implement hook_omniture_variables() for each version.
Comment #7
Dean Reilly commented#5: omniture-use-a-theme-function-317830-5.patch queued for re-testing.
Comment #9
bleen commentedplease ignore the test failures here pending #1866370: add simpletests for 6.x