Please see https://www.drupal.org/node/1619504 for details.

CommentFileSizeAuthor
#4 token-deprecated-role-constants-2932276-4-D8.patch1.06 KBMatroskeen
#2 token-deprecated-role-constants-2932276-2-D8.patch1.07 KBMatroskeen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Matroskeen created an issue. See original summary.

Matroskeen’s picture

Matroskeen
Primary language Ukrainian
Location 🇺🇦 Ukraine, Lutsk
CreditAttribution: Matroskeen as a volunteer and at DevBranch, Drupal Ukraine Community commented
Assigned: Matroskeen » Unassigned
Status: Active » Needs review
FileSize
token-deprecated-role-constants-2932276-2-D8.patch1.07 KB

It could be a simple one.
Patch is here.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Needs work

Thanks. not directly related but lets remove that string cast there, that's pointless, those constants are already strings in Drupal 8.

Matroskeen’s picture

Matroskeen
Primary language Ukrainian
Location 🇺🇦 Ukraine, Lutsk
CreditAttribution: Matroskeen as a volunteer and at DevBranch, Drupal Ukraine Community commented
Status: Needs work » Needs review
FileSize
token-deprecated-role-constants-2932276-4-D8.patch1.06 KB

Fixed, updated the patch.

  • Berdir committed 1354707 on 8.x-1.x authored by Matroskeen 
    Issue #2932276 by Matroskeen: DRUPAL_ANONYMOUS_RID and...
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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