Project

General

Profile

task #9280

Refactor: IKeyTerm.getMessage() --> IKeyTerm.getLabel()

Added by Andreas Kohlbecker 7 months ago. Updated 7 months ago.

Status:
Closed
Priority:
Highest
Category:
cdmlib
Target version:
Start date:
11/05/2020
Due date:
% Done:

100%

Severity:
normal

Description

The property name IKeyTerm.message is misleading and confusing it should be renamed to label


From email communication:

Hallo,

ich finde IKeyTerm.getMessage() immer wieder irritierend. Warum denn Message hier handelt es sich doch eindeutig um ein Label, das seht so auch inm Java-doc.

Wäre schön wir könnten das mal ändern.

Viele Grüße
Andreas

...

Ja, da hast du vermutlich Recht. Ich weiß ehrlich gesagt nicht mehr, warum ich das damals Message genannt habe.
Von mir aus können wir das ändern.

AM

Associated revisions

Revision 18620ac6 (diff)
Added by Andreas Müller 7 months ago

ref #9280 rename IKeyTerm.getMessage() => .getLabel()

Revision 340adc01 (diff)
Added by Andreas Müller 7 months ago

ref #9280 adapt appimport to new Term.getLabel()

Revision 137d0ad0 (diff)
Added by Andreas Müller 7 months ago

ref #9280 adapt TaxEditor to new Term.getLabel()

Revision 2a7ad236 (diff)
Added by Andreas Müller 7 months ago

ref #9280 adapt TaxEditor to new Term.getLabel() (cont.)

Revision cc7051ed (diff)
Added by Andreas Müller 7 months ago

ref #9280 adapt TaxEditor to new Term.getLabel() (cont. 2)

History

#1 Updated by Andreas Kohlbecker 7 months ago

  • Description updated (diff)

#2 Updated by Andreas Müller 7 months ago

  • Priority changed from New to Highest
  • Target version changed from Unassigned CDM tickets to Release 5.18

#3 Updated by Andreas Müller 7 months ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

This has been fixed. As all code compiles now I think review is not necessary. Please review if you think it is needed.

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)