Bug 1082990

Summary: NPE is thrown when event with a timestamp set in the past is modified
Product: [Retired] JBoss BRMS Platform 6 Reporter: Anton Giertli <agiertli>
Component: BREAssignee: Mario Fusco <mfusco>
Status: CLOSED CURRENTRELEASE QA Contact: Tomas Schlosser <tschloss>
Severity: high Docs Contact:
Priority: high    
Version: 6.0.0CC: kverlaen, vigoyal
Target Milestone: ER1   
Target Release: 6.0.2   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
An attempt to modify an event/model in the 'THEN' condition of a rule when that event had a timestamp set in the past (before pesudoclock) with an 'after' operator, resulted in a NullPointerException being thrown. This has now been fixed with the addition of check before executing a job to check whether the job still exists in case the timer for that job has already been fired and no scheduling of a job was done.
Story Points: ---
Clone Of: 1082988 Environment:
Last Closed: 2014-08-06 19:54:13 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1082988    
Bug Blocks:    
Attachments:
Description Flags
maven reproducer
none
non maven reproducer none

Comment 1 Anton Giertli 2014-04-01 09:19:05 UTC
Created attachment 881266 [details]
maven reproducer

Comment 2 Anton Giertli 2014-04-01 09:19:35 UTC
Created attachment 881267 [details]
non maven reproducer

Comment 4 Marek Winkler 2014-04-01 10:29:50 UTC
I have reproduced this with BRMS 6.0.1 CR2 #2, working on a more isolated reproducer.

Comment 5 Marek Winkler 2014-04-01 13:12:10 UTC
*** Bug 1082988 has been marked as a duplicate of this bug. ***

Comment 6 Mario Fusco 2014-04-01 17:28:32 UTC
Fixed by https://github.com/droolsjbpm/drools/commit/14cae3411

Comment 7 Mario Fusco 2014-04-01 17:29:50 UTC
Cherry-picked to 6.0.x branch with https://github.com/droolsjbpm/drools/commit/37d0c2d03

Comment 8 Tomas Schlosser 2014-05-06 12:01:11 UTC
Verified in BRMS 6.0.2 ER2.