Project

General

Profile

Bug #3285

Install procedure

Added by Marko Kastelic almost 9 years ago. Updated over 7 years ago.

Status:
Bug resolved
Priority:
Normal
Category:
-
Target version:
Start date:
20/04/2011
Due date:
% Done:

90%

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

Description

install procedure does not care about overriding existing DB; if you try to set new install with existing target DB, there is no warning, no droping tables/DBs, in the case of previous chamilo DB, new content is appended to the existing tables. So at least course home page look very interesting : all the course tools are doubled, ...


Files

four.png (12.1 KB) four.png Marko Kastelic, 13/06/2012 09:20
six.png (9.12 KB) six.png Marko Kastelic, 13/06/2012 09:20
six_seven.png (33.8 KB) six_seven.png Marko Kastelic, 13/06/2012 09:20
step_7.png (44.2 KB) step_7.png Marko Kastelic, 13/06/2012 10:18

Associated revisions

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

Minor install UI change see #3285

Revision e710e521 (diff)
Added by Yannick Warnier over 7 years ago

Improved messaging in install process by putting database-already-exists message as warning - refs #3285

Revision c82126a1 (diff)
Added by Yannick Warnier over 7 years ago

Improved installation process a little showing current step and please wait message - refs #3285

History

#1

Updated by Yannick Warnier over 7 years ago

I believe this is fixed now and there is a warning saying that it will drop the already-existing databases (at the bottom of page 6 if I remember well).

We should check whether we actually remove the previous databases or not.

#2

Updated by Yannick Warnier over 7 years ago

  • Target version set to 1.9 Beta
#3

Updated by Julio Montoya over 7 years ago

  • Status changed from New to Needs more info

I think it removes the tables right now. requires validation ...

#4

Updated by Julio Montoya over 7 years ago

  • % Done changed from 0 to 50
#5

Updated by Yannick Warnier over 7 years ago

Marko, could you please have a look and let us know if you're alright with the patch?

#6

Updated by Marko Kastelic over 7 years ago

i tried several installations and everything go fine. All procedures are working just as expected.

here are some observations and sugestions:

1) [=CourseTestWasCreated=] variable is missing
2) in step 4 suggested db name is checked against existing. good. but changed name is checked only if the button for checking is pressed. This could be done using onkeypress event on dbname input field without check button.
3) in step 4: message 'Database will be created' or 'A database with the same name already exist' is quite invisible. Should be on red backgroud or yellow as is in step 6.

although it is contained in step 6, i'd like to see additional message about the tables will be erased when the database already exist (no harm doubling)

4)transition between step 6 and step 7 is a bit odd. Creating (and populating) tables can take 30 seconds and more, and there is nothing on the screen to tell what's going on. We should add message : installing, please wait or progress bar or something ..

#7

Updated by Marko Kastelic over 7 years ago

almost forgot:
a little ui change : Step 7 should advance to heading (before Chamilo installation: look toc at the left)

#8

Updated by Yannick Warnier over 7 years ago

  • Target version changed from 1.9 Beta to 1.9 RC1

I strongly agree with all the suggestions, but it can be moved out of the way for the beta (in the end, these are non-structural improvements).

#9

Updated by Yannick Warnier over 7 years ago

  • Assignee set to Marko Kastelic
  • % Done changed from 50 to 90

I think you should be happy now, Marko. I haven't added the animated gif (I didn't do that for a long time) but I added a straightforward "Please wait" message that should cover it for now.

I've also moved and updated the title and update the database message context (transformed into warning). If you approve it, that's one task less in the way to 1.9.

#10

Updated by Marko Kastelic over 7 years ago

  • Assignee changed from Marko Kastelic to Yannick Warnier

just tested. 'please wait' is quite enough, maybe in the future, there will be something Chamilo Conn. does :
creating database blabal ... done
creating tabel blabla11 ... done

before i noticed this demand, i opened another issue for strange back buttons http://support.chamilo.org/issues/5243

so i'm leaving you to close this on.

#11

Updated by Yannick Warnier over 7 years ago

  • Status changed from Needs more info to Bug resolved

The "Creating database..." stuff is anti-usability. We tried it. The average technical level of people installing Chamilo doesn't justify putting them in front of what the system actually does. We could put a small "extensible" window with details (like in Ubuntu packages installator) but that's another story (and another issue, indeed :p)

Also available in: Atom PDF