When saving a field in Field UI, any of its formatter settings under Display Fields are lost.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | 553292-summary-fix-d7.patch | 1008 bytes | andypost |
| #7 | field_ui_wipe_display.patch | 3.51 KB | quicksketch |
| #3 | field_ui_wipe_display-553292-3.patch | 3.28 KB | yched |
Comments
Comment #1
webchickGrrr. I keep doing that!
Comment #2
yched commentedComment #3
yched commentedPatch. Kittens party time: also removes a TODO in this area, and useless explicit includes of field.crud.inc, that's automatically loaded anyway.
Comment #5
webchickOh, testing bot...
Comment #7
quicksketchReroll to accommodate for changes caused by #367595: Translatable fields.
Comment #8
yched commentedThanks for the reroll, Nate.
Dare I bump to RTBC ? Fixing this will ease testing of Filefield / Imagefield patches.
Comment #9
yched commentedEr, and actually doing it.
Comment #10
quicksketchOf course I actually tried this out also. It does indeed correct the saving of formatters when used in combination with the Image field patch.
Comment #11
webchickThanks, committed to HEAD.
Marking needs work because we need tests for this.
Comment #12
sun.core commentedTests don't qualify as critical.
Comment #13
andypostFormatter settings button is hidden for field that has settings but summary returns empty string.
Comment #14
yched commented@andypost - the current behavior is intended.
Might be discussed, but does not belong to this thread. Care to open a separate issue ?
Resetting to closed rather than 'needs work for tests', because there's a separate issue for Field UI tests.
Comment #15
andypostI think we should discuss this at new issue #961412: Formatter settings button lost when summary is empty