Project

General

Profile

Actions
Copy link

feature request #8688

closed

Make OriginalSourceBase.cdmSource visible in TaxEditor

Added by Andreas Müller over 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
New
Assignee:
Katja Luther
Category:
taxeditor
Target version:
Release 5.13
Start date:
Due date:
% Done:

100%

Estimated time:
Severity:
major
Tags:
euro+med transmission-engine-distribution additivity

Description

The field OriginalSourceBase.cdmSource is usually filled during aggregations (for now). Therefore it does not need to be editable but the result should be visible to be able to distinguish and recognize sources.

Possible open questions related: do we want to have a new subclass for OriginalSourceBase which only handles CdmLink-Sources. This subclass does not need to show all other fields (citation, id, namespace as these are probably never used). Or can we distinguish which fields to show simply by the result of (osb.cdmSource == null)? => moved to: #8778

Related issues

Related to EDIT - feature request #8686: Support CdmLink sources in bibliographyClosedAndreas Kohlbecker

Actions
Related to EDIT - bug #8687: Description page must show all DescriptionElement subclassesNewAndreas Müller

Actions
Related to EDIT - feature request #8724: Allow OriginalSource to link to TaxonClosedAndreas Müller

Actions
Related to EDIT - feature request #8778: Have explicit subclass for CdmSourceLink sourcesNewAndreas Müller

Actions
Copied to EDIT - feature request #8777: Make visiblity of CDM Source configurableResolvedKatja Luther

Actions
Actions
Copy link
 #1

Updated by Andreas Müller over 4 years ago

  • Target version changed from Unassigned CDM tickets to Release 5.12

Please decide who could be assignee. It would be nice to have it soon.

Actions
Copy link
 #2

Updated by Andreas Müller over 4 years ago

Actions
Copy link
 #3

Updated by Andreas Müller over 4 years ago

  • Related to bug #8687: Description page must show all DescriptionElement subclasses added
Actions
Copy link
 #4

Updated by Patrick Plitzner over 4 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50
Actions
Copy link
 #5

Updated by Patrick Plitzner over 4 years ago

  • Assignee changed from Patrick Plitzner to Andreas Müller

Current implementation shows the titleCache if it is an IdentifiableEntity. It shows the UUID if it is a CdmBase. Otherwise it calls object.toString().

Actions
Copy link
 #6

Updated by Andreas Müller over 4 years ago

Actions
Copy link
 #7

Updated by Andreas Müller over 4 years ago

Actions
Copy link
 #8

Updated by Andreas Müller over 4 years ago

  • Description updated (diff)
Actions
Copy link
 #9

Updated by Andreas Müller over 4 years ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Katja Luther

The current solution shows titleCache+uuid for identifiable entities. This is too much. For now we need only the titleCache. Please remove uuid (it is needed only for non-identifiable entities.

Also there seems to be an extra empty line at the end which is not needed.

Actions
Copy link
 #10

Updated by Andreas Müller over 4 years ago

Actions
Copy link
 #11

Updated by Katja Luther over 4 years ago

  • Target version changed from Release 5.12 to Release 5.13
Actions
Copy link
 #12

Updated by Katja Luther about 4 years ago

  • Status changed from Feedback to Resolved
  • Assignee changed from Katja Luther to Andreas Müller

now only titleCache (for identifiable entities) or uuid (for cdmBase objects) is shown

Actions
Copy link
 #13

Updated by Andreas Müller about 4 years ago

  • Status changed from Resolved to Closed
  • Assignee changed from Andreas Müller to Katja Luther
  • % Done changed from 50 to 100

I can't test for CdmBase as there is no class implementing the ICdmSource interface yet which is not identifiable.

I tested for Taxon and it worked as expected.

Actions
Copy link

Also available in: Atom PDF