Files: 
CommentFileSizeAuthor
#10 remove_deprecate_drupal_strip_dangerous_protocols_2089327-10.patch6.47 KBCyberschorsch
PASSED: [[SimpleTest]]: [MySQL] 58,409 pass(es).
[ View ]
#4 2089327-4.patch6.43 KBthedavidmeister
PASSED: [[SimpleTest]]: [MySQL] 58,833 pass(es).
[ View ]
#1 2089327-1.patch6.43 KBthedavidmeister
PASSED: [[SimpleTest]]: [MySQL] 59,150 pass(es).
[ View ]

Comments

thedavidmeister’s picture

Status:Active» Needs review
StatusFileSize
new6.43 KB
PASSED: [[SimpleTest]]: [MySQL] 59,150 pass(es).
[ View ]

patch

jibran’s picture

Status:Needs review» Reviewed & tested by the community
Issue tags:+Quick fix, +Novice

I have done quick search unable to find any more occurrence of drupal_strip_dangerous_protocols so RTBC.

webchick’s picture

Title:Convert all core calls to drupal_strip_dangerous_protocols() to \Drupal\Component\Utility\Url::stripDangerousProtocols()» Convert all core calls to drupal_strip_dangerous_protocols() to \\Drupal\\Component\\Utility\\Url::stripDangerousProtocols()
Status:Reviewed & tested by the community» Needs work
Issue tags:+Needs reroll

Patch no longer applies.

thedavidmeister’s picture

Status:Needs work» Needs review
StatusFileSize
new6.43 KB
PASSED: [[SimpleTest]]: [MySQL] 58,833 pass(es).
[ View ]

reroll

thedavidmeister’s picture

Status:Needs review» Reviewed & tested by the community

RTBC as per #2

jibran’s picture

Issue tags:-Needs reroll

RTBC +1

thedavidmeister’s picture

Title:Convert all core calls to drupal_strip_dangerous_protocols() to \\Drupal\\Component\\Utility\\Url::stripDangerousProtocols()» Remove/deprecate calls to drupal_strip_dangerous_protocols() use \\Drupal\\Component\\Utility\\Url::stripDangerousProtocols()
Status:Reviewed & tested by the community» Needs work

Let's actually deprecate drupal_strip_dangerous_protocols()

Cyberschorsch’s picture

Status:Needs work» Reviewed & tested by the community

I don't seem to find anymore occurrence of drupal_strip_dangerous_protocols() so this should be fixed?

webchick’s picture

Status:Reviewed & tested by the community» Needs work

Sorry, doesn't seem to apply. :(

Cyberschorsch’s picture

Status:Needs work» Needs review
StatusFileSize
new6.47 KB
PASSED: [[SimpleTest]]: [MySQL] 58,409 pass(es).
[ View ]

I rerolled the patch.

jibran’s picture

Title:Remove/deprecate calls to drupal_strip_dangerous_protocols() use \\Drupal\\Component\\Utility\\Url::stripDangerousProtocols()» Remove / deprecate calls to drupal_strip_dangerous_protocols() use \Drupal\Component\Utility\Url::stripDangerousProtocols()
Status:Needs review» Reviewed & tested by the community

Back to RTBC.

webchick’s picture

Status:Reviewed & tested by the community» Fixed

Committed and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary:View changes

Updated issue summary.