Project

General

Profile

Actions
Copy link

Bug #2709

closed

The database management page doesn't nicely handle SQL error

Added by Nicolas CHARLES over 11 years ago. Updated over 11 years ago.

Status:
Released
Priority:
1
Assignee:
Nicolas CHARLES
Category:
Web - Maintenance
Target version:
2.4.0~beta3
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

When something goes bad with the database, we have a not nice blank page saying something bad happened

Actions
Copy link
 #1

Updated by Jonathan CLARKE over 11 years ago

  • Subject changed from The database management doesn't nicely handle SQL error to The database management page doesn't nicely handle SQL error
Actions
Copy link
 #2

Updated by Nicolas CHARLES over 11 years ago

  • Status changed from New to Pending technical review
  • % Done changed from 0 to 100
Actions
Copy link
 #3

Updated by François ARMAND over 11 years ago

  • Status changed from Pending technical review to Released

I think DatabaseManagerImpl is not the good place to put try/catch code. That should be part of the contract of ReportRepository, it is that last class that need to handle problem with the outer world with Box.

So, I will let that one as it is for now, because changing ReportRepository methods is rather involving, but I will trace that requirement in ticket: #2777

Actions
Copy link

Also available in: Atom PDF