Skip to content

Commit

Permalink
Merge pull request #1001 from sbenedicadb/mob-20427
Browse files Browse the repository at this point in the history
[MOB-20427] - don't show fullscreenmessage with empty html
  • Loading branch information
cdhoffmann committed Feb 15, 2024
2 parents 81760e5 + 0dddc81 commit 367f883
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions AEPServices/Sources/ui/fullscreen/FullscreenMessage.swift
Original file line number Diff line number Diff line change
Expand Up @@ -345,6 +345,14 @@
}

private func displayWithAnimation(webView: WKWebView) {
// MOB-20427 - only display the WKWebView if we have html to show
guard !self.payload.isEmpty else {
Log.trace(label: self.LOG_PREFIX, "Suppressing the display of a FullscreenMessage because it has no HTML to be shown.")
// reset the monitor so it doesn't think a message is being shown
self.messageMonitor.dismissMessage()
return

Check warning on line 353 in AEPServices/Sources/ui/fullscreen/FullscreenMessage.swift

View check run for this annotation

Codecov / codecov/patch

AEPServices/Sources/ui/fullscreen/FullscreenMessage.swift#L350-L353

Added lines #L350 - L353 were not covered by tests
}

DispatchQueue.main.async {
let keyWindow = UIApplication.shared.getKeyWindow()

Expand Down

0 comments on commit 367f883

Please sign in to comment.