Patch to provide new ClassPathResource in jetty with JUnits

Greg Wilkins added a comment - 12/Dec/06 10:59 AM Matthew, I think I have implemented this without the new class. I have simply added more static methods on Resource in the same way as we have getSystemResource (which is pretty much the same). Is this good enough? checked into head. /* ------------------------------------------------------------ */ /** Find a classpath resource. */ public static Resource newClassPathResource(String resource) { return newClassPathResource(resource,true,false); } /* ------------------------------------------------------------ */ /** Find a classpath resource. The {@java.lang.Class#getResource} method is used to lookup the resource. If it is not found, then the {@link Loader#getResource(Class, String, boolean)} method is used. If it is still not found, then {@link ClassLoader#getSystemResource(String)} is used. Unlike {@link #getSystemResource} this method does not check for normal resources. @param name The relative name of the resouce @param useCaches True if URL caches are to be used. @param checkParents True if forced searching of parent classloaders is performed to work around loaders with inverted priorities @return Resource or null */ public static Resource newClassPathResource(String name,boolean useCaches,boolean checkParents) { URL url=Resource.class.getResource(name); if (url==null) { try { url=Loader.getResource(Resource.class,name,checkParents); } catch(ClassNotFoundException e) { url=ClassLoader.getSystemResource(name); } } if (url==null) return null; return newResource(url,useCaches); }

Matthew Purland added a comment - 12/Dec/06 11:16 AM Yes, this will work. Thanks. Also, have you adapted the JUnits that I have provided to work with this new static method?

Greg Wilkins added a comment - 12/Dec/06 4:45 PM Yes - I have included to updated tests in the build. Thanks