Closed (fixed)
Project:
Field Permissions
Version:
8.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Task
Assigned:
Reporter:
Created:
28 Jun 2016 at 16:18 UTC
Updated:
22 Jul 2016 at 22:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
jhedstromHere's a start. I'd like to pull further permission-type-specific logic into the individual plugins, but the field report makes it difficult.
I've left the 'public' permission type hard coded since it basically means to ignore the field as far as this module is concerned, so firing up a plugin for that seemed like overkill.
Comment #3
jhedstromThat patch was missing an interface.
Comment #8
jhedstromNot sure why these are failing.
Comment #11
jhedstromThese tests continue to be green locally.
Comment #14
jhedstromThe test fails have something to do with plugin discovery. I'm adding some kernel tests to try and figure out why these only fail on Drupal CI.
Comment #17
jhedstromThe issue seems to be that the CI can't find the plugins. I'd guess this has to do with overlooked case-sensitivity, but I can't see any. This patch changes how the custom plugin manager is declared (similar to the Filter plugin manager in core).
Comment #20
jhedstromAdding a unit test to try and suss out the bizarre failures here. I am still unable to replicate these locally.
Comment #23
jhedstromI think it was indeed a case-sensitivity issue. Can't believe I didn't catch this sooner.
Comment #25
jhedstromDone and done.