Why isnt my admob listener called?

gains · September 9, 2014, 8:21pm

Here is part of my build.settings

android = { &nbsp; &nbsp;usesPermissions = &nbsp; &nbsp;{ &nbsp; &nbsp; &nbsp; "android.permission.INTERNET", &nbsp; &nbsp; &nbsp; "android.permission.ACCESS\_NETWORK\_STATE", &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;}, }, plugins = &nbsp; &nbsp; { &nbsp; &nbsp; ["plugin.google.play.services"] = &nbsp; &nbsp; &nbsp; &nbsp; { &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; publisherId = "com.coronalabs" &nbsp; &nbsp; &nbsp; &nbsp; },

Here is part of the scene in which I can succesfully show banner ads from admob but the adlistener isnt being called.

local function adMobListener( event ) sometext.text = "Ad height is "..ads.height() if ( event.isError ) then &nbsp; &nbsp;print("error") &nbsp; &nbsp; &nbsp;&nbsp; &nbsp; end end ads.init( "admob", "ca-app-pub-#", adMobListener )

–I call ads.show inside of scene:create( event )

ads.show( "banner", { x=0, y=0} )

Also ads.height() returns 0 when I tried to call it directly.

How can I solve these issues?

Im using build 2014.2393

bjsorrentino · September 9, 2014, 10:15pm

Where is your listener function scoped? Also inside the :create() event? If so, you should move this up and outside, so Lua recognizes its scope.

Brent

gains · September 9, 2014, 10:43pm

Both the ads.init and the listener is before and outside of scene:create.

davebollinger · September 9, 2014, 11:27pm

can you get a log from your device?

on Android, I get an exception on some devices, after which listener is never called again (bug 34895 submitted) looks like this:

W/Ads &nbsp; &nbsp; (22297): Interrupted during GADSignals creation. W/Ads &nbsp; &nbsp; (22297): java.lang.InterruptedException W/Ads &nbsp; &nbsp; (22297): &nbsp; &nbsp; &nbsp;at java.util.concurrent.locks.AbstractQueuedSynchronizer .doAcquireSharedInterruptibly(AbstractQueuedSynchronizer.java:975) W/Ads &nbsp; &nbsp; (22297): &nbsp; &nbsp; &nbsp;at java.util.concurrent.locks.AbstractQueuedSynchronizer .acquireSharedInterruptibly(AbstractQueuedSynchronizer.java:1282) W/Ads &nbsp; &nbsp; (22297): &nbsp; &nbsp; &nbsp;at java.util.concurrent.CountDownLatch.await(CountDownLa tch.java:207) W/Ads &nbsp; &nbsp; (22297): &nbsp; &nbsp; &nbsp;at zn.a(SourceFile:147) W/Ads &nbsp; &nbsp; (22297): &nbsp; &nbsp; &nbsp;at zn.b(SourceFile:180) W/Ads &nbsp; &nbsp; (22297): &nbsp; &nbsp; &nbsp;at aeg.a(SourceFile:120) W/Ads &nbsp; &nbsp; (22297): &nbsp; &nbsp; &nbsp;at afv.run(SourceFile:14) W/Ads &nbsp; &nbsp; (22297): &nbsp; &nbsp; &nbsp;at afx.run(SourceFile:30) W/Ads &nbsp; &nbsp; (22297): &nbsp; &nbsp; &nbsp;at java.util.concurrent.ThreadPoolExecutor.runWorker(Thr eadPoolExecutor.java:1076) W/Ads &nbsp; &nbsp; (22297): &nbsp; &nbsp; &nbsp;at java.util.concurrent.ThreadPoolExecutor$Worker.run(Th readPoolExecutor.java:569) W/Ads &nbsp; &nbsp; (22297): &nbsp; &nbsp; &nbsp;at java.lang.Thread.run(Thread.java:856) I/Ads &nbsp; &nbsp; (22297): Starting ad request.

rob · September 9, 2014, 11:51pm

Can you post the relevant parts of your code for this? I’m concerned about when you say it’s out side of your scene:create function… Actually while I ask for that, let me be quite clear on how you do this:

main.lua:

local ads = require("ads") &nbsp; local sometext = display.newText("", display.contentCenterX, 10, native.systemFontBold, 20) &nbsp; local function adMobListener(event) &nbsp;&nbsp;&nbsp;&nbsp; .... your event handler code &nbsp;&nbsp;&nbsp;&nbsp; sometext.text = "Height: " .. tostring(ads.height()) end ads.init("admob", yourProviderID, adMobListener)

yourscene.lua

local composer = require ( "composer" ) local scene = composer.newScene() local ads = require( "ads" ) &nbsp; ... &nbsp; function scene:show( event ) &nbsp;&nbsp;&nbsp;&nbsp;&nbsp; local sceneView = self.view &nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if event.phase == "did" then &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ads.show("banner", { x=0, y=0 } ) &nbsp;&nbsp;&nbsp;&nbsp;&nbsp; end end &nbsp; function scene:hide( event ) &nbsp;&nbsp;&nbsp;&nbsp;&nbsp; local sceneView = self.view &nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if event.phase == "will" then &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ads.hide()&nbsp;&nbsp; -- optional if you want to hide ads in between scenes &nbsp;&nbsp;&nbsp;&nbsp;&nbsp; end end

Now you can get the height of the ad inside the admob listener, but it’s going to be difficult to display that as part of the scene’s display hirearchy. Since you likely are not really wanting that in your scene’s view, you can create your text in main.lua and then just reference it that way. The ads.height() should be set following the call to ads.show() in your scene so you can make UI adjustments there.

Rob

bjsorrentino · September 9, 2014, 10:15pm