GoogleCodeExporter
commented
9 years ago I was experiencing the same behavior, and I think I know the source of the
issue.
I downloaded the source (thanks for providing this!) and started stepping
through the
code until I found a potential error inside of the _addIfNotEmpty() function.
The original source looks like this:
private function _addIfNotEmpty( arr:Array, field:String, value:String ):void
{
if(value != "")
{
value = value.split( "+" ).join( "%20" );
value = value.split( " " ).join( "%20" );
arr.push( field + value );
}
}
The issue is that _addIfNotEmpty() assumes that the parameter "value" will
never be
null (but apparently it assumes that value might be empty). I think that
somewhere
along the code path, some of the campaign config params are set to null (since
they
may not exist in certain contexts).
I made the following change to the function and it fixed the problem:
private function _addIfNotEmpty( arr:Array, field:String, value:String ):void
{
if(value && value != "")
{
value = value.split( "+" ).join( "%20" );
value = value.split( " " ).join( "%20" );
arr.push( field + value );
}
}
A simple null reference check solved all the issues I was having and as far as
I can
tell no information is missing from the reporting (I am going to have to dig
deeper,
and it would be nice if a google person could verify).
Just as an FYI, if anyone else wants to use this solution simply download the
source
code from the repository (put it somewhere convenient like your project
directory)
and add it as a classpath in your project's publish settings, make the
recommended
change provided above (to CampaignTracker.as/_addIfNotEmpty()), and voila!
I know this is not ideal so I hope google has a solution in the next release.Original comment by cemon...@gmail.com on 19 May 2009 at 5:07
Original issue reported on code.google.com by
sproutbu...@gmail.comon 24 Feb 2009 at 12:57