Revision: 1211
Author: jhoskens
Date: 2006-06-21 04:54:19 -0700 (Wed, 21 Jun 2006)
ViewCVS: http://svn.sourceforge.net/spring-rich-c/?rev=1211&view=rev
Log Message:
-----------
fix for RCP-361:
If field rows is already null, setRows would return which results in NPE's
afterwards.
Fixed by changing setRows() structure.
Modified Paths:
--------------
trunk/spring-richclient/support/src/main/java/org/springframework/richclient/table/BaseTableModel.java
Modified:
trunk/spring-richclient/support/src/main/java/org/springframework/richclient/table/BaseTableModel.java
===================================================================
---
trunk/spring-richclient/support/src/main/java/org/springframework/richclient/table/BaseTableModel.java
2006-06-21 11:35:11 UTC (rev 1210)
+++
trunk/spring-richclient/support/src/main/java/org/springframework/richclient/table/BaseTableModel.java
2006-06-21 11:54:19 UTC (rev 1211)
@@ -44,12 +44,13 @@
}
public void setRows(List rows) {
+ // first check null, if somehow field was null it may not return (in
second if)
+ if (rows == null) {
+ this.rows = new ArrayList();
+ }
if (this.rows == rows) {
return;
}
- if (rows == null) {
- this.rows = new ArrayList();
- }
this.rows = rows;
fireTableDataChanged();
}
This was sent by the SourceForge.net collaborative development platform, the
world's largest Open Source development site.
_______________________________________________
spring-rich-c-cvs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/spring-rich-c-cvs
