3314 Posts in 890 Topics by 835 members
|Go to End||Next >|
27 March 2009 at 9:57am Last edited: 27 March 2009 9:58am
Decided to test straight away and had issues both with the latest SVN build of SS, and the patch. Decided to post here rather bog up the Trac entry in case I was doing something wrong.
Steps to duplicate errors:
1. Get latest SVN from http://svn.silverstripe.com/open/phpinstaller/trunk
2. Install SS as normal using Blackcandy Theme
3. Add following code to /themes/blackcandy/templates/Page.ss just above $Layout
<% if InSection(home) %>
<% else_if InSection(about-us) %>
<% else %>
Not Home or About Us
<% end_if %>
4. Check out the home page and about us page on new installation.
ERRORS on SS Trunk
1. When on home page, not only is "Home" displayed by the above code, but the <% else_if InSection(about-us) %> statement is not parsed and "Home <% else_if InSection(about-us) %> About Us" is displayed on the page.
2. When on about-us page, only "Not Home or About Us" is displayed.
1. Apply patch with Tortoise SVN (Windows XP). /sapphire/core/SSViewer.php, /sapphire/tests/SSViewerTest.php and /cms/templates/Includes/CommentAdmin_SiteTree.ss all updated.
2. Go to home page and flush=1
ERROR on Patch
Fatal error: Class 'TemplateParser' not found in C:\wamp\www\ss_latest\sapphire\core\SSViewer.php on line 405
I havn't had time to go through the code to figure that out yet.
27 March 2009 at 11:10am Last edited: 27 March 2009 11:11am
My bad. I forgot to include the template parser file in that patch file. Patch file revision 5 has been revised to include that file!
Does it work any better now?
27 March 2009 at 11:33am
No more fatal error. But there is a SS error there:
http://sandbox.ecomgroup.co.nz/aaron_ss/SS_TemplateParserStackTrace.htm (not live, just an HTML dump)
Occurs with or without the test template block in my previous post.
A little over my head without picking through the code and working out how this exactly works.
28 March 2009 at 3:03am
This is valuable feedback. I think the problem is that my PHP install has a later version of the PCRE (Perl Compatible Regular Expressions) library which supports \g<name> syntax. I probably need to rework the expressions to not use that feature.
Would you be able to provide a copy of your server's phpinfo() output (at least the PHP version & PCRE info parts)? With that info, maybe I can find a matching PHP build for me to test on.
28 March 2009 at 3:18am
In the mean time, I posted revision 5.1 which uses a different syntax style in the regular expressions. Maybe it will work on your PHP install.
Revision 5 used Oniguruma style subroutine calls; 5.1 uses Python's style.
30 March 2009 at 9:46am
No luck there sorry. 5.1 fails to parse pretty much everything and just pumps out all the <% %> tags to browser.
I am using WAMP 2.0 on Windows XP.
31 March 2009 at 2:26am
This is strange. To verify that the patch contains all of the necessary files, I just applied it to a fresh copy of truck. The patch worked flawlessly. Would you have time where we could hope on IRC together and troubleshoot this?
10 April 2010 at 4:40am
*Sigh* This still isn't fixed?
Just ran into this issue on a site I'm working on. Seems like a pretty common request and could be useful.
I was the one that originally brought this up when I stumbled across it in SS version 2.2.x
I would have thought/hoped this would be fixed now that we're coming up to 2.4 Any word on this? I see the ticket is still unassigned in trac.
|Go to Top||Next >|