common-workflow-language / common-workflow-language/cwlviewer

Split database entities and response serialized objects (DTO)

Offen
#379 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Java
Sterne
52
Forks
29
Ø Merge
5 Std. 12 Min.
Gemergte PRs (30 T.)
6

Beschreibung

It's normally considered a bad practice to have the database model classes also being used to send data to the user in responses.

It's better - and IMHO easier to maintain - to have the database model classes with its single responsibility, and move all de/serialization specifics to DTO's (or any other acronym like VO/POJO/DO/etc used in the project).

## Description

Part of #254 . Realized I was having to deal with the MongoDB specifics and also with how data is de/serialized :disappointed_relieved: It would have been a bit simpler should these two were separated entities.

## Expected Behavior

Separate classes, separate concerns.

## Actual Behavior

## Possible Fix

## Steps to Reproduce

NA

## Context

## Your Environment

* Version used:
* Environment name and version (e.g. Chrome 39, Java 1.8):

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.