Check last state by weight, not sid

bwo - August 9, 2008 - 10:29
Project:Comment Workflow
Version:5.x-1.1
Component:Miscellaneous
Category:bug report
Priority:normal
Assigned:John Money
Status:closed
Description

I have installed the module and set everything according to help page, did some workflow state changes and the comments behave as they used to, I cannot see any change. What this module is supposed to do exactly? Thanks for help!

#1

John Money - August 9, 2008 - 17:48
Status:active» closed

Quick test case:

  • Node type has comments enabled (e.g. read/write)
  • Node type is assigned workflow with 3 states: created, Review, Approved
  • Node created, advancing workflow state to Review
  • Node state is changed to Approved
  • Comments are now closed (e.g. read only)
  • Node state is changed back to Review
  • Comments are now reopened (e.g. read/write)

Let me know if you need further help.

#2

bwo - August 14, 2008 - 10:11
Title:How to use this module?» Check last state by weight, not sid
Category:support request» bug report
Status:closed» active

OK, I know what's going on. You treat as "last state" the state with the highest sid, i.e. the last created state, which is not always the case. I think the module should the check highest weight, not sid.

#3

John Money - August 21, 2008 - 20:05
Assigned to:Anonymous» John Money
Status:active» closed

Yes, indeed... new release forthcoming. Thanks for report! :)

 
 

Drupal is a registered trademark of Dries Buytaert.