Shao Fan

关于JAVA与软件工程
posts - 31, comments - 71, trackbacks - 0, articles - 4
  BlogJava :: 首页 :: 新随笔 :: 联系 :: 聚合  :: 管理

Struts2客户端验证的一个bug

Posted on 2007-06-03 17:56 shaofan 阅读(2528) 评论(3)  编辑  收藏 所属分类: Java
Struts2默认theme是xhtml,它用表格来对表单中的控件进行排版。它也提供一个客户端的js验证功能,但是它的js脚本却有些问题,在某些情况下,前次验证的提示信息无法被清除,提示信息会不断的累积显示在屏幕上。而按照设计,每次提交表单时应只显示每次验证的出错信息。

它的客户端验证的流程大概是这样,用户提交表单时,对各个控件的输入按预先设置的规则进行验证,如果有问题,则清除表单里原有的出错提示信息,并写入新的提示。其设计的功能是把出错信息写表格里出错控件的上方,以便用户看得更加清楚。问题就出在其用来清除原出错信息的函数,其代码是这样的(在struts.jar的template/xhtml目录下可以找到):

 1 function clearErrorMessages(form) {
 2 
 3     var table = form.childNodes[1];
 4     iftypeof table == "undefined" ) {
 5         table = form.childNodes[0];
 6     }
 7 
 8     // clear out any rows with an "errorFor" attribute
 9     var rows = table.rows;
10     var rowsToDelete = new Array();
11     if (rows == null){
12         return;
13     }
14 
15     for(var i = 0; i < rows.length; i++) {
16         var r = rows[i];
17         if (r.getAttribute("errorFor")) {
18             rowsToDelete.push(r);
19         }
20     }
21 
22     // now delete the rows
23     for (var i = 0; i < rowsToDelete.length; i++) {
24         var r = rowsToDelete[i];
25         table.deleteRow(r.rowIndex);
26         //table.removeChild(rowsToDelete[i]);
27     }
28 }


看这个函数的前三行,它试图取得form的第1个或第2个子节点,并把它作为table来处理(看接下来的几行)。要想清除表格里的错误信息,首先要取得表格本身,这没错,但是如果第1个或第2个子节点不是table的话,脚本就会出错,造成原出错信息无法清除,这样每次提交后的提示信息就会累积在屏幕上。

要解决这个问题有两个办法:
  • 写代码时要小心,保证form的第1或2个子节点是table,不要在生成table前加其他代码。
  • 或,修改xhtml的validation.js,使它总能获得正确的table元素,重新打包到struts.jar。
刚看了一下Struts的JIRA,已经有人报告了这个问题(id WW-1802),而且这个bug在2.1版本中已经解决了。

评论

# re: Struts2客户端验证的一个bug  回复  更多评论   

2008-01-25 11:59 by encodinglife
找了很久 终于找到 错误信息累加 的原因了
多谢了!!!

# re: Struts2客户端验证的一个bug  回复  更多评论   

2008-01-25 12:13 by encodinglife
struts 2.1 发布了吗
刚才到struts home 只有2.0.11版本的
没有 2.1 版本的
这是怎么回事啊?
请博主回复一下

# re: Struts2客户端验证的一个bug[未登录]  回复  更多评论   

2008-01-27 16:13 by shaofan
楼上的提醒,刚去看了一眼,struts还真是没发布到2.1.0,现在最高版本是2.0.11。但是JIRA上确实显示去年5月份这个BUG已经被搞定了,但是不知道为什么修复的版本是2.1.0。也许是写错了??我现在没有用struts在开发了,楼上试试最2.0.11,也许已经修复了?

只有注册用户登录后才能发表评论。


网站导航: