DeepDiff icon indicating copy to clipboard operation
DeepDiff copied to clipboard

Change<T> could use a .map(...) method.

Open jrogccom opened this issue 5 years ago • 1 comments

I made a PR to this end. It can be useful when looking at changes while hiding private implementation. See https://github.com/onmyway133/DeepDiff/pull/33 .

jrogccom avatar Mar 05 '19 17:03 jrogccom

+1 to this Could you apply this pull request? I can't even make it locally in my project cause Insert<T>, Delete<T>, Replace<T> and Move<T> don't have public initializers😌

Beside that reload<T: DiffAware>(changes: ..)doesn't need to require T to be DiffAware. Maybe I'm missing something, but we don't need to store items inside Insert<T>, Delete<T>, Replace<T> and Move<T>, as we only calculate indexes from those.

Why do I need it? I have a data model, that is : DiffAware, but my table wrapper object that configures the cells can't be made : DiffAware.

PhilipDukhov avatar Jan 28 '20 13:01 PhilipDukhov