crabatool 1.0.65 → 1.0.69

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "crabatool",
-  "version": "1.0.65",
+  "version": "1.0.69",
   "description": "crabatool",
   "main": "index.js",
   "scripts": {

package/tool/checkgspx.js

@@ -3,7 +3,7 @@ var config = require('../lib/config.js');
 var fs = require('fs');
 var path = require('path');
 var htmlparser2 = require('htmlparser2');
-var warnTags = ['VPanel', 'HPanel', 'HiddenColumn', 'Hblock', 'Vblock']; // 'div', 'table', 'tr', 'td', 'br','HiddenField', 'CloseButton', 'CancelButton'];
+var warnTags = ['VPanel', 'HPanel', 'HiddenColumn', 'Hblock', 'Vblock', 'div', 'table', 'tr', 'td', 'br', 'iframe', 'HiddenField', 'CloseButton', 'CancelButton'];
 
 module.exports.getResults = function() {
   var data = {

package/tool/checkjs.js

@@ -14,6 +14,24 @@ var sizeBlackList = ['craba.min.js', 'crabaEx.min.js', 'crabaNgp.js', 'echarts-a
 // 语法规则，不能包含这些关键词
 var dangerousJs = ['\\slet\\s', '\\sconst\\s', '\=\>\\s?\\(?\\{', '\\sasync\\s', '\\sawait\\s', '\\.then\\(', '\\sdebugger\\s'];
 
+// gspx无用标签的修改建议
+var warnTagsInfo = {
+  'VPanel': '建议更改为`FlexColumn`或`FlowPanel LayoutDirection="Vert"`',
+  'HPanel': '建议更改为`HBlock`或`FlowPanel`',
+  'HiddenColumn': '写错了，没有这个标签',
+  'Hblock': '写错了，没有这个标签',
+  'Vblock': '写错了，没有这个标签',
+  'div': '平台的Block标签等效于div标签`<Block CssClass="styleA"/>`等于`<div class="styleA"></div>`',
+  'table': '建议更换为`FlowPane`l或`FlexColumn和HBlock的组合`',
+  'tr': '建议更换为`FlowPanel`或`FlexColumn和HBlock的组合`',
+  'td': '建议更换为`FlowPanel`或`FlexColumn和HBlock的组合`',
+  'br': '建议使用`CssStyle="margin:10px 0;"`增加间距或`VSPacer`',
+  'HiddenField': '新平台里面99%的这个标签都没意义，`<HiddenField ID="hd" Value="${hd}"/>`取代方法，不用gspx解析一遍，改为js直接读取变量：`var hd =this.get_context("hd");`',
+  'CloseButton': '如果是`<CloseButton Visible=false/>`这种建议直接去掉，新平台不用加这个标签都可以`Esc`关闭页面，每个页面默认支持Esc关闭',
+  'CancelButton': '如果是`<CancelButton Visible=false/>`这种建议直接去掉，新平台不用加这个标签都可以`Esc`关闭页面，每个页面默认支持Esc关闭',
+  'iframe': '建议使用IFrame'
+}
+
 
 module.exports.start = function() {
   var jsFiles = utils.getFiles({ source: config.webPath, exts: '.js', ignores: config.ignoreCheck });
@@ -101,7 +119,7 @@ module.exports.start = function() {
   info.push(`| 共检查 | ${gspxData.count}个gspx文件 |`);
   info.push(`| 忽略目录和文件 | ${config.ignoreCheck ? config.ignoreCheck.join('、') : '无' + ' |'}`);
 
-  info.push("# 检测报表");
+  info.push("# 检测结果汇总");
   info.push("| 类型 | 数量 |");
   info.push("| ------ | ------ |");
   info.push(`| 语法异常文件数 | ${errList.length}个 |`);
@@ -132,10 +150,10 @@ module.exports.start = function() {
   var detail = [];
   if (errList.length > 0) {
     detail.push("");
-    detail.push('# js语法异常文件清单如下');
+    detail.push('# js语法异常清单如下');
     errList.forEach((s, index) => {
 
-      detail.push(`## ${index + 1}. ${s}`);
+      detail.push(`${index + 1}. ${s}`);
 
       var err = errRegList[index];
       detail.push("| 异常字符 | 位置 |");
@@ -149,9 +167,9 @@ module.exports.start = function() {
 
   if (notUtf8List.length > 0) {
     detail.push("");
-    detail.push('# 非utf8文件清单如下');
+    detail.push('# 非utf8清单如下');
     notUtf8List.forEach((s, index) => {
-      detail.push(`## ${index + 1}. ${s}`);
+      detail.push(`${index + 1}. ${s}`);
 
       sonar.issues.push(getSonarEntity('非utf8', modName, '', s, -1, -1));
     });
@@ -159,9 +177,9 @@ module.exports.start = function() {
 
   if (bigList.length > 0) {
     detail.push("");
-    detail.push(`# 超过${fileSize}kb的文件清单如下`);
+    detail.push(`# 超过${fileSize}kb清单如下`);
     bigList.forEach((s, index) => {
-      detail.push(`## ${index + 1}. ${s}`);
+      detail.push(`${index + 1}. ${s}`);
       var arr = s.split(' ');
       sonar.issues.push(getSonarEntity(`超${fileSize}kb文件`, modName, arr[1], arr[0], -1, -1));
     });
@@ -170,27 +188,37 @@ module.exports.start = function() {
   if (gspxData.errorList.length > 0) {
     detail.push(`# 出错误页面清单如下`);
     gspxData.errorList.forEach((info, index) => {
-      detail.push(`## ${index + 1}. ${info.name}`);
+      detail.push(`${index + 1}. ${info.name}`);
       detail.push(`> 错误标签: ${info.tag}`);
       detail.push(`> 错误描述：${info.error}`);
     });
   }
 
   if (warnPages > 0) {
-    detail.push(`# 建议优化gspx页面清单如下`);
+    detail.push(`# 建议优化gspx清单如下`);
     Object.keys(gspxData.warnList).forEach((key, index) => {
       var tags = gspxData.warnList[key];
 
-      detail.push(`## ${index + 1}. ${key}`); 4
-      detail.push(`> 此页面建议优化标签明细: ${tags.join('、')}`);
+      detail.push(`**${index + 1}. ${key}**`);
+      detail.push(`>建议优化标签明细: \`${tags.join('`、`')}\``);
+      detail.push("");
+      detail.push("| 标签名称 | 优化建议 |");
+      detail.push("| ----- | ------ |");
+      var oks = [];
+      tags.forEach((tag) => {
+        if (oks.includes(tag)) return;
+        oks.push(tag);
+
+        detail.push(`| ${tag} | ${warnTagsInfo[tag] || '-'} |`);
+      });
     });
   }
 
-  detail.push('# 本项目标签使用情况汇总');
-  detail.push("| 标签名称 | 使用次数 |");
-  detail.push("| ----- | ------ |");
+  detail.push('# 项目标签汇总');
+  detail.push("| 标签名称 | 使用次数 | 标签使用建议");
+  detail.push("| ----- | ------ | ---- |");
   gspxData.tagList.forEach((item, index) => {
-    detail.push(`| ${item.name} | ${item.count} |`);
+    detail.push(`| ${item.name} | ${item.count} | ${warnTagsInfo[item.name] || '-'} |`);
   });
 
 
@@ -202,7 +230,7 @@ module.exports.start = function() {
 
   // 推送到钉钉
   if (status != '无异常') { // 检查结果检查就不发到钉钉了； ps:健康也发到钉钉，鼓励开发继续保持；
-    postWebhooks(id, reportUrl, webhookList);
+    //postWebhooks(id, reportUrl, webhookList);
   }
 }