Bug 295841 - MarkerUtil does not create markers for nested statuses
Summary: MarkerUtil does not create markers for nested statuses
Status: RESOLVED FIXED
Alias: None
Product: EMF Services
Classification: Modeling
Component: Validation (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal
Target Milestone: ---   Edit
Assignee: Boris Gruschko CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2009-11-23 05:29 EST by Matthias Schoettle CLA
Modified: 2017-02-24 15:09 EST (History)
0 users

See Also:
boris.gruschko: iplog+

Attachments
Added methods which recursively unwrap all children of multi-statuses and creates markers for them (4.57 KB, patch)
2009-11-23 05:32 EST, Matthias Schoettle CLA 		ahunter.eclipse: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Matthias Schoettle CLA 2009-11-23 05:29:37 EST 
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2
Build Identifier: M20090917-0800

When creating or updating markers using the MarkerUtil only the children of a multi-status on the first level are unwrapped.
This happens when at least one constraint returns a multi-status. Only a marker for the multi-status itself is created then. But it's necessary to create markers for the children.


Reproducible: Always

Steps to Reproduce:
1. Create a Java constraint which returns a multi-status failure using ConstraintStatus.createMultiStatus(...)
2. Run the (Batch)Validator and use the MarkerUtil to create Markers using the results of the validation
Comment 1 Matthias Schoettle CLA 2009-11-23 05:32:55 EST 
Created attachment 152833 [details]
Added methods which recursively unwrap all children of multi-statuses and creates markers for them
Comment 2 Boris Gruschko CLA 2009-11-23 11:15:55 EST 
Thanks for the patch. I'll create a test case and apply it to HEAD
Comment 3 Boris Gruschko CLA 2009-11-24 11:44:24 EST 
patch and testcase submitted to HEAD.