From d60c1d19412f8df4723a174ab3982ea0af4fe6f1 Mon Sep 17 00:00:00 2001 From: Carlos Garcia Date: Tue, 21 May 2024 16:50:49 +0200 Subject: [PATCH] Use `guard`/`if let` statements to avoid using optionals --- .../GutenbergGalleryUploadProcessor.swift | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/WordPress/Classes/ViewRelated/Gutenberg/Processors/GutenbergGalleryUploadProcessor.swift b/WordPress/Classes/ViewRelated/Gutenberg/Processors/GutenbergGalleryUploadProcessor.swift index 6ead457d0b69..3c11743febd5 100644 --- a/WordPress/Classes/ViewRelated/Gutenberg/Processors/GutenbergGalleryUploadProcessor.swift +++ b/WordPress/Classes/ViewRelated/Gutenberg/Processors/GutenbergGalleryUploadProcessor.swift @@ -29,8 +29,10 @@ class GutenbergGalleryUploadProcessor: GutenbergProcessor { } func processImgPostMediaUpload(_ element: Element) { - let imgTags = try? element.select(ImageKeys.name) - imgTags?.forEach {imgTag in + guard let imgTags = try? element.select(ImageKeys.name) else { + return + } + imgTags.forEach {imgTag in guard let imgClass = try? imgTag.attr(ImageKeys.classAttributes) else { return } @@ -57,8 +59,7 @@ class GutenbergGalleryUploadProcessor: GutenbergProcessor { _ = try? imgTag.attr(ImageKeys.dataID, "\(self.serverMediaID)") _ = try? imgTag.attr(ImageKeys.dataFullURL, self.remoteURLString) - let dataLinkAttribute = try? imgTag.attr(ImageKeys.dataLink) - if dataLinkAttribute != nil { + if let _ = try? imgTag.attr(ImageKeys.dataLink) { _ = try? imgTag.attr(ImageKeys.dataLink, self.mediaLink) } } @@ -69,8 +70,10 @@ class GutenbergGalleryUploadProcessor: GutenbergProcessor { } func processLinkPostMediaUpload(_ block: GutenbergParsedBlock) { - let aTags = try? block.elements.select(LinkKeys.name) - aTags?.forEach { aTag in + guard let aTags = try? block.elements.select(LinkKeys.name) else { + return + } + aTags.forEach { aTag in guard let linkOriginalContent = try? aTag.html() else { return }