While book or comment look at the very property they are filling in, not so node. The published checkbox just can't be ticked off on a multistep node form because it gets overwritten by the original. Trivial, I say.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

I did this (with helps of chx)

1) disable js
2) enable poll
3) go to node/add/poll
4) untick published
5) fill in the title and two choices
6) now press add more
7) the published checkbox will be back

With and without patch.

And Chx's patch fixes it ;)
RTBC till bots says no

Status: Reviewed & tested by the community » Needs work

The last submitted patch, x.patch, failed testing.

Status: Needs work » Needs review

Re-test of x.patch from comment @comment was requested by aspilicious.

chx’s picture

Status: Needs review » Needs work

Well, #1 needs to be turned into a test anyways and it won't be too hard, really, I have already described on IRC and aspilicious reposted here the steps to write the test.

chx’s picture

Status: Needs work » Needs review
FileSize
1.48 KB

With a test.

chx’s picture

FileSize
2.1 KB

Right. That should fail because it does not include the original patch. This one does. Posting this way proves that the patch not just works but necessary too.

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

As above (with proper tests)

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Nice find! Nice test! :)

Committed to HEAD.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.