The current field permissions service relies heavily on static methods that call out to global services via \Drupal.

This task is to move as many of those as possible into non-static methods and inject the required services.

CommentFileSizeAuthor
#2 2757431-02.patch28.08 KBjhedstrom

Comments

jhedstrom created an issue. See original summary.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Active » Needs review
StatusFileSize
new 2757431-02.patch28.08 KB

This also removes several methods from the interface that were only used internally, and makes them protected instead of public.

Status: Needs review » Needs work

The last submitted patch, 2: 2757431-02.patch, failed testing.

The last submitted patch, 2: 2757431-02.patch, failed testing.

  • jhedstrom committed e41c1ec on 8.x-1.x 
    Issue #2757431: Move away from static methods in field permission...
jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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