Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Simplify EntityRowMapper #1974

Closed
schauder opened this issue Jan 8, 2025 · 0 comments
Closed

Simplify EntityRowMapper #1974

schauder opened this issue Jan 8, 2025 · 0 comments
Assignees
@schauder
Milestone
3.3.8 (2024.0.8)

Comments

@schauder
Copy link
Contributor

schauder commented Jan 8, 2025

The current version has an unused field and repeats operations, that can be moved into the constructor.
@schauder schauder self-assigned this Jan 8, 2025
@schauder schauder added this to the 3.3.8 (2024.0.8) milestone Jan 8, 2025
schauder added a commit that referenced this issue Jan 8, 2025
@schauder
Simplified EntityRowMapper.
8d4b052 
Removes an unused field and moves operations into  constructor, to make the mapRow method simpler.

Closes #1974
schauder added a commit that referenced this issue Jan 8, 2025
@schauder
Simplified EntityRowMapper.
496f151 
Removes an unused field and moves operations into  constructor, to make the mapRow method simpler.

Closes #1974
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@schauder schauder

Labels
None yet
Projects
None yet
Milestone
3.3.8 (2024.0.8)
Development

No branches or pull requests
1 participant
@schauder