Project

General

Profile

Feature #4694

Add feedback during installation

Added by Laurent Opprecht over 6 years ago. Updated almost 3 years ago.

Status:
New
Priority:
Normal
Category:
-
Target version:
Start date:
03/05/2012
Due date:
% Done:

90%

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

Description

Currently the installation process display the same screen with no information about process completion.
During lengthy upgrade the user don't know whether the upgrade crashes or is still running.

Would be nice to have a bit more feedback like course xxx started course xxx done, etc.

Another issue is that the logging is diffult to read.


Files

file_doctor.class.php (13.6 KB) file_doctor.class.php Laurent Opprecht, 30/05/2012 09:45
student_publication_doctor.class.php (7.42 KB) student_publication_doctor.class.php Laurent Opprecht, 22/06/2012 15:52

History

#1

Updated by Laurent Opprecht over 6 years ago

Replaces error_log by log::error and add a switch to echo that on the screen in verbose mode?

#2

Updated by Laurent Opprecht over 6 years ago

use monolog for handling logs? - this is the solution used by Symphony

#3

Updated by Laurent Opprecht over 6 years ago

  • Assignee set to Laurent Opprecht

Another issue is that information messages are sent to the error log through log_error which add the "error" string to information only messages.

#4

Updated by Laurent Opprecht over 6 years ago

In https://code.google.com/r/laurentopprecht-gva/ until I am done moving things in and out between test and dev.

#5

Updated by Laurent Opprecht over 6 years ago

Another issue is that the level of PHP reporting is to high - some errors are difficult to pinpoint because of that.
While this may only be a wish I am going to try to get rid of warnings.

#6

Updated by Yannick Warnier over 6 years ago

Laurent Opprecht wrote:

Another issue is that the level of PHP reporting is to high - some errors are difficult to pinpoint because of that.
While this may only be a wish I am going to try to get rid of warnings.

Actually we usually reach no-warnings, no-notice levels in stable versions. Quite a feat! It's alright, it's not only a wish, you'll get there quickly, you'll see. It is important to have default warning levels to show NOTICE as well, so we can fix those in time, we avoid memory or CPU leaks because of undefined variables and things like that.

#7

Updated by Yannick Warnier over 6 years ago

Laurent Opprecht wrote:

Another issue is that information messages are sent to the error log through log_error which add the "error" string to information only messages.

In my opinion this is not an issue. Error logs are sent to the error log because they are not useful to normal people, and they can only be of use to experts in Chamilo (they are about migrations, mostly, as a new installation is not generally causing an error after the checks we do on the databas configuration screen). What I will absolutely not accept is the level of detail that Chamilo LCMS or Moodle show to the user. This is frightening most people away (I have experienced that with a series of people).

#8

Updated by Laurent Opprecht over 6 years ago

I should have explained better, my mistake. The purpose is not to display the error log on the screen but to change the error log label when needed from

[error] some info

to

[notice] some info

This issue is linked too to the fact that there are a lot information in the error log. The usual solution is to filter out noice and look at the real errors. But with many information lines having an [error] tag it is difficult to find out what is a real errors.

#9

Updated by Yannick Warnier over 6 years ago

Laurent Opprecht wrote:

This issue is linked too to the fact that there are a lot information in the error log. The usual solution is to filter out noice and look at the real errors. But with many information lines having an [error] tag it is difficult to find out what is a real errors.

Oh, OK then. Just be careful that these error logs are not always managed by Apache, so avoiding making something too Apache-focused is good practice (I say that because I'm not sure how to send the messages to the default error log and giving them the proper error level). It is nice to go to these depths, but I would say don't worry too much about it if you don't need it. In the end, we're probably just 10 people around the world occasionally looking at these logs during the installation process. Your invaluable time would probably be much better spent on memory usage issues :-)

#10

Updated by Laurent Opprecht over 6 years ago

Yep, I am afraid I am part of the 10 people ;-) - I have started testing migration of our rather old Dokeos - 8.5 - so I am fixing issues I need on the go. Won't touch anything before 8.5 though.

#11

Updated by Yannick Warnier over 6 years ago

If you have the problem of moving from ISO to UTF-8 for documents names, let me know, I have a big script to help

#12

Updated by Laurent Opprecht over 6 years ago

  • % Done changed from 0 to 50
#13

Updated by Laurent Opprecht over 6 years ago

Indeed I have plenty of problems with iso/utf8 conversion. Script would be welcomed.

#14

Updated by Laurent Opprecht over 6 years ago

Script to correct lost pathes

#15

Updated by Yannick Warnier over 6 years ago

  • Target version set to 1.9.2
#16

Updated by Laurent Opprecht over 6 years ago

  • % Done changed from 50 to 90
#17

Updated by Laurent Opprecht over 6 years ago

Code to handle sub-directories in student publications. The code rename the folders and relink the files so

folder1/subfolder1

becomes

folder1_subfolder1
#18

Updated by Laurent Opprecht over 6 years ago

Adding Hubert in case he is interested by the code for correcting a few old migration issues.

#19

Updated by Laurent Opprecht over 6 years ago

Note install screen is not much better at the moment as several (migration of the stats) take quite a bit of time but login has been much improved.

#20

Updated by Yannick Warnier over 6 years ago

  • Target version changed from 1.9.2 to 2.0
#21

Updated by Yannick Warnier almost 3 years ago

  • Target version changed from 2.0 to 3.0

Also available in: Atom PDF