]> git.ipfire.org Git - thirdparty/moment.git/commitdiff
Use explicit .valueOf() calls instead of coercion
authorDamian Senn <damian.senn@adfinis-sygroup.ch>
Fri, 8 Jan 2016 12:20:22 +0000 (13:20 +0100)
committerIskren Chernev <iskren.chernev@gmail.com>
Sat, 16 Apr 2016 05:54:40 +0000 (22:54 -0700)
src/lib/create/from-anything.js
src/lib/moment/add-subtract.js
src/lib/moment/compare.js
src/lib/moment/to-type.js
src/lib/units/offset.js

index 7b92587ec7746565f86dbe4c46ab40dc83288ba7..022c6eb172183b1eab2e974050efa6f3cf76de09 100644 (file)
@@ -63,7 +63,7 @@ function configFromInput(config) {
     if (input === undefined) {
         config._d = new Date(hooks.now());
     } else if (isDate(input)) {
-        config._d = new Date(+input);
+        config._d = new Date(input.valueOf());
     } else if (typeof input === 'string') {
         configFromString(config);
     } else if (isArray(input)) {
index dd33333236e7546776173e98b581ff5dd2dd8dbb..588153bbee7295bfc3a94959753590177a008f9c 100644 (file)
@@ -36,7 +36,7 @@ export function addSubtract (mom, duration, isAdding, updateOffset) {
     updateOffset = updateOffset == null ? true : updateOffset;
 
     if (milliseconds) {
-        mom._d.setTime(+mom._d + milliseconds * isAdding);
+        mom._d.setTime(mom._d.valueOf() + milliseconds * isAdding);
     }
     if (days) {
         set(mom, 'Date', get(mom, 'Date') + days * isAdding);
index d196a9806610518f35a0acc11f78c0c18b1afb4a..2e3da51efa4dfa33a6910cc71bde9cd536606bf4 100644 (file)
@@ -10,9 +10,9 @@ export function isAfter (input, units) {
     }
     units = normalizeUnits(!isUndefined(units) ? units : 'millisecond');
     if (units === 'millisecond') {
-        return +this > +localInput;
+        return this.valueOf() > localInput.valueOf();
     } else {
-        return +localInput < +this.clone().startOf(units);
+        return localInput.valueOf() < this.clone().startOf(units).valueOf();
     }
 }
 
@@ -23,9 +23,9 @@ export function isBefore (input, units) {
     }
     units = normalizeUnits(!isUndefined(units) ? units : 'millisecond');
     if (units === 'millisecond') {
-        return +this < +localInput;
+        return this.valueOf() < localInput.valueOf();
     } else {
-        return +this.clone().endOf(units) < +localInput;
+        return this.clone().endOf(units).valueOf() < localInput.valueOf();
     }
 }
 
@@ -41,10 +41,10 @@ export function isSame (input, units) {
     }
     units = normalizeUnits(units || 'millisecond');
     if (units === 'millisecond') {
-        return +this === +localInput;
+        return this.valueOf() === localInput.valueOf();
     } else {
-        inputMs = +localInput;
-        return +(this.clone().startOf(units)) <= inputMs && inputMs <= +(this.clone().endOf(units));
+        inputMs = localInput.valueOf();
+        return this.clone().startOf(units).valueOf() <= inputMs && inputMs <= this.clone().endOf(units).valueOf();
     }
 }
 
index 0fba9990772a93284c593e477671298b29a79eb9..584348da9841730a5e73f152bbbaf46be86787e1 100644 (file)
@@ -1,13 +1,13 @@
 export function valueOf () {
-    return +this._d - ((this._offset || 0) * 60000);
+    return this._d.valueOf() - ((this._offset || 0) * 60000);
 }
 
 export function unix () {
-    return Math.floor(+this / 1000);
+    return Math.floor(this.valueOf() / 1000);
 }
 
 export function toDate () {
-    return this._offset ? new Date(+this) : this._d;
+    return this._offset ? new Date(this.valueOf()) : this._d;
 }
 
 export function toArray () {
index 8723c87b858ccee6d607f5b6c8a963dc9f01ff6c..22305d17eb536f6f006920d0dce0ee11901e5af0 100644 (file)
@@ -61,9 +61,9 @@ export function cloneWithOffset(input, model) {
     var res, diff;
     if (model._isUTC) {
         res = model.clone();
-        diff = (isMoment(input) || isDate(input) ? +input : +createLocal(input)) - (+res);
+        diff = (isMoment(input) || isDate(input) ? input.valueOf() : createLocal(input).valueOf()) - res.valueOf();
         // Use low-level api, because this fn is low-level api.
-        res._d.setTime(+res._d + diff);
+        res._d.setTime(res._d.valueOf() + diff);
         hooks.updateOffset(res, false);
         return res;
     } else {