Skip to content

[stable32] Fix federated reshares#58900

Open
backportbot[bot] wants to merge 2 commits intostable32from
backport/58689/stable32
Open

[stable32] Fix federated reshares#58900
backportbot[bot] wants to merge 2 commits intostable32from
backport/58689/stable32

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Mar 12, 2026

Backport of #58689

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 bug 3. to review Waiting for reviews labels Mar 12, 2026
@backportbot backportbot bot added this to the Nextcloud 32.0.7 milestone Mar 12, 2026
Signed-off-by: provokateurin <kate@provokateurin.de>
…r might on another server

Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin force-pushed the backport/58689/stable32 branch from 4127da0 to d377b66 Compare March 17, 2026 10:16
@provokateurin provokateurin marked this pull request as ready for review March 17, 2026 10:16
@provokateurin provokateurin requested a review from a team as a code owner March 17, 2026 10:16
@provokateurin provokateurin requested review from icewind1991 and removed request for a team March 17, 2026 10:16
@provokateurin provokateurin enabled auto-merge March 17, 2026 10:17
* Remove share from table.
*/
private function removeShareFromTableById($shareId) {
public function removeShareFromTable(string $shareId): void {
Copy link
Contributor

Choose a reason for hiding this comment

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

$shareId here is always an int as the Snowflake changes are not present on 32 and getLastInsertId returns an int.

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.

Just a small patch needed

@skjnldsv skjnldsv mentioned this pull request Mar 18, 2026
20 tasks
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 bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants