Skip to content

Added missing method for mongodb #35

Merged
merged 1 commit into from Jan 3, 2012

3 participants

@giacomomacri

I've noticed that getBoardSize and setBoardSize where missing for mongodb.

@aliasaria aliasaria merged commit 033050e into aliasaria:master Jan 3, 2012
@Waateur
Waateur commented Jun 12, 2012

Which driver mongodb do you use ? i have an error ligne 167 lib/data/mongo.js

this.room.findOne({name:room})
            ^
TypeError: Cannot call method 'findOne' of undefined

And I think it's from the driver :/

@giacomomacri

I've just tried and it works, MongoDB 1.8.1 (it's a little bit older I know.. :-) ) ..

Have you changed config.js for mongo connection? You're running mongo on the right port?

@Waateur
Waateur commented Jun 12, 2012

The mongo connection is fine, I only have a trouble when i resize the board. But post-it and column are nicely saved :D . I think i found a typo and sent a pull request about it but it still doesn't work properly.

/usr/local/stow/scrumblr/node_modules/mongodb/lib/mongodb/collection.js:882
    if(items.length == 1) return callback(null, items[0]);  

It's seems like it's about args give to the findOne function or something like that.
I'm using mongoDB 2.0.5 and drivers from npm mongodb 1.0.1

I will continue looking for the problem :) Thanks for answering :)

@Waateur
Waateur commented Jun 12, 2012

Found what was wrong for me :) as written in my first message there was a mistake on ligne 167 (s missing) but this ligne is useless. I just removed it and everything is fine now :)
Thank you again :)
and by the way Really nice program.

@giacomomacri

I was trying updating MongoDB to 2.0.1 .. :-) for you ... looking at code now.. you're right it's useless!! back to work... bye!!!

@Waateur
Waateur commented Jun 12, 2012

thank's again :)
bye =)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.