You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by sn...@apache.org on 2007/09/12 17:32:19 UTC

svn commit: r574995 - in /roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business: hibernate/HibernateWeblogManagerImpl.java jpa/JPAWeblogManagerImpl.java

Author: snoopdave
Date: Wed Sep 12 08:32:18 2007
New Revision: 574995

URL: http://svn.apache.org/viewvc?rev=574995&view=rev
Log:
Fix for http://opensource.atlassian.com/projects/roller/browse/ROL-1540
"Deleted Entries are still visible on the Weblog to non - authenticated users (not logged in state)."

We weren't updating the weblog's last-modified time on remove-entry, so the cache still thought the page was still up to date.

Modified:
    roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/hibernate/HibernateWeblogManagerImpl.java
    roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/jpa/JPAWeblogManagerImpl.java

Modified: roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/hibernate/HibernateWeblogManagerImpl.java
URL: http://svn.apache.org/viewvc/roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/hibernate/HibernateWeblogManagerImpl.java?rev=574995&r1=574994&r2=574995&view=diff
==============================================================================
--- roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/hibernate/HibernateWeblogManagerImpl.java (original)
+++ roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/hibernate/HibernateWeblogManagerImpl.java Wed Sep 12 08:32:18 2007
@@ -297,9 +297,7 @@
         this.strategy.remove(entry);
         
         // update weblog last modified date.  date updated by saveWebsite()
-        if(entry.isPublished()) {
-            roller.getUserManager().saveWebsite(entry.getWebsite());
-        }
+        roller.getUserManager().saveWebsite(entry.getWebsite());
         
         // remove entry from cache mapping
         this.entryAnchorToIdMap.remove(entry.getWebsite().getHandle()+":"+entry.getAnchor());

Modified: roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/jpa/JPAWeblogManagerImpl.java
URL: http://svn.apache.org/viewvc/roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/jpa/JPAWeblogManagerImpl.java?rev=574995&r1=574994&r2=574995&view=diff
==============================================================================
--- roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/jpa/JPAWeblogManagerImpl.java (original)
+++ roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/jpa/JPAWeblogManagerImpl.java Wed Sep 12 08:32:18 2007
@@ -378,10 +378,8 @@
         // remove entry
         this.strategy.remove(entry);
         
-        // update weblog last modified date.  date updated by saveWebsite()
-        if (entry.isPublished()) {
-            roller.getUserManager().saveWebsite(website);
-        }
+        // update weblog last modified date. date updated by saveWebsite()
+        roller.getUserManager().saveWebsite(website);
         
         // remove entry from cache mapping
         this.entryAnchorToIdMap.remove(website.getHandle() + ":" + entryAnchor);