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

El server se modularizó #96

Closed andresmanelli closed 8 years ago

andresmanelli commented 8 years ago

Favor de verificar que les funcione. Yo no tuve problemas. Traté de separar de manera lógica. Cualquier comentario es bienvenido.

Los archivos que comienzan con guión bajo son los que acceden a la BDD, los demás exponen la API que usa server.js. Qué les parece?

francoa commented 8 years ago

tal vez habría que hacer los mismo con los archivos user(s)

En dicho caso el único cambio sería que user.js pasaría a ser _users.js

en un rato reviso que funcione

andresmanelli commented 8 years ago

Claro, no estaba seguro por las lineas que definen el prototype de User y eso. La verdad no lo entiendo mucho

francoa commented 8 years ago

qué líneas por ejemplo?

andresmanelli commented 8 years ago

Era en user.js

var User = module.exports = function User(_node) {
    this._node = _node;
};

Object.defineProperty(User.prototype, 'id', {
    get: function () { return this._node.id; }
}); 

Object.defineProperty(User.prototype, 'name', {
    get: function () {
        return this._node.data['name'];
    },
    set: function (name) {
        this._node.data['name'] = name;
    }
});
francoa commented 8 years ago

yo hice eso? o_O jajaja

francoa commented 8 years ago

lo debo haber hecho por alguna razón pija que no recuerdo. Son getters y setters así que no pasa nada. Tienen que estar ahí de todas maneras, hasta que redescubra por qué los puse

andresmanelli commented 8 years ago

Cambio ese nombre de archivo entonces? O estás verificando cosas?

francoa commented 8 years ago

cambialo pero dejame que lo testee cuando termine con lo otro