Updated: Comment 0

Problem/Motivation

While working with the ControllerBase I realized that config return an unexpected value.

Proposed resolution

Remaining tasks

User interface changes

API changes

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Status: Active » Needs review
FileSize
662 bytes

.

pwolanin’s picture

Status: Needs review » Reviewed & tested by the community

yes, simple fix.

webchick’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Quick fix, +Needs tests

Hm. Tests? :\

dawehner’s picture

I was indeed confused that alex refused some tests for https://drupal.org/node/2078285#comment-7859707

dawehner’s picture

Status: Needs work » Needs review
FileSize
2.76 KB
2.11 KB

So here is a unit test, take it or drop it :)

dawehner’s picture

#5: controller-2089195-FAIL.patch queued for re-testing.

Berdir’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs tests

Yes, agreed that it makes sense. There are use cases where you need the config factory, but $this->config() should work the same way as Drupal::config().

Unit tests look fine to me, we can extend them for other methods if there's a problem later on.

andypost’s picture

+1 to rtbc, this really need for complex controllers

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed a40399f and pushed to 8.x. Thanks!

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