PureMVC Architects Lounge

PureMVC Manifold => Bug Report => Topic started by: factotvm on February 13, 2009, 03:06:30



Title: [Deprecated port] Bug when calling toString on Notification
Post by: factotvm on February 13, 2009, 03:06:30
Here's the patch I created to fix the issue:

:
Index: src/main/javascript/org/puremvc/js/core/PureMVC_JS_1_0.js
===================================================================
@@ -623,9 +623,9 @@
  };
  o.toString = function()
  {
- var msg = "Notification Name:"+getName();
- msg += "\nBody:"+(( body == null )?"null":body.toString());
- msg += "\nType:"+(( type == null )?"null":type);
+ var msg = "Notification Name:"+this._name;
+ msg += "\nBody:"+(( this._body == null )?"null":this._body.toString());
+ msg += "\nType:"+(( this._type == null )?"null":this._type);
  return msg;
  };
 }


Title: Re: Bug when calling toString on Notification
Post by: Tekool on February 14, 2009, 06:50:33
Thanks again.

It will become :

:
o.toString = function()/*String*/
{
var msg/*String*/ = "Notification Name:" + this.getName();
msg += "\nBody:" + (( this._body == null ) ? "null" : this._body.toString());
msg += "\nType:" + (( this._type == null ) ? "null" : this._type);
return msg;
}

Already fixed in the trunk.

I have added PureMVC_JS/bin/PureMVC_JS_1_0_1.js to the trunk. It is updated with your fixes.

I will check the forum often next days as it seems you are working on something new with it.