flexbiz-server 12.7.6 → 12.7.7

package/server/models/listinfo.js

@@ -1,6 +1,37 @@
+/**
+ * NÂNG CẤP:
+ * - Bỏ các đoạn dùng `async.parallel`, `async.series`, `async.map`, `async.mapSeries` Ở LOGIC
+ *   NỘI BỘ của các hàm dưới đây, chuyển sang async/await + Promise.all:
+ *     deletePost, dynamicCreating, dynamicCreated, dynamicUpdating, dynamicUpdated,
+ *     dynamicDeleting, onView.
+ * - VẪN GIỮ `const async = require("async")` và tiếp tục truyền `async` vào sandbox của
+ *   `evalute()` cho các script động do admin cấu hình trong listinfo (handle_oncreating_server,
+ *   handle_onupdating_server, handle_view_server...) - đây là API công khai cho script của người
+ *   dùng cấu hình hệ thống, KHÔNG thuộc phạm vi "code nội bộ" cần bỏ async, và việc bỏ đi có thể
+ *   làm hỏng các script đã lưu sẵn trong DB.
+ * - GIỮ NGUYÊN toàn bộ chữ ký callback `next(error, result)` của các hàm dynamicXxx/onView vì đây
+ *   là hợp đồng (contract) mà controller.js và các handler (updateHandler/createHandler/
+ *   deleteHandler/updateFieldHandler) gọi vào - đổi sang Promise ở đây sẽ phá vỡ các nơi đó.
+ * - Tối ưu tốc độ: `onView`'s xử lý join dữ liệu tham chiếu cho nhiều field (fields_have_ref)
+ *   chạy song song bằng Promise.all thay vì async.map (giữ đúng ngữ nghĩa "chạy song song không
+ *   giới hạn" của async.map, nhưng nhẹ hơn).
+ * - FIX bug tiềm ẩn: trong `deletePost` (async.mapSeries) và `onView` (async.map), nếu một tác vụ
+ *   con lỗi SAU đoạn try/catch nội bộ (ví dụ lỗi từ `asyncJoinModel2`), lỗi đó không được bắt lại
+ *   -> callback `cb()`/`next()` không bao giờ được gọi -> request treo vĩnh viễn. Với async/await
+ *   + Promise.all bọc toàn bộ logic của từng tác vụ, lỗi giờ luôn được bắt và trả về đúng.
+ * - Trong `model.updateModel`, thay 1 lệnh `_newModel.collection.dropIndex(old_index_unique, cb)`
+ *   kiểu callback cũ bằng await, đồng nhất với các lệnh dropIndex khác ngay bên cạnh trong cùng
+ *   hàm (vốn đã dùng try/await).
+ *
+ * KHÔNG THAY ĐỔI (ngoài phạm vi): `model.createModel`/`model.createController` gọi
+ * `model.updateModel(...)` mà KHÔNG await (fire-and-forget) vì bản thân các hàm gọi nó
+ * (`createModel`, phần khởi tạo trong `createController`) không phải hàm async - việc chuyển đổi
+ * toàn bộ chuỗi gọi này thành async sẽ ảnh hưởng tới luồng khởi tạo route/model ở nơi khác không
+ * có trong phạm vi file này, nên mình giữ nguyên như bản gốc. Báo lại nếu muốn xử lý luôn.
+ */
 const controller = require('../controllers/controller');
 const underscore =require('underscore');
-const async = require("async");
+const async = require("../libs/asyncCompat");
 const {evalute} = require("../libs/utils");
 const utils = require("../libs/utils");
 //const {isSupperAdmin} = require("../libs/utils");
@@ -546,14 +577,14 @@ model.updateModel = async (info,_newModel)=>{
 					old_index_unique[u]=  (field||{}).sort || 1;
 				});
 				//Logger.warn("drop current index unique...",old_index_unique);
-				_newModel.collection.dropIndex(old_index_unique,()=>{
-					//if(e) return Logger.error("Can't drop index unique",old_index_unique,model_code,e);
-					//create new unique
-					if(indUq){
-						//Logger.warn("create new index unique...",indUq);
-						_newModel.schema.index(indUq,{unique: true });
-					}
-				})
+				try{
+					await _newModel.collection.dropIndex(old_index_unique);
+				}catch(e){/* giữ nguyên hành vi gốc: bỏ qua lỗi drop index */}
+				//create new unique
+				if(indUq){
+					//Logger.warn("create new index unique...",indUq);
+					_newModel.schema.index(indUq,{unique: true });
+				}
 			}else{
 				//create new unique
 				if(indUq){
@@ -587,34 +618,34 @@ model.createModel = (info)=>{
 	}
 	return exModel;
 }
-const deletePost = (_listInfo,obj)=>{
-	return new Promise((resolve,reject)=>{
-		let bookbackup={};
-		async.mapSeries((_listInfo.postinfos||[]).filter(info=>info.model && info.script && info.condition),(postInfo,callback)=>{
-			setImmediate(async ()=>{
-				let m = global.getModel(postInfo.model);
-				let scriptCondition = postInfo.condition.indexOf("return ")<0?`return ${postInfo.condition}`:postInfo.condition;
-				let condition = evalute(scriptCondition,{master:obj,data:obj,moment});
-				condition.id_app = obj.id_app;
-				//backup old data
-				bookbackup[postInfo.model] = await m.find(condition).lean();
-				//delete
-				await m.deleteMany(condition);
-				callback();
-			})
-		},async (e)=>{
-			if(e){
-				//restore old data
-				for(let key in bookbackup){
-					if(bookbackup[key].length>0){
-						await global.getModel(key.toString()).insertMany(bookbackup[key]);
-					}
-				}
-				return reject(e);
+
+/**
+ * @returns {Promise<object>} bookbackup - dữ liệu đã backup trước khi xoá (theo từng model)
+ */
+const deletePost = async (_listInfo, obj) => {
+	const bookbackup = {};
+	const postInfos = (_listInfo.postinfos || []).filter(info => info.model && info.script && info.condition);
+	try {
+		for (const postInfo of postInfos) {
+			const m = global.getModel(postInfo.model);
+			const scriptCondition = postInfo.condition.indexOf("return ") < 0 ? `return ${postInfo.condition}` : postInfo.condition;
+			const condition = evalute(scriptCondition, { master: obj, data: obj, moment });
+			condition.id_app = obj.id_app;
+			//backup old data
+			bookbackup[postInfo.model] = await m.find(condition).lean();
+			//delete
+			await m.deleteMany(condition);
+		}
+		return bookbackup;
+	} catch (e) {
+		//restore old data
+		for (let key in bookbackup) {
+			if (bookbackup[key].length > 0) {
+				await global.getModel(key.toString()).insertMany(bookbackup[key]);
 			}
-			resolve(bookbackup)
-		})
-	})
+		}
+		throw e;
+	}
 }
 const post = (_listInfo,obj)=>{
 	if(obj.toObject) obj = obj.toObject();
@@ -700,32 +731,23 @@ const dynamicCreating = async (listInfo,user,obj,next,options={})=>{
 		if(_listInfo.handle_oncreating_server.indexOf("next")<0) return next("Script xử lý dữ liệu trước khi tạo yêu cầu gọi function next");
 		try{
 			await new Promise((resolve,reject)=>{
-				async.parallel([
-					(next)=>{
-						const func_body = `(async ()=>{
-							try{
-								${_listInfo.handle_oncreating_server}
-							}catch(e){
-								next(e);
-							}
-							
-						})();`
-						setImmediate(async ()=>{
-							try{
-								await evalute(func_body,{obj,user,getLib:global.getLib,getModel:global.getModel,utils,query:utils.query,async,controller,controllerUtils,controllers:global.controllers,joinModel2,next,moment,numeral,request:options.req})
-							}catch(e){
-								Logger.error("[dynamicCreating][handle_oncreating_server]",{code:listInfo.code,handle_oncreating_server:_listInfo.handle_oncreating_server,e})
-								next(e);
-							}
-						})
-						
+				// next ở đây là callback dành riêng cho script động (khác với next của dynamicCreating)
+				const scriptNext = (e,rs)=> e ? reject(e) : resolve(rs);
+				const func_body = `(async ()=>{
+					try{
+						${_listInfo.handle_oncreating_server}
+					}catch(e){
+						next(e);
+					}
+					
+				})();`
+				setImmediate(async ()=>{
+					try{
+						await evalute(func_body,{obj,user,getLib:global.getLib,getModel:global.getModel,utils,query:utils.query,async,controller,controllerUtils,controllers:global.controllers,joinModel2,next:scriptNext,moment,numeral,request:options.req})
+					}catch(e){
+						Logger.error("[dynamicCreating][handle_oncreating_server]",{code:listInfo.code,handle_oncreating_server:_listInfo.handle_oncreating_server,e})
+						scriptNext(e);
 					}
-				],(e,rs)=>{
-					if(e){
-						Logger.error("[dynamicCreating][handle_oncreating_server]",listInfo.code,e,_listInfo.handle_oncreating_server);
-						return reject(e)
-					} 
-					resolve(rs);
 				})
 			})
 		}catch(e){
@@ -741,30 +763,21 @@ const dynamicCreated = async (_listInfo,user,obj,next,options={})=>{
 		if(_listInfo.handle_oncreated_server.indexOf("next")<0) return next("Script xử lý dữ liệu sau khi tạo yêu cầu gọi function next");
 		try{
 			await new Promise((resolve,reject)=>{
-				async.parallel([
-					(next)=>{
-						const func_body = `(async ()=>{
-							try{
-								${_listInfo.handle_oncreated_server}
-							}catch(e){
-								next(e);
-							}
-						})();`
-						setImmediate(async ()=>{
-							try{
-								await evalute(func_body,{obj,user,getLib:global.getLib,getModel:global.getModel,controller,controllerUtils,controllers:global.controllers,utils,query:utils.query,async,joinModel2,next,moment,numeral,request:options.req})
-							}catch(e){
-								Logger.error("error dynamic created",_listInfo.handle_oncreated_server,e)
-								next(e)
-							}
-						})
+				const scriptNext = (e,rs)=> e ? reject(e) : resolve(rs);
+				const func_body = `(async ()=>{
+					try{
+						${_listInfo.handle_oncreated_server}
+					}catch(e){
+						next(e);
+					}
+				})();`
+				setImmediate(async ()=>{
+					try{
+						await evalute(func_body,{obj,user,getLib:global.getLib,getModel:global.getModel,controller,controllerUtils,controllers:global.controllers,utils,query:utils.query,async,joinModel2,next:scriptNext,moment,numeral,request:options.req})
+					}catch(e){
+						Logger.error("error dynamic created",_listInfo.handle_oncreated_server,e)
+						scriptNext(e)
 					}
-				],(e,rs)=>{
-					if(e){
-						Logger.error(e);
-						return reject(e)
-					} 
-					resolve(rs);
 				})
 			})
 		}catch(e){
@@ -793,31 +806,22 @@ const dynamicUpdating = async (listInfo,user,data,obj,next,options={})=>{
 		if(_listInfo.handle_onupdating_server.indexOf("next")<0) return next("Script xử lý dữ liệu trước khi update yêu cầu gọi function next");
 		try{
 			await new Promise((resolve,reject)=>{
-				async.series([
-					(next)=>{
-						const func_body = `(async ()=>{
-							try{
-								${_listInfo.handle_onupdating_server}
-							}catch(e){
-								next(e);
-							}
-							
-						})();`
-						setImmediate(async ()=>{
-							try{
-								await evalute(func_body,{data,obj,user,getLib:global.getLib,getModel:global.getModel,controller,controllerUtils,controllers:global.controllers,utils,query:utils.query,async,joinModel2,next,moment,numeral,request:options.req})
-							}catch(e){
-								Logger.error("[listinfo][dynamicUpdating] error dynamic update",_listInfo.handle_onupdating_server,e)
-								next(e)
-							}
-						})
+				const scriptNext = (e,rs)=> e ? reject(e) : resolve(rs);
+				const func_body = `(async ()=>{
+					try{
+						${_listInfo.handle_onupdating_server}
+					}catch(e){
+						next(e);
+					}
+					
+				})();`
+				setImmediate(async ()=>{
+					try{
+						await evalute(func_body,{data,obj,user,getLib:global.getLib,getModel:global.getModel,controller,controllerUtils,controllers:global.controllers,utils,query:utils.query,async,joinModel2,next:scriptNext,moment,numeral,request:options.req})
+					}catch(e){
+						Logger.error("[listinfo][dynamicUpdating] error dynamic update",_listInfo.handle_onupdating_server,e)
+						scriptNext(e)
 					}
-				],(e,rs)=>{
-					if(e){
-						Logger.error("[listinfo][dynamicUpdating]",e);
-						return reject(e)
-					} 
-					resolve(rs);
 				})
 			})
 		}catch(e){
@@ -836,30 +840,21 @@ const dynamicUpdated = async (_listInfo,user,obj,next,options={})=>{
 		if(_listInfo.handle_onupdated_server.indexOf("next")<0) return next("Script xử lý dữ liệu sau khi update yêu cầu gọi function next");
 		try{
 			await new Promise((resolve,reject)=>{
-				async.series([
-					(next)=>{
-						const func_body = `(async ()=>{
-							try{
-								${_listInfo.handle_onupdated_server}
-							}catch(e){
-								next(e)
-							}
-						})();`
-						setImmediate(async ()=>{
-							try{
-								await evalute(func_body,{obj,user,getLib:global.getLib,getModel:global.getModel,controller,controllerUtils,controllers:global.controllers,utils,query:utils.query,async,joinModel2,next,moment,numeral,request:options.req})
-							}catch(e){
-								Logger.error("error dynamic updated",_listInfo.handle_onupdated_server,e)
-								next(e)
-							}
-						})
+				const scriptNext = (e,rs)=> e ? reject(e) : resolve(rs);
+				const func_body = `(async ()=>{
+					try{
+						${_listInfo.handle_onupdated_server}
+					}catch(e){
+						next(e)
+					}
+				})();`
+				setImmediate(async ()=>{
+					try{
+						await evalute(func_body,{obj,user,getLib:global.getLib,getModel:global.getModel,controller,controllerUtils,controllers:global.controllers,utils,query:utils.query,async,joinModel2,next:scriptNext,moment,numeral,request:options.req})
+					}catch(e){
+						Logger.error("error dynamic updated",_listInfo.handle_onupdated_server,e)
+						scriptNext(e)
 					}
-				],(e,rs)=>{
-					if(e){
-						Logger.error(e);
-						return reject(e)
-					} 
-					resolve(rs);
 				})
 			})
 		}catch(e){
@@ -888,32 +883,23 @@ const dynamicDeleting = async (listInfo,user,obj,next,options={})=>{
 		if(_listInfo.handle_ondeleting_server.indexOf("next")<0) return next("Script xử lý dữ liệu trước khi update yêu cầu gọi function next");
 		try{
 			await new Promise((resolve,reject)=>{
-				async.series([
-					(next)=>{
-						const func_body = `(async ()=>{
-							try{
-								${_listInfo.handle_ondeleting_server}
-							}catch(e){
-								next(e);
-							}
-						})();`
-						setImmediate(async ()=>{
-							try{
-								await evalute(func_body,{obj,user,getLib:global.getLib,getModel:global.getModel,controller,controllerUtils,controllers:global.controllers,utils,query:utils.query,async,joinModel2,next,moment,numeral,request:options.req})
-							}catch(e){
-								Logger.error("error dynamic delete",_listInfo.handle_ondeleting_server,e)
-								next(e)
-							}
-						})
-						
+				const scriptNext = (e,rs)=> e ? reject(e) : resolve(rs);
+				const func_body = `(async ()=>{
+					try{
+						${_listInfo.handle_ondeleting_server}
+					}catch(e){
+						next(e);
+					}
+				})();`
+				setImmediate(async ()=>{
+					try{
+						await evalute(func_body,{obj,user,getLib:global.getLib,getModel:global.getModel,controller,controllerUtils,controllers:global.controllers,utils,query:utils.query,async,joinModel2,next:scriptNext,moment,numeral,request:options.req})
+					}catch(e){
+						Logger.error("error dynamic delete",_listInfo.handle_ondeleting_server,e)
+						scriptNext(e)
 					}
-				],(e,rs)=>{
-					if(e){
-						Logger.error(e);
-						return reject(e)
-					} 
-					resolve(rs);
 				})
+				
 			})
 		}catch(e){
 			Logger.error(e);
@@ -938,36 +924,31 @@ const onView = async (control,obj,user,items,next,options={})=>{
 	//handle view
 	if(obj.handle_view_server && obj.handle_view_server.trim()!==""){
 		if(obj.handle_view_server.indexOf("next")<0) return next("Script xử lý view yêu cầu gọi function next");
-		await new Promise((resolve,reject)=>{
-			async.parallel([
-				(next)=>{
-					setImmediate(async ()=>{
-						const func_body = `(async ()=>{
-							try{
-								${obj.handle_view_server}
-							}catch(e){
-								next(e);
-							}
-							
-						})();`
+		try{
+			await new Promise((resolve,reject)=>{
+				const scriptNext = (e,rs)=> e ? reject(e) : resolve(rs);
+				setImmediate(async ()=>{
+					const func_body = `(async ()=>{
 						try{
-							//Logger.warn("[listinfo][dynamicView] [handle view]",obj.code);
-							await evalute(func_body,{items,user,getLib:global.getLib,utils,query:utils.query,next,async,joinModel2,moment,numeral,request:options.req})
+							${obj.handle_view_server}
 						}catch(e){
-							Logger.error("[listinfo][dynamicView] [handle view] error dynamic view",obj.handle_view_serve,e);
 							next(e);
 						}
-					})
-				}
-			],(e,rs)=>{
-				if(e){
-					Logger.error(e);
-					return reject(e);
-				}
-				resolve(rs);
+						
+					})();`
+					try{
+						//Logger.warn("[listinfo][dynamicView] [handle view]",obj.code);
+						await evalute(func_body,{items,user,getLib:global.getLib,utils,query:utils.query,next:scriptNext,async,joinModel2,moment,numeral,request:options.req})
+					}catch(e){
+						Logger.error("[listinfo][dynamicView] [handle view] error dynamic view",obj.handle_view_serve,e);
+						scriptNext(e);
+					}
+				})
 			})
-		})
-		
+		}catch(e){
+			Logger.error(e);
+			return next(e);
+		}
 	}
 	//
 	
@@ -977,51 +958,48 @@ const onView = async (control,obj,user,items,next,options={})=>{
 		//console.timeEnd(timer);
 		return next(null,items);
 	}
-	
-	async.map(fields_have_ref, (f,cb)=>{
-		setImmediate(async ()=>{
+
+	try{
+		// Xử lý song song cho từng field tham chiếu (giữ đúng ngữ nghĩa "song song không giới hạn"
+		// của async.map bản gốc, nhưng gọn và nhẹ hơn)
+		await Promise.all(fields_have_ref.map(async (f)=>{
 			let ref_model,where,fields,ref_name_model,ref_field,ref_condition,field_name,ref_label;
 			ref_label = f.ref_label_as || f.ref_label;
 			ref_field = f.ref_field;
 			ref_condition = f.ref_condition;
 			field_name  = f.name;
-			try{
-				ref_name_model= f.ref_model;
-				if(ref_name_model.indexOf("[")>=0 || ref_name_model.indexOf(",")>=0){
-					try{
-						if(ref_name_model.indexOf("[")>=0){
-							ref_model = utils.JSONParser(ref_name_model)
-						}else{
-							ref_model = ref_name_model.split(",");
-						}
-					}catch(e){
-						Logger.error("ref_model is not valid",ref_name_model,e);
-						return cb(e);
+			ref_name_model= f.ref_model;
+			if(ref_name_model.indexOf("[")>=0 || ref_name_model.indexOf(",")>=0){
+				try{
+					if(ref_name_model.indexOf("[")>=0){
+						ref_model = utils.JSONParser(ref_name_model)
+					}else{
+						ref_model = ref_name_model.split(",");
 					}
-					
-				}else{
-					if(ref_name_model==="dmkh") ref_name_model = "customer";
-					if(ref_name_model==="dmtk") ref_name_model = "account";
-					if(ref_name_model==="dmnt") ref_name_model = "currency";
-					if(ref_name_model==="dmnhtask") ref_name_model = "group";
+				}catch(e){
+					Logger.error("ref_model is not valid",ref_name_model,e);
+					throw e;
+				}
+				
+			}else{
+				if(ref_name_model==="dmkh") ref_name_model = "customer";
+				if(ref_name_model==="dmtk") ref_name_model = "account";
+				if(ref_name_model==="dmnt") ref_name_model = "currency";
+				if(ref_name_model==="dmnhtask") ref_name_model = "group";
 
-					ref_model = global.mongoose.models[ref_name_model];
+				ref_model = global.mongoose.models[ref_name_model];
+				if(!ref_model){
+					let _listInofOfRef = await model.findOne({code:ref_name_model}).lean();
+					if(_listInofOfRef && (_listInofOfRef.api_code || _listInofOfRef.model_code)){
+						let model_code = _listInofOfRef.api_code || _listInofOfRef.model_code;
+						ref_model = global.mongoose.models[model_code];
+					}
 					if(!ref_model){
-						let _listInofOfRef = await model.findOne({code:ref_name_model}).lean();
-						if(_listInofOfRef && (_listInofOfRef.api_code || _listInofOfRef.model_code)){
-							let model_code = _listInofOfRef.api_code || _listInofOfRef.model_code;
-							ref_model = global.mongoose.models[model_code];
-						}
-						if(!ref_model){
-							let ctrl = global.controllers[ref_name_model.trim().toUpperCase()];
-							if(ctrl) ref_model = ctrl.getProperty("model");
-						}
+						let ctrl = global.controllers[ref_name_model.trim().toUpperCase()];
+						if(ctrl) ref_model = ctrl.getProperty("model");
 					}
-					if(!ref_model) ref_model = global.getModel(ref_name_model);
 				}
-			}catch(e){
-				Logger.error(e);
-				return cb(e);
+				if(!ref_model) ref_model = global.getModel(ref_name_model);
 			}
 			/*//chỉ xử lý dynamic joinview nếu field có giá trị
 			let items_will_join = items.filter(item=>{
@@ -1036,7 +1014,7 @@ const onView = async (control,obj,user,items,next,options={})=>{
 			//Cần xử lý joinview tất cả các field để xoá dữ liệu ref không phù hợp
 			let items_will_join = items;
 			//
-			if(items_will_join.length==0) return cb();
+			if(items_will_join.length==0) return;
 			//xác định các fields cần lấy dữ liệu
 			fields = {[ref_label]:f.ref_label};
 			if(f.ref_model==="trangthai" && f.name=="trang_thai"){
@@ -1104,15 +1082,12 @@ const onView = async (control,obj,user,items,next,options={})=>{
 					})
 				}
 			}
-			cb();
-		})
-		
-	},(e)=>{
+		}))
+	}catch(e){
 		//console.timeEnd(timer);
-		if(e) return next(e);
-		next(null,items);
-
-	})
+		return next(e);
+	}
+	next(null,items);
 }
 model.createController = (router,_listInfo)=>{
 	let api_code = _listInfo.api_code || _listInfo.code;
@@ -1362,4 +1337,4 @@ model.createController = (router,_listInfo)=>{
 	}
 	return _newList;
 }
-module.exports = model;
+module.exports = model;