Discuz 7.2坑爹集锦-js篇

在调试DZ过程中firebug经常提示common.js出错，什么“Error: s is null”，“$ is not defined”，“BROWSER not defined”。其实多半和XML有关：当ajax操作时后台PHP出现错误（包括notice）导致前台JS在解析期望中XML格式返回值时出错。也许是DZ的ajax调用对返回值处理不够完善。解决办法是编辑php.ini设置display_error = off。
DZ的js文件没细看，主要是解决几个调试过程中遇到兼容性问题。

-------------------------------------------------------------------

类型：??? ??? 过时函数
坑爹指数：??? ★
代码：??? ??? include/js/common.js 10-14
BROWSER.ie = window.ActiveXObject && USERAGENT.indexOf('msie') != -1 && USERAGENT.substr(USERAGENT.indexOf('msie') + 5, 3);
BROWSER.firefox = document.getBoxObjectFor && USERAGENT.indexOf('firefox') != -1 && USERAGENT.substr(USERAGENT.indexOf('firefox') + 8, 3);
BROWSER.chrome = window.MessageEvent && !document.getBoxObjectFor && USERAGENT.indexOf('chrome') != -1 && USERAGENT.substr(USERAGENT.indexOf('chrome') + 7, 10);
BROWSER.opera = window.opera && opera.version();
BROWSER.safari = window.openDatabase && USERAGENT.indexOf('safari') != -1 && USERAGENT.substr(USERAGENT.indexOf('safari') + 7, 8);
点评：??? ??? 测试过程中某天发现FF下登录窗口账户名只能选择用户名，而在chrome下面可选UID及email。当时一直没找到原因，某天测试时修改到此处，顺便用slice()替换substr()发现FF的登录窗问题正常了。猛然想起以前看过对于substr的一个注解：“ECMAscript 没有对该方法进行标准化，因此反对使用它”。在Firefox8以前都没这个问题，看来FF开始放弃对此函数的兼容了。另外Firefox9也不支持document.getBoxObjectFor对象了。
FIX：???
BROWSER.ie = window.ActiveXObject && USERAGENT.indexOf('msie') != -1 && USERAGENT.slice(USERAGENT.indexOf('msie') + 5, USERAGENT.indexOf('msie') + 8) || false;
BROWSER.firefox = USERAGENT.indexOf('firefox') != -1 && USERAGENT.slice(USERAGENT.indexOf('firefox') + 8, USERAGENT.indexOf('firefox') + 11) || false;
BROWSER.chrome = window.MessageEvent && !document.getBoxObjectFor && USERAGENT.indexOf('chrome') != -1 && USERAGENT.slice(USERAGENT.indexOf('chrome') + 7, USERAGENT.indexOf('chrome') + 17) || false;
BROWSER.opera = window.opera && opera.version() || false;
BROWSER.safari = window.openDatabase && USERAGENT.indexOf('safari') != -1 && USERAGENT.slice(USERAGENT.indexOf('safari') + 7, USERAGENT.indexOf('safari') + 15) || false;

-------------------------------------------------------------------

类型：??? ??? 操作符顺序
坑爹指数：??? ★
代码：??? ??? include/js/common.js=1112
function ajaxget(url, showid, waitid, loading, display, recall) {
??? waitid = typeof waitid == 'undefined' || waitid === null ? showid : waitid;
??? ...
}
点评：??? ??? 怎么看这个赋值表达式也没问题，可是当minify之后就时不时会在这儿出错，提示“Uncaught ReferenceError: $ is not defined”，用小括号包裹之后就正常了。也蛮奇怪的……
FIX：??? ??? waitid = (typeof waitid == 'undefined' || waitid === null) ? showid : waitid;

-------------------------------------------------------------------

类型：??? ??? 代码规范
坑爹指数：??? ☆
代码：??? ??? include/js/common.js=834
??? if($('fwin_dialog_submit')) $('fwin_dialog_submit').onclick = function() {
??? ??? if(typeof func == 'function') func();
??? ??? else eval(func);
??? ??? hideMenu(menuid, 'dialog')
??? };
点评：??? ??? 缺少分号结尾，跑到if判断后去了。不过js还是很宽容的，这两个小毛病不会导致啥问题。

?

ADD：有朋友说修正分号 后 反而会导致发帖编辑器上按钮（图片、附件等）全部失效了。此处请谨慎修改

-------------------------------------------------------------------

类型：??? ??? 代码规范
坑爹指数：??? ☆
代码：??? ???
include/js/common.js=1140
???? if(x.autogoto) scroll(0, x.showId.offsetTop);
include/js/common.js=1150
??? if(!evaled) evalscript(s);
include/js/common.js=1199
??? if(!evaled) evalscript(s);
include/js/common.js=1234
???? if(isUndefined(timeout)) timeout = 3000;
??? if(isUndefined(cache)) cache = 1;
??? if(isUndefined(pos)) pos = '43';
??? if(isUndefined(duration)) dura