martofeld/melibook

melibook's Issues

Utilizar Gravatar para los avatares

además de poder subir imágenes como lo estan haciendo ahora, si el usuario tiene una cuenta de gravatar Y no subió una imagen propia, en lugar de mostrar la imagen por default, mostrar la imagen configurada en gravatar.

Si no tiene imagen en gravatar y no subio su propio avatar, mostrar el default.

Mover los javascript a archivos javascript.

Lamentablemente no se puede usar createLink en un archivo JS, por lo tanto la generacion del link queda en el gsp, y se asigna a uan varialbe gobal, que luego se consume desde el .JS

Lo que yo suelo hacer es crear una única variable global llamada urls o endpoints que es un mapa con las urls que necesito.

var urls = { create: "${createLink()}", update: bla bla bla..

Refactor de SearchController

Lo que hace parece rebuscado y podría hacerse de formas más simples.
Para resolverlo, hagamos el siguiente procedimiento.

Varios tests para verificar toda las posibilidades que hay en este código. Por ejemplo un test que entre al if user == null, otro que no entre ahí.. demáses... (pueden ver la cobertura usando un plugin de grails llamado coverage)

Luego, con el tests hecho.. y sin modificar el test.. modificar el codigo para que quede más simple y prolijo, y aún así pasen los tests.

No guardar imagenes en DB

Si bien se puede, hace crecer mucho el tamaño de la db, y no siempre funciona como es deseado. Además que esto dificulta algunas tareas de optimización.

Lo ideal es guardar en la db la ubicacion y el nombre de la imagen.. pero la imagen subirla a una carpeta del disco comun y corriente.

Que es el archivo Find Results?

Esto sucede por usar "add ." o commit -a sin realmente revisar que se está commiteando.
https://github.com/martofeld/meliBook/blob/master/grails-app/domain/melibook/Find%20Results

Además, por favor, UN UNICO FIX POR COMMIT Si no se respeta esa, no tan simple rengla, es muy complicado poder deshacer un único cambio.

La carpeta target.. bye bye

Crear TagLib para mostrar avatars de los usuarios

La parte que muestra un avatar o una imagen por default sería mejor hacerla utilizando TagLibs!

Arreglar y optimizar getLastUsers

El método indexController:getLastUsers tiene varios issues..

1. No es un action.
2. En lo personal me parece que pertenece a la clase User, asi que lo crearía como un método estático de User.
   3.No debe iterar por toda la base de datos. Debe retornar los últimos 3 directamente. Para eso se puede usar tanto findAll o list, ambos métodos aceptan por parámetro un ordenamiento Y un limite de registros a retornar.

PostController:refreshPosts

Hay algo raro en lo que retorna ese método.. retorna una vista o hace un render?
En todo caso pareciera que la app retorna en un action que está pensado para ser para ser llamado desde ajax retorna un template..... es mas lindo que retorne json.

Borrar vistas que no se usan

Por ejemplo, me dicen que la vista conversation/view.gsp no se usa más..Entonces borrarla! 😠

Ordeamiento de los POSTS refactorizar

Area.findByName("all").posts.sort {
it.timestamp
}.reverse()

En todos lados se hace el mismo sort... sería conveniente ponerlo en la clase de dominio como un método de instancia que haga siepre eso.