Fix bug 1186547 - mReadOnly and mDisabled do not reflect readOnly and disabled properties - missing return. r=MakeMyDay,a=philipp
authorPhilipp Kewisch <mozilla@kewis.ch>
Wed, 22 Jul 2015 21:21:25 +0200
changeset 22663 3a6adf92dcac5b028aa1e013c8a7751aceaad6cf
parent 22662 18d4427d42034f71b24539482ae0b750aab47f62
child 22664 25d952d59bda146f5d6311edf6718b932799c1bd
child 22666 fc9b574f3f850bf17de602cfd21414fe30444157
push id1435
push usermozilla@kewis.ch
push dateWed, 22 Jul 2015 19:22:11 +0000
treeherdercomm-beta@3a6adf92dcac [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersMakeMyDay, philipp
bugs1186547
Fix bug 1186547 - mReadOnly and mDisabled do not reflect readOnly and disabled properties - missing return. r=MakeMyDay,a=philipp
calendar/providers/caldav/calDavCalendar.js
--- a/calendar/providers/caldav/calDavCalendar.js
+++ b/calendar/providers/caldav/calDavCalendar.js
@@ -1874,16 +1874,17 @@ calDavCalendar.prototype = {
             if (resourceType == kDavResourceTypeCalendar) {
                 // If this calendar was previously offline we want to recover
                 if (thisCalendar.getProperty("disabled")) {
                     thisCalendar.setProperty("disabled", false);
                     thisCalendar.readOnly = false;
                 }
                 thisCalendar.setCalHomeSet(true);
                 thisCalendar.checkServerCaps(aChangeLogListener);
+                return;
             }
 
             // If we get here something must have gone wrong. Abort with a
             // general error to avoid an endless loop.
             thisCalendar.completeCheckServerInfo(aChangeLogListener, Components.results.NS_ERROR_FAILURE);
         };
 
         this.sendHttpRequest(this.makeUri(), queryXml, MIME_TEXT_XML, null, (channel) => {