Angular synchronous http loop to update progress bar

Question

I'm trying to update a progress bar with a sequential http request inside a foreach, this works, but it's not synchronous by on complete, progress bar is being sync'd by the http call, what am I doing wrong?

angular.forEach(queue, function (item) {
    if (item.uid) {
        d.item = item;
        $http({
            url: BASE_URL + 'upp', 
            method: 'post',
            data: d                  
        }).then(function(res){
            x++;
            update_progress((x/queue_count)*100);
        });
    }
});

I would like to call the update_progress function just when the http returns as finished (200 OK), so the progress bar shows the actual progress correctly. Thanks!

Edit:

I tried checking the response status, before calling the *update_progress* function and it still does not work as expected. I wasn't aware that 200 was dispatched before the request is completed :| By logic, the res obj shouldn't be the response of the http request? I mean, if it's 200 and not an error code, that shouldn't mean that the request was completed?

angular.forEach(queue, function (item) {
if (item.uid) {
    d.item = item;
    $http({
        url: BASE_URL + 'upp', 
        method: 'post',
        data: d                  
    }).then(function(res){
        if(res.status == 200) {
          x++;
          update_progress((x/queue_count)*100);
        }
    });
}

Reading more on promises atm to see if I can make it work as stated by @josh-strange

Edit 2:

So promises was the way to do it, all requests are sent sequentially so the progress bar works as expected, here's the code:

var promise = $q.all({}); 
// Had to init $q.all with an empty obj, 'cause null was trowing errors

angular.forEach(queue, function(item){
  if (item.uid) {        
    promise = promise.then(function(){
      d.item = item;
      return $http({
        url: BASE_URL + 'upp', 
        method: 'post',
        data: d 
      }).then(function(res){
        x++;
        update_progress((x/queue_count)*100);
      });
    });
  }
});

promise.then(function(){
  end_js_update();
});

Thanks @josh-strange

Don't use forEach, manually loop through queue? Not sure that will work either. Async can be such a pain sometimes. — aet, Sep 26 '13 at 20:10
You [cannot](http://stackoverflow.com/questions/13088153/how-to-http-synchronous-call-with-angularjs) make synchronous calls in AngularJS but you can make sequential http calls (Reg 1, then Reg 2, etc..). Let me try to get a Plunker together for you. — JoshStrange, Sep 26 '13 at 20:32
[Here](http://plnkr.co/edit/HMccomnJooWpIHFMO4H8) is an VERY simple example of sequential HTTP requests, I will post an answer in a minute with it cleaned up. — JoshStrange, Sep 26 '13 at 20:47
@josh-strange trying to implement the logic on my code. Gonna update & accept if works as expected. Never had to create a promise before, so I'm reading the docs. — Antonio Max, Sep 27 '13 at 18:32

JoshStrange · Accepted Answer · 2013-09-27T13:09:37.643

Here is a working Plunker of a working example of sequential http requests. You could obviously package this up very nicely in a service but for your purposes I just put a simple example of it in a controller.

Here is the "meat" of the code:

var app = angular.module('testApp',[]);

app.controller('myController', function($scope, $http, $q){

  $scope.responses = [];
  $scope.doneLoading = false;
  var urls = [
    'http://httpbin.org/ip',
    'http://httpbin.org/user-agent',
    'http://httpbin.org/headers'
  ];

  var promise = $q.all(null);

  angular.forEach(urls, function(url){
    promise = promise.then(function(){
      return $http({
        method: 'GET', 
        url:url 
      }).then(function(res){
        $scope.responses.push(res.data);
      });
    });
  });

  promise.then(function(){
    //This is run after all of your HTTP requests are done
    $scope.doneLoading = true;
  })

});

EDIT: As Mike P says below: This is an example of chaining promises see $q documentation.

Just for completeness, this is an example of chaining promises together (see $q documentation @ http://docs.angularjs.org/api/ng.$q). — Mike Pugh, Sep 27 '13 at 02:11

score 0 · Answer 2 · answered Sep 26 '13 at 20:34

0

You could try something like this. Just wait to send the next request until the previous is finished?

var sendRequest = function(index) {
  if (queue[index].uid) {
    d.item = queue[index];
    $http({
      url: BASE_URL + 'upp', 
      method: 'post',
      data: d                  
    }).then(function(res){
      if (index != queue.length - 1) {
        x++;
        update_progress((x / queue_count) * 100);
        sendRequest(++index);
      }
    });
  }
}

answered Sep 26 '13 at 20:34

Zack Argyle

7,447
4
27
36

thank you, but @josh-strange answer was better because it uses promises, which I believe now is the right way to do it! – Antonio Max Sep 27 '13 at 20:38

Angular synchronous http loop to update progress bar

2 Answers2

Linked