Remove GeoLocation Event Listener

[beppo-ivel]

If I open app the event-listener is added and the function recordis executed constantly, but if I click on the label the event is not removed from scope. The function is still executed.

views/index.xml

<Alloy>
	<Window class="container" >
		<Label id="label" onClick="doClick">Hello, World</Label>
	</Window>
</Alloy>

lib/util.js

class Util{
  static record(e){
    Ti.API.error("running");
  }
}
module.exports = Util;

controllers/index.js

let Util = require('/util');
function doClick(e) {
  Ti.Geolocation.removeEventListener('location', (e) => Util.record(e));
	alert($.label.text);
}

if (OS_ANDROID) {
	$.index.addEventListener('open', function() {
		if (!Ti.Geolocation.hasLocationPermissions()) {
	
		}else{
			Ti.Geolocation.accuracy = Ti.Geolocation.ACCURACY_HIGH;
   		        Ti.Geolocation.addEventListener('location', (e) => Util.record(e));
		}
	});
}
$.index.open();

[ewanharris]

@beppo-ivel In order to remove an event listener you need to use a function declaration not an anonymous function, so for example the below JS should work

let Util = require('/util');
function doClick(e) {
	Ti.Geolocation.removeEventListener('location', locationListener);
	alert($.label.text);
}

function locationListener() {
	Util.record(e)
}

if (OS_ANDROID) {
	$.index.addEventListener('open', function() {
		if (!Ti.Geolocation.hasLocationPermissions()) {
	
		} else {
			Ti.Geolocation.accuracy = Ti.Geolocation.ACCURACY_HIGH;
			Ti.Geolocation.addEventListener('location', locationListener);
		}
	});
}
$.index.open();

[beppo-ivel]

@ewanharris Is this the only pattern to pass a class function as callback? And thanks I didn't recognized that the arrow calls are anonymous function calls.

[ewanharris]

@beppo-ivel Ahh I missed you were wrapping the static function on the class. You should be able to just pass that to both calls I believe like below and do away with the wrapping function

let Util = require('/util');
function doClick(e) {
	Ti.Geolocation.removeEventListener('location', Util.record);
	alert($.label.text);
}

if (OS_ANDROID) {
	$.index.addEventListener('open', function() {
		if (!Ti.Geolocation.hasLocationPermissions()) {
	
		} else {
			Ti.Geolocation.accuracy = Ti.Geolocation.ACCURACY_HIGH;
			Ti.Geolocation.addEventListener('location', Util.record);
		}
	});
}
$.index.open();

[beppo-ivel]

Ti.Geolocation.addEventListener('location', Util.record); was my first try and it run into issues. But now it works. Thanks.