-
Notifications
You must be signed in to change notification settings - Fork 988
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
Fixed issue 18821: Public url is not used for SURVEYURL #3371
Fixed issue 18821: Public url is not used for SURVEYURL #3371
Conversation
gabrieljenik
commented
Aug 18, 2023
- Test for cretaePublicUrl and getPublicBaseUrl
- Porting fix for LSYiiValidatorsTest::testXssFilterAttributeForSuperAdmin()
- Porting fix about urls
Expected url generated dinamically so that tests stick to site configuration.
This reverts commit ac0bac2.
$_SERVER attributes initialized to avoid errors.
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.
My point of view : (see comment) : publicurl must be always used eve if it's broken.
$parameters = array('param_one' => 1, 'param_two' => 2); | ||
$url = Yii::app()->createPublicUrl('controller/action', $parameters); | ||
|
||
$expectedRelativeUrl = Yii::app()->createUrl('controller/action', $parameters); |
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.
👍
This test is already merged on master. |
OK, i think its a new test here :) |