rss MDL-25319 replicating my optimization made to forum rss in glossa…

…ry rss and database rss
anovikov · Nov 19, 2010 · c02f338 · c02f338
1 parent e3c21e8
commit c02f338
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/mod/data/rsslib.php b/mod/data/rsslib.php
@@ -34,8 +34,9 @@ function data_rss_get_feed($context, $args) {
         if (file_exists($cachedfilepath)) {
             $cachedfilelastmodified = filemtime($cachedfilepath);
         }
-
-        if (data_rss_newstuff($data, $cachedfilelastmodified)) {
+        //if the cache is more than 60 seconds old and there's new stuff
+        $dontrecheckcutoff = time()-60;
+        if ( $dontrecheckcutoff > $cachedfilelastmodified && data_rss_newstuff($data, $cachedfilelastmodified)) {
             require_once($CFG->dirroot . '/mod/data/lib.php');
 
             // Get the first field in the list  (a hack for now until we have a selector)

diff --git a/mod/glossary/rsslib.php b/mod/glossary/rsslib.php
@@ -37,8 +37,9 @@ function glossary_rss_get_feed($context, $args) {
         if (file_exists($cachedfilepath)) {
             $cachedfilelastmodified = filemtime($cachedfilepath);
         }
-
-        if (glossary_rss_newstuff($glossary, $cachedfilelastmodified)) {
+        //if the cache is more than 60 seconds old and there's new stuff
+        $dontrecheckcutoff = time()-60;
+        if ( $dontrecheckcutoff > $cachedfilelastmodified && glossary_rss_newstuff($glossary, $cachedfilelastmodified)) {
             if (!$recs = $DB->get_records_sql($sql, array(), 0, $glossary->rssarticles)) {
                 return null;
             }