1

What I'm trying to solve is: preserving the order of my array of Ids with $in using this suggested method (mapReduce): Does MongoDB's $in clause guarantee order

I've done my homework, and saw it's ideal to convert them to strings: Comparing mongoose _id and strings.

Code: 

var dataIds = [ '57a1152a4d124a4d1ad12d80',
     '57a115304d124a4d1ad12d81',
     '5795316dabfaa62383341a79',
     '5795315aabfaa62383341a76',
     '57a114d64d124a4d1ad12d7f',
     '57953165abfaa62383341a78' ];

CollectionSchema.statics.all = function() {

      var obj = {};

      //adds dataIds to obj.scope as inputs , to be accessed in obj.map
      obj.scope = {'inputs': dataIds};

      obj.map = function() {
        //used toString method as suggested in other SO answer, but still get -1 for Id.
        var order = inputs.indexOf(this._id.toString());
        emit(order, {
          doc : this
        });
      };

      obj.reduce = function() {};

      obj.out = {inline: 1};

      obj.query = {"_id": {"$in": dataIds } };

      obj.finalize = function(key, value) {
        return value;
      };

      return Product
        .mapReduce(obj)
        .then(function(products){
          console.log('map products : ', products)
        })
     };

This is what I keep getting back for my console.log in the products promise : 

[{ _id: -1, value: null } ]

Which, leads me to believe it's not able to match the ObjectId from this, with an index of dataIds. However, if I just use the $in clause within a .find(), the correct products are returned -- but, in the incorrect order.

Update:

getting unexpected behavior with this. 

  obj.map = function() {
    for(var i = 0; i < inputs.length; i++){
      if(inputs[i] == this._id.toString()){
      }
      emit(inputs[i], this);
    }
  };

emits: 

 [ { _id: '5795315aabfaa62383341a76', value: null },
  { _id: '57953165abfaa62383341a78', value: null },
  { _id: '5795316dabfaa62383341a79', value: null },
  { _id: '57a114d64d124a4d1ad12d7f', value: null },
  { _id: '57a1152a4d124a4d1ad12d80', value: null },
  { _id: '57a115304d124a4d1ad12d81', value: null } ]

  obj.map = function() {
    for(var i = 0; i < inputs.length; i++){
      if(inputs[i] == this._id.toString()){
        var order = i;
      }
      emit(this._id.toString(), this);
    }
  };

emits: 

[ { _id: 'ObjectId("5795315aabfaa62383341a76")', value: null },
  { _id: 'ObjectId("57953165abfaa62383341a78")', value: null },
  { _id: 'ObjectId("5795316dabfaa62383341a79")', value: null },
  { _id: 'ObjectId("57a114d64d124a4d1ad12d7f")', value: null },
  { _id: 'ObjectId("57a1152a4d124a4d1ad12d80")', value: null },
  { _id: 'ObjectId("57a115304d124a4d1ad12d81")', value: null } ]

Now, how do I get rid of the ObjectId() wrapper? Preferably, something more clean than str.slice(), which would work -- However, I feel there must be a more "mongo" / safer way of converting the Id.

I checked out the docs, but it only mentions the toString() method, which does not seem to be working correctly within map: https://docs.mongodb.com/manual/reference/method/ObjectId.toString/

Community
  • 1
  • 1
NoobSter
  • 1,150
  • 1
  • 16
  • 39

1 Answers1

1

figured it out: 

  obj.map = function() {
    for(var i = 0; i < inputs.length; i++){
      if(this._id.equals(inputs[i])) {
        var order = i;
      }
    }
    emit(order, {doc: this});
  };
NoobSter
  • 1,150
  • 1
  • 16
  • 39