Commit Graph
3 Commits
Author SHA1 Message Date
Steven 9af5cff974 $.parseJSON should return any instead of Object
Here is my test:
```js
var i = JSON.parse('1');
var a = JSON.parse('[1]');
var o = JSON.parse('{"foo":"bar"}');
var s = JSON.parse('"string"');
var n = JSON.parse('null');

i instanceof Object; // false
a instanceof Object; // true
o instanceof Object; // true
s instanceof Object; // false
n instanceof Object; // false
```

JSON.parse returns the `any` type so I think this method should do the same.

More of the discussion regarding this pull request can be found on the original commit 8c23d04a5b

https://github.com/borisyankov/DefinitelyTyped/commit/8c23d04a5b4a0505df042a82be90f95adff02d87
2014-05-14 18:00:49 -07:00
Steven b8e5189925 Changing the context param to be OPTIONAL
The compile function returns a template function. The template function takes two parameters, `context` and `options`. These are both optional parameters. This commit fixes `context` to be optional.
2014-01-23 16:08:20 -08:00
Steven ef2926042e RefreshHints are optional
The filters in RefreshHints should be optional so there is no error when updating a single one such as:

this.dataView.setRefreshHints({
    isFilterUnchanged: true
});
2013-12-20 18:47:38 -08:00