Project

General

Profile

Bug #4219

chamilo's installation

Added by Juan Carlos Raña Trabado over 7 years ago. Updated over 7 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
11/01/2012
Due date:
% Done:

70%

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

Description

Hay unos comportamientos inadecuados en la instalación de Chamilo que podría considerarse como un bug ya que anteriormente estas cuestiones estaban resueltas.
1. Cuando se instala Chamilo como nueva instalación, si ya existe anteriormente una base de datos llamada chamilo, en lugar de sobreescribirla no deja seguir adelante y aparenta no poder crear la nueva.
2. Cuando se instala Chamilo como nueva instalación y no se introduce el password correcto de acceso a la base de datos, Chamilo no impide seguir con los siguientes pasos de instalación, con lo que el usuario piensa que este paso está bien pero al final algo no funcionará. Si se hace la comprobación de conexión con la base de datos mediante el botón sí nos da esta información, pero lo suyo es que si esto no es correcto que nos informe de ello en el mismo momento de querer seguir adelante en la instalación.


Files

Captura_curso_strict_standards.PNG (159 KB) Captura_curso_strict_standards.PNG Juan Carlos Raña Trabado, 28/01/2012 19:36

Associated revisions

Revision 90879512 (diff)
Added by Julio Montoya over 7 years ago

Should fix bug during the db check in the installation see #4219

Revision 670fbb9a (diff)
Added by Julio Montoya over 7 years ago

Improving installation DB errors messages see #4219

Revision 1b5bf9ec (diff)
Added by Julio Montoya over 7 years ago

Fixing "Non static method should not be called statically" bug see #4219

History

#1

Updated by Julio Montoya over 7 years ago

  • Status changed from New to Assigned
  • Assignee set to Julio Montoya
#2

Updated by Julio Montoya over 7 years ago

  • Status changed from Assigned to Needs more info
  • Assignee deleted (Julio Montoya)
  • % Done changed from 0 to 50

1. creo que esta bien que no deje instalar si ya existe una base de datos con el mismo nombre ... aunque esto implicaría problemas cuando uno tiene un hosting compartido en el que hay que crear la BD primero. Bien visto debería estar arreglado ahora

2. he agregado un disabled en el button de next por lo menos esto impedirá una serie de problemas

ver :

http://code.google.com/p/chamilo/source/detail?r=a561ce1a8f1740065890452c206d0442b3777b9a&repo=classic

#3

Updated by Juan Carlos Raña Trabado over 7 years ago

Lo suyo sería que presentara un mensaje diciendo que ya está creada una base de datos con el mismo nombre y avisara que si se continúa se borrarán todos los datos antiguos. El deshabilitar el botón no le da información al usuario de qué está ocurriendo.
Por otro lado, con los cambios ahora si en la fase en que se me deshabilita el botón pulso el botón comprobar la conexión con la base de datos, entonces el botón deshabilitado deja de estarlo y ahora sí que puedo seguir adelante como se supone que sé lo que estoy haciendo la base de datos antigua debe machacarse con la nueva, pero no estaría en absoluto de más para usuarios menos experimentados que apareciera previamente un mensaje avisando de que esto ocurrirá.
En suma, mejor que deshabilitar el botón presentar un mensaje con un texto parecido a

"En el servidor ya existe una base de datos con este nombre, si sigue adelante todos los datos que contenga serán borrados. Si no desea que esto ocurra, cancele la instalación, saque una copia de seguridad de sus datos y en lugar de hacer una nueva instalación realice una actualización de Chamilo"

#4

Updated by Juan Carlos Raña Trabado over 7 years ago

Bueno parece que realmente no borra todo, he intentado crear un curso y me dice que ya existe uno con el mismo código. Esto en una instalación nueva no debería de suceder.

#5

Updated by Julio Montoya over 7 years ago

he agregado algunos mensajes cuando para validar si el usuario tiene acceso a la base de datos.

Tienes razón en una instalación nueva debería de limpiar la base de datos drop y create pero esto depende si el usuario tiene permisos de creación de BD, otra salida sería de eliminar cada tabla o agregar algo como "drop if exists create table"

#6

Updated by Julio Montoya over 7 years ago

  • Target version set to 1.9 Stable
  • % Done changed from 50 to 70
#7

Updated by Juan Carlos Raña Trabado over 7 years ago

In a new installation, when I have finished the last step, the platform give this error

Strict Standards: Non-static method HTML_QuickForm::registerElementType() should not be called statically in C:\xampp\htdocs\chamilo190\main\inc\lib\pear\HTML\QuickForm\advmultiselect.php on line 1185

#8

Updated by Juan Carlos Raña Trabado over 7 years ago

also in course home page, see attached clip

--

xampp over Windows 7

#9

Updated by Juan Carlos Raña Trabado over 7 years ago

Related Bug #3915

#10

Updated by Yannick Warnier over 7 years ago

Juan Carlos Raña wrote:

In a new installation, when I have finished the last step, the platform give this error

Strict Standards: Non-static method HTML_QuickForm::registerElementType() should not be called statically in C:\xampp\htdocs\chamilo190\main\inc\lib\pear\HTML\QuickForm\advmultiselect.php on line 1185

This is because we don't support the "Strict" message level in the php.ini's error_reporting variable (should be E_ALL & ~E_STRICT or something like that). Strict errors are really hard to deal with because in this case, for example, it's something wrong inside a PEAR library, not inside the Chamilo code.

#11

Updated by Julio Montoya over 7 years ago

he cambiado mis settings a

error_reporting = E_ALL | E_STRICT

y he visto algunos errores

he corregido y silenciado el error en pear con "@"

ver:

http://code.google.com/p/chamilo/source/detail?r=7bf6b961036d5e334f1d836105a3f9e0c5fc2504&repo=classic

#12

Updated by Julio Montoya over 7 years ago

  • Status changed from Needs more info to Bug resolved

Also available in: Atom PDF