Merge pull request #736 from ConvertKit/broadcasts-importer-image-cap…

…tion-markup

Broadcasts Importer: Retain `figure` and `figcaption`

includes/class-convertkit-broadcasts-importer.php

@@ -577,6 +577,8 @@ private function permitted_html_tags( $disable_styles = false ) {
 			's',
 			'a',
 			'img',
+			'figure',
+			'figcaption',
 			'br',
 			'style', // Deliberate; we'll use DOMDocument to remove inline styles and their contents.
 		);

tests/wpunit/BroadcastsImportTest.php

@@ -319,6 +319,12 @@ private function assertImportValid($post)
 		// Confirm tracking image has been removed.
 		$this->assertStringNotContainsString('<img src="https://preview.convertkit-mail2.com/open" alt="">', $post->post_content);
 
+		// Confirm images included retain their <figure> and <figcaption> elements.
+		$this->assertStringContainsString('<figure', $post->post_content);
+		$this->assertStringContainsString('<figcaption', $post->post_content);
+		$this->assertStringContainsString('A Sample Caption</figcaption>', $post->post_content);
+		$this->assertStringContainsString('</figure>', $post->post_content);
+
 		// Confirm unsubscribe link section has been removed.
 		$this->assertStringNotContainsString('<div class="ck-section ck-hide-in-public-posts"', $post->post_content);