Related change notice: https://www.drupal.org/node/2498803

CommentFileSizeAuthor
#2 masquerade-lazy-builder-2543678-02.patch2.23 KBjhedstrom
#1 masquerade-lazy-builder-2543678-01.patch0 bytesjhedstrom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom at Phase2 commented
Status: Active » Needs review
FileSize
masquerade-lazy-builder-2543678-01.patch0 bytes

Some tests are failing, but this gets rid of the fatal.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom at Phase2 commented
FileSize
masquerade-lazy-builder-2543678-02.patch2.23 KB

A non-empty patch.

Status: Needs review » Needs work

The last submitted patch, 2: masquerade-lazy-builder-2543678-02.patch, failed testing.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost at Skilld commented
Status: Needs work » Reviewed & tested by the community
Parent issue: » #1836516: Port Masquerade to D8 and rewrite + simplify it into a new 2.x series
Related issues: +#2552881: Changes necessary from core decoupling of sessions and cookies

will commit soon

  • andypost committed e128371 on authored by jhedstrom 
    Issue #2543678 by jhedstrom:   Call to undefined method     Drupal\Core\...
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost at Skilld commented
Status: Reviewed & tested by the community » Fixed

Thanx a lot, nice clean-up, pushed

Status: Fixed » Closed (fixed)

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