jQuery Ajax, async : false, return undefined

Question

I have this function:

GetMessage: function (code) {
            var msg = localStorage.getItem(code);
            if (msg == null || msg == 'undefined') {
                $.ajax({
                    type: 'GET',
                    url: CTHelper.ApiUrl() + '/Api/Case/GetMessage?code=' + code,
                    async : false,
                    success: function (result) {
                        localStorage.setItem(code, result);
                        return result;
                    },
                    error: function (result) {
                        return '';
                    }
                });
            }
            else {
                return msg;
            }
        },

The first time it is invoked, it goes to the server ($.ajax) and stores a value in the localStorage, so the second and so on invocations get the value from the cache/storage.

Now, it works well all the times but the first invocation, it seems like return result in the success function returns always undefined.

I thought setting the async : false would fix the issue but it didn't

Below an image that illustrates my problem

See this: stackoverflow.com/a/2592780/1623249

Maria Ines Parnisari
– Maria Ines Parnisari

2017-09-28 23:50:59 +00:00
Commented Sep 28, 2017 at 23:50 — Maria Ines Parnisari
– Maria Ines Parnisari, Commented Sep 28, 2017 at 23:50

Victor Hugo Terceros · Accepted Answer · 2017-09-29 02:06:51Z

4

The return statements inside the 2 callback functions success and error don't return your GetMessage function - they return from the callbacks. If you want your function to return the correct message, assign the result to msg and always return it at the end of GetMessage:

GetMessage: function(code) {
  var msg = localStorage.getItem(code);
  if (msg == null || msg == 'undefined') {
    $.ajax({
      type: 'GET',
      url: CTHelper.ApiUrl() + '/Api/Case/GetMessage?code=' + code,
      async: false,
      success: function(result) {
        localStorage.setItem(code, result);
        msg = result;
      },
      error: function(result) {
        msg = '';
      }
    });
  }

  return msg;
},

See the result below

edited Sep 29, 2017 at 2:06

Victor Hugo Terceros

3,1893 gold badges21 silver badges33 bronze badges

answered Sep 28, 2017 at 23:53

AVAVT

7,1732 gold badges25 silver badges45 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Victor Hugo Terceros Over a year ago

Excelent answer, very helpful +1

Collectives™ on Stack Overflow

jQuery Ajax, async : false, return undefined

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related