Project

General

Profile

feature request #8656

[DISCUSS] Should only characters be allowed for DescriptiveDataSets/Character Matrix?

Added by Patrick Plitzner 9 months ago. Updated 7 months ago.

Status:
Closed
Priority:
Priority14
Category:
taxeditor
Target version:
Start date:
11/07/2019
Due date:
% Done:

0%

Severity:
normal
Tags:

Description

du filterst in aggregateDescription() nur DescriptionElements die Characters als Feature haben. Das

 description.getElements().stream()

 //filter out elements that do not have a Character as Feature

.filter(element->HibernateProxyHelper.isInstanceOf(element.getFeature(), Character.class))

.forEach(ele->addCharacterToMap(featureToElementMap, ele));

Das verstehe ich nicht ganz. Grundsätzlich können normale Features doch auch aggregiert werden. Dass sie als Characters mit Sturcture und Property erzeugt wurden oder nicht ist doch glaube ich nicht zwingend. Was willst du damit verhindern?

Mir kommt es eher so vor, als ob wir checken sollten, ob das Feature/Character im DescriptiveDataSet.descriptiveSystem enthalten ist UND dass es entweder Categorical oder Quantitativ ist. Habe das jetzt aber nicht bis zuletzt gecheckt.

auch hier verstehe ich nicht ganz, warum es unbedingt Characters sein müssen. Features tuns doch auch. Hauptsache sie sind im FeatureTree.

Übrigens, nach dieser Meldung schließt der MatrixEditor sich nicht sondern zeigt dauerhaft „Loading matrix…“, was irritierend ist, da man nicht weiß, ob das nur eine Warnung ist und er wirklich noch läd, oder ob er keine Funktion mehr hat.

picture188-1.png View (6.76 KB) Patrick Plitzner, 11/07/2019 12:13 PM

Associated revisions

Revision 390ef5ac (diff)
Added by Patrick Plitzner 9 months ago

ref #8656 Update label if matrix loadiung failed

Revision 3a88b57a (diff)
Added by Patrick Plitzner 9 months ago

ref #8656 i18n for loading label

History

#1 Updated by Andreas Müller 9 months ago

  • Target version changed from 287 to Release 5.12

#2 Updated by Patrick Plitzner 9 months ago

  • Target version changed from Release 5.12 to 287

#3 Updated by Andreas Müller 8 months ago

  • Status changed from New to Resolved

#4 Updated by Andreas Müller 8 months ago

  • Target version changed from 287 to Release 5.12

#5 Updated by Andreas Müller 8 months ago

  • Assignee changed from Patrick Plitzner to Andreas Müller

#6 Updated by Andreas Müller 7 months ago

  • Status changed from Resolved to Closed
  • Priority changed from New to Priority14

This is fixed with the new implementation.

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)