This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 150210 - Unable to create DOM tree scaner
Summary: Unable to create DOM tree scaner
Status: RESOLVED FIXED
Alias: None
Product: xml
Classification: Unclassified
Component: Code (show other bugs)
Version: 6.x
Hardware: All All
: P1 blocker (vote)
Assignee: Sonali Kochar
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-15 12:29 UTC by Michael Nazarov
Modified: 2008-10-17 19:06 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
java2/modules/org-netbeans-modules-xml-tools-java.jar (271.03 KB, application/octet-stream)
2008-10-15 19:39 UTC, Samaresh Panda
Details
Fixed with new jar (346.98 KB, image/png)
2008-10-15 22:18 UTC, tonybeckham
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Nazarov 2008-10-15 12:29:09 UTC
Product Version: NetBeans IDE Dev (Build 200810150101)
Java: 1.6.0_10-beta; Java HotSpot(TM) Client VM 11.0-b12
System: Windows XP version 5.1 running on x86; Cp1251; ru_RU (nb)

1. Create Java Application.
2. Add Purchase order sample schema.
3. Generate sample XML for created schema with purchaseOrder as root element.
4. Generate DTD for XML.
5. Generate DOM tree scanner for DTD.

NB will produce completely empty Java source file NewPurchaseOrderScanner.java.
Comment 1 Samaresh Panda 2008-10-15 14:31:07 UTC
The first thing I want to know is, do you have automated tests for this and if they are part of commit validation? If
not, why not? We have to have more and more automated tests so that we don't discover these issues after code-freeze.
Comment 2 Samaresh Panda 2008-10-15 14:32:24 UTC
I almost forgot, Sonali is on vacation. I'll look into it.
Comment 3 Michael Nazarov 2008-10-15 14:52:01 UTC
Yes we have automated tests for this and this issue found using them.
Do you think we need to replace short commit validation with all existing tests?
Comment 4 Samaresh Panda 2008-10-15 15:18:51 UTC
No. I would pick tests on various actions and not the fine-grained tests such as creating a substitution-group in a
schema. I think all top-level actions should be included in the CV.
Comment 5 Samaresh Panda 2008-10-15 15:21:09 UTC
> Yes we have automated tests for this and this issue found using them.
Are you saying that, this was only found in 200810150101 and was working in all earlier builds?
Comment 6 Michael Nazarov 2008-10-15 15:23:51 UTC
Commit validation tests we have now created long time ago and nobody care about incomplete content of them. The only
time you asked about new test was about two week ago and test added next day if you remember.
Anyway it's incorrect to move test into commit validation in case of any found issue. This is not purpose of CV.

Well, if you have concerns about CV set we can discuss.
Comment 7 Michael Nazarov 2008-10-15 15:29:26 UTC
>> Yes we have automated tests for this and this issue found using them.
> Are you saying that, this was only found in 200810150101 and was working in all earlier builds?

No, this is not true. We have kind of RC1 today so I recheck tests more closer then usually.
Well, I think you agree this is not typical regression: file created but content is empty so there was no such check in
then corresponding test just because it is impossible to forecast all kinds of errors, so some of them can pass checking
for some time. But finally one new issue found. It's better then one not found.
Comment 8 Samaresh Panda 2008-10-15 18:45:16 UTC
I wonder if this was ever tested! In any case, I have a fix that will work. Stay tuned...
Comment 9 Michael Nazarov 2008-10-15 18:47:59 UTC
Depends on that do you mean by "this".
Comment 10 Samaresh Panda 2008-10-15 19:35:10 UTC
This may not be the best possible solution but this will do the job.
Fixed in trunk: http://hg.netbeans.org/main/rev/71f96b4c6841.
Comment 11 Samaresh Panda 2008-10-15 19:36:40 UTC
Michael, you could use the attached jar to test this. You do not have to wait for a build, but it is up to you.
Comment 12 Michael Nazarov 2008-10-15 19:39:03 UTC
I believe this jar will help, but my worry is about time this fix will come into RC.
Comment 13 Samaresh Panda 2008-10-15 19:39:08 UTC
Created attachment 71897 [details]
java2/modules/org-netbeans-modules-xml-tools-java.jar
Comment 14 Samaresh Panda 2008-10-15 19:42:22 UTC
You may want to add the keyword 65_HR_FIX. As soon as you verify, I'll send an approval email.
Comment 15 Michael Nazarov 2008-10-15 19:44:35 UTC
Okey, I see.
Comment 16 tonybeckham 2008-10-15 22:13:43 UTC
Confirmed in RC1...will test patch now.

Product Version: NetBeans IDE 6.5 RC1 (Build 200810151402)
Java: 1.6.0_07; Java HotSpot(TM) 64-Bit Server VM 1.6.0_07-b06-57
System: Mac OS X version 10.5.5 running on x86_64; MacRoman; en_US (nb)
Comment 17 tonybeckham 2008-10-15 22:17:58 UTC
Verified that the patch works (see screen shot)

I approve the fix.
Comment 18 tonybeckham 2008-10-15 22:18:41 UTC
Created attachment 71912 [details]
Fixed with new jar
Comment 19 tonybeckham 2008-10-15 22:22:07 UTC
The new jar file fixes the issue.
Comment 20 Quality Engineering 2008-10-16 04:17:47 UTC
Integrated into 'main-golden', will be available in build *200810160201* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/71f96b4c6841
User: samaresh@netbeans.org
Log: #150210 Unable to create DOM tree scaner
Comment 21 Petr Blaha 2008-10-16 08:08:59 UTC
Please, push the fix in release65 clone.
Comment 22 Samaresh Panda 2008-10-16 17:44:00 UTC
Sorry, forgot to update the issue. It has been done.
http://hg.netbeans.org/release65?cmd=changeset;node=f7fc281624ce.
Comment 23 tonybeckham 2008-10-17 19:06:35 UTC
Fix Verified

Product Version: NetBeans IDE 6.5 RC1 (Build 200810171318)
Java: 1.6.0_07; Java HotSpot(TM) 64-Bit Server VM 1.6.0_07-b06-57
System: Mac OS X version 10.5.5 running on x86_64; MacRoman; en_US (nb)