Remove usages of .done/.fail/.always and use promises A/A+ instead (.then/.catch)
Noticed when signing off T170812: Return promises from QUnit tests
Remove usages of .done/.fail/.always and use promises A/A+ instead (.then/.catch)
Noticed when signing off T170812: Return promises from QUnit tests
Change 373054 had a related patch set uploaded (by Jhernandez; owner: Jhernandez):
[mediawiki/extensions/Popups@master] Hygiene: Use promises A/A everywhere
Change 373054 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] Hygiene: Use promises A/A everywhere
@Jdlrobson I saw that but .fail is not compatible with .catch straight forward. Specially when chaining promises like on this case where after that there is a $.when with to promises.
We need to be sure the semantics of the change are the same when we migrate that line to .catch.
Change 373327 had a related patch set uploaded (by Jhernandez; owner: Jhernandez):
[mediawiki/extensions/Popups@master] Hygiene: Don't rely on .fail, use Promises/A
Change 373327 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] Hygiene: Don't rely on .fail, use Promises/A