search

pombreda / cdpedia

Automatically exported from code.google.com/p/cdpedia
0 stars 0 forks source link

404 for some /static/skins/js/wikibits.js resources #153

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
En log del crawler
14:35:41 Error:  "NOT FOUND" (404) at link localhost:8000/sort_up.gif (from 
localhost:8000/static/skins/js/wikibits.js)
14:35:41 Error:  "NOT FOUND" (404) at link localhost:8000/sort_down.gif (from 
localhost:8000/static/skins/js/wikibits.js)
14:35:41 Error:  "NOT FOUND" (404) at link localhost:8000/sort_none.gif (from 
localhost:8000/static/skins/js/wikibits.js)

Veo files con esos nombres en
cdpedia/assets/static/skins/css/images

aunque no sé si el .js quiere indicar esos files.

El .js construye dinámicamente el path, y pareciera que se esta equivocando o 
algo lo confunde.

Original issue reported on code.google.com by ccanepacc@gmail.com on 7 Oct 2012 at 10:51

GoogleCodeExporter commented 9 years ago 
Estos son recursos usados para la presentacion de tablas de datos.
Son las flechitas que aparecen en el header de la tabla, usadas para ordenar 
por una columna especifica.

Ejemplo:
http://localhost:8000/wiki/Portugal_en_el_Festival_de_la_OTI

Con el codigo de launchpad r393 si apuntamos a esa pagina con Opera o FF se vé 
que las imagenes de las flechitas no se obtuvieron y el navegador usa el texto 
alt para rendir; IE8 en cambio muestra el pic de 'img not present'.
Si clickeamos un par de veces en el target lo que sale en cdpedia.log es:

127.0.0.1 - - [19/Oct/2012 00:08:49] "GET /skins/css/images/sort_none.gif 
HTTP/1.1" 404 -
127.0.0.1 - - [19/Oct/2012 00:08:51] "GET /skins/css/images/sort_up.gif 
HTTP/1.1" 404 -
127.0.0.1 - - [19/Oct/2012 00:08:51] "GET /skins/css/images/sort_down.gif 
HTTP/1.1" 404 -

Al path del log le falta un /static adelante para que cuadre con el esquema de 
recursos que se esta usando.

La URL es generada en wikibits.js, con una linea
var ts_image_path = stylepath + '/skins/css/images/';

Una busqueda por 'stylepath' muestra un unico setter :
  D:\tmp\cd_mini_3\cdpedia\assets\static\skins\js\header.js (1 hits)
    Line 2: stylepath="",

Qué conviene modificar aquí? el valor de stylepath o agregarle un '/static' 
adelante a la constante ?

Si lo agrego a la constante, el recurso se encuentra. (Opera, FF, IE8)
Si cambio el valor de 'stylepath' en header.js el log dice
127.0.0.1 - - [19/Oct/2012 01:06:32] "GET 
/wiki/static/skins/css/images/sort_none.gif HTTP/1.1" 404 -

Esta url no tiene sentido en el esquema que estamos usando.

Asi que lo correcto pareceria ser cambiar de 
    var ts_image_path = stylepath + '/skins/css/images/';
a
    var ts_image_path = stylepath + '/static/skins/css/images/';

Original comment by ccanepacc@gmail.com on 19 Oct 2012 at 5:15

GoogleCodeExporter commented 9 years ago 
please review

https://code.launchpad.net/~ccanepacc/cdpedia/table-resouces/+merge/130481

Original comment by ccanepacc@gmail.com on 19 Oct 2012 at 5:21