Cuenta atrás para pelisalacarta 2.14

[jurrabi]

Ahora salta el mensaje. Antes daba un error de script. Por eso digo que el canal debería controlarlo...

[bandavi]

Jurrabi dijo:
Insisto es que ese error debería servir sólo para controlar problemas de conexión. Los problemas con páginas concretas debería controlarlos cada canal.

@jurrabi, en todo caso ese "except URLError" que pusiste en pelisalacarta.py esta mal puesto, no deberia ir ahi. Seguramente el error de conexion que te saltó fue porque, el plugin cuando lo inicias este verifica tu version conectandose a internet y eso lo hace con el archivo updater.py, podrias haberlo puesto ahi para que no pille otros errores de URl y lance un mensaje incorrecto como le pasa a Lily, que seguramente aunque lo arregle en su canal igual salte ese mensaje de no hay conexion.Tambien deberia haberse puesto un "except HTTPError" antes de ese except, para errores con codigo de respuesta como los tipicos 404.

En realidad el control para errores de URL y HTTP deberian hacerse en el scrapertools puesto que todos los canales utilizan esa libreria.
Si Jesus le puede hechar una revisada a eso o si gusta lo puedo hacer yo.

@Lily, el otro mensaje de "Video Borrado" podrias decirme en que archivo esta ese mensaje y el canal, seccion y video que lo produce.

[jurrabi]

A ver si puedo hacerme entender.

1. Me parece bien que se cambie el texto a lo que se quiera.

2. Yo puse ese control a nivel global porque había diversos puntos del plugin que daban error de script y dejaban al usuario sin saber que pasaba. Decidí poner el control global como red de seguridad última para que el plugin no finalice con un error de script y la gente no sepa que ha pasado.

3. Esta red en realidad no debería llegarse nunca, ya que los propios canales y módulos de utilidad deberían controlar los errores que se puedan producir y dar un mensaje adecuado al efecto.


@bandavi. Estoy contigo en que un punto de control sería en scrapertools, pero también en cierto que ante un error de http scrapertools poco sabe del contexto por lo que el mensaje no sería el más adecuado. De hecho, la excepción es una forma de respuesta válida ante un error de este tipo por parte de scrapertools. Los canales deben prepararse para un error de internet o intentarían seguir y tendrían comportamientos impredecibles.

Yo de momento cambiaría el mensaje como sugiere Lily. Personalmente estoy que no llego con el canal de tvshack y la opción de auto-actualizar series en la biblioteca que me gustarían que estuvieran para el domingo.

[bandavi]

@jurrabi a ver si me entiendes tu, anda haz el esfuerzo un poco
Te entiendo porque pusiste eso ahi, para solucionar un error de conexion a internet con un mensaje concreto, pero a veces la solucion puede ser peor que el problema, es preferible que te salga el error de script y poder verificar en el log, dónde esta el problema, hablando de errores de respuesta http, pero el "except URLError:" que pusiste te coje cualquier error, excepto algunos claro y te da un mensaje de error de conexion a internet cuando el error es otro, un ejemplo que he pillado hace poco en el canal: "dospuntocerovision|Programas de TV|american choper", intenta ver que mensaje sale ante un error de codigo 404 que deberia decir "Pagina no encontrada".

Dices que cada canal deberia controlar los errores que puedan producir en eso tienes razon pero es algo que no solemos hacer y a estas alturas querer cambiar todos los screapers de los canales seria una locura, por eso digo que deberia hacerse en el mismo screapertools.

Parece que el problema podria estar en el mensaje de texto pero creo que podria hacer algo mas.
Vere si puedo resolver eso, con el permiso de Jesus claro.

[jurrabi]

Claro que te entiendo bandavi. Creo que cambiar el mensaje como sugiere Lily sería la mejor solución a corto plazo.

No estoy de acuerdo contigo en que es mejor el fallo de script ya que eso siempre generará mensajes en el foro de me falla el script a los que hay que responder dime que ponen en el log, seguido de donde se mira eso... etc etc.
Con un mensaje fallo de internet o la página no está disponible la gente ya está más encaminada hacia lo que pasa.

Controlar en xbmctools por otro lado, mi pregunta sigue en pie. ¿y que puede hacer xbmctools para que un canal que no está preparado para manejar el error se entere? Si no devuelve una excepción el canal intentará seguir como si nada y tendrá un comportamiento impredecible. Yo veo mejor (tal y como está hecho todo) que xbmc de la excepción, que el canal la deje pasar y que en pelis la tratemos...
Lo que si se puede mejorar seguramente es analizar la información adicional de la excepción para distinguir (o al menos intentarlo) el tipo concreto de error. De todas formas me da en la nariz que un 404 no se puede distinguir la mayoría de las veces de una falta de conexión (hasta los exploradores responden igual muchas veces).

Por eso toda esta diatriba se resume en mi línea inicial de que la mejor solución por el momento me parece la sugerencia de Lily (contra la que nunca he estado, aunque lo pareciara. Al veces al escribir me doy cuenta de que no sueno como quería, tendréis que perdonarme)...

[bandavi]

Jurrabi dijo:
Controlar en xbmctools por otro lado, mi pregunta sigue en pie. ¿y que puede hacer xbmctools para que un canal que no está preparado para manejar el error se entere? Si no devuelve una excepción el canal intentará seguir como si nada y tendrá un comportamiento impredecible. Yo veo mejor (tal y como está hecho todo) que xbmc de la excepción, que el canal la deje pasar y que en pelis la tratemos...

Depende de lo que se le envie al canal, si solo se devuelve (simple return) al canal este seguira mandando error de script y no continuará la accion.
En cambio si se devuelve comillas(return "") el canal finalizará la accion, porque ahi se controla sola.
De todas maneras te voy hacer caso y el scrapertools no lo voy a tocar y lo voy hacer en pelisalacarta.py, lo hago y lo subo al svn para que veas que tal.
Lastima que Lily no tenga acceso al svn para que pueda verificar los mensajes de error a ver que tal le parecen .

[jurrabi]

En el apartado de tipo de error con código conocido yo indicaría algo como:
"Se ha detectado un error de red"
o
"Se ha detectado un error de conexión"
o
"Se ha detectado un error de Internet"

No sé... pero algo que cualifique un poco el error... quizá Lily quiera ofrecer su inspirada... inspiración

[Lily]

@Lily, el otro mensaje de "Video Borrado" podrias decirme en que archivo esta ese mensaje y el canal, seccion y video que lo produce.

Por ejemplo en Anime (Foros)/AstroteamRG(...)/Kochikame(...)/ en los capítulos 82, 83... salta la alerta al seleccionar calidad baja, pero en calidad alta se reproducen correctamente, antes pasaba sistemáticamente en todos los capítulos recién subidos, pero ahora parece que los últimos si tienen enlace a megavídeo... (esos dos capítulos no son los únicos en ese listado es bastante frecuente que no se pueda reproducir en calidad baja...). Ese mensaje también me ha salido para Megavideo, alguna vez, en películas de películasyonkis....

No tengo muchas ganas de rollos o enrollos , pero yo he comentado lo de los mensajes porque realmente me parece que no son apropiados sino confusos, esto simplemente como usuaria del plugin... porque uno me dice que el capi que quiero ver de Kochikame está borrado que pruebe con otro, cuando no está borrado y lo puedo ver en calidad alta , y el otro cuando le cae un rayo al servidor de la página que tiene que cargar, me dice poco menos que vaya a mirar el router o compruebe el ptr

No quiero por favor, que nadie me de explicaciones de lo de arriba que si el mensaje dice eso o lo deja decir, o si está para eso o para lo otro,.. porque lo he escrito en tono jocoso como un ejemplo...

Bueno, a estar bien

Saludos

[jurrabi]

Creo que me toca pedir perdón, porque han sido mis mensajes los que han iniciado el embrollo.

El punto de vista que aporta Lily (en mi opinión) es el único que cuenta. Es decir, el del usuario final.
Ésa era mi intención cuando decidí interceptar los errores http en pelisalacarta porque entendía que un error de script era algo demasiado críptico y genérico para la mayoría de los usuarios. Puse el mensaje de mirar la conexión de internet porque asumí (erróneamente) que el resto de errores se interceptarían correctamente en cada canal.

Puesto que la realidad nos muestra que no es así, los cambios que proponía inicialmente lily me parece de lo más acertado. De hecho, bandabi, creo que las ventanas de mensajes deben quedar como decía lily. Y luego, para nosotros si quieres, podemos mandar al log los parámetros técnicos (que de nada le sirven al usuario).

Si estás de acuerdo y si quieres lo cambio yo...

nuevamente, perdón por embrollar algo que estaba cristalino como el agua.

[jesus]

Vaya lio más gracioso que tenéis con los mensajes

Propongo cambiar todos los mensajes por uno que ponga "Lo siento pero algo no va bien" je je je