Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
The text for the "force transition" checkbox states that this will allow the transition to occur even if workflow permissions disallow it. However, setting force transition currently will also allow the transition to occur even if the transition doesn't actually exist. This should give an error in all cases to avoid nodes being in an undefined state.
Patch enclosed.
Comment | File | Size | Author |
---|---|---|---|
workflow.HEAD_.notid_.patch | 579 bytes | gnassar | |
Comments
Comment #1
jvandyk CreditAttribution: jvandyk commentedI would prefer patches against 5.x-2.x-dev at this point.
Comment #2
gnassar CreditAttribution: gnassar commentedMakes sense. I recall reading about the additions made for Drupal 6 in another issue, now that you mention it.
Only one problem: the code was actually originally written against DRUPAL-5. I re-diff'ed it against HEAD and saw no change, so I submitted it as HEAD.
So I went back and checked, and there *is* no 2.x-dev branch (DRUPAL-5--2) that I could see. There are 2.x releases, but that's all. That would imply that DRUPAL-5 is the active devel branch for the 5.x version of the project (2.x, I'd assume) -- which would be fine, except for some reason it's currently identical to HEAD. Am I confused in this?
This is related to another issue I entered, and then moved over to Actions: #263392: DRUPAL-5 branch incompatible with DRUPAL-5 Workflow
Now would probably be a good time to put up DRUPAL-6, maybe, to help with confusion?
Comment #3
jvandyk CreditAttribution: jvandyk commentedFixed in upcoming 5.x-2.3 release. This fix will also make it into the 6.x release.
Comment #4
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #5
j4h8 CreditAttribution: j4h8 commentedHi there,
contrary to #3, apparently the fix did not make it into the 6.x release. I am running 6-1.1 and line 506 of workflow.module still reads
if (!$tid && !$force) {
Is this going to be fixed?
Thanks!
Jutta
Comment #6
Bastlynn CreditAttribution: Bastlynn commentedSince this request is over 2 years old, I'm going to assume a solution was found or you've moved on. If not, please get updated to the latest versions of all modules and make a request for it against Drupal 7 and I'll be glad to take a look at it.