MaDDoGo / metadata.filmaffinity.com

Scraper de filmaffinity para XBMC
26 stars 7 forks source link

No funciona la ultima version #53

Closed raynar closed 10 years ago

raynar commented 10 years ago

Hola chicos, al parecer filmaffinity ha vuelto a actualizar y se ha vuelto a romper el plugin.

Tengo la ultima version publicada y hoy no funciona. Se bajan sin titulo ni año ni nada.

Saludos!

pancheto commented 10 years ago

yo estoy trabajando en una modificación que lo solucione, pero aunque es funcional recupera un error de XBMC de hace unos meses por el que no detecta correctamente la codificación de los resultados de filmaffinity, y hace que los caracteres especiales se visualicen incorrectamente en muchas ocasiones (ni siquiera es en todas, de ahí que no se pueda programar una solución general). he preguntado en un par de sitios por este problema y cuando obtenga respuesta volveré sobre mi desarrollo. mientras tanto lo he puesto en dropbox por si alguien quiere trastear con él: https://www.dropbox.com/s/gwbiytb9vbh1h1g/filmaffinity.xml pero insisto, hace falta que XBMC solucione el problema de detección de la codificación correcta de resultados, o no se mostrarán correctamente (título, sinopsis,...) en la biblioteca.

MaDDoGo commented 10 years ago

Perdonad chicos que estoy muy desconectado del tema...

Ha vuelto el error en las nightlies o vuelve a fallar en la versión estable? :S

Cuando falló en las nightlies se solucionó sólo, quizás el problema es de la página web

pancheto commented 10 years ago

el error se dio en la salida de Gotham 13.0, lo reportaste allá por enero, y efectivamente en las nightlies se solucionó. entiendo que en la 13.1 estaría todo integrado, y que la 13.2 actual debería tener esa funcionalidad intacta, pero se ve que no. mira las 2 últimas páginas del hilo del scraper en xbmcspain y verás la descripción del error, que es sin más que XBMC no es capaz de detectar la codificación correcta de los resultados obtenidos. lo he reportado en Track y en el hilo del foro propio de XBMC en el que tú mismo avisaste del problema. por el momento no hay movimiento: http://forum.xbmc.org/showthread.php?tid=180262 http://trac.xbmc.org/ticket/15355#comment:1%3E

MaDDoGo commented 10 years ago

Que cosa más rara. Bueno, a ver si la gente de XBMC contesta (que acostumbran a contestar bastante rápido) y se arregla sin tocar gran cosa.

Edito:

Acabo de ver que en el trac han posteado añadiendo a más gente al lío, a ver que contesta

matthias / karlson2k Did you guys work in this area lately? Not sure if already fixed in Helix

pancheto commented 10 years ago

fallar falla tanto la 13.1 estable como la 13.2 beta 2 actual. no me he metido con la 14 nightly porque asumo que un error tan gordo tienen que solucionarlo dentro de la versión 13 y no esperar a la 14. a ver qué nos dicen.

que conste que yo intenté introducir cosas a nivel de código para intentar ayudar a la detección de UTF-8, pero sin éxito. a lo mejor se puede ir en esa dirección si la solución se retrasa, aunque yo creo que esa es labor de los desarrolladores de XBMC porque es un problema de base en su gestión de la información obtenida por scrapers.

MaDDoGo commented 10 years ago

Si, el cambio a nivel de código sirve de más bien poco, te lo digo por experiencia que cuando ocurrio el error por primera vez me pelee mucho con el tema.

En teoría solucionaron el error, pero puede que filmaffinity haga las cosas tan mal o tan bien que el scrapper se vuelve loco.

A ver si contestan los entendidos en codificación de carácteres y le podemos meter mano al tema

pancheto commented 10 years ago

la verdad es que no sé qué puede facilitarle más las cosas a XBMC desde filmaffinity, porque en la sección de todos sus resultados tienen un tag como un mundo. a saber qué más necesitan que eso. una de mis pruebas fue intentar forzar el cambio "charset" por "encoding", por si fueran así de limitados, pero no conseguí nada.

MaDDoGo commented 10 years ago

El problema creo que está en la detección automática... Pero a ver qué dicen cuando despierten

raynar commented 10 years ago

Hola de nuevo! He probado tu version @pancheto y no funciona, falla igual que la otra.

Hay alguna forma de debuguear eso y mandarte mas info?

raynar commented 10 years ago

cerraron el ticket :(

eso quiere decir que estamos jodidos? xD

MaDDoGo commented 10 years ago

Si cierran el ticket es porque no han detectado / han corregido el bug, hay que leerse los comentarios El 02/08/2014 13:33, "raynar" notifications@github.com escribió:

cerraron el ticket :(

eso quiere decir que estamos jodidos? xD

— Reply to this email directly or view it on GitHub https://github.com/MaDDoGo/metadata.filmaffinity.com/issues/53#issuecomment-50960643 .

pancheto commented 10 years ago

cerraron el ticket porque dicen que XBMC hace bien su trabajo, y que el problema está en la incorrecta codificación de caracteres de la web de FA, así que la única solución que proponen es hablar con el equipo de FA para que lo solucionen.

como soy un culo inquieto una respuesta así no me valía, y le he dado unas cuantas vueltas al tema hasta darme cuenta que podíamos tirar de la web móvil de FA, porque las versiones móviles siempre son más estrictas con los estándares por pura obligación. y ha funcionado! así que he propuesto una versión beta en XBMCSPAIN (que incluye un par de cambios de XBMCERO que aún hay que pulir un poco) para que la gente la pruebe durante el verano, y según el feedback que reciba la idea sería hacerla pública en septiembre tras poder asegurar su validez. pasaos por allí y echadle un ojo.

MaDDoGo commented 10 years ago

Perfecto!!

Luego lo miro ;)

Eres el alma del scrapper, así da gusto :)

2014-08-02 17:36 GMT+02:00 Lechu notifications@github.com:

cerraron el ticket porque dicen que XBMC hace bien su trabajo, y que el problema está en la incorrecta codificación de caracteres de la web de FA, así que la única solución que proponen es hablar con el equipo de FA para que lo solucionen.

como soy un culo inquieto una respuesta así no me valía, y le he dado unas cuantas vueltas al tema hasta darme cuenta que podíamos tirar de la web móvil de FA, porque las versiones móviles siempre son más estrictas con los estándares por pura obligación. y ha funcionado! así que he propuesto una versión beta en XBMCSPAIN (que incluye un par de cambios de XBMCERO que aún hay que pulir un poco) para que la gente la pruebe durante el verano, y según el feedback que reciba la idea sería hacerla pública en septiembre tras poder asegurar su validez. pasaos por allí y echadle un ojo.

— Reply to this email directly or view it on GitHub https://github.com/MaDDoGo/metadata.filmaffinity.com/issues/53#issuecomment-50965918 .

Enric Soler Rastrollo

pancheto commented 10 years ago

qué exagerado! seguimos al pie del cañón!

la verdad es que he tenido que revisar todos los patrones de búsqueda de FA como si fueran nuevos, y aunque la gente parece indicar que funcionan aún estoy pendiente de que alguien encuentre casos raros de esos a los que FA nos tiene acostumbrados. hoy reportaron un par de ellos en el foro de XBMCSPAIN que intentaré corregir el lunes, y si puedo lo voy colgando en github para que sea más visible. lo malo es que incluye unos cambios para tirar más y mejor de IMDB y de TMDB que implican más cosas que el filmaffinity.xml y aún no he testado su validez, pero seguro que XBMCERO pronto lo podrá tener listo. desde luego para proponer una versión 1.6.0 final para septiembre nos debería dar seguro.

pancheto commented 10 years ago

al parecer la web de FA arregló por sí misma el problema, así que podríamos cerrar esta incidencia directamente. de todas formas la última versión del scraper 1.6.0 que acabo de publicar como pull request del branch master del scraper de maddogo solucionaría este problema de volver a aparecer, y aparte incluye muchas mejoras a todos los niveles. cierro pues este hilo.