DO NOT REPLY [Bug 51964] Documentation of RewriteRule flag skip unclear

[bu...@apache.org]

https://issues.apache.org/bugzilla/show_bug.cgi?id=51964

Filipus Klutiero <ch...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |VERIFIED

--- Comment #2 from Filipus Klutiero <ch...@gmail.com> 2012-04-10 15:47:55 UTC ---
Ah, I see now. Thank you, that is a lot clearer.

I would still suggest some clarifications.
"This can be thought of as a goto statement in your rewrite ruleset."
could be
"This can be thought of as a forward-only goto statement in your rewrite
ruleset."

Also, the formulation "The last rule of the then-clause becomes skip=N, where N
is the number of rules in the else-clause:" for the if-then-else example would
make me think of the following if clause:

RewriteRule (.*\.gif) images.php?$1
RewriteRule (.*\.html) docs.php?$1 [S=1] # Skip past the "else" stanza.

Instead, we give

RewriteRule (.*\.gif) images.php?$1
RewriteRule (.*\.html) docs.php?$1
# Skip past the "else" stanza.
RewriteRule .? - [S=1]

I would present this form this way:
"A skip=N rule is added at the end of the then-clause, where N is the number of
rules in the else-clause:"

Finally, it would be nice to start by explaining the argument (N).

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: docs-unsubscribe@httpd.apache.org
For additional commands, e-mail: docs-help@httpd.apache.org