Bug 138846 - Offer XML MIME resolvers with just o-o-filesystems.jar in CP
Offer XML MIME resolvers with just o-o-filesystems.jar in CP
Status: RESOLVED FIXED
Product: platform
Classification: Unclassified
Component: Filesystems
6.x
All All
: P3 (vote)
: 6.x
Assigned To: Jesse Glick
issues@platform
: API, API_REVIEW_FAST
Depends on: 139253
Blocks:
  Show dependency treegraph
 
Reported: 2008-07-02 17:59 UTC by Jesse Glick
Modified: 2008-12-22 12:14 UTC (History)
4 users (show)

See Also:
Issue Type: DEFECT
:


Attachments
Proposed patch (still needs testing) (37.14 KB, patch)
2008-07-02 19:23 UTC, Jesse Glick
Details | Diff
Y01 and Y03 addressed; also fixing FileAssociationsModel's expectation that Lookup<MIMEResolver> will be useful here (40.41 KB, patch)
2008-07-08 23:16 UTC, Jesse Glick
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jesse Glick 2008-07-02 17:59:45 UTC
We have been talking recently about making our APIs more test-friendly. One sore point is that the Filesystems API
documents that XML MIME resolvers are available, and even supplies their DTD, yet does not actually make them work if
you add them to your layer. (I don't think this is analogous to other API/SPI situations where there is a documented
pluggable SPI implemented in some other module and the API guarantees nothing. The API here offers an SPI - MIMEResolver
- yet also declares that a given XML format may be used instead.) Moving the resolver impl to Filesystems would solve
the issue of writing unit tests involving XML subtypes (cf. issue #138173).
Comment 1 Jesse Glick 2008-07-02 19:23:25 UTC
Created attachment 63840 [details]
Proposed patch (still needs testing)
Comment 2 Jesse Glick 2008-07-02 19:25:05 UTC
Please review this proposed change.
Comment 3 Jesse Glick 2008-07-02 19:34:25 UTC
Note changes from current behavior (theoretically incompatible):

1. The old impl would have interpreted a resolver (with the correct doctype) in any location beneath Services. (Actually
a resolver would offer an InstanceCookie<MIMEResolver> in any location, but outside Services they would not have been
used in MIME resolution.) The new impl expects resolvers to be Services/MIMEResolver/*.xml, as in practice they always
are, and does not bother to verify that the public ID is set to the expected value in the doctype.

2. The old impl would have inserted the declarative resolvers in default lookup (assuming you were running with the
module system started and thus FolderLookup[Services included). The new impl does not.

The current patch does not attempt to reuse the parse of a FileObject between refreshes of the resolver list cache,
though this should be simple enough to add. It should also listen to changes in the resolver folder (new files, etc.)
and refresh the cache.
Comment 4 Jaroslav Tulach 2008-07-07 14:20:55 UTC
Y01 Do not use org/openide/filesystems/declmime package, use org.nb.modules.openide.fs.declmime
Y02 This is not really compatible change. If you want to sacrify compatibility, please give something important back. 
For example, speed: change the code to create only one MimeResolverImpl and make the decision process in the impl 
faster.
Y03 Eliminate public static ArrayList<String[]> getExtensionsAndMIMETypes(FileObject fo) somehow, please (core.xyz has 
impl dependency on openide.fs and could access such method?)
Y04 Fix apisupport "File Type" template to use this
Comment 5 Jesse Glick 2008-07-08 14:44:12 UTC
To Y04 - just means no unit test template is necessary any more; see issue #139253.
Comment 6 Jesse Glick 2008-07-08 21:47:30 UTC
Y01 - can do.


Y02 - the advantage is clear: better testability. If I see any opportunity for easy optimization I will take it, but I
doubt it will have any real benefit, since we already order resolvers according to expected frequency of matches.


Y03 - no one has an impl dep on filesystems currently. The new method is needed from core.ui (for the new Options dialog
UI). Could perhaps duplicate a sufficient portion of the parser in core.ui to extract this information.
Comment 7 Jesse Glick 2008-07-08 23:16:27 UTC
Created attachment 64115 [details]
Y01 and Y03 addressed; also fixing FileAssociationsModel's expectation that Lookup<MIMEResolver> will be useful here
Comment 8 Jesse Glick 2008-07-11 05:35:28 UTC
Now reusing parses, and some other impl changes. core-main #d65ff2111b86
Comment 9 Quality Engineering 2008-07-17 04:34:43 UTC
Integrated into 'main-golden', available in NB_Trunk_Production #324 build
Changeset: http://hg.netbeans.org/main/rev/d65ff2111b86
User: Jesse Glick <jglick@netbeans.org>
Log: Issue #138846: move the implementation of declarative MIME resolvers to the Filesystems API.


By use of this website, you agree to the NetBeans Policies and Terms of Use. © 2012, Oracle Corporation and/or its affiliates. Sponsored by Oracle logo