Comments on: A View Stream with Zend_View

By: Rob...

Rob... — Fri, 28 Mar 2008 08:06:08 +0000

Dinoboff,

Very interesting, Thanks!

Regards,

Rob…

By: Dinoboff

Dinoboff — Fri, 28 Mar 2008 04:42:57 +0000

Here a version with caching:
http://code.google.com/p/simple-php-template/source/browse/trunk/src/library/Spt/Stream/View.php

It doesn't add the escape method. A subclass of Zend_View escape the variables by default instead.

By: Dinoboff

Dinoboff — Thu, 27 Mar 2008 16:55:36 +0000

Finally, about the ternary operator:

become:

escape($this->var ? $this->var : '-unknown-'); ?>
escape($var ? $var : '-unknown-'); ?>

Even if it is not really what you write, isn't it valid php?

By: Dinoboff

Dinoboff — Thu, 27 Mar 2008 16:40:46 +0000

We also need to escape result of functions:

$find = '/<\?=\s*([^;]*);?\s*\?>/';
$replace = "escape($1); ?>";
$this->data = preg_replace($find, $replace, $this->data);

By: Dinoboff

Dinoboff — Thu, 27 Mar 2008 16:33:56 +0000

Try again…
$find = '/<\?=\s*(\$[a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)\s*;?\s*\?>/';
$replace = "escape($1); ?>";
$this->data = preg_replace($find, $replace, $this->data);

By: Dinoboff

Dinoboff — Thu, 27 Mar 2008 16:31:05 +0000

Nevermind, I had to test the solution to understand the details. ps: make it to work I had to use the following regex:



$find = '//';

            $replace = "escape($1); ?>";

            $this->data = preg_replace($find, $replace, $this->data);

The regex for a viriable name is from http://www.php.net/language.variables

By: Rob...

Rob... — Thu, 27 Mar 2008 08:06:07 +0000

Dinoboff,

Have you example code that breaks?

Thanks,

Rob…

By: Dinoboff

Dinoboff — Thu, 27 Mar 2008 04:50:34 +0000

@rob: I don't think it can works because inside a loop same variables shouldn't be changed into an instance property (key and value of the array element).

By: Hendi

Hendi — Wed, 26 Mar 2008 13:34:29 +0000

Argh… So Wordpress also ate my post. So here's my next try:

Find the lines starting with "$find" and "$replace". In the first line, there has to be a backslash between the opening angle bracket and the first question mark and one between the asterisk and the question mark at the end.

In the second line, a backslash is needed in front of the second dollar sign (that's the one followed by "this").

By: Hendi

Hendi — Wed, 26 Mar 2008 11:41:28 +0000

There are some errors in your code, I blame WordPress for them, which eats backslashes.

Here is the updated part, replace ### with a backsplash:

$find = '/]*?|[^;?]*?)[; ]*###?>/';
$replace = "escape($1); ?>";