-
-
Notifications
You must be signed in to change notification settings - Fork 946
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
[ticket/15351] Makes confirm_works in a router context (app.php) #4932
Conversation
e6343ef
to
5256426
Compare
phpBB/phpbb/path_helper.php
Outdated
@@ -496,4 +496,17 @@ public function get_valid_page($page, $mod_rewrite = false) | |||
|
|||
return $page; | |||
} | |||
|
|||
/** | |||
* Tells if the router is currently in use (if the current page is a route or note) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo
// Script name URI (e.g. phpBB/app.php) | ||
$script_name = $this->symfony_request->getScriptName(); | ||
|
||
return basename($script_name) === 'app.' . $this->php_ext; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this also work when the .htaccess redirection is in place?
Bugs that are connected to this are nasty.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes it works. The script name should always be defined
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Surely it's always defined but does it also always point to the app.php when routing through it but is redirected by .htaccess?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep
5256426
to
98f7612
Compare
98f7612
to
c368d17
Compare
[ticket/15351] Makes confirm_works in a router context (app.php)
PHPBB3-15351
Checklist:
Tracker ticket (set the ticket ID to your ticket ID):
https://tracker.phpbb.com/browse/PHPBB3-15351