Closed (fixed)
Project:
Drupal core
Version:
8.2.x-dev
Component:
configuration system
Priority:
Critical
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
31 Jul 2016 at 20:00 UTC
Updated:
6 Sep 2016 at 11:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
alexpottAdding all the related issues.
Comment #3
alexpottComment #5
alexpottDiscussed with @xjm, @webchick, @effulgentsia and @Cottser. We agreed to keep this critical because:
Comment #6
xjmComment #7
alexpottHere's the whack it with a hammer upgrade path approach.
Comment #8
xjmThe beta allowed changes include critical issues, so the beta target tag is not applicable.
Comment #9
dawehnerI agree with alex, its not really worth to try to optimize the update here. In case we miss any case, it will hurt much more, than a little bit more time to run the update.
Can we use something like
assertNull?Its weird to not have splitted up this into two assertions. It makes debugging harder than it would have to be
Comment #10
alexpottRe #9.2 I did that because isNew() should be enough and doing it on separate lines looks weird to me. But so be it... here's the change.
Comment #11
dawehnerThank you alex!
Comment #14
catchCommitted/pushed to 8.3.x and cherry-picked to 8.2.x. Thanks!
Comment #15
alexpottLet's hope no one is doing anything made on config save... mind you if they are config import would probably be broken too.
Comment #16
gábor hojtsyWow, thanks!