-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
To clarify: Mock test #4871
base: master
Are you sure you want to change the base?
To clarify: Mock test #4871
Conversation
Issues
======
- Added 3
Complexity increasing per file
==============================
- src/main/java/com/owncloud/android/ui/dialog/Controller.java 4
See the complete overview on Codacy |
|
||
public void verifyAndCreateFolder(OCFile mParentFolder, String newFolderName) { | ||
if (newFolderName.length() <= 0) { | ||
throw new RuntimeException(String.valueOf(R.string.filename_empty)); |
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.
Issue found: Avoid throwing raw exception types.
if (which == AlertDialog.BUTTON_POSITIVE) { | ||
|
||
if (newFolderName.length() <= 0) { | ||
throw new RuntimeException(String.valueOf(R.string.filename_empty)); |
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.
Issue found: Avoid throwing raw exception types.
if (newFolderName.length() <= 0) { | ||
throw new RuntimeException(String.valueOf(R.string.filename_empty)); | ||
} | ||
boolean serverWithForbiddenChars = true; |
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.
Signed-off-by: tobiasKaminsky <[email protected]>
Signed-off-by: tobiasKaminsky <[email protected]>
471ec0c
to
ef92ca7
Compare
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/12739.apk |
Unit test failed, but no output was generated. Maybe a preliminary stage failed. |
IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/12739-IT |
Codacy354Lint
SpotBugs (new)
SpotBugs (master)
SpotBugs increased! |
No description provided.