Author: psharples
Date: Fri Jan 13 14:42:11 2012
New Revision: 1231097
URL: http://svn.apache.org/viewvc?rev=1231097&view=rev
Log:
Fix for where the parser kept the original wgt file (found in uploads) open &
so file locked. See WOOKIE-302.
Modified:
incubator/wookie/trunk/parser/java/src/org/apache/wookie/w3c/W3CWidgetFactory.java
Modified:
incubator/wookie/trunk/parser/java/src/org/apache/wookie/w3c/W3CWidgetFactory.java
URL:
http://svn.apache.org/viewvc/incubator/wookie/trunk/parser/java/src/org/apache/wookie/w3c/W3CWidgetFactory.java?rev=1231097&r1=1231096&r2=1231097&view=diff
==============================================================================
---
incubator/wookie/trunk/parser/java/src/org/apache/wookie/w3c/W3CWidgetFactory.java
(original)
+++
incubator/wookie/trunk/parser/java/src/org/apache/wookie/w3c/W3CWidgetFactory.java
Fri Jan 13 14:42:11 2012
@@ -257,7 +257,7 @@ public class W3CWidgetFactory {
* @throws BadManifestException
*/
private W3CWidget processWidgetPackage(File zipFile, String
defaultIdentifier) throws BadWidgetZipFileException, BadManifestException{
- ZipFile zip;
+ ZipFile zip;
try {
zip = new ZipFile(zipFile);
} catch (IOException e) {
@@ -302,7 +302,14 @@ public class W3CWidgetFactory {
throw e;
} catch (Exception e){
throw new BadManifestException(e);
+ } finally {
+ try {
+ zip.close();
+ } catch (IOException e) {
+ _logger.error("Unable to close wgt file:" + e.getMessage());
+ }
}
+
}
else{
// no manifest file found in zip archive
