PureMVC Architects Lounge

PureMVC Manifold => Bug Report => Topic started by: m@vrtis on January 01, 2009, 02:11:48



Title: [ FIXED ] - Error in MacroCommand and Notifier
Post by: m@vrtis on January 01, 2009, 02:11:48
the Notifier is calling the parent::__construct() but the Notifier doesn't extend any class

  public function __construct()
  {
    $this->facade = Facade::getInstance();
    parent::__construct();
  }

The MacroCommand is missing the parent::__construct() and I was not able reference to the facade when i tried to extend the MacroCommand.

I removed the parent::__construct() from Notifier and added it to MacroCommand.

I could have done something wrong, but since I've made the changes it has been working fine.

Nice work on the 1.0.1 release
 
--matt



Title: Re: possible error in MacroCommand and Notifier - pureMVC 1.0.1 - PHP Version 5
Post by: hasan on January 02, 2009, 04:16:00
Hi Matt,

Thanks for the insight. I've updated trunk with those changes. Let us know if you have a demo you'd like to share with the community ;)

Happy New Year!!



Title: Re: possible error in MacroCommand and Notifier - pureMVC 1.0.1 - PHP Version 5.2.1
Post by: puremvc on January 03, 2009, 02:37:13
Hasan, send me an email with the new version info and I'll bag n tag the new release and update the wiki.

Thanks and Appy New Year. (Kinda nerdy pun, huh?)

Cheers,
-=Cliff>


Title: Re: possible error in MacroCommand and Notifier - pureMVC 1.0.1 - PHP Version 5
Post by: hasan on January 13, 2009, 09:46:47
@Cliff - Email sent. Guess I'm officially a nerd because I immediately got it...:)

Appy New Year!!