Futurescale, Inc. PureMVC Home

The PureMVC Framework Code at the Speed of Thought


Over 10 years of community discussion and knowledge are maintained here as a read-only archive.

New discussions should be taken up in issues on the appropriate projects at https://github.com/PureMVC

Pages: [1]
Print
Author Topic: [Deprecated port] Bug when calling toString on Notification  (Read 4380 times)
factotvm
Newbie
*
Posts: 2


View Profile Email
« 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;
  };
 }
« Last Edit: February 15, 2010, 05:05:17 by puremvc » Logged
Tekool
Moderator
Sr. Member
*****
Posts: 192


View Profile WWW Email
« Reply #1 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.
Logged
Pages: [1]
Print