Author: j16sdiz
Date: 2008-12-29 01:18:24 +0000 (Mon, 29 Dec 2008)
New Revision: 24810

Modified:
   trunk/plugins/XMLSpider/XMLSpider.java
Log:
mark as succeededonly if really succeeded

Modified: trunk/plugins/XMLSpider/XMLSpider.java
===================================================================
--- trunk/plugins/XMLSpider/XMLSpider.java      2008-12-29 01:18:09 UTC (rev 
24809)
+++ trunk/plugins/XMLSpider/XMLSpider.java      2008-12-29 01:18:24 UTC (rev 
24810)
@@ -421,7 +421,6 @@
                }
 
                FreenetURI uri = state.getURI();
-               page.status = Status.SUCCEEDED; // Content filter may throw, 
but we mark it as success anyway
 
                try {
                                // Page may be refetched if added manually
@@ -463,14 +462,18 @@
                                } finally {
                                        data.free();
                                }
-               } finally {
+                               
                        synchronized (this) {
-                               runningFetch.remove(page);
+                               page.status = Status.SUCCEEDED;
                                page.lastChange = System.currentTimeMillis();
                                db.store(page);
+                               db.commit();
                        }
+               } finally {
+                       synchronized (this) {
+                               runningFetch.remove(page);
+                       }
                        if (!stopped) {
-                               db.commit();
                                startSomeRequests();
                        } else {
                                db.rollback();

_______________________________________________
cvs mailing list
[email protected]
http://emu.freenetproject.org/cgi-bin/mailman/listinfo/cvs

Reply via email to