From ddfdb662061120db34f9faf7f1ccaec9dc0a3740 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Fri, 10 Apr 2020 23:05:16 +0300
Subject: [PATCH] added log rotation....

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 ui (gen4)/features/core.js | 48 ++++++++++++++++++++++++++++++--------
 1 file changed, 38 insertions(+), 10 deletions(-)

diff --git a/ui (gen4)/features/core.js b/ui (gen4)/features/core.js
index badbd9b5..01dada3c 100755
--- a/ui (gen4)/features/core.js	
+++ b/ui (gen4)/features/core.js	
@@ -219,7 +219,14 @@ if(typeof(window) != 'undefined'){
 
 // XXX add log filtering...
 var LoggerActions = actions.Actions({
+	config: {
+		// NOTE: if set to 0 no log limit is applied...
+		'log-size': 10000,
+	},
+
 	Logger: object.Constructor('BaseLogger', {
+		doc: `Logger object constructor...`,
+
 		__context: null,
 		get context(){
 			return this.__context || this.root.__context },
@@ -228,6 +235,28 @@ var LoggerActions = actions.Actions({
 		get isRoot(){
 			return this === this.root },
 
+		__path: null,
+		get path(){
+			return (this.__path = 
+				this.__path == null ? 
+					[] 
+					: this.__path) },
+		set path(value){
+			this.__path = value },
+
+		// NOTE: if set to 0 no log limit is applied...
+		// NOTE: writing to this will modify .context.config['log-size']
+		// 		if a available and .__max_size otherwise...
+		__max_size: null,
+		get max_size(){
+			return this.__max_size != null ?
+				this.__max_size
+				// this.context.config['log-size']
+				: ((this.context || {}).config || {})['log-size'] || 10000 },
+		set max_size(value){
+			return this.context ?
+				(this.context.config['log-size'] = value)
+				: this.__max_size = value },
 		// NOTE: to disable log retention in .log set this to false...
 		__log: null,
 		get log(){
@@ -239,15 +268,6 @@ var LoggerActions = actions.Actions({
 					(this.__log = this.__log || []) 
 				: this.root.log },
 
-		__path: null,
-		get path(){
-			return (this.__path = 
-				this.__path == null ? 
-					[] 
-					: this.__path) },
-		set path(value){
-			this.__path = value },
-
 
 		// log management...
 		clear: function(){
@@ -291,8 +311,16 @@ var LoggerActions = actions.Actions({
 					}) },
 
 		emit: function(status, ...rest){
+			// write to log...
 			this.log !== false
 				&& this.log.push([this.path, status, rest])
+			// maintain log size...
+			this.log !== false
+				&& this.log.push([this.path, status, rest])
+				&& (this.max_size > 0 
+					&& this.log.length > this.max_size 
+					&& this.log.splice(0, this.log.length - this.max_size))
+			// call context log handler...
 			this.context
 				&& this.context.handleLogItem
 				&& this.context.handleLogItem(this.path, status, ...rest)
@@ -313,7 +341,7 @@ var LoggerActions = actions.Actions({
 			this.__logger 
 				|| this.Logger(this)) },
 
-	// XXX move this to console-logger... (???)
+	// XXX move this to console-logger???
 	handleLogItem: ['- System/',
 		function(path, status, ...rest){
 			console.log(