Skip to content

Do not prefix unsafe order by expressions with table prefix. #1513

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.

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

Closed
wants to merge 2 commits into from

Conversation

schauder
Copy link
Contributor

@schauder schauder commented May 12, 2023

Such expressions now get passed on unchanged.

This also deprecates org.springframework.data.r2dbc.query.QueryMapper.getMappedObject(Sort, RelationalPersistentEntity<?>).
It was only used in tests and translates an Order into another Order, which sounds wrong.

Closes #1512

schauder added 2 commits May 12, 2023 15:42
Such expressions now get passed on unchanged.

This also deprecates org.springframework.data.r2dbc.query.QueryMapper.getMappedObject(Sort, RelationalPersistentEntity<?>).
It was only used in tests and translates an Order into another Order, which sounds wrong.

Closes #1512
@schauder schauder requested a review from mp911de May 12, 2023 14:15
@mp911de mp911de added the type: bug A general bug label May 17, 2023
mp911de pushed a commit that referenced this pull request May 17, 2023
Such expressions now get passed on unchanged.

This also deprecates org.springframework.data.r2dbc.query.QueryMapper.getMappedObject(Sort, RelationalPersistentEntity<?>).
It was only used in tests and translates an Order into another Order, which sounds wrong.

Closes #1512
Original pull request: #1513
mp911de added a commit that referenced this pull request May 17, 2023
Reformat code.

See #1512
Original pull request: #1513
mp911de pushed a commit that referenced this pull request May 17, 2023
Such expressions now get passed on unchanged.

This also deprecates org.springframework.data.r2dbc.query.QueryMapper.getMappedObject(Sort, RelationalPersistentEntity<?>).
It was only used in tests and translates an Order into another Order, which sounds wrong.

Closes #1512
Original pull request: #1513
@mp911de mp911de added this to the 3.1.1 (2023.0.1) milestone May 17, 2023
mp911de added a commit that referenced this pull request May 17, 2023
Reformat code.

See #1512
Original pull request: #1513
@mp911de mp911de closed this May 17, 2023
@mp911de mp911de deleted the issue/1512-drop-table-prefix-for-unsafe branch May 17, 2023 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unsafe SqlOrder should not be prefixed by table name.
2 participants