PureMVC Architects Lounge

PureMVC Manifold => Bug Report => Topic started by: miroslav on January 28, 2008, 01:00:31



Title: [ FIXED ] Small Bug in Notification
Post by: miroslav on January 28, 2008, 01:00:31
I've noticed very small bug in Notification class (v1.7.1) in toString method.

On lines 115 and 116 it's:
msg += "\nBody:"+( body == null )?"null":body.toString();
msg += "\nType:"+( type == null )?"null":type;


but it should be:
msg += "\nBody:"+(( body == null )?"null":body.toString());
msg += "\nType:"+(( type == null )?"null":type);


so you get string concatenation instead of using "\nBody" and "\nType" strings as conditional statement for conditional operator.


Title: [ FIXED ] Re: Small Bug in Notification
Post by: puremvc on January 29, 2008, 09:52:57
Excellent timing with this report!

The release of 2.0 is slightly delayed as all the demos and docs are updated.

So this fix has been incorporated into 2.0 and a corresponding test has been added to the test suite.

Cheers,
-=Cliff>