I suppose conditions are handled by ConditionAccessResolverTrait.

CommentFileSizeAuthor
#2 2783619-2-removed_condigion_plugin_manager_service.patch2.55 KBChi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Chi created an issue. See original summary.

Chi’s picture

Chi CreditAttribution: Chi commented
Status: Active » Needs review
FileSize
2783619-2-removed_condigion_plugin_manager_service.patch2.55 KB
dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner at Chapter Three commented
Version: 8.2.x-dev » 8.3.x-dev
Status: Needs review » Reviewed & tested by the community

I think this is a 8.3.x only change now, but it looks perfect at it is. Small bits matter over time.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: 2783619-2-removed_condigion_plugin_manager_service.patch, failed testing.

Chi’s picture

Chi CreditAttribution: Chi commented
Status: Needs work » Needs review
dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner at Chapter Three commented
Status: Needs review » Reviewed & tested by the community

Back to RTBC

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

Committed d32fe80 and pushed to 8.3.x. Thanks!

  • alexpott committed d32fe80 on 8.3.x 
    Issue #2783619 by Chi: Remove unused manager service from...

Status: Fixed » Closed (fixed)

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