WebGUI
      Click here to register.
      
PBWG Banner


     Report a Bug > WebGUI Bug Tracker

Spectre Should Check That It Has a Valid Workflow
User jamestolley
Date 6/19/2007 7:40 pm
Severity Minor (annoying, but not harmful)
Version WebGUI 7.3.17
Views 109
Rating 1    Rate [
|
]
Karma Rank 0.000000
Previous · Next
User Message
jamestolley
Date: 6/19/2007 7:40 pm   ·   Subject: Spectre Should Check That It Has a Valid Workflow   ·   Rating: 1

WG::Op::Spectre.pm does not check that a workflow is valid before calling get() on it. The workflowIds in this case were empty, not NULL.

Back to Top
Rate [
|
]
 
 
colink
Date: 6/19/2007 10:30 pm   ·   Subject: Re: Spectre Should Check That It Has a Valid Workflow   ·   Rating: -1

There is no WebGUI::Operation::Spectre.pm.

Do you by chance mean WebGUI::Operation::Workflow::www_runWorkflow?
Back to Top
Rate [
|
]
 
 
jamestolley
Date: 6/20/2007 1:11 pm   ·   Subject: Re: Spectre Should Check That It Has a Valid Workflow   ·   Rating: 1

Unless I'm missing something...

https://svn.webgui.org/svnweb/plainblack/view/WebGUI/lib/WebGUI/Operation/Spectre.pm 

 

This line has caused problems:

			next unless $instance->getWorkflow->get("enabled");

 
Back to Top
Rate [
|
]
 
 
colink
Date: 6/22/2007 12:56 pm   ·   Subject: Re: Spectre Should Check That It Has a Valid Workflow   ·   Rating: -2

My bad.

Fixed in 7.3.19, and 7.4.0

getAllInstances will now always return valid objects only, the array ref won't contain undef.  Instead, it will log instanceIds that return an undef object. 
Back to Top
Rate [
|
]
 
 
     Report a Bug > WebGUI Bug Tracker