hola-server 0.3.8 → 0.3.10

package/core/meta.js

@@ -74,7 +74,7 @@ const validate_field = (meta, field) => {
     const keys = Object.keys(field);
     keys.forEach(key => {
         if (!field_attrs.includes(key)) {
-            throw new Error("The attribute [" + key + "] isn't supported now for field:" + field + " and meta:" + meta.collection);
+            throw new Error("The attribute [" + key + "] isn't supported now for field:" + JSON.stringify(field) + " and meta:" + meta.collection);
         }
     });
 }

package/db/db.js

@@ -27,7 +27,7 @@ const log_msg = (category, level, msg) => {
   const path = req ? req.originalUrl : "";
   const user = req && req.session && req.session.user ? req.session.user.id : "";
 
-  db.create(col_log, { time: time, category: category, level: level, msg: msg, user: user, path: path }).then(() => {});
+  db.create(col_log, { time: time, category: category, level: level, msg: msg, user: user, path: path }).then(() => { });
 };
 
 const is_log_debug = () => {
@@ -124,6 +124,7 @@ const bulk_update = async (col, items, attrs) => {
     attrs.forEach(function (attr) {
       query[attr] = item[attr];
     });
+    delete item["_id"];
     bulk.find(query).upsert().update({ $set: item }, true);
   }
   return await bulk.execute();
@@ -152,6 +153,7 @@ class DB {
    */
   create(code, obj) {
     const col = this.db[code];
+    delete obj["_id"];
     return col.insert(obj, { checkKeys: false });
   }
 
@@ -168,6 +170,7 @@ class DB {
     }
 
     const col = this.db[code];
+    delete obj["_id"];
     return col.update(query, { $set: obj }, { upsert: true, multi: true });
   }
 

package/db/entity.js

@@ -297,7 +297,7 @@ class Entity {
     * @param {param obj from user input} param_obj
     * @returns object with code and err
     */
-    async clone_entity(param_obj) {
+    async clone_entity(_id, param_obj) {
         const fields = this.meta.clone_fields
         const { obj, error_field_names } = convert_type(param_obj, fields);
         if (error_field_names.length > 0) {
@@ -309,7 +309,7 @@ class Entity {
         }
 
         if (this.meta.before_clone) {
-            const { code, err } = await this.meta.before_clone(this, obj);
+            const { code, err } = await this.meta.before_clone(_id, this, obj);
             if (err || code != SUCCESS) {
                 if (is_log_error()) {
                     log_error(LOG_ENTITY, "before_clone error:" + JSON.stringify(err) + ", with code:" + code);
@@ -342,7 +342,7 @@ class Entity {
         }
 
         if (this.meta.clone) {
-            const { code, err } = await this.meta.clone(this, obj);
+            const { code, err } = await this.meta.clone(_id, this, obj);
             if (err || code != SUCCESS) {
                 if (is_log_error()) {
                     log_error(LOG_ENTITY, "clone error:" + JSON.stringify(err) + ", with code:" + code);
@@ -360,7 +360,7 @@ class Entity {
         }
 
         if (this.meta.after_clone) {
-            const { code, err } = await this.meta.after_clone(this, obj);
+            const { code, err } = await this.meta.after_clone(_id, this, obj);
             if (err || code != SUCCESS) {
                 if (is_log_error()) {
                     log_error(LOG_ENTITY, "after_clone error:" + JSON.stringify(err) + ", with code:" + code);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hola-server",
-  "version": "0.3.8",
+  "version": "0.3.10",
   "description": "a meta programming framework used to build nodejs restful api",
   "main": "index.js",
   "scripts": {

package/router/clone.js

@@ -1,8 +1,8 @@
 const { set_file_fields, save_file_fields_to_db } = require('../db/gridfs');
-const { SUCCESS } = require('../http/code');
+const { SUCCESS, NO_PARAMS } = require('../http/code');
 const { get_session_userid } = require('../http/session');
 const { wrap_http } = require('../http/error');
-const { post_params } = require('../http/params');
+const { post_params, required_post_params } = require('../http/params');
 const { has_value } = require('../core/validate');
 const { Entity } = require('../db/entity');
 
@@ -19,6 +19,12 @@ const init_clone_router = function (router, meta) {
     const cp_upload = meta.upload_fields.length > 0 ? upload_file.fields(meta.upload_fields) : upload_file.none();
 
     router.post('/clone', cp_upload, wrap_http(async function (req, res) {
+        let params = required_post_params(req, ["_id"]);
+        if (params === null) {
+            res.json({ code: NO_PARAMS, err: '[_id] checking params are failed!' });
+            return;
+        }
+
         const param_obj = post_params(req, meta.field_names);
         set_file_fields(meta, req, param_obj);
 
@@ -30,7 +36,7 @@ const init_clone_router = function (router, meta) {
             param_obj[meta.user_field] = user_id;
         }
 
-        const { code, err } = await entity.clone_entity(param_obj);
+        const { code, err } = await entity.clone_entity(params["_id"], param_obj);
         if (!has_value(code)) {
             throw new Error("the method should return code");
         }