Project

General

Profile

Bug #8362

Nuevo instalador aparece con estilos faltantes

Added by Yannick Warnier over 3 years ago. Updated over 3 years ago.

Status:
Needs testing
Priority:
Normal
Category:
Installation / Migration
Target version:
-
Start date:
26/07/2016
Due date:
% Done:

50%

Estimated time:
Spent time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

Estoy instalando un Chamilo 1.11.x (con composer update hecho) encima de un Chamilo 1.9 y el instalador me aparece así:

Además, al ver la fuente, no es HTML válido:

Finalmente, en el último paso del instalador (confirmación) algo raro hace que no sea factible presionar el botón verde (como si hubiera una zona hover invisible encima). El botón para acceder a la documentación sí funciona, y cuando está en pantalla ancha también el botón verde funciona - solo se bloquea en vista menos ancha:

Luego hay más problemas, como esta lista que ya no aparece como pestañas, pero parece solucionarse con un CTRL+F5... depende de una librería JS con el mismo nombre que antes?


Files

instalador.png (33.7 KB) instalador.png Yannick Warnier, 26/07/2016 01:54
instalador-fuente.png (7.58 KB) instalador-fuente.png Yannick Warnier, 26/07/2016 01:54
instalador-fin.png (20.6 KB) instalador-fin.png Yannick Warnier, 26/07/2016 02:16
lista.png (20.9 KB) lista.png Yannick Warnier, 26/07/2016 02:20

Related issues

Related to Chamilo LMS - Bug #8393: Corregir estilos para mensajes de error en ChamiloBug resolved12/08/2016

Actions

Associated revisions

Revision 96dfd2ea (diff)
Added by jmontoyaa over 3 years ago

Fix url + css paths see #8362

History

#1

Updated by Yannick Warnier over 3 years ago

#2

Updated by Yannick Warnier over 3 years ago

  • Description updated (diff)
#3

Updated by Yannick Warnier over 3 years ago

#4

Updated by Yannick Warnier over 3 years ago

  • Description updated (diff)
#5

Updated by Julio Montoya over 3 years ago

+1

#6

Updated by Alex Aragon over 3 years ago

  • Status changed from Assigned to Needs testing
  • Assignee changed from Alex Aragon to Yannick Warnier

Corregido, pero el problema de los estilos del instalador es cuando esta fuera de una url donde no reconoce la ruta de los estilos.

#7

Updated by Yannick Warnier over 3 years ago

  • Status changed from Needs testing to Assigned
  • Assignee changed from Yannick Warnier to Julio Montoya

Para solucionarlo (el problema es que cuando estoy en main/admin/ intenta cargar los CSS desde web/assets/ y no se encuentran porque lo traduce en main/admin/web/assets/), habría que hacer que el global_error_message.inc.php haga un redirect (en caso del error de no instalado) en vez de mostrar el mensaje directamente ahí. Julio, que opinas?

#8

Updated by Yannick Warnier over 3 years ago

  • % Done changed from 0 to 50
#9

Updated by Yannick Warnier over 3 years ago

  • Target version deleted (1.11.0)

Esta tarea realmente no importa mucho. La estoy desasignando de la 1.11

#10

Updated by Julio Montoya over 3 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Julio Montoya to Yannick Warnier

Ahora funciona correctamente.

#11

Updated by Julio Montoya over 3 years ago

He encontrado un error cuando global_error_message.inc.php:

Cuando se ingresa a un portal ya instalado y se ingresa a /main/install/ no aparecen los estilos.
Ahora si aparecen. El error estaba en este commit:

https://github.com/chamilo/chamilo-lms/commit/ec843c12248fad15d36b6cb74dc26b6bf0a9e2e4

#12

Updated by Julio Montoya over 3 years ago

Voy ha eliminar fontawesome de global_error_message.inc.php, pues hay conflictos por los paths donde se encuentras los iconos. Y debido a que es una página de error, no creo que sea urgente de mostrar iconos.

Comentarios?

#13

Updated by Yannick Warnier over 3 years ago

  • Status changed from Needs more info to Needs testing

Asignando a Alex para que el verifique que no está rompiendo nada de otros lados.

Ver commit https://github.com/chamilo/chamilo-lms/commit/96dfd2eadcc7bad1272f3801c05037aa36bd4d73

#14

Updated by Yannick Warnier over 3 years ago

  • Assignee changed from Yannick Warnier to Alex Aragon
#15

Updated by Alex Aragon over 3 years ago

Se elimino el font awesome y esta tarea esta relacionada con la #8393

#16

Updated by Alex Aragon over 3 years ago

  • Assignee changed from Alex Aragon to Yannick Warnier
#17

Updated by Julio Montoya over 3 years ago

Bueno al parecer han elimnado mis cambios aqui:

https://github.com/chamilo/chamilo-lms/commit/ec843c12248fad15d36b6cb74dc26b6bf0a9e2e4#commitcomment-18641068

Los archivos css tienen que ser llamados usando "file_get_contents" de lo contrario cuando el archivo global_error_message.inc.php es usado en otros lugares en chamilo la ruta asi no funciona:

$css_def .= '<link rel="stylesheet" href="'.$css_base_chamilo_file.'">'.PHP_EOL;
#18

Updated by Julio Montoya over 3 years ago

Los cambios siguen ahi, falsa alarma :)

Also available in: Atom PDF