Project

General

Profile

Actions

feature request #8503

closed

Create abstract base UI from RegistrationUI and use it for all UIs

Added by Andreas Kohlbecker over 4 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
New
Category:
cdm-vaadin
Target version:
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

Copied to EDIT - feature request #9860: base all UIs on the AbstractUI NewAndreas Kohlbecker

Actions
Actions #1

Updated by Andreas Müller over 4 years ago

Especially it needs to be available for the distribution editor

Actions #2

Updated by Andreas Kohlbecker over 4 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.

Actions #3

Updated by Andreas Kohlbecker over 2 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50
Actions #4

Updated by Andreas Kohlbecker over 2 years ago

Actions #5

Updated by Andreas Kohlbecker over 2 years ago

  • Status changed from Resolved to Closed
Actions #6

Updated by Andreas Kohlbecker over 2 years 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

Actions #7

Updated by Andreas Kohlbecker over 2 years ago

  • Target version changed from Unassigned CDM tickets to Release 5.45
Actions #8

Updated by Andreas Müller about 2 years ago

  • Target version changed from Release 5.45 to Release 5.29
Actions

Also available in: Atom PDF