#15845 closed defect (fixed)
dojo/store/Cache third argument (options) is no longer optional
Reported by: | Kris Zyp | Owned by: | bill |
---|---|---|---|
Priority: | undecided | Milestone: | 1.8.1 |
Component: | Data | Version: | 1.8.0 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
In [28515] on line 35 of Cache.js, the line that provides a default object for options so that the options argument can be optional was inadvertently removed. This regression breaks code (including a couple tests in dgrid) that don't pass a third argument to the Cache constructor.
Change History (6)
comment:1 Changed 10 years ago by
Owner: | set to bill |
---|---|
Status: | new → assigned |
comment:2 Changed 10 years ago by
Component: | General → Data |
---|---|
Milestone: | tbd → 1.8.1 |
comment:5 Changed 10 years ago by
For anyone else googling for the error this bug causes, the message reported tends to be:
Uncaught TypeError: Cannot read property 'isLoaded' of undefined
I spend ages looking into this, unable to find a reference online. An example of the failure: http://dojo-sandbox.net/neek/ccd8d/0
Note: See
TracTickets for help on using
tickets.
Ah sorry I'll put that back.
Looks like you are missing tests for this though (in dojo/tests/store/Cache.js), and tests for the isLoaded callback.