Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
field system
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
17 Nov 2014 at 16:02 UTC
Updated:
5 Dec 2014 at 10:34 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
yched commentedPatch.
Comment #2
andypostseems there's no tests...
Comment #3
yched commentedTest added.
Comment #4
yched commentedSide note: #2368349: Entity view and form display configuration schemas are too verbose / key ones missing adds a missing config schema for the current *field* setting.
If it gets committed first, the patch here will need to be updated to change the config schema accordingly (-> storage setting)
Comment #8
yched commentedSide note: It's just the "test only" patch that failed, as it should.
Comment #9
andypostSure, that's why I call re-test
PS:
ContainerAwareEventDispatcherTestseems a bot flux that we see whole dayComment #10
alexpott#2368349: Entity view and form display configuration schemas are too verbose / key ones missing got in
Comment #11
yched commentedActually, #2368349: Entity view and form display configuration schemas are too verbose / key ones missing landed, so as mentioned in #4, we now also need to update the config schema.
[edit: heh, crosspost]
Comment #13
yched commentedBleh. Never mind #10, something weird happened in the copy/paste.
Patch with config schema change, and interdiff with #3
(also, using a diff algorithm that better reflects the actual change)
Comment #15
yched commented"SchemaIncompleteException: Schema errors for field.storage.entity_test.h7quxlds with the following errors: field.storage.entity_test.h7quxlds:settings.unsigned missing schema"
Weird - 'unsigned' is indeed missing from the schema (#2370305: Refactor field type configuration schemas for DX, easier to find errors is fixing that), but that doesn't cause an exception in NumberItemTest on my setup.
Well, adding it here as well if that's what it takes to make the testbot happy - this patch and #2370305: Refactor field type configuration schemas for DX, easier to find errors will conflict anyway.
Comment #16
andypostLooks good to go
Comment #17
alexpottThis issue addresses a major bug and is allowed per https://www.drupal.org/core/beta-changes. Committed d3a7d13 and pushed to 8.0.x. Thanks!