Skip to content

Commit

Permalink
Merge pull request abrensch#705 from afischerdev/new-apk
Browse files Browse the repository at this point in the history
Prevent exceptions when importing profiles
  • Loading branch information
afischerdev authored Jun 3, 2024
2 parents be0aa77 + 2a77f71 commit e011343
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -73,13 +73,19 @@ private void parseIntent(Intent intent) {
// URI example ==> dat=content://me.bluemail.mail.attachmentprovider/a2939069-76b5-44e4-8cbd-94485d0fd4ff/cc32b61d-97a6-4871-b67f-945d1d1d43c8/VIEW
String filename = null;
long filesize = 0L;

try (Cursor cursor = this.getContentResolver().query(intent.getData(), new String[]{
OpenableColumns.DISPLAY_NAME, OpenableColumns.SIZE}, null, null, null)) {
if (cursor != null && cursor.moveToFirst()) {
filename = cursor.getString(cursor.getColumnIndexOrThrow(OpenableColumns.DISPLAY_NAME));
filesize = cursor.getLong(cursor.getColumnIndexOrThrow(OpenableColumns.SIZE));
}
} catch (Exception e) {
resultMessage.append("ERROR: File not accessible\n");
displayMessage(resultMessage.toString());
return;
}

// is the file extention ".brf" in the file name
if (filename == null || isInvalidProfileFilename(filename)) {
resultMessage.append("ERROR: File extention must be \".brf\"\n");
Expand Down
3 changes: 3 additions & 0 deletions docs/revisions.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,9 @@ Profiles

- update gravel profile


- Minor bug fixes

[Solved issues](https://github.com/abrensch/brouter/issues?q=is%3Aissue+milestone%3A%22Version+1.7.5%22+is%3Aclosed)


Expand Down

0 comments on commit e011343

Please sign in to comment.