feat(editor): publish: check feature range

Author: yyc-git

contributes/meta3d-action-mod-career-publish-to-game/package.json

@@ -1,6 +1,6 @@
 {
     "name": "meta3d-action-mod-career-publish-to-game",
-    "version": "1.3.58",
+    "version": "1.3.60",
     "publisher": "meta3d",
     "protocol": {
         "name": "meta3d-action-mod-career-publish-to-game-protocol"

contributes/meta3d-action-mod-career-publish-to-game/src/Main.ts

@@ -158,12 +158,54 @@ let _buildIconId = (state: state, isChinese) => {
     return `${_buildUniqueName(state, isChinese)}_icon`
 }
 
+let _unifyCareerFeatureValueToArray = (value) => {
+    if (Array.isArray(value)) {
+        return value
+    }
+
+    return [value]
+}
+
+let _range = (a, b) => {
+    let res = []
+    for (let i = a; i <= b; i++) {
+        res.push(i)
+    }
+
+    return res
+}
+
+let _isFeaturesInRange = (api: api, features) => {
+    return features.reduce((result, { values, minValue, maxValue }) => {
+        if (!result) {
+            return result
+        }
+
+        let valueCount = values.count()
+
+        minValue = api.nullable.getWithDefault(
+            api.nullable.map(_unifyCareerFeatureValueToArray, minValue),
+            _range(0, valueCount - 1).map(_ => -Infinity)
+        )
+        maxValue = api.nullable.getWithDefault(
+            api.nullable.map(_unifyCareerFeatureValueToArray, maxValue),
+            _range(0, valueCount - 1).map(_ => Infinity)
+        )
+
+        return values.reduce((result, value, i) => {
+            if (!result) {
+                return result
+            }
+
+            return value >= minValue[i] && value <= maxValue[i]
+        }, result)
+    }, true)
+}
+
 let _check = (api: api, state: state, isChinese, features) => {
     let {
         displayName_cn,
         displayName_en,
-        // repoLink,
-        author,
         readme,
     } = state
 
@@ -184,6 +226,9 @@ let _check = (api: api, state: state, isChinese, features) => {
     else if (features.count() <= 0) {
         message = api.nullable.return(isChinese ? "请选择职业特性" : "Please select the career features")
     }
+    else if (!_isFeaturesInRange(api, features)) {
+        message = api.nullable.return(isChinese ? "职业特性值超出范围" : "Career feature's value exceed the specified range")
+    }
 
     return api.nullable.getWithDefault(
         api.nullable.map((message) => {

doc/1.3.1.org

@@ -1222,6 +1222,27 @@ TODO need gem should limit
 TODO whole->jump miss username
 
 
+* DONE update
+
+TODO check value range when publish
+
+
+
+# 第一个问题：
+# 在电脑端创建职业模组时，可编辑数值的职业预设特性在进行手动输入时，最终值会默认在后面增加两个0（例如设定值为100，最终值会变为10000）。并且会导致模组无法发布
+
+
+
+TODO update cn
+TODO update en
+
+
+
+
+
+
+
+
 * TODO update
 
 
@@ -1238,13 +1259,12 @@ TODO publish in package assemble: name, description should can be empty
 
 
 
-第一个问题：
-在电脑端创建职业模组时，可编辑数值的职业预设特性在进行手动输入时，最终值会默认在后面增加两个0（例如设定值为100，最终值会变为10000）。并且会导致模组无法发布
-
-
+还有，在发布模组时，总是出现这个:
+发布失败
 
+  TODO add handleNetwork api
+    retry 3 times
 
+    rewrite error message
 
 
-还有，在发布模组时，总是出现这个:
-发布失败