Skip to content

improve `LocalRepository` class:

Andrii Bezuglyi requested to merge feat/LocalRepository_maxDate into master

Improve the LocalRepository class in the following way

  • no need to pass rowCount
  • can pass '#maxDate' for "mi_deleteDate" attributes

This improvement intend to help avoid mistakes in code:

  • it is a common case when LocalRepository need to be used with entities, which might be safe deleted
  • developer tend to do mistakes, when they create maxDate values by hand: missing that months in the Date constructor are zero-based, put time in the value, while UB server uses value without time, etc.
  • also there is no point to pass rowCount, in absolutely majority (or even all?) of the cases it is the same as data.length

Merge request reports