Skip to content

[stable32] fix(Propagator): rollback transaction if it fails#58131

Open
backportbot[bot] wants to merge 1 commit intostable32from
backport/57733/stable32
Open

[stable32] fix(Propagator): rollback transaction if it fails#58131
backportbot[bot] wants to merge 1 commit intostable32from
backport/57733/stable32

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Feb 6, 2026

Backport of #57733

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

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

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

@backportbot backportbot bot added the 3. to review Waiting for reviews label Feb 6, 2026
@backportbot backportbot bot added this to the Nextcloud 32.0.6 milestone Feb 6, 2026
@CarlSchwan CarlSchwan force-pushed the backport/57733/stable32 branch from 685b611 to 7bbd260 Compare February 6, 2026 10:02
@CarlSchwan CarlSchwan marked this pull request as ready for review February 6, 2026 10:02
@CarlSchwan CarlSchwan requested a review from a team as a code owner February 6, 2026 10:02
@CarlSchwan CarlSchwan requested review from Altahrim, ArtificialOwl, icewind1991 and leftybournes and removed request for a team February 6, 2026 10:02
try {
$this->connection->beginTransaction();

$query = $this->connection->getQueryBuilder();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The alignment here is off

Copy link
Contributor

@salmart-dev salmart-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Besides a formatting issue LGTM

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan force-pushed the backport/57733/stable32 branch from 7bbd260 to 481cd6a Compare February 6, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants