Conversation
Notices
-
@litox Sí, sería cambiarle el nombre al filtro y hacer algunas pruebas a ver si funciona :) @david
-
@david @manuel Estoy investigando el tema de los avatares en wordpress y como funciona wp-gnusocial. Creo que he detectado algo que se podría optimizar, pero no estoy totalmente seguro.
Si en vez de ponerle un filtro a getavatar, se lo ponéis a preget_avatar y os ahorráis que consulte a gravatar antes.
En el caso de que queráis dejar la opción por defecto devolvéis nulo y arreando.
¿Voy bien encaminado?
He estado mirando esto:
https://lamatriz.org/url/250198
http://git.lasindias.club/lasindias/wp-gnusocial/src/master/includes/wp-gnusocial-klaso.php
-
@litox @manuel mmm me suena a que se podría hacer de gravatar un sustitutivo en caso de que no hubiera imagen disponible en lamatriz.org... gracias!
-
@litox Super bien encaminado! mucho más eficiente solo comunicarnos con gravatar en los casos estrictos en que el plugin no puede obtener el avatar del nodo de !gnusocial Lo has probado en local? Tienes un pull request :D
Con esto que comentas y unos cambios que hicimos a mano el otro día con @david Tenemos nueva versión de !wpgnusocial muy pronto!!!
-
@manuel @david Ahora en un rato (tengo que ir a comprar antes de que cierren) envío un pull request, en principio con el cambio funciona igual que antes, aún así los avatares no me funcionan últimamente muy bien. Con un plugin de cache supongo que se arreglaría bastante.
Pero no me he quedado en eso si no que he hecho una limpieza de esa parte de código en lo que me chirriaba un poco. No más alusiones a gravatar, si wordpress quiere usar gravatar que lo use, pero nosotros no vamos a obligar a ello.
En fin, que lo probáis y me decís si notáis mejora o ha sido para mantener la coherencia y ya.
Ahora que ya empiezo a entender como funciona iremos mejorándolo.
-
@litox Buenísimo! Excelentes noticias! muchas gracias! es un gustazo te la compañía de tu teclado ;) @david
-
@litox Listo! ya he aceptado tu pull request en el repo de !wpgnusocial Mañana hacer un par de pruebas, agregamos lo que corregimos el otro día y lanzamos nueva versión :) @david