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

[stable26] Rename numericStorageId to numericExternalStorageId in PersonalMount.php #38317

Merged
merged 1 commit into from
May 17, 2023

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #38115

…unt.php

so that it doesn't override `numericStorageId` in the `MountPoint` baseclass

fixes #37473
fixes #37414

Signed-off-by: Daniel <7558512+DerGenaue@users.noreply.github.com>
@@ -51,7 +51,7 @@
public function __construct(
UserStoragesService $storagesService,
StorageConfig $storageConfig,
$storageId,
$externalStorageId,

Check notice

Code scanning / Psalm

MissingParamType

Parameter $externalStorageId has no provided type
@DerGenaue
Copy link
Contributor

No idea what tests failed here;
Can this still go into 26.0.2?
I'd like to update my instance to that one so I don't have to keep it running in the broken state

@blizzz blizzz mentioned this pull request May 17, 2023
@blizzz blizzz requested review from ArtificialOwl, icewind1991, a team and nfebe and removed request for a team May 17, 2023 11:50
@blizzz blizzz added the 3. to review Waiting for reviews label May 17, 2023
@blizzz
Copy link
Member

blizzz commented May 17, 2023

failing tests are unrelated, but another review is needed

@blizzz blizzz merged commit dfd4f58 into stable26 May 17, 2023
@blizzz blizzz deleted the backport/38115/stable26 branch May 17, 2023 17:04
@DerGenaue
Copy link
Contributor

🥳 Thanks!

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.

3 participants