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

AdWhirl/iPhone concurrency - does not send appId #42

Closed
GoogleCodeExporter opened this issue Aug 30, 2015 · 5 comments
Closed

AdWhirl/iPhone concurrency - does not send appId #42

GoogleCodeExporter opened this issue Aug 30, 2015 · 5 comments

Comments

@GoogleCodeExporter
Copy link

It looks like if people are using multiple adwhirl spots within one application 
it will cause some issue with the appid and send (null) instead. The same thing 
happened for android and I think I fixed it - so I think these are coming from 
the iPhone version

I'm seeing a decent amount of these in the logs:

GET 
/exmet.php?appid=(null)&nid=26fe866b88e74d6ba5d1be984151c978&type=1&uuid=2B8DEFE
ADAC20ADAFFF6647813794E22&country_code=en_US&appver=223&client=1 HTTP/1.1

Original issue reported on code.google.com by jpin...@google.com on 22 Jun 2010 at 9:45

@GoogleCodeExporter
Copy link
Author

For the appid, the AdWhirlView simply calls the delegate's 
adWhirlApplicationKey method. The only way I can think of that it can be null 
is when the delegate method return nil. The AdWhirl code is very simple and I 
can't think of any situation, multiple adwhirl spots or not, that can cause the 
appid to be nil other than that the app delegate returns nil for 
adWhirlApplicationKey. In which case, it's the delegates responsibility to 
return a proper appid. So for this it doesn't seem like there's anything for us 
to fix.

Original comment by nigelc...@google.com on 13 Jul 2010 at 7:16

  • Changed state: WontFix

@GoogleCodeExporter
Copy link
Author

Original comment by nigelc...@google.com on 13 Jul 2010 at 11:21

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

Original comment by nigelc...@google.com on 13 Aug 2010 at 6:54

  • Added labels: Component-SDK-iPhone, Priority-Low
  • Removed labels: Priority-Medium

@GoogleCodeExporter
Copy link
Author

Original comment by nigelc...@google.com on 7 Oct 2010 at 5:59

  • Changed state: Started

@GoogleCodeExporter
Copy link
Author

Fixed in 2.6.1

Original comment by nigelc...@google.com on 8 Oct 2010 at 12:36

  • Changed state: Fixed

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

No branches or pull requests

1 participant