MecatronicaUncu / Red-Social-Asociacion

A small open source social network for any small community
GNU General Public License v2.0
3 stars 1 forks source link

Eliminar Mongo #84

Closed fcladera closed 8 years ago

fcladera commented 8 years ago

Hola! Estoy tratando de sacar Mongo para hacer el Dockerfile, pero veo que todavía hay código activo de Mongo en varias partes

server/routes/mongo.js
3:// Módulo del server que se comunica la base de datos MongoDB //
7:var mongo = require('mongodb').MongoClient;
13:/** @type {String} URL de conexión a la base de datos MongoDB */
14:var mdburl = 'mongodb://localhost:27017/test';
15:/** @type {MongoDB} Objeto que hace referencia a la base de datos utilizada */
19: * Realiza la conexión con la base de datos MongoDB local y guarda la referencia
26:mongo.connect(mdburl, function(err, mdb) {
28:             console.log('Error connecting to mongodb server');
30:             console.log('Connected correctly to mongodb server');
80:                             res.send(500,'Error MONGO getTimes');
104:                    res.send(500,'Error MONGO getConfig 1');
110:                                    res.send(500,'Error MONGO getConfig 2');
135:                    res.send(500,'Error MONGO getPlaces');
167:                res.send(500,'Error MONGO newActivity');
217://     * TODO: Interrumpir el SignUp si no se registra en MongoDB, para
259://                  res.send(500,'Error MONGO getProfile');
287://                  res.send(500,'Error MONGO changeProfile');
307://                  res.send(500,'Error MONGO getPlaces');
323://                  res.send(500,'Error MONGO getThey');
356:                    res.send(500,'Error MONGO getNodeTypes');
387:                    res.send(500,'Error MONGO getRelTypes');
416:                    res.send(500,'Error MONGO getRelTypes');
423:                                    res.send(500,'Error MONGO getNodeTypes');
server/routes/users.js
5:var Mongo = require('./mongo.js');
498:            //Mongo.getNodeContentsData(req, res, contents);
server/server.js
7:    , mongo = require('./routes/mongo.js')
108:app.get('/acttypes', mongo.getTypes);
111:app.get('/edtplaces', mongo.getPlaces);
155:app.get('/fields/:label', mongo.getFields);
180:app.get('/nodereltypes', users.extractCookieData, mongo.getNodeRelTypes);

Pareciera ser solamente para conectarse a la db, no hay otras cosas. Pero debería romperse cuando lo saque.

andresmanelli commented 8 years ago

Si. En la PR que hice ayer eliminé una dep. Estoy trabajando para sacarlo. No queda mucho.

andresmanelli commented 8 years ago

@fcladera esto se puede cerrar con la PR #83 ?

fcladera commented 8 years ago

On Sunday, March 20, 2016 11:26:57 AM ART, Andrés wrote:

@fcladera esto se puede cerrar?

Yo creo que sí, si ya mergeamos las otras ramas.