From 24745ffda3ec9709bf1110812902a6778c084955 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Sat, 27 Apr 2019 04:15:03 +0300
Subject: [PATCH] experimenting with .search(..) -- .get(..) alternative based
 on .walk(..)...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 ui (gen4)/lib/widget/browse2.js | 69 +++++++++++++++++++++++++++++++++
 1 file changed, 69 insertions(+)

diff --git a/ui (gen4)/lib/widget/browse2.js b/ui (gen4)/lib/widget/browse2.js
index 855366d9..b152564b 100755
--- a/ui (gen4)/lib/widget/browse2.js	
+++ b/ui (gen4)/lib/widget/browse2.js	
@@ -836,6 +836,7 @@ var BaseBrowserPrototype = {
 	//
 	//
 	//
+	// XXX should we move the defaults to .config???
 	// XXX make item access by index lazy... 
 	// 		- index nested stuff and lengths... (.sublist_length)
 	// 		- stop when target reached... (control callback???)
@@ -888,6 +889,74 @@ var BaseBrowserPrototype = {
 	},
 
 
+	// XXX EXPERIMENTAL...
+	// XXX make this a bit smarter and accept an index or a path...
+	// XXX can this replace .get(..)
+	search: function(func, options){
+		var that = this
+
+		// XXX can we avoid this in every client???
+		var args = [...arguments]
+		func = args[0] instanceof Function ? 
+			args.shift() 
+			: undefined
+		var i = typeof(args[0]) == typeof(123) ?
+			args.shift()
+			: 0
+		var path = (args[0] instanceof Array 
+				|| typeof(args[0]) == typeof('str')) ?
+			args.shift()
+			: []
+		var options = args.pop() || {}
+
+		// XXX better name...
+		var allMatching = options.allMatching
+
+		var Stop = new Error('Stop search exception...')
+		var res = []
+
+		try {
+			this.walk(
+				function(i, path, elem, doNested){
+					// check the element and if it matches then break the search...
+					func.call(that, elem, i, path, that) ?
+						// XXX returning path and i might be a good idea...
+						res.push(elem)
+						: null
+
+					if(res.length > 0 && !allMatching){
+						throw Stop
+					}
+
+					return []
+				}, 
+				// XXX for path tests use this to narrow down the options...
+				function(_, i, path, sublist, options){
+					// NOTE: this needs to call the actual func that the user
+					// 		gave us and not the constructed function that we 
+					// 		pass to .walk(..) above...
+					return sublist.search(func, i, path, options) || [] },
+				// XXX this is essentially the only place we need parsed arguments...
+				// 		...can we either pass in the args in some standard format 
+				// 		or spec the format and let walk handle it???
+				i,
+				path,
+				options)
+
+			// nothing found...
+			return res
+
+		} catch(e){
+			// we got a result...
+			if(e === Stop){
+				return res[0]
+			}
+			// error...
+			throw e
+		}
+	},
+
+
 	// XXX BROKEN...
 	// Sublist map functions...
 	//