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

BugFix - Use Long For X-Activity-Last-Given #1672

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alperozturk96
Copy link
Contributor

@alperozturk96 alperozturk96 commented Feb 20, 2025

Issue : X-Activity-Last-Given, exceeding the integer limit.

@tobiasKaminsky In Java, if a constructor expects a long primitive type, you can still pass an int primitive type, and it will compile successfully. See the video below for reference. Therefore, there is no need to modify the Android Files Client.

Screen.Recording.2025-03-03.at.14.46.00.mp4

@alperozturk96 alperozturk96 changed the title BugFix - Use Long X-Activity-Last-Given BugFix - Use Long For X-Activity-Last-Given Feb 20, 2025
Copy link
Contributor

SpotBugs

CategoryBaseNew
Bad practice3535
Correctness3434
Dodgy code2626
Internationalization66
Malicious code vulnerability4949
Multithreaded correctness33
Performance88
Total161161

Copy link
Member

@joshtrichards joshtrichards left a comment

Choose a reason for hiding this comment

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

Signed-off-by: alperozturk <[email protected]>
@alperozturk96 alperozturk96 force-pushed the bugfix/use-long-for-GetActivitiesRemoteOperation branch from 86363c1 to ee1fb0c Compare March 3, 2025 13:37
Copy link
Contributor

github-actions bot commented Mar 3, 2025

SpotBugs

CategoryBaseNew
Bad practice3535
Correctness3434
Dodgy code2626
Internationalization66
Malicious code vulnerability4949
Multithreaded correctness33
Performance88
Total161161

@nextcloud-android-bot
Copy link
Collaborator

@nextcloud-android-bot
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sharing files not possible: java.lang.NumberFormatException: For input string: in GetActivitiesRemoteOperation
3 participants