Skip to content

Added an article about private console commands #7051

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 16 commits into from

Conversation

javiereguiluz
Copy link
Member

This documents symfony/symfony#20029.

GeertDD and others added 15 commits October 26, 2014 14:50
This PR was submitted for the master branch but it was merged into the 2.4 branch instead (closes symfony#4343).

Discussion
----------

Compiled PHP for customized route matching

The second parameter of `rtrim()` is a character mask. By checking if `rtrim($pathinfo, '/contact') === ''`, `$pathinfo` could be any URL containing those characters and not only "/contact" which was configured as the route path, for example:

    $pathinfo = '/octocat';
    var_dump(rtrim($pathinfo, '/contact') === ''); // bool(true)

Commits
-------

fb5e820 Compiled PHP for customized route matching
@javiereguiluz javiereguiluz force-pushed the private_commands branch 2 times, most recently from 2092802 to 78c9293 Compare October 10, 2016 08:39
@javiereguiluz
Copy link
Member Author

Closed in favor of #7052. Excuse me for the noise.

@javiereguiluz javiereguiluz deleted the private_commands branch May 24, 2018 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants