-
Notifications
You must be signed in to change notification settings - Fork 156
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
fix variable name in error object #6413
Conversation
33cf66d
to
6a02019
Compare
Results for oC10SharingExternalRoot https://drone.owncloud.com/owncloud/web/22648/42/1
|
Results for oC10SharingPermToRoot https://drone.owncloud.com/owncloud/web/22648/35/1
|
Results for oCISFiles1 https://drone.owncloud.com/owncloud/web/22648/55/1
|
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.
Good find and fix, thank you! 💪 Small small nitpicks, see comments below.
6a02019
to
7b90589
Compare
7b90589
to
33a101b
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Results for oCISSharingAndUpload https://drone.owncloud.com/owncloud/web/22673/66/1
|
Description
the variable name should be
error.resourceName
,error.resource
is not used anywhereRelated Issue
Motivation and Context
found when cleaning up unit test log messages:
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist:
Open tasks: