[ FIXED ] - Error in MacroCommand and Notifier

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

PureMVC Architects Lounge

PureMVC Manifold

Port to PHP

Standard Version

Bug Report (: hasan)

[ FIXED ] - Error in MacroCommand and Notifier

Pages: [1]

« previous next »

Author

Topic: [ FIXED ] - Error in MacroCommand and Notifier (Read 7094 times)

m@vrtis

Newbie

Posts: 1

[ FIXED ] - Error in MacroCommand and Notifier

« 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


« Last Edit: January 14, 2009, 12:25:24 by puremvc »	Logged

hasan

Moderator
Newbie

Posts: 8

Re: possible error in MacroCommand and Notifier - pureMVC 1.0.1 - PHP Version 5

« Reply #1 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!!


	Logged

P E A C E

Hasan

puremvc

Global Moderator
Hero Member

Posts: 2871

Re: possible error in MacroCommand and Notifier - pureMVC 1.0.1 - PHP Version 5.2.1

« Reply #2 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>


	Logged