Problem/Motivation

Maybe related to #2271005: Rename Language module's LanguageInterface to ConfigurableLanguageInterface and Language to ConfigurableLanguage but maybe not.
We dont need this alias in the use.

Proposed resolution

Remaining tasks

User interface changes

No.

API changes

No.

CommentFileSizeAuthor
#2 base-2337849-1.patch7.99 KBmartin107
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

martin107’s picture

martin107 CreditAttribution: martin107 commented
Assigned: Unassigned » martin107

I am working on this now

martin107’s picture

martin107 CreditAttribution: martin107 commented
Assigned: martin107 » Unassigned
Status: Active » Needs review
FileSize
base-2337849-1.patch7.99 KB

Without the alias these files read much better.

YesCT’s picture

YesCT CreditAttribution: YesCT commented
Status: Needs review » Reviewed & tested by the community

read the whole patch. looks good. no unintended changes.
(if there are other use as aliases we can get rid of as a result of #2271005: Rename Language module's LanguageInterface to ConfigurableLanguageInterface and Language to ConfigurableLanguage, we can do them separately)

[edit]
I also applied the patch and did a search:
ag -Q 'use Drupal\Core\Language\LanguageInterface as' core/*
and got no results. (which is good)

alexpott’s picture

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

Committed f30b126 and pushed to 8.0.x. Thanks!

  • alexpott committed f30b126 on 8.0.x 
    Issue #2337849 by martin107 | YesCT: Dont need alias in use for use...

Status: Fixed » Closed (fixed)

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