Mongoose Adapter Single Index support #125

merged 2 commits into from Sep 26, 2012


None yet

2 participants


It is a minor fix with the mongoose adapter to support index in mongo..

@anatoliychakkaev anatoliychakkaev and 1 other commented on an outdated diff Sep 20, 2012
@@ -48,8 +48,12 @@ function MongooseAdapter(client) {
MongooseAdapter.prototype.define = function (descr) {
var props = {};
Object.keys( (key) {
- props[key] =[key].type;
- if (props[key].name === 'Text' || props[key].name === 'JSON') props[key] = String;
+ props[key] = {};
anatoliychakkaev Sep 20, 2012 Collaborator

Unnecessary spaces between =

anatoliychakkaev Sep 20, 2012 Collaborator

Thanks. When I was a ruby programmer this style was common practice in our team. After years I've managed that it's not true code beauty. And finnaly I have heard same thoughs from other developers and such gurus as Steve McConnell. Sorry for pain.

nashadalam Sep 20, 2012

No probs...I understand...

@anatoliychakkaev anatoliychakkaev merged commit 2cbce0d into 1602:master Sep 26, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment