[api] Move notification backend calls

Moved the Backend::Connection call from UpdateNotificationEvents to Backend::Api
openSUSE · Aug 28, 2017 · 21263ce · 21263ce
1 parent 2dae904
commit 21263ce
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/api/app/models/update_notification_events.rb b/src/api/app/models/update_notification_events.rb
@@ -55,7 +55,7 @@ def perform
         nr = 1 if nr.zero?
 
         begin
-          @last = Xmlhash.parse(Backend::Connection.get("/lastnotifications?start=#{nr}&block=1").body)
+          @last = Xmlhash.parse(Backend::Api.last_notifications(nr))
         rescue Net::ReadTimeout, EOFError, ActiveXML::Transport::Error
           return
         end

diff --git a/src/api/lib/backend/api.rb b/src/api/lib/backend/api.rb
@@ -94,5 +94,10 @@ def self.merge_service(project, package, login)
     def self.run_service(project, package, login)
       Backend::Connection.post("/source/#{CGI.escape(project)}/#{CGI.escape(package)}?cmd=runservice&user=#{CGI.escape(login)}")
     end
+
+    # Returns the latest notifications specifying a starting point
+    def self.last_notifications(start)
+      Backend::Connection.get("/lastnotifications?start=#{CGI.escape(start.to_s)}&block=1").body
+    end
   end
 end