mobservable - npm Package Compare versions

Comparing version 1.2.4 to 1.2.5

CHANGELOG.md

@@ -0,1 +1,6 @@
+# 1.2.5
+
+* Map no longer throws when `.has`, `.get` or `.delete` is invoked with an invalid key (#116)
+* Files are now compiled without sourcemap to avoid issues when loading mobservable in a debugger when `src/` folder is not available.
+
 # 1.2.4
@@ -2,0 +7,0 @@

lib/core.js

@@ -498,2 +498,1 @@ var dnode_1 = require('./dnode');
 exports.observe = observe;
-//# sourceMappingURL=core.js.map

lib/dnode.js

@@ -273,2 +273,1 @@ var __extends = (this && this.__extends) || function (d, b) {
 var simpleeventemitter_1 = require('./simpleeventemitter');
-//# sourceMappingURL=dnode.js.map

lib/extras.js

@@ -113,2 +113,1 @@ var dnode_1 = require('./dnode');
 exports.trackTransitions = trackTransitions;
-//# sourceMappingURL=extras.js.map

lib/index.js

@@ -52,2 +52,1 @@ function __export(m) {
 };
-//# sourceMappingURL=index.js.map

lib/observablearray.js

@@ -335,2 +335,1 @@ var __extends = (this && this.__extends) || function (d, b) {
 reserveArrayBuffer(1000);
-//# sourceMappingURL=observablearray.js.map

lib/observablemap.d.ts

@@ -30,2 +30,3 @@ import { ObservableValue } from './observablevalue';
     toJs(): KeyValueMap<V>;
+    private isValidKey(key);
     private assertValidKey(key);
@@ -32,0 +33,0 @@ toString(): string;

lib/observablemap.js

@@ -31,3 +31,4 @@ var observablevalue_1 = require('./observablevalue');
     ObservableMap.prototype.has = function (key) {
-        this.assertValidKey(key);
+        if (!this.isValidKey(key))
+            return false;
         if (this._hasMap[key])
@@ -71,3 +72,2 @@ return this._hasMap[key].get();
         var _this = this;
-        this.assertValidKey(key);
         if (this._has(key)) {
@@ -104,3 +104,2 @@ var oldValue = this._data[key]._value;
     ObservableMap.prototype.get = function (key) {
-        this.assertValidKey(key);
         if (this.has(key))
@@ -153,6 +152,11 @@ return this._data[key].get();
     };
-    ObservableMap.prototype.assertValidKey = function (key) {
+    ObservableMap.prototype.isValidKey = function (key) {
         if (key === null || key === undefined)
-            throw new Error("[mobservable.map] Invalid key: '" + key + "'");
+            return false;
         if (typeof key !== "string" && typeof key !== "number")
+            return false;
+        return true;
+    };
+    ObservableMap.prototype.assertValidKey = function (key) {
+        if (!this.isValidKey(key))
             throw new Error("[mobservable.map] Invalid key: '" + key + "'");
@@ -170,2 +174,1 @@ };
 exports.ObservableMap = ObservableMap;
-//# sourceMappingURL=observablemap.js.map

lib/observableobject.js

@@ -82,2 +82,1 @@ var core_1 = require('./core');
 exports.ObservableObject = ObservableObject;
-//# sourceMappingURL=observableobject.js.map

lib/observablevalue.js

@@ -54,2 +54,1 @@ var __extends = (this && this.__extends) || function (d, b) {
 exports.ObservableValue = ObservableValue;
-//# sourceMappingURL=observablevalue.js.map

lib/observableview.js

@@ -84,2 +84,1 @@ var __extends = (this && this.__extends) || function (d, b) {
 exports.ObservableView = ObservableView;
-//# sourceMappingURL=observableview.js.map

lib/simpleeventemitter.js

@@ -44,2 +44,1 @@ var utils_1 = require('./utils');
 exports.default = SimpleEventEmitter;
-//# sourceMappingURL=simpleeventemitter.js.map

lib/transform.js

@@ -36,2 +36,1 @@ var observableview_1 = require('./observableview');
 }
-//# sourceMappingURL=transform.js.map

lib/utils.js

@@ -114,2 +114,1 @@ function once(func) {
 var observablearray_1 = require('./observablearray');
-//# sourceMappingURL=utils.js.map

package.json

 {
     "name": "mobservable",
-    "version": "1.2.4",
+    "version": "1.2.5",
     "description": "Observable data. Reactive functions. Simple code.",
@@ -17,4 +17,4 @@ "main": "lib/index.js",
         "clean-build": "npm run clean && npm run build",
-        "build-dist": "npm run build && webpack && uglifyjs --in-source-map dist/mobservable.js.map -s dist/mobservable.js --source-map dist/mobservable.min.js.map -o dist/mobservable.min.js",
-        "buildtest": "npm run build && npm run build-typescript-tests && npm run build-babel-tests",
+        "build-dist": "tsc && webpack && uglifyjs --source-map dist/mobservable.min.js.map -o dist/mobservable.min.js dist/mobservable.js",
+        "buildtest": "tsc --sourceMap && npm run build-typescript-tests && npm run build-babel-tests",
         "build-typescript-tests": "tsc -m commonjs -t es5 --experimentalDecorators --noImplicitAny --outDir test test/typescript-tests.ts",
@@ -21,0 +21,0 @@ "build-babel-tests": "babel test/babel/babel-tests.js -o test/babel-tests.js"

No alert changes

Improved metrics

Lines of code

2236

increased by0.22%

Worsened metrics

Total package byte prevSize

106424

decreased by-42.46%

Number of package files

32

decreased by-28.89%

No dependency changes