search

MaDDoGo / metadata.filmaffinity.com

Scraper de filmaffinity para XBMC
26 stars 7 forks source link

revisión de búsqueda en HDTrailers #44

Closed pancheto closed 11 years ago

pancheto commented 11 years ago

por error, hasta ahora la búsqueda de HDTrailers se hacia siempre, cuando realmente hay un condicional que la controla. ahora sólo se realiza la búsqueda cuando este condicional está activado en la configuración del scraper.

he actualizado el filmaffinity.xml y el changelog.txt, dejando que sea MaDDoGo el que actualice el addon.xml adecuadamente

MaDDoGo commented 11 years ago

Dios mío, que verguenza :( Acabo de ver este PR de hace dos meses que no había mergeado, lo siento mucho @pancheto he estado bastante liado este tiempo y se me había pasado completamente.

Voy a intentar hacer todo el papeleo para que se haga el update del scrapper pero todavía estoy liado con examenes y el curro. (malditos Compiladores)

pancheto commented 11 years ago

tranqui, que te entiendo perfectamente. yo también tuve un final de curso lectivo fino, así que ni siquiera me acordé de recordártelo.

el cambio es poca cosa, pero puede ayudar a acelerar sensiblemente el scraper cuando se trate con librerías grandes, así que habrá más de uno que lo agradecerá.

el próximo PR estaré yo mismo también pendiente, por si es cuestión de reavivar el tema pasado un tiempo prudencial. yo ahora debería tener algo más de tiempo para esto, así que si hay que echar más manos, se echan.

xbmcspain commented 11 years ago

Vergüenza ninguna... Está claro que antes está la obligación que la devoción... (dioooss, cada día me parezco más a mi padre...), y como dice @pancheto, hasta que no lo ha hecho davidmuma, tampoco nadie se había acordado de recordártelo.

Por cierto... ¿vas a hacer también el merge en el ramal master?? Aviso de que el addon.xml se ha quedado en la versión 1.5.6.

Salu2!

pancheto commented 11 years ago

yo estoy de vacaciones hasta la semana que viene, así que no accederé a github hasta entonces. lo único que cambié fue el filmaffinity.xml, y en el addon.xml incluí la explicación del cambio, nada más. si lo pudieras hacer tú... ;-)

MaDDoGo commented 11 years ago

Ah, alguien me lo ha dicho por el foro???? No me ha llegado notificacion :( lo he visto de milagro esta mañana entre los correos enterrados, he visto el de github...

Necesito un "algo" de notas o tengo la memoria de un pez...

A ver si el jueves puedo mirarmelo que ya acabo (despues de 1.5 casi 2 meses) los p*&%s examenes...

Saludos y disculpas por el olvido ;) El 13/07/2013 21:19, "Lechu" notifications@github.com va escriure:

yo estoy de vacaciones hasta la semana que viene, así que no accederé a github hasta entonces. lo único que cambié fue el filmaffinity.xml, y en el addon.xml incluí la explicación del cambio, nada más. si lo pudieras hacer tú... ;-)

— Reply to this email directly or view it on GitHubhttps://github.com/MaDDoGo/metadata.filmaffinity.com/pull/44#issuecomment-20925065 .

xbmcspain commented 11 years ago

@pancheto, pero si yo hago el pr encantado!! Por eso preguntaba lo del master, para no meter la pata...

En 5º??? Pues ánimo @MaDDoGo que en ná licenciado!

MaDDoGo commented 11 years ago

Ya lo tenemos hecho todo, actualizadas las 3 branch :)