refactor: DSRecordHistory >> #records -> #events (with accessors)#52
Conversation
|
Why not, but I don't know if there is a reason behind this difference... We should ask Steven before changing this name attribute (cause we can argue for having In the meantime, is there a need of uniformity between |
|
Yes, I would need them to correspond for a refactoring about the records presenters, the idea would be that every item used for the presenter's roots would define an 'events' method that would return the events, that is already the case for the windows and jumps, and I would like it to be the same for the history. |
|
I'm not sure to understand, when do we have multiple histories displayed in your presenter? |
|
We have only one history displayed of course, but we have 3 presenters that displays records :
The idea would be to generalize a table presenter that displays records and takes a parameter that defines a method that returns the events, which could be 'events' or 'records' |
I propose to refactor this as some other classes use the name 'events' for their attributes so it would be uniformed.