Setter needs to return true even when the scan pointer is NULL

(e.g. when run as a timed event) - this bug was introduced in the "Fix sub[] property setter" crash bug fix 2 commits ago.
This fixes the "TypeError: sub.code is undefined" error.

src/sbbs3/js_msg_area.c

@@ -315,7 +315,7 @@ static JSBool js_sub_set(JSContext *cx, JSObject *obj, jsid id, JSBool strict, j
 
 	subscan_t*	scan = p->subscan;
 	if(scan == NULL)
-		return JS_FALSE;
+		return JS_TRUE;
 
     JS_IdToValue(cx, id, &idval);
     tiny = JSVAL_TO_INT(idval);