-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Add command to list mounts for user #42817
Conversation
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Psalm found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
namespace OCP\Files\Storage; | ||
|
||
interface IStorageDebugInfo { | ||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
^what psalm says
* | ||
* @return string | ||
*/ | ||
function debugInfo(): string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also here what psalm says
return null; | ||
default: | ||
return "Unknown (" . $share->getShareType() . ")"; | ||
if ($storage->instanceOfStorage(IStorageDebugInfo::class)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
technically the complaint is right, the interface is not an IStorage. Does instance of
not work here? Or method_exists in worst case?
Can be useful when debugging filesystem issues.
Includes a new extention interface to provide a debug description of storages instead of the previous hard coding of a few mounts in the
info:file
command.