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.
Summary: | XDM access problem when running Ant task inside NetBeans | ||
---|---|---|---|
Product: | xml | Reporter: | Jun Qian <jqian> |
Component: | XDM | Assignee: | issues@xml <issues> |
Status: | VERIFIED FIXED | ||
Severity: | blocker | ||
Priority: | P1 | ||
Version: | 5.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Attachments: |
patch for hybrid mode
Patch to allow overriding of default access provider |
Description
Jun Qian
2006-10-06 00:37:03 UTC
Created attachment 34998 [details]
patch for hybrid mode
This problem was caused by a design change to the classpath for the ANT task. Previously, the classpath was isolated from the IDE's classpath and the XDM service was not available so the models performed as they would outside the IDE. There was a recent change to resolve a memory leak, where the IDE classpath is not available and thus the XDMModelAccessProvider is on the classpath. The suggested fix is to allow a client to override the default DocumentAccessProvider via the ModelSource lookup. A client which requires read-only access could add the following to the lookup to override the default Model Access Support: org.netbeans.modules.xml.xam.dom.ReadOnlyAccess.Provider.getInstance() The fix to XAM is to look at the current ModelSource's lookup before using the default implementation. Created attachment 35004 [details]
Patch to allow overriding of default access provider
fixed in release55_dev Good change for added flexibility for model source to tell which model access provider to use. Code looks goods, change would have no effects on existing client code and existing behaviour. fixed in release55. jqian: This issue submitted by you is marked as resolved. Since it relates to some source code specifics, QA is unable to verify it. Could you please verify this issue and mark it as verified or reopen it? Verified. |