» Details |
---|
|
» Comment |
I'm glad to finally see a standardized notification system. I'm probably going to start using it today. :-) - Member vars should have docblocks with @access (if not public) and (short) description of what they are (Notification.php) - the @VER@ entry doesn't seem to be altered in the files when installing the package. (Hmmm, there's no package.xml. I didn't realize that PEAR would install without a package.xml. I'm further surprised that it put them in the Event folder. Even the tests went in the right place. Interesting. ^_^ Nice job, installer author(s)!) |