[CLK-283] Table should nullify rowList property

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Component/s: core
• Environment: Click 1.4RC2
• Assignee: Bob Schellink
• Reporter: Bob Schellink
• Created: 09/Jan/08 12:18
• Updated: 11/Feb/08 03:56
• Resolved: 10/Jan/08 10:11

Description

Hi Malcolm,

We use a cache layer on top of read-only DAO's. The result of the DAO methods are placed in unmodifiable collections. However since the table calls rowList.clear() an exception is thrown. If we then pass in the modifiable versions, the actual cache objects are cleared.

I think a better and less error prone way is to dereference rowList instead:

public void onDestroy() {
if (getNullifyRowListOnDestroy())

{ setRowList(null); }

...

What do you think?