Issue 427 - Remove non-standard function and getter syntax

lib/subscriptionClasses.js

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2014 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 50 matching lines @@
   filters: null,
 
   _title: null,
   _fixedTitle: false,
   _disabled: false,
 
   /**
    * Title of the filter subscription
    * @type String
    */
-  get title() { return this._title; },

[Wladimir Palant, 2014/05/15 07:12:38]

Nit: Please add proper line breaks here.

+  get title()
+  {
+    return this._title;
+  },
   set title(value)
   {
     if (value != this._title)
     {
       let oldValue = this._title;
       this._title = value;
       FilterNotifier.triggerListeners("subscription.title", this, value, oldValue);
     }
     return this._title;
   },
 
   /**
    * Determines whether the title should be editable
    * @type Boolean
    */
-  get fixedTitle() { return this._fixedTitle; },

[Wladimir Palant, 2014/05/15 07:12:38]

Nit: Please add proper line breaks here.

+  get fixedTitle()
+  {
+    return this._fixedTitle;
+  },
   set fixedTitle(value)
   {
     if (value != this._fixedTitle)
     {
       let oldValue = this._fixedTitle;
       this._fixedTitle = value;
       FilterNotifier.triggerListeners("subscription.fixedTitle", this, value, oldValue);
     }
     return this._fixedTitle;
   },
 
   /**
    * Defines whether the filters in the subscription should be disabled
    * @type Boolean
    */
-  get disabled() { return this._disabled; },

[Wladimir Palant, 2014/05/15 07:12:38]

Nit: Please add proper line breaks here.

+  get disabled()
+  {
+    return this._disabled;
+  },
   set disabled(value)
   {
     if (value != this._disabled)
     {
       let oldValue = this._disabled;
       this._disabled = value;
       FilterNotifier.triggerListeners("subscription.disabled", this, value, oldValue);
     }
     return this._disabled;
   },
@@ skipping 245 matching lines @@
 {
   __proto__: Subscription.prototype,
 
   _homepage: null,
   _lastDownload: 0,
 
   /**
    * Filter subscription homepage if known
    * @type String
    */
-  get homepage() { return this._homepage; },

[Wladimir Palant, 2014/05/15 07:12:38]

Nit: Please add proper line breaks here.

+  get homepage()
+  {
+    return this._homepage;
+  },
   set homepage(value)
   {
     if (value != this._homepage)
     {
       let oldValue = this._homepage;
       this._homepage = value;
       FilterNotifier.triggerListeners("subscription.homepage", this, value, oldValue);
     }
     return this._homepage;
   },
 
   /**
    * Time of the last subscription download (in seconds since the beginning of the epoch)
    * @type Number
    */
-  get lastDownload() { return this._lastDownload; },

[Wladimir Palant, 2014/05/15 07:12:38]

Nit: Please add proper line breaks here.

+  get lastDownload()
+  {
+    return this._lastDownload;
+  },
   set lastDownload(value)
   {
     if (value != this._lastDownload)
     {
       let oldValue = this._lastDownload;
       this._lastDownload = value;
       FilterNotifier.triggerListeners("subscription.lastDownload", this, value, oldValue);
     }
     return this._lastDownload;
   },
@@ skipping 55 matching lines @@
   __proto__: RegularSubscription.prototype,
 
   _downloadStatus: null,
   _lastCheck: 0,
   _errors: 0,
 
   /**
    * Status of the last download (ID of a string)
    * @type String
    */
-  get downloadStatus() { return this._downloadStatus; },

[Wladimir Palant, 2014/05/15 07:12:38]

Nit: Please add proper line breaks here.

+  get downloadStatus()
+  {
+    return this._downloadStatus;
+  },
   set downloadStatus(value)
   {
     let oldValue = this._downloadStatus;
     this._downloadStatus = value;
     FilterNotifier.triggerListeners("subscription.downloadStatus", this, value, oldValue);
     return this._downloadStatus;
   },
 
   /**
    * Time of the last successful download (in seconds since the beginning of the
    * epoch).
    */
   lastSuccess: 0,
 
   /**
    * Time when the subscription was considered for an update last time (in seconds
    * since the beginning of the epoch). This will be used to increase softExpiration
    * if the user doesn't use Adblock Plus for some time.
    * @type Number
    */
-  get lastCheck() { return this._lastCheck; },

[Wladimir Palant, 2014/05/15 07:12:38]

Nit: Please add proper line breaks here.

+  get lastCheck()
+  {
+    return this._lastCheck;
+  },
   set lastCheck(value)
   {
     if (value != this._lastCheck)
     {
       let oldValue = this._lastCheck;
       this._lastCheck = value;
       FilterNotifier.triggerListeners("subscription.lastCheck", this, value, oldValue);
     }
     return this._lastCheck;
   },
 
   /**
    * Hard expiration time of the filter subscription (in seconds since the beginning of the epoch)
    * @type Number
    */
   expires: 0,
 
   /**
    * Soft expiration time of the filter subscription (in seconds since the beginning of the epoch)
    * @type Number
    */
   softExpiration: 0,
 
   /**
    * Number of download failures since last success
    * @type Number
    */
-  get errors() { return this._errors; },

[Wladimir Palant, 2014/05/15 07:12:38]

Nit: Please add proper line breaks here.

+  get errors()
+  {
+    return this._errors;
+  },
   set errors(value)
   {
     if (value != this._errors)
     {
       let oldValue = this._errors;
       this._errors = value;
       FilterNotifier.triggerListeners("subscription.errors", this, value, oldValue);
     }
     return this._errors;
   },
@@ skipping 33 matching lines @@
     if (this.softExpiration)
       buffer.push("softExpiration=" + this.softExpiration);
     if (this.errors)
       buffer.push("errors=" + this.errors);
     if (this.version)
       buffer.push("version=" + this.version);
     if (this.requiredVersion)
       buffer.push("requiredVersion=" + this.requiredVersion);
   }
 };