Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
language system
Priority:
Minor
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
5 Oct 2014 at 15:54 UTC
Updated:
26 Oct 2014 at 21:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dawehnerComing from #2304403: Convert language:weight into a protected property
Comment #2
dawehner.
Comment #3
rpayanmWorking on this...
Comment #4
yesct commentedThanks. That looks good.
was there something else you wanted to work on? unassigning since it was green and want to be able to continue to move this issue forward. comment here or ping me irc if you were in the middle of something.
Comment #5
yesct commentedI'm not sure how many places we want to look for patterns like that, but I think we can do both that were in #2304403: Convert language:weight into a protected property.
There is another in ConfigurableLanguageManager.
Comment #6
rpayanmfixed :D
Comment #7
yesct commentedthanks.
--
I took a look at the patch.
out of scope for this issue.
--
the rest looks ok.
--
Note https://drupal.org/documentation/git/interdiff is really nice when working on an issue that has a previous patch. It really helps reviewers.
Comment #8
rpayanmsorry, here the patch and interdiff for you last correction :)
Comment #9
yesct commentedthis looks super. improves just the two from the other patch that @dawehner spotted.
Comment #10
alexpottCommitted 8b24674 and pushed to 8.0.x. Thanks!