The same as #2350507: \Drupal\Core\Url has no __toString() magic method.

CommentFileSizeAuthor
#2 tostring-magic-method-2707853-2.patch1.34 KBnicolas.rafaelli
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BR0kEN created an issue. See original summary.

nicolas.rafaelli’s picture

nicolas.rafaelli CreditAttribution: nicolas.rafaelli at Globant commented
Status: Active » Needs review
FileSize
tostring-magic-method-2707853-2.patch1.34 KB

I hope this helps.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer and at Chapter Three commented

Right, but where do you actually need one? Is it that important?

Status: Needs review » Needs work

The last submitted patch, 2: tostring-magic-method-2707853-2.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer and at Chapter Three commented

IMHO if we gonna change that we should change it for both URL and LINK but well, we decided to make the conversion explicit. I'm not sure then why this is a bug.

Version: 8.2.x-dev » 8.3.x-dev

Drupal 8.2.0-beta1 was released on August 3, 2016, which means new developments and disruptive changes should now be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.3.x-dev » 8.4.x-dev

Drupal 8.3.0-alpha1 will be released the week of January 30, 2017, which means new developments and disruptive changes should now be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer and at Chapter Three commented
Status: Needs work » Closed (works as designed)