Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] add xslt_stylesheet_param directive

Laurence Rowe
February 07, 2012 08:04PM
On 4 February 2012 16:47, SamB <ja.nginx@mailnull.com> wrote:
> Hello,
>
>    I've modified my patch, according to comments in
> http://mailman.nginx.org/pipermail/nginx-devel/2012-January/001751.html.
>
>    Directive has been renamed to xslt_param, taking 2 parameters - name and
> value separated (similarly to set directive),
>    therefore it's now also possible to pass ':' character as it had been
> requested before.
>    I think it's better and easier to have it this way + there is no need to
> do that ugly parsing in runtime ;)

Great! Thanks for working on this. One other thing that occurs to me,
it would be useful to be able to pass arbitrary string paramaters.
Perhaps an xslt_string_param directive could also be added?

XPath quoting gets complicated when a string contains both single and
double quotes (see
http://stackoverflow.com/questions/642125/encoding-xpath-expressions-with-both-single-and-double-quotes),
so you should use
http://xmlsoft.org/XSLT/html/libxslt-variables.html#xsltQuoteOneUserParam
on those values.

Laurence

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH] add xslt_stylesheet_param directive Attachments

SamB 1658 January 24, 2012 05:04PM

Re: [PATCH] add xslt_stylesheet_param directive

Maxim Dounin 638 January 30, 2012 08:54PM

Re: [PATCH] add xslt_stylesheet_param directive Attachments

SamB 520 February 04, 2012 11:50AM

Re: [PATCH] add xslt_stylesheet_param directive

Laurence Rowe 578 February 07, 2012 08:04PM

Re: [PATCH] add xslt_stylesheet_param directive Attachments

SamB 514 February 08, 2012 07:58AM

Re: [PATCH] add xslt_stylesheet_param directive

Maxim Dounin 543 February 09, 2012 06:14PM

Re: [PATCH] add xslt_stylesheet_param directive

SamB 715 February 10, 2012 12:50PM

Re: [PATCH] add xslt_stylesheet_param directive

Maxim Dounin 642 February 09, 2012 06:14PM

Re: [PATCH] add xslt_stylesheet_param directive

SamB 675 February 10, 2012 12:02PM

Re: [PATCH] add xslt_stylesheet_param directive Attachments

SamB 608 February 26, 2012 10:18AM

Re: [PATCH] add xslt_stylesheet_param directive

Maxim Dounin 429 March 04, 2012 08:00AM

Re: [PATCH] add xslt_stylesheet_param directive

SamB 588 March 05, 2012 05:34PM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 91
Record Number of Users: 6 on February 13, 2018
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready