Browse files

Add UnderscoreJS as a Bob dependency

  • Loading branch information...
1 parent bdb47fb commit e75caf8bb54d560f5eaf6374805f66efcea04777 @vitorbaptista vitorbaptista committed Feb 27, 2013
Showing with 3 additions and 3 deletions.
  1. +3 −3 app/bob/bob.js
6 app/bob/bob.js
@@ -8,7 +8,8 @@ var osw = OpenSpending.Widgets;
osw.QueryBuilder = function(elem, callback, finish, context, spec) {
var self = this;
- var resources = ["//",
+ var resources = ["//",
+ "//",
OpenSpending.scriptRoot + "/app/bob/css/query-builder/jquery-ui-1.8.18.custom.css",
OpenSpending.scriptRoot + "/lib/vendor/handlebars.js"
@@ -17,8 +18,7 @@ osw.QueryBuilder = function(elem, callback, finish, context, spec) {
self.hasFinish = finish instanceof Function;
self.noFinish = !self.hasFinish;
- self.context = _.extend({
- }, context);
+ self.context = $.extend({}, context);

Is there a specific reason why you change this from _ to $?

Nice catch. I tried to remove all dependencies on Underscore, as (just in Bob) they are all trivial, and we're already loading jQuery anyway. But then I hit a roadblock that I don't remember. Some stuff was harder than I expected, so I changed my mind. I probably forgot to checkout this line, and it got in :/

Thanks for fixing! I wanted to be entirely sure I didn't break anything when I updated openspendingjs.

Hrm, I don't think you pushed a fix. I'm going to push a fix for this now. Please use a pull request in future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
self.serialize = function() {
state = {};

0 comments on commit e75caf8

Please sign in to comment.