Re: HTML_SetupWizard and more ....

Hi Lorenzo, > I finally had the time to have a look at this package. > I must say it's amazing, the amount of documentation > produced should be an example for us all. > I'm +1 on having this in PEAR. Thanks a lot, it's make me very happy :o) > - what about other template engines? Can we switch > to the one we like (I haven't checked the sources > about this subject)? Is template-management detached > from the main class? A template "plugin" model would be great. Well, it was one of my objectives, and wanted to do it easily but only switching one engine to another with simple code like : $wiz = new HTML_SetupWizard(); $wiz->setOption ( array('templateEngine' => 'Sigma') ); But finally, it's not really well implemented; the option do nothing right now, until we have a universal template renders/plugin as suggest in another thread of the pear-dev list . Great Idea i'm +1 for this option ! > - In the examples provided, the main index.php has a > few NOTICES: > "${node1_ .$i}" should be "${'node1_'.$i}" (notice the quotes). > Similarly for ${nodeX .$i} I'm surprised; You've probably used the wrong examples package! i've fixed this error on SW4P_demos-1.0RC1 : http://farell.free.fr/en/pear/bin/SW4P_demos-1.0RC1.tgz > - in function registerElementWizard, you should use the > DIRECTORY_SEPARATOR constant instead of hard-coding a '/': > $dir = dirname(__FILE__) . > DIRECTORY_SEPARATOR.'SetupWizard'.DIRECTORY_SEPARATOR.'GUI'; > > and a few lines below: > $file = $dir.DIRECTORY_SEPARATOR.$entry; OK i've well noticed it , thanks Best regards, Laurent