Actions
feature request #8503
closedCreate abstract base UI from RegistrationUI and use it for all UIs
Start date:
Due date:
% Done:
50%
Estimated time:
Severity:
normal
Description
The code which configures the error hanlder should also go into the abstract base class:
DelegatingErrorHandler delegatingErrorHander = new DelegatingErrorHandler();
WindowErrorHandler errorHandler = new WindowErrorHandler(this, RegistrationUIDefaults.ERROR_CONTACT_MESSAGE_LINE + "</br></br>"
+ "<i>To help analyzing the problem please describe your actions that lead to this error and provide the error details from below in your email. "
+ "You also might want to add a sreenshot of the browser page in error.</i>");
delegatingErrorHander.registerHandler(new ErrorTypeErrorHandlerWrapper<PermissionDeniedException>(PermissionDeniedException.class, new PermissionDeniedErrorHandler(this)));
delegatingErrorHander.registerHandler(new ErrorTypeErrorHandlerWrapper<Exception>(Exception.class, errorHandler));
setErrorHandler(delegatingErrorHander);
VaadinSession.getCurrent().setErrorHandler(delegatingErrorHander);
also other code part of the init method
To be implemented for
- DistributionStatusUI URGENT: this UI is already based on the UI interface only
- StatusEditorUI: still based on the AbstractAuthenticatedUI
- ConceptRelationshipUI: still based on the AbstractAuthenticatedUI
Additional
- drop AbstractAuthenticatedUI and CdmBaseUI once done
Related issues
Updated by Andreas Müller over 3 years ago
Especially it needs to be available for the distribution editor
Updated by Andreas Kohlbecker over 3 years ago
- Description updated (diff)
Especially it needs to be available for the distribution editor
agreed! This means it need to be done for the DistributionStatusUI first.
Updated by Andreas Kohlbecker over 1 year ago
- Status changed from New to Resolved
- % Done changed from 0 to 50
Applied in changeset cdm-vaadin|4f8aab68e114332f01013b52b7e4f1d17a6582e8.
Updated by Andreas Kohlbecker over 1 year ago
- Copied to feature request #9860: base all UIs on the AbstractUI added
Updated by Andreas Kohlbecker over 1 year ago
- Status changed from Resolved to Closed
Updated by Andreas Kohlbecker over 1 year ago
migration of existing UIs:
- DistributionStatusUI URGENT: this UI is already based on the UI interface only
- StatusEditorUI: still based on the AbstractAuthenticatedUI
- ConceptRelationshipUI: still based on the AbstractAuthenticatedUI
moved to new ticket
Updated by Andreas Kohlbecker over 1 year ago
- Target version changed from Unassigned CDM tickets to Release 5.38
Updated by Andreas Müller about 1 year ago
- Target version changed from Release 5.38 to Release 5.29
Actions