fix(android): navigationWindow open/close event

android/modules/ui/src/java/ti/modules/titanium/ui/NavigationWindowProxy.java

@@ -30,11 +30,12 @@ public NavigationWindowProxy()
 	public void open(@Kroll.argument(optional = true) Object arg)
 	{
 		// FIXME: Shouldn't this complain/blow up if window isn't specified?
-		if (getProperties().containsKeyAndNotNull(TiC.PROPERTY_WINDOW)) {
+		if (!opened && getProperties().containsKeyAndNotNull(TiC.PROPERTY_WINDOW)) {
 			opened = true;
 			Object rootView = getProperties().get(TiC.PROPERTY_WINDOW);
 			if (rootView instanceof WindowProxy || rootView instanceof TabGroupProxy) {
 				openWindow(rootView, arg);
+				fireEvent(TiC.EVENT_OPEN, null);
 			}
 			return;
 		}
@@ -56,8 +57,12 @@ public void popToRootWindow(@Kroll.argument(optional = true) Object arg)
 	@Kroll.method
 	public void close(@Kroll.argument(optional = true) Object arg)
 	{
-		popToRootWindow(arg);
-		closeWindow(windows.get(0), arg); // close the root window
+		if (opened) {
+			opened = false;
+			popToRootWindow(arg);
+			closeWindow(windows.get(0), arg); // close the root window
+			fireEvent(TiC.EVENT_CLOSE, null);
+		}
 		super.close(arg);
 	}
 

tests/Resources/ti.ui.navigationwindow.addontest.js

@@ -0,0 +1,39 @@
+/*
+ * Appcelerator Titanium Mobile
+ * Copyright (c) 2017-Present by Axway. All Rights Reserved.
+ * Licensed under the terms of the Apache Public License
+ * Please see the LICENSE included with this distribution for details.
+ */
+/* eslint-env mocha */
+/* eslint no-unused-expressions: "off" */
+'use strict';
+
+describe.windowsMissing('Titanium.UI.NavigationWindow', _ => {
+	let nav;
+
+	this.timeout(10000);
+
+	afterEach(function () {
+		if (nav) {
+			nav.close();
+		}
+		nav = null;
+	});
+
+	it('open/close events', finish => {
+        const window = Ti.UI.createWindow();
+
+		nav = Ti.UI.createNavigationWindow({
+            window: window
+        });
+
+		nav.addEventListener('open', _ => {
+			navigation.close();
+		});
+		nav.addEventListener('close', _ => {
+			finish();
+        });
+
+		nav.open();
+	});
+});