Add support of `js-data-sql` adapter by LoicMahieu · Pull Request #328 · js-data/js-data-angular
In browserify context, if:
require('js-data')
window.DSHttpAdapter = require('js-data-http')
window.DSSqlAdapter = require('js-data-sql')
require('js-data-angular')
There is a bug in adapter registration:
null "new data store created" Defaults {…} null null
null "registerAdapter" "http" DSHttpAdapter {defaults: Defaults} Object {}
null "default adapter is http" null null null
null "registerAdapter" "sql" DSHttpAdapter {defaults: Defaults} Object {}
null "default adapter is http" null null null
null "registerAdapter" "http" DSHttpAdapter {defaults: Defaults} Object {default: true}
null "default adapter is http" null null null
In adapter registration loop (https://github.com/js-data/js-data-angular/blob/master/src/index.js#L205-L214), if the first adapter is well registrated, it continue to loop over next adapter when with a wrong object.
Adding a break fix the issue. (in 846c4c0)
(I added a commit to current PR, if separate PR is preferable, just tell me)