Opened 5 years ago

Closed 4 years ago

#1901 closed defect (done)

CollectionItemChangedEventArgs use null to indicate that no oldItems were present

Reported by: mkommend Owned by: mkommend
Priority: medium Milestone: HeuristicLab 3.3.8
Component: Collections Version: 3.3.8
Keywords: Cc:

Description

As the oldItems are saved as an IEnumerable<T> one would not expect that this field can be null. It is better to use Enumerable.Empty<T>() to indicate the absence of oldItems.

Change History (5)

comment:1 Changed 5 years ago by mkommend

  • Status changed from new to accepted

comment:2 Changed 5 years ago by mkommend

  • Owner changed from mkommend to swagner
  • Status changed from accepted to reviewing

r8317: Changed CollectionItemChangedEventArgs to use empty enumerables instead of null to indicate the absence of items.

comment:3 Changed 4 years ago by mkommend

  • Owner changed from swagner to ascheibe

comment:4 Changed 4 years ago by ascheibe

  • Owner changed from ascheibe to mkommend
  • Status changed from reviewing to readytorelease

Thanks!

comment:5 Changed 4 years ago by swagner

  • Resolution set to done
  • Status changed from readytorelease to closed
  • Version changed from 3.3.7 to 3.3.8
Note: See TracTickets for help on using tickets.