Bug 8100 - Error handling on some pages hides all content
Summary: Error handling on some pages hides all content
Status: NEW
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Web Administration (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: LowPrio
Assignee: Bugzilla mail exporter
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-02-17 10:54 CET by Linn
Modified: 2023-04-18 13:25 CEST (History)
0 users

See Also:
Acceptance Criteria:


Attachments

Description Linn cendio 2023-02-17 10:54:19 CET
In the template for pages Load and Sessions in Web Admin, nothing on the page is shown if an error occurs (except for the error message itself).

With the introduction of the popup in bug 8075, we want to have the option to show some error messages in the popup. To achieve this, the error handling for pages constructed like this have to change.

One issue is that it is unclear why this restrictive error handling was done, and if errors could actually break things on the page or not.
Comment 2 Linn cendio 2023-03-16 12:57:49 CET
When working on the sessions page as part of bug 8083, errors for trying to terminate a session without having the checkbox checked are now shown in the popup. These "termination errors" are shown by adding a red text to the checkbox label:

* Label when no error:
   Yes, really terminate session 

* Label when error:
   Yes, really terminate session (* Required)

This is the same behaviour as "delete errors", seen when e.g. trying to delete a profile without checking the checkbox.


On the session page, all other errors occur either when retrieving session data or when we received an invalid display number. In both cases, we can't present the session data properly, and it makes sense to keep the current behaviour of only showing the error message and nothing else for these cases.

Note You need to log in before you can comment on or make changes to this bug.