#2184653: Rename Drupal\Component\Utility\Url to UrlHelper missed one important thing when moving the file... Renaming the class!

CommentFileSizeAuthor
url-helper-test.patch701 bytestim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs review » Reviewed & tested by the community

hah

Status: Reviewed & tested by the community » Needs work

The last submitted patch, url-helper-test.patch, failed testing.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Needs work » Needs review

url-helper-test.patch queued for re-testing.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Sydney, Australia
CreditAttribution: jibran commented
Status: Needs review » Reviewed & tested by the community

Back to RTBC

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Priority: Normal » Minor
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

  • Commit a68f86e on 8.x by webchick: 
    Issue #2228627 by tim.plunkett: Core/tests/Drupal/Tests/Component/...

Status: Fixed » Closed (fixed)

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