Re: SCB1381 - Scarab summary page

Hi :)

yes, it works now. Everything works, except the follwing:

We did a multi user test right now:
If user A changes an issue in module 1, scarab clears the report values 
cached for module 2 from user B, too.

cheers

Völker Ronny schrieb:
> Oops, you are right.
>
> In this code all instances ReportTableModel share the same cached values, which is obviously wrong.
> But our problem remains, that we don't have a ReportTableModel-instance when we clear the cache.
>
> As far as I know, the MethodResultCache does not support directly removing the results of a method for all instances of a class.
> A workaround is the following:
> getMethodResult().get(REPORT_TABLE_MODEL, GET_VALUEAT, this, new Integer(row),new Integer(column));
> getMethodResult().put(contents, REPORT_TABLE_MODEL,GET_VALUEAT, this, new Integer(row),new Integer(column));
> getMethodResult().removeAll(REPORT_TABLE_MODEL,GET_VALUEAT);
> You can think of this as ReportTableModel would have a static method getValueAt, which takes an instance of ReportTableModel as first argument.
>
> Btw, the two constants are better located in ScarabReportTable or ReportManager because they don't make sense elsewhere.
>
> Ronny
>
> -----Original Message-----
> From: Johannes Hoechstaedter [mailto:[email protected]] 
> Sent: Wednesday, March 05, 2008 11:15 AM
> To: [email protected]
> Subject: Re: SCB1381 - Scarab summary page
>
> Hi,
>
> ok I introudced some finals in ScarabConstants. It works. My question is 
> now, is scarab still ready for multi user usage?
>
> The code in ScarabReportTable looks like this now:
>
> ReportManager.getMethodResult().get(ScarabConstants.REPORT_TABLE_MODEL,ScarabConstants.GET_VALUE_AT,new 
> Integer(row),new Integer(column));
>
> And in ActivitySetManager like this:
>
> ReportManager.getMethodResult().removeAll(ScarabConstants.REPORT_TABLE_MODEL,ScarabConstants.GET_VALUE_AT);
>
> cheers
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>
>
>