search

mdumrauf / tacs-guybrush

Social torrent tracker that groups torrents and serves them to bittorrent clients.
1 stars 0 forks source link

Get feeds impl #25

Closed mdumrauf closed 11 years ago

ghost commented 11 years ago

Ahí estuve revisando, hay un par de cosas que no entendí. 1) El por qué del objeto guybrushApp en js. 2) Por qué por ejemplo la clase Feed necesita un método estático newFeed que cree una instancia de esa clase. ¿No convendría directamente hacer new Feed(userId, title, description) desde donde se lo necesite instanciar?

mdumrauf commented 11 years ago

1) Para que haya un objeto que conozca todas las funciones y se puedan invocar desde cualquier js, por ejemplo application.js usa funciones de fbscripts.js y viceversa. Para que esté más modularizado. 2) No lo necesita. Es un helper. No quise sobrescribir el constructor default porque no sé cómo va a reaccionar Slim3 con eso, ya que por default usa el contrato Java Bean. Lo ideal sería hacer un Builder, pero no tuve tiempo. Se aceptan PR con eso! :)

2013/6/22 guidok notifications@github.com

Ahí estuve revisando, hay un par de cosas que no entendí. 1) El por qué del objeto guybrushApp en js. 2) Por qué por ejemplo la clase Feed necesita un método estático newFeed que cree una instancia de esa clase. ¿No convendría directamente hacer new Feed(userId, title, description) desde donde se lo necesite instanciar?

— Reply to this email directly or view it on GitHubhttps://github.com/mdumrauf/tacs-guybrush/pull/25#issuecomment-19865597 .