Skip to content

refactor: DSRecordHistory >> #records -> #events (with accessors)#52

Merged
Julesboul merged 1 commit intoPharo-XP-Tools:P14from
nicolasp025:refactor/history-records-to-events
Apr 23, 2026
Merged

refactor: DSRecordHistory >> #records -> #events (with accessors)#52
Julesboul merged 1 commit intoPharo-XP-Tools:P14from
nicolasp025:refactor/history-records-to-events

Conversation

@nicolasp025
Copy link
Copy Markdown

@nicolasp025 nicolasp025 commented Apr 22, 2026

I propose to refactor this as some other classes use the name 'events' for their attributes so it would be uniformed.

@nicolasp025 nicolasp025 requested a review from Julesboul April 22, 2026 12:59
@nicolasp025 nicolasp025 marked this pull request as ready for review April 22, 2026 12:59
@Julesboul
Copy link
Copy Markdown
Contributor

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 records also).

In the meantime, is there a need of uniformity between DSRecordHistory, DSWindowRecord and DSWindowActivityRecord?

@nicolasp025
Copy link
Copy Markdown
Author

nicolasp025 commented Apr 23, 2026

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.
It could also be 'records' instead, I would just want it to be unified for history, windows and jumps as possible

@Julesboul
Copy link
Copy Markdown
Contributor

I'm not sure to understand, when do we have multiple histories displayed in your presenter?
I though that there is one history displayed (linked to the selected file).

@nicolasp025
Copy link
Copy Markdown
Author

We have only one history displayed of course, but we have 3 presenters that displays records :

  • the records tab (that comes from history)
  • the selected window's records in windows tab
  • the selected jump's records in the activity tab

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'

@Julesboul Julesboul merged commit 3b11c98 into Pharo-XP-Tools:P14 Apr 23, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants