libs/web: make apply_xhr less aggressive, only rerequest status if previous response...
authorJo-Philipp Wich <jow@openwrt.org>
Wed, 17 Nov 2010 15:09:18 +0000 (15:09 +0000)
committerJo-Philipp Wich <jow@openwrt.org>
Wed, 17 Nov 2010 15:09:18 +0000 (15:09 +0000)
libs/web/luasrc/view/cbi/apply_xhr.htm

index 11851ad8100314b2e5e5e877b1522ee27b1a67af..d519a5f59c39300e71772ce9bad894d9ea34ac45 100644 (file)
@@ -20,35 +20,33 @@ $Id$
 
                apply_xhr.get('<%=luci.dispatcher.build_url("servicectl", "restart", table.concat(configs, ","))%>', null,
                        function() {
-                               var intv = window.setInterval(
-                                       function() {
-                                               apply_xhr.abort();
-                                               apply_xhr.get('<%=luci.dispatcher.build_url("servicectl", "status")%>', null,
-                                                       function(x) {
-                                                               if( x.responseText == 'finish' )
+                               var checkfinish = function() {
+                                       apply_xhr.get('<%=luci.dispatcher.build_url("servicectl", "status")%>', null,
+                                               function(x) {
+                                                       if( x.responseText == 'finish' )
+                                                       {
+                                                               var e = document.getElementById('cbi-apply-<%=id%>-status');
+                                                               if( e )
                                                                {
-                                                                       window.clearInterval(intv);
-
-                                                                       var e = document.getElementById('cbi-apply-<%=id%>-status');
-                                                                       if( e )
-                                                                       {
-                                                                               e.innerHTML = '<%:Configuration applied.%>';
-                                                                               window.setTimeout(function() {
-                                                                                       e.parentNode.style.display = 'none';
-                                                                                       <% if redirect then %>location.href='<%=redirect%>';<% end %>
-                                                                               }, 1000);
-                                                                       }
+                                                                       e.innerHTML = '<%:Configuration applied.%>';
+                                                                       window.setTimeout(function() {
+                                                                               e.parentNode.style.display = 'none';
+                                                                               <% if redirect then %>location.href='<%=redirect%>';<% end %>
+                                                                       }, 1000);
                                                                }
-                                                               else
-                                                               {
-                                                                       var e = document.getElementById('cbi-apply-<%=id%>-status');
-                                                                       if( e && x.responseText ) e.innerHTML = x.responseText;
+                                                       }
+                                                       else
+                                                       {
+                                                               var e = document.getElementById('cbi-apply-<%=id%>-status');
+                                                               if( e && x.responseText ) e.innerHTML = x.responseText;
 
-                                                               }
+                                                               window.setTimeout(checkfinish, 1000);
                                                        }
-                                               );
-                                       }, 1000
-                               )
+                                               }
+                                       );
+                               }
+
+                               window.setTimeout(checkfinish, 1000);
                        }
                );
        ]]></script>