Error: collection name must be a String in node js

Question

I am trying to run following code on server

var express = require("express"),
    app = express(),
    bodyParser = require('body-parser'),
    errorHandler = require('errorhandler'),
    methodOverride = require('method-override'),
    hostname = process.env.HOSTNAME || 'localhost',
    port = parseInt(process.env.PORT, 10) || 4004,
    publicDir = process.argv[2] || __dirname + '/public';
var exec = require('child_process').exec;
var fs = require('fs');
 var mongodb = require('mongodb'),
    serverdb = new mongodb.Server('127.0.0.1', 27017, {}),
    dbName = new mongodb.Db('prisync', serverdb, {});
var url = "urlinfo";


//Show homepage
app.get("/", function (req, res) {
  //res.redirect("/index.html");
  console.log("shubham ");
  dbName.open(function (error, client){
      var collection = new mongodb.Collection(client , url);   //line 24 ====
      collection.find().limit(20).toArray(function (err, dataObjArr){
        var data = '';
        var dataArr = [];
        var i = dataObjArr.length;
        //check for error
        if(err){return res.end('error!'+err);}
        //Data
        if(dataObjArr){
          while(i--){
            dataArr[i] = dataObjArr[i]._id;
          }
          data = dataArr.join(' ');
          res.render('/index.html',{ returnedData : data });
        }else{
          res.end();
        }

      });


      });
  });



app.get("/search", function (req, res){
  console.log("shubham batra");
   var pro_name = req.query.name;
   var pro_code = req.query.code;
   var pro_category = req.query.category;
   var pro_brand = req.query.brand;

 pro_name = pro_name+"";
 pro_code = pro_code+"";
 pro_brand = pro_brand+"";
 pro_category = pro_category+"";



  MongoClient.connect('mongodb://127.0.0.1:27017/prisync', function(err, db) {
  if (err) throw err;
    console.log("Connected to Database");


    var documen = {name:pro_name, code:pro_code , category:pro_category, brand:pro_brand };
     //documen = JSON.stringify(documen);
  //insert record
  db.collection('urlinfo').insert(documen, function(err, records) {
    if (err) throw err;

    });

  res.redirect("/index.html");

});
});
//Search page
app.use(methodOverride());
app.use(bodyParser.json());
app.use(bodyParser.urlencoded({
  extended: true
}));
app.use(express.static(publicDir));
app.use(errorHandler({
  dumpExceptions: true,
  showStack: true
}));

console.log("Server showing %s listening at http://%s:%s", publicDir, hostname, port);
app.listen(port);

but it gives following error while try to load localhost:4004

Server showing /home/shubham/Music/pricesync/server/public listening at http://localhost:4004
shubham 

/home/shubham/node_modules/mongodb/lib/server.js:274
      process.nextTick(function() { throw err; })
                                          ^
Error: collection name must be a String
    at Error (<anonymous>)
    at checkCollectionName (/home/shubham/node_modules/mongodb/lib/utils.js:69:11)
    at new Collection (/home/shubham/node_modules/mongodb/lib/collection.js:57:3)
    at /home/shubham/Music/pricesync/server/server.js:24:24
    at /home/shubham/node_modules/mongodb/lib/db.js:221:5
    at connectHandler (/home/shubham/node_modules/mongodb/lib/server.js:272:7)
    at g (events.js:180:16)
    at emit (events.js:95:17)
    at /home/shubham/node_modules/mongodb/node_modules/mongodb-core/lib/topologies/server.js:399:23
    at /home/shubham/node_modules/mongodb/node_modules/mongodb-core/lib/topologies/server.js:756:13

swill734 · Accepted Answer · 2018-02-26 19:20:26Z

16

I had the same error, for me it was caused by my timestamp being incorrectly placed.

INCORRECT CODE:

    }),{timestamps:true});

CORRECTED CODE:

var mongoose = require('mongoose');
var ObjectId = mongoose.Schema.Types.ObjectId;

var User = mongoose.model('User', new mongoose.Schema({
    email:{type:String, required:true},
    password:{type:String, required:true},

    listings:[{type:ObjectId,ref:"Listings"}]

},{timestamps:true}));

edited Feb 26, 2018 at 19:20

answered Feb 26, 2018 at 13:29

swill734

1611 silver badge5 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Richard Andreu · Accepted Answer · 2015-09-03 12:51:09Z

0

Assuming the collection name is in 'url' varialbe (i.e. 'urlinfo'), the right way to instantiate the collection would be:

var collection = dbName.collection(url); //line 24

If you look at documentation, it states specifically that Collection class should not be instantiated directly : https://mongodb.github.io/node-mongodb-native/api-generated/collection.html

edited Sep 3, 2015 at 12:51

answered Sep 3, 2015 at 12:35

Richard Andreu

12 bronze badges

1 Comment

abarisone Over a year ago

Could you please elaborate more your answer adding a little more description about the solution you provide?

Peter · Accepted Answer · 2015-12-13 21:35:09Z

0

I got stuck at the same situation, while solving the exercise from Guillermos Smashing Node.JS book...

I followed Richard Andreus directions and after few attempts, succesfully established the server.

I newly defined

var Db = require('mongodb').Db,
    Server = require('mongodb').Server;

instead of existing variable definitions. As next I've defined the variable db.

var db = new Db('test', new Server('localhost', 27017));

Then open the connection to db, or in your case dbName.

db.open(function(err, db) {
  var collection = db.collection(url);
// in my case I defined app.users instead of collection

NOTE: Don't forget your definition at dbName

answered Dec 13, 2015 at 21:35

Peter

1,6892 gold badges21 silver badges27 bronze badges

Collectives™ on Stack Overflow

Error: collection name must be a String in node js

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related