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.

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

[stable29] fix: Remove bogus code from query builder and fix parameter name #45103

Closed
wants to merge 1 commit into from

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Apr 30, 2024

Backport of #44533

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@come-nc come-nc changed the title [stable29] build(deps): bump vimeo/psalm from 5.9.0 to 5.23.1 in /vendor-bin/psalm [stable29] fix: Remove bogus code from query builder and fix parameter name Apr 30, 2024
@backportbot backportbot bot force-pushed the backport/44533/stable29 branch from 7c5c834 to 762c377 Compare April 30, 2024 08:43
This was broken by last codestyle update, maybe even before because of
 operator precedence. Now using vars to make it readable and correct at
 the same time.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@backportbot backportbot bot force-pushed the backport/44533/stable29 branch from 762c377 to 2733aa1 Compare April 30, 2024 08:45
@come-nc come-nc enabled auto-merge April 30, 2024 08:45
@come-nc come-nc disabled auto-merge April 30, 2024 08:45
@come-nc
Copy link
Contributor

come-nc commented Apr 30, 2024

Broke this by trying to backport another commit, the backport bot uses the same branch name -_-

@come-nc come-nc closed this Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants