-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 warnings for OCFileListFragment #12112
Conversation
alperozturk96
commented
Oct 31, 2023
•
edited
Loading
edited
- Tests written, or not not needed
@@ -1786,38 +1734,8 @@ private void encryptFolder(OCFile folder, | |||
} | |||
} | |||
|
|||
@Subscribe(threadMode = ThreadMode.BACKGROUND) | |||
public void onMessageEvent(FileLockEvent event) { |
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.
Unused function
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.
Like mentioned in the other PR, I expect the event bus to make use of them, so this should actually be a false-positive, best to check with @tobiasKaminsky
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.
Yeah, this is still used.
You can check where FileLockEvent is being used:
Within FileOperationsHelper there is: EventBus.getDefault().post(new FileLockEvent(file.getRemotePath(), shouldBeLocked));
This is to send the event, and the other function receives it.
In long term I would like to get rid of it…
59f9233
to
fe98ecc
Compare
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
fe98ecc
to
b6286ff
Compare
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.
Re-add event bus function.
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: github-actions <[email protected]>
Signed-off-by: alperozturk <[email protected]>
app/src/main/java/com/owncloud/android/ui/fragment/OCFileListFragment.java
Outdated
Show resolved
Hide resolved
@@ -1786,6 +1731,7 @@ private void encryptFolder(OCFile folder, | |||
} | |||
} | |||
|
|||
// Do not delete this function |
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.
// Do not delete this function |
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.
IDE will always show as unused function better to keep that comment
app/src/main/java/com/owncloud/android/ui/fragment/OCFileListFragment.java
Outdated
Show resolved
Hide resolved
…ragment.java Co-authored-by: Tom <[email protected]> Signed-off-by: Alper Öztürk <[email protected]>
…ragment.java Co-authored-by: Tom <[email protected]> Signed-off-by: Alper Öztürk <[email protected]>
/rebase |
Signed-off-by: alperozturk <[email protected]>
/rebase |
Signed-off-by: github-actions <[email protected]>
/rebase |
/rebase |
Signed-off-by: alperozturk <[email protected]>
/rebase |
Signed-off-by: github-actions <[email protected]>
Signed-off-by: alperozturk <[email protected]>
/rebase |
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: github-actions <[email protected]>
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/12112.apk |
blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed. |
Out of date due to conflicts |