Project

General

Profile

Actions

bug #6842

closed

NPE in Registration constructor when name is null

Added by Andreas Kohlbecker almost 7 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
New
Category:
cdmlib
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Severity:
normal
Found in Version:

Related issues

Related to EDIT - task #6258: Add Registration to cdm modelClosedAndreas Kohlbecker

Actions
Actions #1

Updated by Andreas Kohlbecker almost 7 years ago

  • Related to task #6258: Add Registration to cdm model added
Actions #2

Updated by Andreas Kohlbecker almost 7 years ago

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

Updated by Andreas Kohlbecker almost 7 years ago

  • Assignee changed from Andreas Kohlbecker to Andreas Müller
Actions #4

Updated by Andreas Müller almost 7 years ago

hmm, should we better fix the setName() method instead the NewInstance method?

Actions #5

Updated by Andreas Kohlbecker almost 7 years ago

agreed and fixed

Actions #6

Updated by Andreas Müller almost 7 years ago

  • Assignee changed from Andreas Müller to Andreas Kohlbecker

Looks like there were still some bugs in setName().

  • if name was null there was still an NPE in the last line
  • existing registration was removed from name.getRegistrations but should be removed from this.name.getRegistrations()
  • maybe some more

I adapted the method and wrote test to proof correct behavior.

Please review.

Actions #7

Updated by Andreas Kohlbecker almost 7 years ago

  • Status changed from Resolved to Closed
  • % Done changed from 50 to 100
Actions

Also available in: Atom PDF