Subject: | Good job. |
Summary: | Package rating comment |
Messages: | 2 |
Author: | Przemek Berezowski |
Date: | 2011-03-28 12:41:38 |
Update: | 2011-03-28 18:01:58 |
|
|
|
Przemek Berezowski rated this package as follows:
Utility: | Good |
Consistency: | Good |
Examples: | Sufficient |
|
 Przemek Berezowski - 2011-03-28 12:41:38
Good job.
Please consider use of singleton pattern for PageHeader. Then you can call this in any part of your application eg in controllers you could write action related metas eg kewords, on layout you could write page global settings like doctype, on view you can load eg js scripts needed by a page. On the other hand, PageHeader object can be stored in some registry obcject, what give the same functionality :)
 Leandro Antonello - 2011-03-28 18:01:58 - In reply to message 1 from Przemek Berezowski
Hi Przemek,
I really appreciated your comments and your ideas, but i don't like the MVC software architecture, personally.
But I will consider creating a version with singleton pattern for PageHeader class soon.
If you want you can create it at will.
Thanks.
|