[CLOSED] Replace .isResolved & .isRejected with .state

[core-ai-bot]

Issue by TuckerWhitehouse
Monday Apr 29, 2013 at 19:55 GMT
Originally opened as adobe/brackets#3665

---

This is a step towards the eventually upgrade to jQuery 2.0 (#3123).

Both .isResolved & .isRejected were deprecated in jQuery 1.7 and removed in 1.8 (http://api.jquery.com/deferred.isResolved/).

---

TuckerWhitehouse included the following code: https://github.com/adobe/brackets/pull/3665/commits

[core-ai-bot]

Comment by TuckerWhitehouse
Monday Apr 29, 2013 at 20:01 GMT

---

Also, should the wiki/Brackets-Coding-Conventions be updated to advise people to use .state instead of .isResolved or .isRejected?

[core-ai-bot]

Comment by jasonsanjose
Tuesday Apr 30, 2013 at 21:05 GMT

---

Reviewing

[core-ai-bot]

Comment by jasonsanjose
Tuesday Apr 30, 2013 at 21:47 GMT

---

@TuckerWhitehouse I don't think we necessarily need a note on the wiki to avoid deprecated or soon-to-be-deprecated APIs. Your changes look good and passes tests. Merging.