On fields ui, when creating a new field, this odd behavior prevents further changes to machine name.
After editing a field machine name, then re-ordering the field, the machine name field disappears.
Here are screenshots:
This bug definitely applies to 7.x-dev, and I assume to 8.x as well since field ui doesn't seem to have changed all that much.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | core-js-machine-name_tabledrag-1700210-1-D7.patch | 678 bytes | jfhovinne |
| #1 | core-js-machine-name_tabledrag-1700210-1-D7-do-not-test.patch | 678 bytes | nod_ |
| #1 | core-js-machine-name_tabledrag-1700210-1.patch | 706 bytes | nod_ |
| disappearing-machine-name_step2.png | 50.82 KB | aaronbauman | |
| disappearing-machine-name_step1.png | 39.27 KB | aaronbauman |
Comments
Comment #1
nod_That's a pretty crappy bug.
Don't forget the JavaScript tag next time :)
Comment #2
swentel commentedWorks perfect, for both versions.
Comment #3
webchickCommitted and pushed to 8.x. Thanks!
For whatever reason this didn't apply cleanly to 7.x, so moving to backport.
Comment #4
nod_tag
Comment #5
jfhovinne commentedTested core-js-machine-name_tabledrag-1700210-1-D7-do-not-test.patch from #1, which applies cleanly against 7.x-dev and fixes the bug, so renaming the file and submitting for testing.
Comment #6
aaronbaumanpatch in #5 solves the issue for me.
nice work, folks
Comment #7
nod_Sorry putting back as NW since we need a test scenario added to this page http://drupal.org/node/1777342 and hopefully some testswarm code.
Comment #8
ZenDoodles commentedHey @nod_
I'm assuming the testswarm tests should be for 8x first? Then we can backport the whole kit and caboodle.
Comment #9
nod_owww :( testing tabledrag suck big time.
But yes, the testing page http://drupal.org/node/1777342 needs updating.
Comment #10
s_leu commentedThis is no longer a problem in D8 since the tabledrag is now only available in the manage form display tab where no only already saved fields will appear.
Comment #13
sutharsan commentedRemoving Novice tag, 'Needs JS testing' does not qualify for novice.
Comment #17
xjmComment #18
poker10 commentedI think this can be closed as fixed in 8.x-dev (it was commited in #3 and is no longer relevant for 9.4.x-dev as there is no tabledrag on that page). According to the backport policy we should continue with the D7 fix in the separate D7 issue here: #2649926: [D7] Field UX: machine name disappears.