Merge pull request #201 from 10up/fix/large-svg-items

Add option to enable large SVGs

composer.json

@@ -11,7 +11,7 @@
 			"email": "[email protected]",
 			"homepage": "https://10up.com/",
 			"role": "Developer"
-		},		
+		},
 		{
 			"name": "Daryll Doyle",
 			"email": "[email protected]",

includes/safe-svg-settings.php

@@ -25,6 +25,15 @@ public function __construct() {
 	 */
 	public function settings_init() {
 		register_setting( 'media', 'safe_svg_upload_roles', [ $this, 'sanitize_safe_svg_roles' ] );
+		register_setting(
+			'media',
+			'safe_svg_large_svg',
+			[
+				'type'              => 'integer',
+				'default'           => 0,
+				'sanitize_callback' => 'absint',
+			]
+		);
 
 		add_settings_section(
 			'safe_svg_settings',
@@ -40,6 +49,14 @@ public function settings_init() {
 			'media',
 			'safe_svg_settings'
 		);
+
+		add_settings_field(
+			'safe_svg_large_svg',
+			__( 'Large Files', 'safe-svg' ),
+			[ $this, 'safe_svg_large_svg_cb' ],
+			'media',
+			'safe_svg_settings'
+		);
 	}
 
 	/**
@@ -162,4 +179,15 @@ public function update_capability( $new_roles, $old_roles ) {
 		return $new_roles;
 	}
 
+	/**
+	 * Large SVG files field callback function.
+	 */
+	public function safe_svg_large_svg_cb() {
+		?>
+		<label>
+			<input type="checkbox" name="safe_svg_large_svg" value="1" <?php checked( get_option( 'safe_svg_large_svg' ), 1 ); ?> /> <?php esc_html_e( 'Allow large SVG files', 'safe-svg' ); ?>
+		</label>
+		<p class="description"><?php esc_html_e( 'Turning this on will allow SVG files larger than 10MB to be uploaded. This can impact performance and is not recommended unless needed.', 'safe-svg' ); ?></p>
+		<?php
+	}
 }

safe-svg.php

@@ -330,6 +330,11 @@ protected function sanitize( $file ) {
 				}
 			}
 
+			// Allow large SVGs if the setting is on.
+			if ( get_option( 'safe_svg_large_svg' ) ) {
+				$this->sanitizer->setAllowHugeFiles( true );
+			}
+
 			/**
 			 * Load extra filters to allow devs to access the safe tags and attrs by themselves.
 			 */