anotherim/anotherim-desktop
2
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Fix regression in FileItem generation

Browse source 
fixes #1019
This commit is contained in:
fiaxh 2021-03-23 13:45:52 +01:00
parent e70b7c1222
commit 7ce49d7d68
2 changed files with 17 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
libdino/src/service/content_item_store.vala
View file

@ -56,9 +56,13 @@ public class ContentItemStore : StreamInteractionModule, Object {
} }
break; break;
case 2: case 2:
FileTransfer? file_transfer = stream_interactor.get_module(FileTransferStorage.IDENTITY).get_call_by_id(foreign_id); FileTransfer? file_transfer = stream_interactor.get_module(FileTransferStorage.IDENTITY).get_file_by_id(foreign_id, conversation);
if (file_transfer != null) { if (file_transfer != null) {
var file_item = new FileItem(file_transfer, conversation, row[db.content_item.id]); Message? message = null;
if (file_transfer.provider == 0 && file_transfer.info != null) {
message = stream_interactor.get_module(MessageStorage.IDENTITY).get_message_by_id(int.parse(file_transfer.info), conversation);
}
var file_item = new FileItem(file_transfer, conversation, row[db.content_item.id], message);
items.add(file_item); items.add(file_item);
} }
break; break;

17
libdino/src/service/file_transfer_storage.vala
View file

@ -27,10 +27,10 @@ namespace Dino {
public void add_file(FileTransfer file_transfer) { public void add_file(FileTransfer file_transfer) {
file_transfer.persist(db); file_transfer.persist(db);
cache_call(file_transfer); cache_file(file_transfer);
} }
public FileTransfer? get_call_by_id(int id) { public FileTransfer? get_file_by_id(int id, Conversation conversation) {
FileTransfer? file_transfer = files_by_db_id[id]; FileTransfer? file_transfer = files_by_db_id[id];
if (file_transfer != null) { if (file_transfer != null) {
return file_transfer; return file_transfer;
@ -38,15 +38,20 @@ namespace Dino {
RowOption row_option = db.file_transfer.select().with(db.file_transfer.id, "=", id).row(); RowOption row_option = db.file_transfer.select().with(db.file_transfer.id, "=", id).row();
return create_call_from_row_opt(row_option); return create_file_from_row_opt(row_option, conversation);
} }
private FileTransfer? create_call_from_row_opt(RowOption row_opt) { private FileTransfer? create_file_from_row_opt(RowOption row_opt, Conversation conversation) {
if (!row_opt.is_present()) return null; if (!row_opt.is_present()) return null;
try { try {
FileTransfer file_transfer = new FileTransfer.from_row(db, row_opt.inner, FileManager.get_storage_dir()); FileTransfer file_transfer = new FileTransfer.from_row(db, row_opt.inner, FileManager.get_storage_dir());
cache_call(file_transfer);
if (conversation.type_.is_muc_semantic()) {
file_transfer.ourpart = conversation.counterpart.with_resource(file_transfer.ourpart.resourcepart);
}
cache_file(file_transfer);
return file_transfer; return file_transfer;
} catch (InvalidJidError e) { } catch (InvalidJidError e) {
warning("Got file transfer with invalid Jid: %s", e.message); warning("Got file transfer with invalid Jid: %s", e.message);
@ -54,7 +59,7 @@ namespace Dino {
return null; return null;
} }
private void cache_call(FileTransfer file_transfer) { private void cache_file(FileTransfer file_transfer) {
files_by_db_id[file_transfer.id] = file_transfer; files_by_db_id[file_transfer.id] = file_transfer;
} }
} }