Title: [opsview] [12209] Fixed number of items in RSS feed, off by one
Revision
12209
Author
tvoon
Date
2013-05-01 14:17:15 +0100 (Wed, 01 May 2013)

Log Message

Fixed number of items in RSS feed, off by one

Modified Paths


Modified: trunk/CHANGES
===================================================================
--- trunk/CHANGES	2013-05-01 13:16:09 UTC (rev 12208)
+++ trunk/CHANGES	2013-05-01 13:17:15 UTC (rev 12209)
@@ -33,6 +33,7 @@
     Fixed SNMP polling of device interfaces with no ifDescr set
     Fixed paths to images in events view and dashboard splash page for a rehomed web app
     Fixed adding custom contact variables from notification methods into environment variables for notification scripts to use
+    Fixed notify_by_rss's maximum number of items, which was off by one
 
 3.20130304
     ENHANCEMENTS:

Modified: trunk/opsview-core/notifications/notify_by_rss
===================================================================
--- trunk/opsview-core/notifications/notify_by_rss	2013-05-01 13:16:09 UTC (rev 12208)
+++ trunk/opsview-core/notifications/notify_by_rss	2013-05-01 13:17:15 UTC (rev 12209)
@@ -153,9 +153,6 @@
     $output .= "<br /><br />Acknowledged by: $author<br />Comment: $ackcomment";
 }
 
-# remove more than max entries from the head of the array
-@$entries = splice @$entries, 0, $atom_max_items;
-
 my $found = 0;
 if ($atom_collapsed) {
 
@@ -240,6 +237,9 @@
     }
 }
 
+# remove more than max entries from the head of the array
+@$entries = splice @$entries, 0, $atom_max_items;
+
 open( STORE, ">", $storefile );
 flock( STORE, LOCK_EX );
 store_fd( $entries, *STORE );

_______________________________________________
Opsview-checkins mailing list
Opsview-checkins@lists.opsview.org
http://lists.opsview.org/lists/listinfo/opsview-checkins

Reply via email to