From 3358795145c658309c46d87c0518eb35b8e2dcc5 Mon Sep 17 00:00:00 2001
From: rswindell <>
Date: Fri, 26 Oct 2018 03:25:27 +0000
Subject: [PATCH] bbs.menu() now returns Boolean (the menu file was displayed
 successfully)

---
 src/sbbs3/js_bbs.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/sbbs3/js_bbs.cpp b/src/sbbs3/js_bbs.cpp
index 03272d0434..e42fa07245 100644
--- a/src/sbbs3/js_bbs.cpp
+++ b/src/sbbs3/js_bbs.cpp
@@ -1172,11 +1172,11 @@ js_menu(JSContext *cx, uintN argc, jsval *arglist)
 	if(!menu)
 		return JS_FALSE;
 	rc=JS_SUSPENDREQUEST(cx);
-	sbbs->menu(menu);
+	bool result = sbbs->menu(menu);
 	free(menu);
 	JS_RESUMEREQUEST(cx, rc);
 
-	JS_SET_RVAL(cx, arglist, JSVAL_VOID);
+	JS_SET_RVAL(cx, arglist, result ? JSVAL_TRUE : JSVAL_FALSE);
 
     return(JS_TRUE);
 }
@@ -4135,7 +4135,7 @@ static jsSyncMethodSpec js_bbs_functions[] = {
 	,314
 	},
 	/* menuing */
-	{"menu",			js_menu,			1,	JSTYPE_VOID,	JSDOCSTR("base_filename")
+	{"menu",			js_menu,			1,	JSTYPE_BOOLEAN,	JSDOCSTR("base_filename")
 	,JSDOCSTR("display a menu file from the text/menu directory")
 	,310
 	},
-- 
GitLab