Skip to content

Commit

Permalink
fix for non-printable file name of an attachment
Browse files Browse the repository at this point in the history
  • Loading branch information
or-else committed Dec 17, 2024
1 parent 698d181 commit 88b9bfb
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
4 changes: 4 additions & 0 deletions app/src/main/java/co/tinode/tindroid/MessagesAdapter.java
Original file line number Diff line number Diff line change
Expand Up @@ -1262,8 +1262,12 @@ private boolean clickAttachment(Map<String, Object> data) {
}
}

if (fname != null) {
fname = fname.trim();
}
if (TextUtils.isEmpty(fname)) {
fname = mActivity.getString(R.string.default_attachment_name);
fname += Long.toString(System.currentTimeMillis() % 10000);
}

AttachmentHandler.enqueueDownloadAttachment(mActivity,
Expand Down
3 changes: 3 additions & 0 deletions app/src/main/java/co/tinode/tindroid/VideoViewFragment.java
Original file line number Diff line number Diff line change
Expand Up @@ -298,6 +298,9 @@ public boolean onMenuItemSelected(@NonNull MenuItem item) {
String filename = args.getString(AttachmentHandler.ARG_FILE_NAME);
String mime = args.getString(AttachmentHandler.ARG_MIME_TYPE);

if (filename != null) {
filename = filename.trim();
}
if (TextUtils.isEmpty(filename)) {
filename = getResources().getString(R.string.tinode_video);
filename += Long.toString(System.currentTimeMillis() % 10000);
Expand Down

0 comments on commit 88b9bfb

Please sign in to comment.