keejef
/
session-ios
mirror of https://github.com/oxen-io/session-ios
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Merge branch 'charlesmchen/heicHeifFixes'

Browse Source
pull/1/head
Matthew Chen 7 years ago
parent 11680958de 7132179c5c
commit 1fb170715b
1 changed files with 84 additions and 82 deletions
Show Stats Download Patch File Download Diff File

166
SignalMessaging/attachments/SignalAttachment.swift
Unescape Escape View File

@ -229,7 +229,7 @@ public class SignalAttachment: NSObject {
if let cachedImage = cachedImage { if let cachedImage = cachedImage {
return cachedImage return cachedImage
} }
guard let image = UIImage(data:dataSource.data()) else { guard let image = UIImage(data: dataSource.data()) else {
return nil return nil
} }
cachedImage = image cachedImage = image
@ -285,7 +285,7 @@ public class SignalAttachment: NSObject {
if let filename = sourceFilename { if let filename = sourceFilename {
let fileExtension = (filename as NSString).pathExtension let fileExtension = (filename as NSString).pathExtension
if fileExtension.count > 0 { if fileExtension.count > 0 {
if let mimeType = MIMETypeUtil.mimeType(forFileExtension:fileExtension) { if let mimeType = MIMETypeUtil.mimeType(forFileExtension: fileExtension) {
// UTI types are an imperfect means of representing file type; // UTI types are an imperfect means of representing file type;
// file extensions are also imperfect but far more reliable and // file extensions are also imperfect but far more reliable and
// comprehensive so we always prefer to try to deduce MIME type // comprehensive so we always prefer to try to deduce MIME type
@ -344,7 +344,7 @@ public class SignalAttachment: NSObject {
if dataUTI == kUnknownTestAttachmentUTI { if dataUTI == kUnknownTestAttachmentUTI {
return "unknown" return "unknown"
} }
guard let fileExtension = MIMETypeUtil.fileExtension(forUTIType:dataUTI) else { guard let fileExtension = MIMETypeUtil.fileExtension(forUTIType: dataUTI) else {
return nil return nil
} }
return fileExtension return fileExtension
@ -443,8 +443,8 @@ public class SignalAttachment: NSObject {
if UIPasteboard.general.numberOfItems < 1 { if UIPasteboard.general.numberOfItems < 1 {
return false return false
} }
let itemSet = IndexSet(integer:0) let itemSet = IndexSet(integer: 0)
guard let pasteboardUTITypes = UIPasteboard.general.types(forItemSet:itemSet) else { guard let pasteboardUTITypes = UIPasteboard.general.types(forItemSet: itemSet) else {
return false return false
} }
let pasteboardUTISet = Set<String>(pasteboardUTITypes[0]) let pasteboardUTISet = Set<String>(pasteboardUTITypes[0])
@ -493,57 +493,57 @@ public class SignalAttachment: NSObject {
return nil return nil
} }
// If pasteboard contains multiple items, use only the first. // If pasteboard contains multiple items, use only the first.
let itemSet = IndexSet(integer:0) let itemSet = IndexSet(integer: 0)
guard let pasteboardUTITypes = UIPasteboard.general.types(forItemSet:itemSet) else { guard let pasteboardUTITypes = UIPasteboard.general.types(forItemSet: itemSet) else {
return nil return nil
} }
let pasteboardUTISet = Set<String>(pasteboardUTITypes[0]) let pasteboardUTISet = Set<String>(pasteboardUTITypes[0])
for dataUTI in inputImageUTISet { for dataUTI in inputImageUTISet {
if pasteboardUTISet.contains(dataUTI) { if pasteboardUTISet.contains(dataUTI) {
guard let data = dataForFirstPasteboardItem(dataUTI:dataUTI) else { guard let data = dataForFirstPasteboardItem(dataUTI: dataUTI) else {
owsFail("\(TAG) Missing expected pasteboard data for UTI: \(dataUTI)") owsFail("\(TAG) Missing expected pasteboard data for UTI: \(dataUTI)")
return nil return nil
} }
let dataSource = DataSourceValue.dataSource(with:data, utiType: dataUTI) let dataSource = DataSourceValue.dataSource(with: data, utiType: dataUTI)
// Pasted images _SHOULD _NOT_ be resized, if possible. // Pasted images _SHOULD _NOT_ be resized, if possible.
return attachment(dataSource : dataSource, dataUTI : dataUTI, imageQuality:.medium) return attachment(dataSource: dataSource, dataUTI: dataUTI, imageQuality: .medium)
} }
} }
for dataUTI in videoUTISet { for dataUTI in videoUTISet {
if pasteboardUTISet.contains(dataUTI) { if pasteboardUTISet.contains(dataUTI) {
guard let data = dataForFirstPasteboardItem(dataUTI:dataUTI) else { guard let data = dataForFirstPasteboardItem(dataUTI: dataUTI) else {
owsFail("\(TAG) Missing expected pasteboard data for UTI: \(dataUTI)") owsFail("\(TAG) Missing expected pasteboard data for UTI: \(dataUTI)")
return nil return nil
} }
let dataSource = DataSourceValue.dataSource(with:data, utiType: dataUTI) let dataSource = DataSourceValue.dataSource(with: data, utiType: dataUTI)
return videoAttachment(dataSource : dataSource, dataUTI : dataUTI) return videoAttachment(dataSource: dataSource, dataUTI: dataUTI)
} }
} }
for dataUTI in audioUTISet { for dataUTI in audioUTISet {
if pasteboardUTISet.contains(dataUTI) { if pasteboardUTISet.contains(dataUTI) {
guard let data = dataForFirstPasteboardItem(dataUTI:dataUTI) else { guard let data = dataForFirstPasteboardItem(dataUTI: dataUTI) else {
owsFail("\(TAG) Missing expected pasteboard data for UTI: \(dataUTI)") owsFail("\(TAG) Missing expected pasteboard data for UTI: \(dataUTI)")
return nil return nil
} }
let dataSource = DataSourceValue.dataSource(with:data, utiType: dataUTI) let dataSource = DataSourceValue.dataSource(with: data, utiType: dataUTI)
return audioAttachment(dataSource : dataSource, dataUTI : dataUTI) return audioAttachment(dataSource: dataSource, dataUTI: dataUTI)
} }
} }
let dataUTI = pasteboardUTISet[pasteboardUTISet.startIndex] let dataUTI = pasteboardUTISet[pasteboardUTISet.startIndex]
guard let data = dataForFirstPasteboardItem(dataUTI:dataUTI) else { guard let data = dataForFirstPasteboardItem(dataUTI: dataUTI) else {
owsFail("\(TAG) Missing expected pasteboard data for UTI: \(dataUTI)") owsFail("\(TAG) Missing expected pasteboard data for UTI: \(dataUTI)")
return nil return nil
} }
let dataSource = DataSourceValue.dataSource(with:data, utiType: dataUTI) let dataSource = DataSourceValue.dataSource(with: data, utiType: dataUTI)
return genericAttachment(dataSource : dataSource, dataUTI : dataUTI) return genericAttachment(dataSource: dataSource, dataUTI: dataUTI)
} }
// This method should only be called for dataUTIs that // This method should only be called for dataUTIs that
// are appropriate for the first pasteboard item. // are appropriate for the first pasteboard item.
private class func dataForFirstPasteboardItem(dataUTI: String) -> Data? { private class func dataForFirstPasteboardItem(dataUTI: String) -> Data? {
let itemSet = IndexSet(integer:0) let itemSet = IndexSet(integer: 0)
guard let datas = UIPasteboard.general.data(forPasteboardType:dataUTI, inItemSet:itemSet) else { guard let datas = UIPasteboard.general.data(forPasteboardType: dataUTI, inItemSet: itemSet) else {
owsFail("\(TAG) Missing expected pasteboard data for UTI: \(dataUTI)") owsFail("\(TAG) Missing expected pasteboard data for UTI: \(dataUTI)")
return nil return nil
} }
@ -569,12 +569,12 @@ public class SignalAttachment: NSObject {
assert(dataUTI.count > 0) assert(dataUTI.count > 0)
assert(dataSource != nil) assert(dataSource != nil)
guard let dataSource = dataSource else { guard let dataSource = dataSource else {
let attachment = SignalAttachment(dataSource : DataSourceValue.emptyDataSource(), dataUTI: dataUTI) let attachment = SignalAttachment(dataSource: DataSourceValue.emptyDataSource(), dataUTI: dataUTI)
attachment.error = .missingData attachment.error = .missingData
return attachment return attachment
} }
let attachment = SignalAttachment(dataSource : dataSource, dataUTI: dataUTI) let attachment = SignalAttachment(dataSource: dataSource, dataUTI: dataUTI)
guard inputImageUTISet.contains(dataUTI) else { guard inputImageUTISet.contains(dataUTI) else {
attachment.error = .invalidFileFormat attachment.error = .invalidFileFormat
@ -597,45 +597,47 @@ public class SignalAttachment: NSObject {
Logger.verbose("\(TAG) Sending raw \(attachment.mimeType) to retain any animation") Logger.verbose("\(TAG) Sending raw \(attachment.mimeType) to retain any animation")
return attachment return attachment
} else { } else {
guard let image = UIImage(data:dataSource.data()) else { guard let image = UIImage(data: dataSource.data()) else {
attachment.error = .couldNotParseImage attachment.error = .couldNotParseImage
return attachment return attachment
} }
attachment.cachedImage = image attachment.cachedImage = image
if isValidOutputImage(image: image, dataSource: dataSource, dataUTI: dataUTI, imageQuality:imageQuality) { let isValidOutput = isValidOutputImage(image: image, dataSource: dataSource, dataUTI: dataUTI, imageQuality: imageQuality)
if let sourceFilename = dataSource.sourceFilename,
let sourceFileExtension = sourceFilename.fileExtension, if let sourceFilename = dataSource.sourceFilename,
["heic", "heif"].contains(sourceFileExtension.lowercased()) { let sourceFileExtension = sourceFilename.fileExtension,
["heic", "heif"].contains(sourceFileExtension.lowercased()) {
// If a .heic file actually contains jpeg data, update the extension to match.
// // If a .heic file actually contains jpeg data, update the extension to match.
// Here's how that can happen: //
// In iOS11, the Photos.app records photos with HEIC UTIType, with the .HEIC extension. // Here's how that can happen:
// Since HEIC isn't a valid output format for Signal, we'll detect that and convert to JPEG, // In iOS11, the Photos.app records photos with HEIC UTIType, with the .HEIC extension.
// updating the extension as well. No problem. // Since HEIC isn't a valid output format for Signal, we'll detect that and convert to JPEG,
// However the problem comes in when you edit an HEIC image in Photos.app - the image is saved // updating the extension as well. No problem.
// in the Photos.app as a JPEG, but retains the (now incongruous) HEIC extension in the filename. // However the problem comes in when you edit an HEIC image in Photos.app - the image is saved
assert(dataUTI == kUTTypeJPEG as String) // in the Photos.app as a JPEG, but retains the (now incongruous) HEIC extension in the filename.
Logger.verbose("\(self.TAG) changing extension: \(sourceFileExtension) to match jpg uti type") assert(dataUTI == kUTTypeJPEG as String || !isValidOutput)
Logger.verbose("\(self.TAG) changing extension: \(sourceFileExtension) to match jpg uti type")
let baseFilename = sourceFilename.filenameWithoutExtension
dataSource.sourceFilename = baseFilename.appendingFileExtension("jpg") let baseFilename = sourceFilename.filenameWithoutExtension
} dataSource.sourceFilename = baseFilename.appendingFileExtension("jpg")
}
if isValidOutput {
Logger.verbose("\(TAG) Sending raw \(attachment.mimeType)") Logger.verbose("\(TAG) Sending raw \(attachment.mimeType)")
return attachment return attachment
} else {
Logger.verbose("\(TAG) Compressing attachment as image/jpeg, \(dataSource.dataLength()) bytes")
return compressImageAsJPEG(image: image, attachment: attachment, filename: dataSource.sourceFilename, imageQuality: imageQuality)
} }
Logger.verbose("\(TAG) Compressing attachment as image/jpeg, \(dataSource.dataLength()) bytes")
return compressImageAsJPEG(image : image, attachment : attachment, filename:dataSource.sourceFilename, imageQuality:imageQuality)
} }
} }
// If the proposed attachment already conforms to the // If the proposed attachment already conforms to the
// file size and content size limits, don't recompress it. // file size and content size limits, don't recompress it.
private class func isValidOutputImage(image: UIImage?, dataSource: DataSource?, dataUTI: String, imageQuality: TSImageQuality) -> Bool { private class func isValidOutputImage(image: UIImage?, dataSource: DataSource?, dataUTI: String, imageQuality: TSImageQuality) -> Bool {
guard let image = image else { guard image != nil else {
return false return false
} }
guard let dataSource = dataSource else { guard let dataSource = dataSource else {
@ -644,7 +646,6 @@ public class SignalAttachment: NSObject {
guard SignalAttachment.outputImageUTISet.contains(dataUTI) else { guard SignalAttachment.outputImageUTISet.contains(dataUTI) else {
return false return false
} }
if doesImageHaveAcceptableFileSize(dataSource: dataSource, imageQuality: imageQuality) && if doesImageHaveAcceptableFileSize(dataSource: dataSource, imageQuality: imageQuality) &&
dataSource.dataLength() <= kMaxFileSizeImage { dataSource.dataLength() <= kMaxFileSizeImage {
return true return true
@ -663,7 +664,7 @@ public class SignalAttachment: NSObject {
guard let image = image else { guard let image = image else {
let dataSource = DataSourceValue.emptyDataSource() let dataSource = DataSourceValue.emptyDataSource()
dataSource.sourceFilename = filename dataSource.sourceFilename = filename
let attachment = SignalAttachment(dataSource:dataSource, dataUTI: dataUTI) let attachment = SignalAttachment(dataSource: dataSource, dataUTI: dataUTI)
attachment.error = .missingData attachment.error = .missingData
return attachment return attachment
} }
@ -671,18 +672,19 @@ public class SignalAttachment: NSObject {
// Make a placeholder attachment on which to hang errors if necessary. // Make a placeholder attachment on which to hang errors if necessary.
let dataSource = DataSourceValue.emptyDataSource() let dataSource = DataSourceValue.emptyDataSource()
dataSource.sourceFilename = filename dataSource.sourceFilename = filename
let attachment = SignalAttachment(dataSource : dataSource, dataUTI: dataUTI) let attachment = SignalAttachment(dataSource: dataSource, dataUTI: dataUTI)
attachment.cachedImage = image attachment.cachedImage = image
Logger.verbose("\(TAG) Writing \(attachment.mimeType) as image/jpeg") Logger.verbose("\(TAG) Writing \(attachment.mimeType) as image/jpeg")
return compressImageAsJPEG(image : image, attachment : attachment, filename:filename, imageQuality:imageQuality) return compressImageAsJPEG(image: image, attachment: attachment, filename: filename, imageQuality: imageQuality)
} }
private class func compressImageAsJPEG(image: UIImage, attachment: SignalAttachment, filename: String?, imageQuality: TSImageQuality) -> SignalAttachment { private class func compressImageAsJPEG(image: UIImage, attachment: SignalAttachment, filename: String?, imageQuality: TSImageQuality) -> SignalAttachment {
assert(attachment.error == nil) assert(attachment.error == nil)
if imageQuality == .original && if imageQuality == .original &&
attachment.dataLength < kMaxFileSizeGeneric { attachment.dataLength < kMaxFileSizeGeneric &&
outputImageUTISet.contains(attachment.dataUTI) {
// We should avoid resizing images attached "as documents" if possible. // We should avoid resizing images attached "as documents" if possible.
return attachment return attachment
} }
@ -690,19 +692,19 @@ public class SignalAttachment: NSObject {
var imageUploadQuality = imageQuality.imageQualityTier() var imageUploadQuality = imageQuality.imageQualityTier()
while true { while true {
let maxSize = maxSizeForImage(image: image, imageUploadQuality:imageUploadQuality) let maxSize = maxSizeForImage(image: image, imageUploadQuality: imageUploadQuality)
var dstImage: UIImage! = image var dstImage: UIImage! = image
if image.size.width > maxSize || if image.size.width > maxSize ||
image.size.height > maxSize { image.size.height > maxSize {
dstImage = imageScaled(image, toMaxSize: maxSize) dstImage = imageScaled(image, toMaxSize: maxSize)
} }
guard let jpgImageData = UIImageJPEGRepresentation(dstImage, guard let jpgImageData = UIImageJPEGRepresentation(dstImage,
jpegCompressionQuality(imageUploadQuality:imageUploadQuality)) else { jpegCompressionQuality(imageUploadQuality: imageUploadQuality)) else {
attachment.error = .couldNotConvertToJpeg attachment.error = .couldNotConvertToJpeg
return attachment return attachment
} }
guard let dataSource = DataSourceValue.dataSource(with:jpgImageData, fileExtension:"jpg") else { guard let dataSource = DataSourceValue.dataSource(with: jpgImageData, fileExtension: "jpg") else {
attachment.error = .couldNotConvertToJpeg attachment.error = .couldNotConvertToJpeg
return attachment return attachment
} }
@ -713,7 +715,7 @@ public class SignalAttachment: NSObject {
if doesImageHaveAcceptableFileSize(dataSource: dataSource, imageQuality: imageQuality) && if doesImageHaveAcceptableFileSize(dataSource: dataSource, imageQuality: imageQuality) &&
dataSource.dataLength() <= kMaxFileSizeImage { dataSource.dataLength() <= kMaxFileSizeImage {
let recompressedAttachment = SignalAttachment(dataSource : dataSource, dataUTI: kUTTypeJPEG as String) let recompressedAttachment = SignalAttachment(dataSource: dataSource, dataUTI: kUTTypeJPEG as String)
recompressedAttachment.cachedImage = dstImage recompressedAttachment.cachedImage = dstImage
Logger.verbose("\(TAG) Converted \(attachment.mimeType) to image/jpeg, \(jpgImageData.count) bytes") Logger.verbose("\(TAG) Converted \(attachment.mimeType) to image/jpeg, \(jpgImageData.count) bytes")
return recompressedAttachment return recompressedAttachment
@ -810,7 +812,7 @@ public class SignalAttachment: NSObject {
private class func videoAttachment(dataSource: DataSource?, dataUTI: String) -> SignalAttachment { private class func videoAttachment(dataSource: DataSource?, dataUTI: String) -> SignalAttachment {
guard let dataSource = dataSource else { guard let dataSource = dataSource else {
let dataSource = DataSourceValue.emptyDataSource() let dataSource = DataSourceValue.emptyDataSource()
let attachment = SignalAttachment(dataSource:dataSource, dataUTI: dataUTI) let attachment = SignalAttachment(dataSource: dataSource, dataUTI: dataUTI)
attachment.error = .missingData attachment.error = .missingData
return attachment return attachment
} }
@ -846,7 +848,7 @@ public class SignalAttachment: NSObject {
Logger.debug("\(self.TAG) in \(#function)") Logger.debug("\(self.TAG) in \(#function)")
guard let url = dataSource.dataUrl() else { guard let url = dataSource.dataUrl() else {
let attachment = SignalAttachment(dataSource : DataSourceValue.emptyDataSource(), dataUTI: dataUTI) let attachment = SignalAttachment(dataSource: DataSourceValue.emptyDataSource(), dataUTI: dataUTI)
attachment.error = .missingData attachment.error = .missingData
return (Promise(value: attachment), nil) return (Promise(value: attachment), nil)
} }
@ -854,7 +856,7 @@ public class SignalAttachment: NSObject {
let asset = AVAsset(url: url) let asset = AVAsset(url: url)
guard let exportSession = AVAssetExportSession(asset: asset, presetName: AVAssetExportPresetMediumQuality) else { guard let exportSession = AVAssetExportSession(asset: asset, presetName: AVAssetExportPresetMediumQuality) else {
let attachment = SignalAttachment(dataSource : DataSourceValue.emptyDataSource(), dataUTI: dataUTI) let attachment = SignalAttachment(dataSource: DataSourceValue.emptyDataSource(), dataUTI: dataUTI)
attachment.error = .couldNotConvertToMpeg4 attachment.error = .couldNotConvertToMpeg4
return (Promise(value: attachment), nil) return (Promise(value: attachment), nil)
} }
@ -875,7 +877,7 @@ public class SignalAttachment: NSObject {
guard let dataSource = DataSourcePath.dataSource(with: exportURL) else { guard let dataSource = DataSourcePath.dataSource(with: exportURL) else {
owsFail("Failed to build data source for exported video URL") owsFail("Failed to build data source for exported video URL")
let attachment = SignalAttachment(dataSource : DataSourceValue.emptyDataSource(), dataUTI: dataUTI) let attachment = SignalAttachment(dataSource: DataSourceValue.emptyDataSource(), dataUTI: dataUTI)
attachment.error = .couldNotConvertToMpeg4 attachment.error = .couldNotConvertToMpeg4
fulfill(attachment) fulfill(attachment)
return return
@ -949,10 +951,10 @@ public class SignalAttachment: NSObject {
// NOTE: The attachment returned by this method may not be valid. // NOTE: The attachment returned by this method may not be valid.
// Check the attachment's error property. // Check the attachment's error property.
private class func audioAttachment(dataSource: DataSource?, dataUTI: String) -> SignalAttachment { private class func audioAttachment(dataSource: DataSource?, dataUTI: String) -> SignalAttachment {
return newAttachment(dataSource : dataSource, return newAttachment(dataSource: dataSource,
dataUTI : dataUTI, dataUTI: dataUTI,
validUTISet : audioUTISet, validUTISet: audioUTISet,
maxFileSize : kMaxFileSizeAudio) maxFileSize: kMaxFileSizeAudio)
} }
// MARK: Oversize Text Attachments // MARK: Oversize Text Attachments
@ -962,11 +964,11 @@ public class SignalAttachment: NSObject {
// NOTE: The attachment returned by this method may not be valid. // NOTE: The attachment returned by this method may not be valid.
// Check the attachment's error property. // Check the attachment's error property.
private class func oversizeTextAttachment(text: String?) -> SignalAttachment { private class func oversizeTextAttachment(text: String?) -> SignalAttachment {
let dataSource = DataSourceValue.dataSource(withOversizeText:text) let dataSource = DataSourceValue.dataSource(withOversizeText: text)
return newAttachment(dataSource : dataSource, return newAttachment(dataSource: dataSource,
dataUTI : kOversizeTextAttachmentUTI, dataUTI: kOversizeTextAttachmentUTI,
validUTISet : nil, validUTISet: nil,
maxFileSize : kMaxFileSizeGeneric) maxFileSize: kMaxFileSizeGeneric)
} }
// MARK: Generic Attachments // MARK: Generic Attachments
@ -976,17 +978,17 @@ public class SignalAttachment: NSObject {
// NOTE: The attachment returned by this method may not be valid. // NOTE: The attachment returned by this method may not be valid.
// Check the attachment's error property. // Check the attachment's error property.
private class func genericAttachment(dataSource: DataSource?, dataUTI: String) -> SignalAttachment { private class func genericAttachment(dataSource: DataSource?, dataUTI: String) -> SignalAttachment {
return newAttachment(dataSource : dataSource, return newAttachment(dataSource: dataSource,
dataUTI : dataUTI, dataUTI: dataUTI,
validUTISet : nil, validUTISet: nil,
maxFileSize : kMaxFileSizeGeneric) maxFileSize: kMaxFileSizeGeneric)
} }
// MARK: Voice Messages // MARK: Voice Messages
@objc @objc
public class func voiceMessageAttachment(dataSource: DataSource?, dataUTI: String) -> SignalAttachment { public class func voiceMessageAttachment(dataSource: DataSource?, dataUTI: String) -> SignalAttachment {
let attachment = audioAttachment(dataSource : dataSource, dataUTI : dataUTI) let attachment = audioAttachment(dataSource: dataSource, dataUTI: dataUTI)
attachment.isVoiceMessage = true attachment.isVoiceMessage = true
return attachment return attachment
} }
@ -1012,21 +1014,21 @@ public class SignalAttachment: NSObject {
@objc @objc
public class func attachment(dataSource: DataSource?, dataUTI: String, imageQuality: TSImageQuality) -> SignalAttachment { public class func attachment(dataSource: DataSource?, dataUTI: String, imageQuality: TSImageQuality) -> SignalAttachment {
if inputImageUTISet.contains(dataUTI) { if inputImageUTISet.contains(dataUTI) {
return imageAttachment(dataSource : dataSource, dataUTI : dataUTI, imageQuality:imageQuality) return imageAttachment(dataSource: dataSource, dataUTI: dataUTI, imageQuality: imageQuality)
} else if videoUTISet.contains(dataUTI) { } else if videoUTISet.contains(dataUTI) {
return videoAttachment(dataSource : dataSource, dataUTI : dataUTI) return videoAttachment(dataSource: dataSource, dataUTI: dataUTI)
} else if audioUTISet.contains(dataUTI) { } else if audioUTISet.contains(dataUTI) {
return audioAttachment(dataSource : dataSource, dataUTI : dataUTI) return audioAttachment(dataSource: dataSource, dataUTI: dataUTI)
} else { } else {
return genericAttachment(dataSource : dataSource, dataUTI : dataUTI) return genericAttachment(dataSource: dataSource, dataUTI: dataUTI)
} }
} }
@objc @objc
public class func empty() -> SignalAttachment { public class func empty() -> SignalAttachment {
return SignalAttachment.attachment(dataSource : DataSourceValue.emptyDataSource(), return SignalAttachment.attachment(dataSource: DataSourceValue.emptyDataSource(),
dataUTI: kUTTypeContent as String, dataUTI: kUTTypeContent as String,
imageQuality:.original) imageQuality: .original)
} }
// MARK: Helper Methods // MARK: Helper Methods
@ -1039,12 +1041,12 @@ public class SignalAttachment: NSObject {
assert(dataSource != nil) assert(dataSource != nil)
guard let dataSource = dataSource else { guard let dataSource = dataSource else {
let attachment = SignalAttachment(dataSource : DataSourceValue.emptyDataSource(), dataUTI: dataUTI) let attachment = SignalAttachment(dataSource: DataSourceValue.emptyDataSource(), dataUTI: dataUTI)
attachment.error = .missingData attachment.error = .missingData
return attachment return attachment
} }
let attachment = SignalAttachment(dataSource : dataSource, dataUTI: dataUTI) let attachment = SignalAttachment(dataSource: dataSource, dataUTI: dataUTI)
if let validUTISet = validUTISet { if let validUTISet = validUTISet {
guard validUTISet.contains(dataUTI) else { guard validUTISet.contains(dataUTI) else {

Write Preview
Loading…
Cancel
Save