From 446e86d4ad798655aa1a83aebc86f9f2431484df Mon Sep 17 00:00:00 2001 From: fiaxh Date: Sat, 29 Dec 2018 19:15:25 +0100 Subject: Check FileTransfer's mime_type for null (unknown) when handeling it fixes #479 --- main/src/ui/conversation_selector/conversation_row.vala | 5 +++-- main/src/ui/conversation_summary/content_item_widget_factory.vala | 2 +- main/src/ui/notifications.vala | 5 +++-- 3 files changed, 7 insertions(+), 5 deletions(-) (limited to 'main') diff --git a/main/src/ui/conversation_selector/conversation_row.vala b/main/src/ui/conversation_selector/conversation_row.vala index 88234006..2d41d044 100644 --- a/main/src/ui/conversation_selector/conversation_row.vala +++ b/main/src/ui/conversation_selector/conversation_row.vala @@ -150,10 +150,11 @@ public class ConversationRow : ListBoxRow { nick_label.label = transfer.direction == Message.DIRECTION_SENT ? _("Me") + ": " : ""; } + bool file_is_image = transfer.mime_type != null && transfer.mime_type.has_prefix("image"); if (transfer.direction == Message.DIRECTION_SENT) { - message_label.label = "" + (transfer.mime_type.has_prefix("image") ? _("Image sent") : _("File sent") ) + ""; + message_label.label = "" + (file_is_image ? _("Image sent") : _("File sent") ) + ""; } else { - message_label.label = "" +(transfer.mime_type.has_prefix("image") ? _("Image received") : _("File received") ) + ""; + message_label.label = "" + (file_is_image ? _("Image received") : _("File received") ) + ""; } break; } diff --git a/main/src/ui/conversation_summary/content_item_widget_factory.vala b/main/src/ui/conversation_summary/content_item_widget_factory.vala index d8d21ebd..4a625cc5 100644 --- a/main/src/ui/conversation_summary/content_item_widget_factory.vala +++ b/main/src/ui/conversation_summary/content_item_widget_factory.vala @@ -194,7 +194,7 @@ public class FileItemWidgetGenerator : WidgetGenerator, Object { private Widget getDefaultWidget(FileTransfer file_transfer) { Box main_box = new Box(Orientation.HORIZONTAL, 4) { halign=Align.START, visible=true }; - string? icon_name = ContentType.get_generic_icon_name(file_transfer.mime_type); + string? icon_name = file_transfer.mime_type != null ? ContentType.get_generic_icon_name(file_transfer.mime_type) : null; Image content_type_image = new Image.from_icon_name(icon_name, IconSize.DND) { visible=true }; main_box.add(content_type_image); diff --git a/main/src/ui/notifications.vala b/main/src/ui/notifications.vala index 77a290a0..277a7a09 100644 --- a/main/src/ui/notifications.vala +++ b/main/src/ui/notifications.vala @@ -62,10 +62,11 @@ public class Notifications : Object { FileItem file_item = content_item as FileItem; FileTransfer transfer = file_item.file_transfer; + bool file_is_image = transfer.mime_type != null && transfer.mime_type.has_prefix("image"); if (transfer.direction == Message.DIRECTION_SENT) { - text = transfer.mime_type.has_prefix("image") ? _("Image sent") : _("File sent"); + text = file_is_image ? _("Image sent") : _("File sent"); } else { - text = transfer.mime_type.has_prefix("image") ? _("Image received") : _("File received"); + text = file_is_image ? _("Image received") : _("File received"); } break; } -- cgit v1.2.3-54-g00ecf