Project

General

Profile

bug #7250

add null checks to TeamDefaultCacheStrategy

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

Status:
Closed
Priority:
New
Category:
cdmlib
Target version:
Start date:
02/06/2018
Due date:
% Done:

50%

Severity:
normal
Found in Version:
Tags:

Description

team members are not being null-checked.

In UIs members can be null at a point in the application life cycle when the cache needs to be generated (e.g.: updating ui element labels). Thses Situations are sometimes determined by the UI framework and thus out of the control of the application developer, so we must handle null values in the cache

Associated revisions

Revision 2ab12e51 (diff)
Added by Andreas Kohlbecker 11 months ago

null check in TeamDefaultCacheStrategy needed sometimes in UIs

Revision 02f58c70 (diff)
Added by Andreas Kohlbecker 10 months ago

fix #7250 adding remaining null checks to TeamDefaultCacheStrategy

History

#1 Updated by Andreas Kohlbecker 10 months ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50

#2 Updated by Andreas Kohlbecker 10 months ago

Andreas Müller, please review the code changes are correct. I was a bit unclear about the role of paramter i in concatString(Team team, List<Person> teamMembers, int i)

#3 Updated by Andreas Müller 10 months ago

  • Status changed from Resolved to Closed
  • Assignee changed from Andreas Müller to Andreas Kohlbecker

looks ok

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)