Add support (sort of) for gifv attachments

See #98

Pachyderm/Model/Status.swift

@@ -8,7 +8,7 @@
 
 import Foundation
 
-public final class Status: StatusProtocol, Decodable {
+public final class Status: /*StatusProtocol,*/ Decodable {
     public let id: String
     public let uri: String
     public let url: URL?
@@ -23,8 +23,8 @@ public final class Status: StatusProtocol, Decodable {
 //    public let repliesCount: Int
     public let reblogsCount: Int
     public let favouritesCount: Int
-    public let reblogged: Bool
+    public let reblogged: Bool?
-    public let favourited: Bool
+    public let favourited: Bool?
     public let muted: Bool?
     public let sensitive: Bool
     public let spoilerText: String

Tusker.xcodeproj/project.pbxproj

@@ -134,6 +134,8 @@
 		D64D0AB12128D9AE005A6F37 /* OnboardingViewController.swift in Sources */ = {isa = PBXBuildFile; fileRef = D64D0AB02128D9AE005A6F37 /* OnboardingViewController.swift */; };
 		D64D8CA92463B494006B0BAA /* CachedDictionary.swift in Sources */ = {isa = PBXBuildFile; fileRef = D64D8CA82463B494006B0BAA /* CachedDictionary.swift */; };
 		D64F80E2215875CC00BEF393 /* XCBActionType.swift in Sources */ = {isa = PBXBuildFile; fileRef = D64F80E1215875CC00BEF393 /* XCBActionType.swift */; };
+		D6531DEE246B81C9000F9538 /* GifvAttachmentView.swift in Sources */ = {isa = PBXBuildFile; fileRef = D6531DED246B81C9000F9538 /* GifvAttachmentView.swift */; };
+		D6531DF0246B867E000F9538 /* GifvAttachmentViewController.swift in Sources */ = {isa = PBXBuildFile; fileRef = D6531DEF246B867E000F9538 /* GifvAttachmentViewController.swift */; };
 		D6538945214D6D7500E3CEFC /* TableViewSwipeActionProvider.swift in Sources */ = {isa = PBXBuildFile; fileRef = D6538944214D6D7500E3CEFC /* TableViewSwipeActionProvider.swift */; };
 		D65A37F321472F300087646E /* SwiftSoup.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = D6BED16E212663DA00F02DA0 /* SwiftSoup.framework */; };
 		D65F613423AEAB6600F3CFD3 /* OnboardingTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = D65F613323AEAB6600F3CFD3 /* OnboardingTests.swift */; };
@@ -428,6 +430,8 @@
 		D64D0AB02128D9AE005A6F37 /* OnboardingViewController.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = OnboardingViewController.swift; sourceTree = "<group>"; };
 		D64D8CA82463B494006B0BAA /* CachedDictionary.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = CachedDictionary.swift; sourceTree = "<group>"; };
 		D64F80E1215875CC00BEF393 /* XCBActionType.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = XCBActionType.swift; sourceTree = "<group>"; };
+		D6531DED246B81C9000F9538 /* GifvAttachmentView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = GifvAttachmentView.swift; sourceTree = "<group>"; };
+		D6531DEF246B867E000F9538 /* GifvAttachmentViewController.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = GifvAttachmentViewController.swift; sourceTree = "<group>"; };
 		D6538944214D6D7500E3CEFC /* TableViewSwipeActionProvider.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = TableViewSwipeActionProvider.swift; sourceTree = "<group>"; };
 		D65F612D23AE990C00F3CFD3 /* Embassy.framework */ = {isa = PBXFileReference; explicitFileType = wrapper.framework; path = Embassy.framework; sourceTree = BUILT_PRODUCTS_DIR; };
 		D65F613023AE99E000F3CFD3 /* Ambassador.framework */ = {isa = PBXFileReference; explicitFileType = wrapper.framework; path = Ambassador.framework; sourceTree = BUILT_PRODUCTS_DIR; };
@@ -597,6 +601,7 @@
 			children = (
 				04D14BAE22B34A2800642648 /* GalleryViewController.swift */,
 				D647D92724257BEB0005044F /* AttachmentPreviewViewController.swift */,
+				D6531DEF246B867E000F9538 /* GifvAttachmentViewController.swift */,
 			);
 			path = "Attachment Gallery";
 			sourceTree = "<group>";
@@ -1216,6 +1221,7 @@
 			isa = PBXGroup;
 			children = (
 				D6C94D882139E6EC00CB5196 /* AttachmentView.swift */,
+				D6531DED246B81C9000F9538 /* GifvAttachmentView.swift */,
 				D6C7D27C22B6EBF800071952 /* AttachmentsContainerView.swift */,
 			);
 			path = Attachments;
@@ -1650,6 +1656,7 @@
 			buildActionMask = 2147483647;
 			files = (
 				D626493523BD94CE00612E6E /* CompositionAttachmentData.swift in Sources */,
+				D6531DF0246B867E000F9538 /* GifvAttachmentViewController.swift in Sources */,
 				D6285B5321EA708700FE4B39 /* StatusFormat.swift in Sources */,
 				D6DD353D22F28CD000A9563A /* ContentWarningCopyMode.swift in Sources */,
 				0427033A22B31269000D31B6 /* AdvancedPrefsView.swift in Sources */,
@@ -1702,6 +1709,7 @@
 				D6B053A623BD2D0C00A066FA /* AssetCollectionViewController.swift in Sources */,
 				04DACE8E212CC7CC009840C4 /* ImageCache.swift in Sources */,
 				D627FF7B217E951500CC0648 /* DraftsTableViewController.swift in Sources */,
+				D6531DEE246B81C9000F9538 /* GifvAttachmentView.swift in Sources */,
 				D6370B9C24421FF30092A7FF /* Tusker.xcdatamodeld in Sources */,
 				D6A3BC8F2321FFB900FD64D5 /* StatusActionAccountListTableViewController.swift in Sources */,
 				D6AEBB4823216B1D00E5038B /* AccountActivity.swift in Sources */,

Tusker/Controllers/MastodonController.swift

@@ -30,7 +30,8 @@ class MastodonController {
         }
     }
     
-    private(set) lazy var persistentContainer = MastodonCachePersistentStore(for: self)
+    private let transient: Bool
+    private(set) lazy var persistentContainer = MastodonCachePersistentStore(for: accountInfo, transient: transient)
 
     let instanceURL: URL
     var accountInfo: LocalData.UserAccountInfo?
@@ -40,10 +41,11 @@ class MastodonController {
     var account: Account!
     var instance: Instance!
 
-    init(instanceURL: URL) {
+    init(instanceURL: URL, transient: Bool = false) {
         self.instanceURL = instanceURL
         self.accountInfo = nil
         self.client = Client(baseURL: instanceURL)
+        self.transient = transient
     }
     
     func run<Result>(_ request: Request<Result>, completion: @escaping Client.Callback<Result>) {
@@ -82,11 +84,19 @@ class MastodonController {
             run(request) { response in
                 guard case let .success(account, _) = response else { fatalError() }
                 self.account = account
-                self.persistentContainer.addOrUpdate(account: account)
+                self.persistentContainer.backgroundContext.perform {
+                    if let accountMO = self.persistentContainer.account(for: account.id, in: self.persistentContainer.backgroundContext) {
+                        accountMO.updateFrom(apiAccount: account, container: self.persistentContainer)
+                    } else {
+                        // the first time the user's account is added to the store,
+                        // increment its reference count so that it's never removed
+                        self.persistentContainer.addOrUpdate(account: account, incrementReferenceCount: true)
+                    }
                     completion?(account)
                 }
             }
         }
+    }
     
     func getOwnInstance(completion: ((Instance) -> Void)? = nil) {
         if let instance = self.instance {

Tusker/CoreData/AccountMO.swift

@@ -32,6 +32,7 @@ public final class AccountMO: NSManagedObject, AccountProtocol {
     @NSManaged public var locked: Bool
     @NSManaged public var movedCD: Bool
     @NSManaged public var note: String
+    @NSManaged public var referenceCount: Int
     @NSManaged public var statusesCount: Int
     @NSManaged public var url: URL
     @NSManaged public var username: String
@@ -46,12 +47,30 @@ public final class AccountMO: NSManagedObject, AccountProtocol {
     public var bot: Bool? { botCD }
     public var moved: Bool? { movedCD }
     
+    func incrementReferenceCount() {
+        referenceCount += 1
+    }
+    
+    func decrementReferenceCount() {
+        referenceCount -= 1
+        if referenceCount <= 0 {
+            managedObjectContext!.delete(self)
+        }
+    }
+    
+    public override func prepareForDeletion() {
+        super.prepareForDeletion()
+        movedTo?.decrementReferenceCount()
+    }
+
 }
 
 extension AccountMO {
     convenience init(apiAccount account: Pachyderm.Account, container: MastodonCachePersistentStore, context: NSManagedObjectContext) {
         self.init(context: context)
         self.updateFrom(apiAccount: account, container: container)
+        
+        movedTo?.incrementReferenceCount()
     }
     
     func updateFrom(apiAccount account: Pachyderm.Account, container: MastodonCachePersistentStore) {

Tusker/CoreData/MastodonCachePersistentStore.swift

@@ -23,8 +23,17 @@ class MastodonCachePersistentStore: NSPersistentContainer {
     let statusSubject = PassthroughSubject<String, Never>()
     let accountSubject = PassthroughSubject<String, Never>()
 
-    init(for controller: MastodonController) {
+    init(for accountInfo: LocalData.UserAccountInfo?, transient: Bool = false) {
-        super.init(name: "\(controller.accountInfo!.id)_cache", managedObjectModel: MastodonCachePersistentStore.managedObjectModel)
+        if transient {
+            super.init(name: "transient_cache", managedObjectModel: MastodonCachePersistentStore.managedObjectModel)
+            
+            let storeDescription = NSPersistentStoreDescription()
+            storeDescription.type = NSInMemoryStoreType
+            persistentStoreDescriptions = [storeDescription]
+        } else {
+            super.init(name: "\(accountInfo!.id)_cache", managedObjectModel: MastodonCachePersistentStore.managedObjectModel)
+        }
+        
         loadPersistentStores { (description, error) in
             if let error = error {
                 fatalError("Unable to load persistent store: \(error)")
@@ -96,18 +105,25 @@ class MastodonCachePersistentStore: NSPersistentContainer {
     }
     
     @discardableResult
-    private func upsert(account: Account) -> AccountMO {
+    private func upsert(account: Account, incrementReferenceCount: Bool) -> AccountMO {
         if let accountMO = self.account(for: account.id, in: self.backgroundContext) {
             accountMO.updateFrom(apiAccount: account, container: self)
+            if incrementReferenceCount {
+                accountMO.incrementReferenceCount()
+            }
             return accountMO
         } else {
-            return AccountMO(apiAccount: account, container: self, context: self.backgroundContext)
+            let accountMO = AccountMO(apiAccount: account, container: self, context: self.backgroundContext)
+            if incrementReferenceCount {
+                accountMO.incrementReferenceCount()
+            }
+            return accountMO
         }
     }
     
-    func addOrUpdate(account: Account, completion: ((AccountMO) -> Void)? = nil) {
+    func addOrUpdate(account: Account, incrementReferenceCount: Bool, completion: ((AccountMO) -> Void)? = nil) {
         backgroundContext.perform {
-            let accountMO = self.upsert(account: account)
+            let accountMO = self.upsert(account: account, incrementReferenceCount: incrementReferenceCount)
             if self.backgroundContext.hasChanges {
                 try! self.backgroundContext.save()
             }
@@ -118,7 +134,7 @@ class MastodonCachePersistentStore: NSPersistentContainer {
     
     func addAll(accounts: [Account], completion: (() -> Void)? = nil) {
         backgroundContext.perform {
-            accounts.forEach { self.upsert(account: $0) }
+            accounts.forEach { self.upsert(account: $0, incrementReferenceCount: true) }
             if self.backgroundContext.hasChanges {
                 try! self.backgroundContext.save()
             }
@@ -130,9 +146,11 @@ class MastodonCachePersistentStore: NSPersistentContainer {
     func addAll(notifications: [Pachyderm.Notification], completion: (() -> Void)? = nil) {
         backgroundContext.perform {
             let statuses = notifications.compactMap { $0.status }
-            let accounts = notifications.map { $0.account }
+            // filter out mentions, otherwise we would double increment the reference count of those accounts
+            // since the status has the same account as the notification
+            let accounts = notifications.filter { $0.kind != .mention }.map { $0.account }
             statuses.forEach { self.upsert(status: $0, incrementReferenceCount: true) }
-            accounts.forEach { self.upsert(account: $0) }
+            accounts.forEach { self.upsert(account: $0, incrementReferenceCount: true) }
             if self.backgroundContext.hasChanges {
                 try! self.backgroundContext.save()
             }
@@ -148,7 +166,7 @@ class MastodonCachePersistentStore: NSPersistentContainer {
             var updatedStatuses = [String]()
             
             block(self.backgroundContext, { (accounts) in
-                accounts.forEach { self.upsert(account: $0) }
+                accounts.forEach { self.upsert(account: $0, incrementReferenceCount: true) }
                 updatedAccounts.append(contentsOf: accounts.map { $0.id })
             }, { (statuses) in
                 statuses.forEach { self.upsert(status: $0, incrementReferenceCount: true) }

Tusker/CoreData/StatusMO.swift

@@ -82,6 +82,7 @@ public final class StatusMO: NSManagedObject, StatusProtocol {
     public override func prepareForDeletion() {
         super.prepareForDeletion()
         reblog?.decrementReferenceCount()
+        account.decrementReferenceCount()
     }
 
 }
@@ -92,6 +93,7 @@ extension StatusMO {
         self.updateFrom(apiStatus: status, container: container)
         
         reblog?.incrementReferenceCount()
+        account.incrementReferenceCount()
     }
     
     func updateFrom(apiStatus status: Pachyderm.Status, container: MastodonCachePersistentStore) {
@@ -106,7 +108,7 @@ extension StatusMO {
         self.content = status.content
         self.createdAt = status.createdAt
         self.emojis = status.emojis
-        self.favourited = status.favourited
+        self.favourited = status.favourited ?? false
         self.favouritesCount = status.favouritesCount
         self.hashtags = status.hashtags
         self.inReplyToAccountID = status.inReplyToAccountID
@@ -115,7 +117,7 @@ extension StatusMO {
         self.mentions = status.mentions
         self.muted = status.muted ?? false
         self.pinnedInternal = status.pinned ?? false
-        self.reblogged = status.reblogged
+        self.reblogged = status.reblogged ?? false
         self.reblogsCount = status.reblogsCount
         self.sensitive = status.sensitive
         self.spoilerText = status.spoilerText

Tusker/CoreData/Tusker.xcdatamodeld/Tusker.xcdatamodel/contents

@@ -15,6 +15,7 @@
         <attribute name="locked" attributeType="Boolean" usesScalarValueType="YES"/>
         <attribute name="movedCD" optional="YES" attributeType="Boolean" usesScalarValueType="YES"/>
         <attribute name="note" attributeType="String"/>
+        <attribute name="referenceCount" optional="YES" attributeType="Integer 64" defaultValueString="0" usesScalarValueType="YES"/>
         <attribute name="statusesCount" attributeType="Integer 64" defaultValueString="0" usesScalarValueType="YES"/>
         <attribute name="url" attributeType="URI"/>
         <attribute name="username" attributeType="String"/>
@@ -49,7 +50,6 @@
         <attribute name="uri" attributeType="String"/>
         <attribute name="url" optional="YES" attributeType="URI"/>
         <attribute name="visibilityString" attributeType="String"/>
-
         <relationship name="account" maxCount="1" deletionRule="Nullify" destinationEntity="Account"/>
         <relationship name="reblog" optional="YES" maxCount="1" deletionRule="Nullify" destinationEntity="Status"/>
         <uniquenessConstraints>
@@ -59,7 +59,7 @@
         </uniquenessConstraints>
     </entity>
     <elements>
-        <element name="Account" positionX="169.21875" positionY="78.9609375" width="128" height="313"/>
+        <element name="Account" positionX="169.21875" positionY="78.9609375" width="128" height="328"/>
         <element name="Status" positionX="-63" positionY="-18" width="128" height="418"/>
     </elements>
 </model>

Tusker/Screens/Attachment Gallery/GalleryViewController.swift

@@ -75,6 +75,8 @@ class GalleryViewController: UIPageViewController, UIPageViewControllerDataSourc
                 let vc = AVPlayerViewController()
                 vc.player = AVPlayer(url: $0.url)
                 return vc
+            case .gifv:
+                return GifvAttachmentViewController(attachment: $0)
             default:
                 fatalError()
             }

Tusker/Screens/Attachment Gallery/GifvAttachmentViewController.swift

@@ -0,0 +1,33 @@
+//
+//  GifvAttachmentViewController.swift
+//  Tusker
+//
+//  Created by Shadowfacts on 5/12/20.
+//  Copyright © 2020 Shadowfacts. All rights reserved.
+//
+
+import UIKit
+import Pachyderm
+import AVFoundation
+
+class GifvAttachmentViewController: UIViewController {
+
+    private let attachment: Attachment
+    
+    init(attachment: Attachment) {
+        precondition(attachment.kind == .gifv)
+        self.attachment = attachment
+        
+        super.init(nibName: nil, bundle: nil)
+    }
+    
+    required init?(coder: NSCoder) {
+        fatalError("init(coder:) has not been implemented")
+    }
+
+    override func loadView() {
+        let asset = AVURLAsset(url: attachment.url)
+        self.view = GifvAttachmentView(asset: asset, gravity: .resizeAspect)
+    }
+
+}

Tusker/Screens/Preferences/LocalAccountAvatarView.swift

@@ -10,19 +10,27 @@ import SwiftUI
 
 struct LocalAccountAvatarView: View {
     let localAccountInfo: LocalData.UserAccountInfo
-    @State
+    @State var avatarImage: UIImage? = nil
-    var avatarImage: UIImage? = nil
+    @ObservedObject var preferences = Preferences.shared
     
     var body: some View {
         let image: Image
         if avatarImage == nil {
-            image = Image(systemName: "person.crop.square")
+            let imageName: String
+            switch preferences.avatarStyle {
+            case .circle:
+                imageName = "person.crop.circle"
+            case .roundRect:
+                imageName = "person.crop.square"
+            }
+            image = Image(systemName: imageName).resizable()
         } else {
             image = Image(uiImage: self.avatarImage!).renderingMode(.original)
         }
         return image
             .resizable()
             .frame(width: 30, height: 30)
+            .cornerRadius(preferences.avatarStyle.cornerRadiusFraction * 30)
             .onAppear(perform: self.loadImage)
     }
     

Tusker/Screens/Profile/ProfileTableViewController.swift

@@ -61,6 +61,15 @@ class ProfileTableViewController: EnhancedTableViewController {
         fatalError("init(coder:) has not been implemeneted")
     }
     
+    deinit {
+        if let id = accountID {
+            let container = mastodonController.persistentContainer
+            container.backgroundContext.perform {
+                container.account(for: id, in: container.backgroundContext)?.decrementReferenceCount()
+            }
+        }
+    }
+    
     override func viewDidLoad() {
         super.viewDidLoad()
 
@@ -90,7 +99,7 @@ class ProfileTableViewController: EnhancedTableViewController {
                         }
                         return
                     }
-                    self.mastodonController.persistentContainer.addOrUpdate(account: account) { (_) in
+                    self.mastodonController.persistentContainer.addOrUpdate(account: account, incrementReferenceCount: true) { (_) in
                         DispatchQueue.main.async {
                             self.updateAccountUI()
                             self.tableView.reloadData()

Tusker/Screens/Search/SearchResultsViewController.swift

@@ -134,11 +134,10 @@ class SearchResultsViewController: EnhancedTableViewController {
             var snapshot = NSDiffableDataSourceSnapshot<Section, Item>()
             
             self.mastodonController.persistentContainer.performBatchUpdates({ (context, addAccounts, addStatuses) in
-                // todo: reference count accounts
+                oldSnapshot.itemIdentifiers(inSection: .accounts).forEach { (item) in
-//                oldSnapshot.itemIdentifiers(inSection: .accounts).forEach { (item) in
+                    guard case let .account(id) = item else { return }
-//                    guard case let .account(id) = item else { return }
+                    self.mastodonController.persistentContainer.account(for: id, in: context)?.decrementReferenceCount()
-//                    self.mastodonController.persistentContainer.account(for: id, in: context)?.decrementReferenceCount()
+                }
-//                }
                 
                 oldSnapshot.itemIdentifiers(inSection: .statuses).forEach { (item) in
                     guard case let .status(id, _) = item else { return }
@@ -158,7 +157,6 @@ class SearchResultsViewController: EnhancedTableViewController {
                     snapshot.appendSections([.statuses])
                     snapshot.appendItems(results.statuses.map { .status($0.id, .unknown) }, toSection: .statuses)
                     addStatuses(results.statuses)
-                    addAccounts(results.statuses.map { $0.account })
                 }
             }, completion: {
                 DispatchQueue.main.async {

Tusker/Screens/Status Action Account List/StatusActionAccountListTableViewController.swift

@@ -58,6 +58,17 @@ class StatusActionAccountListTableViewController: EnhancedTableViewController {
         fatalError("init(coder:) has not been implemented")
     }
     
+    deinit {
+        if let accountIDs = self.accountIDs {
+            let container = self.mastodonController.persistentContainer
+            container.backgroundContext.perform {
+                for id in accountIDs {
+                    container.account(for: id, in: container.backgroundContext)?.decrementReferenceCount()
+                }
+            }
+        }
+    }
+    
     override func viewDidLoad() {
         super.viewDidLoad()
 
@@ -71,7 +82,11 @@ class StatusActionAccountListTableViewController: EnhancedTableViewController {
         
         tableView.tableHeaderView = UIView(frame: CGRect(x: 0, y: 0, width: 0, height: CGFloat.leastNormalMagnitude))
         
-        if accountIDs == nil {
+        if let accountIDs = accountIDs {
+            accountIDs.forEach { (id) in
+                self.mastodonController.persistentContainer.account(for: id)?.incrementReferenceCount()
+            }
+        } else {
             // account IDs haven't been set, so perform a request to load them
             guard let status = mastodonController.persistentContainer.status(for: statusID) else {
                 fatalError("Missing cached status \(statusID)")

Tusker/Screens/Timeline/InstanceTimelineViewController.swift

@@ -37,7 +37,7 @@ class InstanceTimelineViewController: TimelineTableViewController {
         self.instanceURL = url
 
         // the timeline VC only stores a weak reference to it, so we need to store a strong reference to make sure it's not released immediately
-        instanceMastodonController = MastodonController(instanceURL: url)
+        instanceMastodonController = MastodonController(instanceURL: url, transient: true)
 
         super.init(for: .public(local: true), mastodonController: instanceMastodonController)
         

Tusker/TuskerNavigationDelegate.swift

@@ -210,7 +210,7 @@ extension TuskerNavigationDelegate where Self: UIViewController {
         let bookmarked = status.bookmarked ?? false
         customActivites.insert(bookmarked ? UnbookmarkStatusActivity() : BookmarkStatusActivity(), at: 0)
 
-        if status.account.id == apiController.account.id {
+        if apiController.account != nil, status.account.id == apiController.account.id {
             let pinned = status.pinned ?? false
             customActivites.insert(pinned ? UnpinStatusActivity() : PinStatusActivity(), at: 1)
         }

Tusker/Views/Attachments/AttachmentView.swift

@@ -81,6 +81,8 @@ class AttachmentView: UIImageView, GIFAnimatable {
             loadVideo()
         case .audio:
             loadAudio()
+        case .gifv:
+            loadGifv()
         default:
             preconditionFailure("invalid attachment type")
         }
@@ -111,9 +113,9 @@ class AttachmentView: UIImageView, GIFAnimatable {
             let asset = AVURLAsset(url: attachmentURL)
             let generator = AVAssetImageGenerator(asset: asset)
             generator.appliesPreferredTrackTransform = true
-            guard let image = try? generator.copyCGImage(at: CMTime(seconds: 0, preferredTimescale: 1), actualTime: nil) else { return }
+            guard let image = try? generator.copyCGImage(at: .zero, actualTime: nil) else { return }
-            DispatchQueue.main.async {
+            DispatchQueue.main.async { [weak self] in
-                guard self.attachment.url == attachmentURL else { return }
+                guard let self = self, self.attachment.url == attachmentURL else { return }
                 self.image = UIImage(cgImage: image)
             }
         }
@@ -150,6 +152,30 @@ class AttachmentView: UIImageView, GIFAnimatable {
         ])
     }
     
+    func loadGifv() {
+        let attachmentURL = self.attachment.url
+        let asset = AVURLAsset(url: attachmentURL)
+        DispatchQueue.global(qos: .userInitiated).async {
+            let generator = AVAssetImageGenerator(asset: asset)
+            generator.appliesPreferredTrackTransform = true
+            guard let image = try? generator.copyCGImage(at: .zero, actualTime: nil) else { return }
+            DispatchQueue.main.async { [weak self] in
+                guard let self = self, self.attachment.url == attachmentURL else { return }
+                self.image = UIImage(cgImage: image)
+            }
+        }
+        
+        let gifvView = GifvAttachmentView(asset: asset, gravity: .resizeAspectFill)
+        gifvView.translatesAutoresizingMaskIntoConstraints = false
+        addSubview(gifvView)
+        NSLayoutConstraint.activate([
+            gifvView.leadingAnchor.constraint(equalTo: leadingAnchor),
+            gifvView.trailingAnchor.constraint(equalTo: trailingAnchor),
+            gifvView.topAnchor.constraint(equalTo: topAnchor),
+            gifvView.bottomAnchor.constraint(equalTo: bottomAnchor),
+        ])
+    }
+    
     override func display(_ layer: CALayer) {
         updateImageIfNeeded()
     }

Tusker/Views/Attachments/AttachmentsContainerView.swift

@@ -11,7 +11,7 @@ import Pachyderm
 
 class AttachmentsContainerView: UIView {
 
-    static let supportedAttachmentTypes = [Attachment.Kind.image, .video, .audio]
+    static let supportedAttachmentTypes = [Attachment.Kind.image, .video, .audio, .gifv]
     
     weak var delegate: AttachmentViewDelegate?
     

Tusker/Views/Attachments/GifvAttachmentView.swift

@@ -0,0 +1,49 @@
+//
+//  GifvAttachmentView.swift
+//  Tusker
+//
+//  Created by Shadowfacts on 5/12/20.
+//  Copyright © 2020 Shadowfacts. All rights reserved.
+//
+
+import UIKit
+import AVFoundation
+
+class GifvAttachmentView: UIView {
+    
+    override class var layerClass: AnyClass {
+        return AVPlayerLayer.self
+    }
+    
+    private var playerLayer: AVPlayerLayer {
+        layer as! AVPlayerLayer
+    }
+
+    private let item: AVPlayerItem
+    private let player: AVPlayer
+    
+    init(asset: AVAsset, gravity: AVLayerVideoGravity) {
+        item = AVPlayerItem(asset: asset)
+        player = AVPlayer(playerItem: item)
+
+        super.init(frame: .zero)
+        
+        playerLayer.player = player
+        playerLayer.videoGravity = gravity
+        player.play()
+        
+        NotificationCenter.default.addObserver(self, selector: #selector(restartItem), name: .AVPlayerItemDidPlayToEndTime, object: item)
+    }
+    
+    required init?(coder: NSCoder) {
+        fatalError("init(coder:) has not been implemented")
+    }
+    
+    @objc func restartItem() {
+        item.seek(to: .zero) { (success) in
+            guard success else { return }
+            self.player.play()
+        }
+    }
+
+}

Tusker/Views/Status/BaseStatusTableViewCell.swift

@@ -261,7 +261,7 @@ class BaseStatusTableViewCell: UITableViewCell {
         mastodonController.run(request) { response in
             DispatchQueue.main.async {
                 if case let .success(newStatus, _) = response {
-                    self.favorited = newStatus.favourited
+                    self.favorited = newStatus.favourited ?? false
                     self.mastodonController.persistentContainer.addOrUpdate(status: newStatus, incrementReferenceCount: false)
                     UIImpactFeedbackGenerator(style: .light).impactOccurred()
                 } else {
@@ -286,7 +286,7 @@ class BaseStatusTableViewCell: UITableViewCell {
         mastodonController.run(request) { response in
             DispatchQueue.main.async {
                 if case let .success(newStatus, _) = response {
-                    self.reblogged = newStatus.reblogged
+                    self.reblogged = newStatus.reblogged ?? false
                     self.mastodonController.persistentContainer.addOrUpdate(status: newStatus, incrementReferenceCount: false)
                     UIImpactFeedbackGenerator(style: .light).impactOccurred()
                 } else {