Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#6675] - Handle DB errors in stats_admin module

? Success

User tests: Successful: Unsuccessful:

avatar alikon
alikon
6 Apr 2015

updated to handle DB errors like #6612 ,#6632, #6665, #6666 and for #6591

Testing info
simulate a db error

avatar alikon alikon - open - 6 Apr 2015
avatar joomla-cms-bot joomla-cms-bot - change - 6 Apr 2015
Labels Added: ?
avatar alikon alikon - reference | - 6 Apr 15
avatar zero-24 zero-24 - change - 6 Apr 2015
Category ⇒ Modules SQL
avatar bertmert
bertmert - comment - 7 Apr 2015

@test Success but why no message here?

Control panel where statistics module is shown.
Before patch.
Incomplete page.
statistics-module1

After patch
Just statistics module incomplete.
statistics-module2

@alikon But why in this PR no messages to the user like this?
statistics-module3

Like in PR #6676

avatar alikon
alikon - comment - 7 Apr 2015

i've just considered that is only an informative module so don't need to bother the user with a msg error
but we can change this as desidered

avatar alikon alikon - reference | - 7 Apr 15
avatar alikon alikon - reference | - 8 Apr 15
avatar alikon alikon - reference | - 8 Apr 15
avatar alikon alikon - reference | - 8 Apr 15
avatar alikon alikon - reference | - 8 Apr 15
avatar alikon alikon - reference | - 8 Apr 15
avatar nonumber
nonumber - comment - 10 Apr 2015

Not sure if this should get merged with regards to discussion on #6710

avatar Bakual Bakual - change - 16 Jun 2015
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2015-06-16 11:32:00
Closed_By ⇒ Bakual
avatar Bakual Bakual - close - 16 Jun 2015

Add a Comment

Login with GitHub to post a comment