+global\Jetpack_Carousel::add_data_to_container()

Complexity

Conditions	11
Paths	4

Size

Total Lines

87

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
	function add_data_to_container( $html ) {
		global $post;
		if (
			class_exists( 'Jetpack_AMP_Support' )
			&& Jetpack_AMP_Support::is_amp_request()
		) {
			return $html;
		}

		if ( isset( $post ) ) {
			$blog_id = (int) get_current_blog_id();

			$extra_data = array(
				'data-carousel-extra' => array(
					'blog_id'   => $blog_id,
					'permalink' => get_permalink( $post->ID ),
				),
			);

			/**
			 * Filter the data added to the Gallery container.
			 *
			 * @module carousel
			 *
			 * @since 1.6.0
			 *
			 * @param array $extra_data Array of data about the site and the post.
			 */
			$extra_data = apply_filters( 'jp_carousel_add_data_to_container', $extra_data );

			foreach ( (array) $extra_data as $data_key => $data_values ) {
				// Do not go any further if DOMDocument is disabled on the server.
				if ( ! class_exists( 'DOMDocument' ) ) {
					return $html;
				}

				// Let's grab all containers from the HTML.
				$dom_doc = new DOMDocument();

				/*
				 * The @ is not enough to suppress errors when dealing with libxml,
				 * we have to tell it directly how we want to handle errors.
				 */
				$old_libxml_disable_entity_loader = libxml_disable_entity_loader( true );
				$old_libxml_use_internal_errors   = libxml_use_internal_errors( true );
				@$dom_doc->loadHTML( $html ); // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

				libxml_use_internal_errors( $old_libxml_use_internal_errors );
				libxml_disable_entity_loader( $old_libxml_disable_entity_loader );

				// Let's look for lists and divs.
				$ul_tags  = $dom_doc->getElementsByTagName( 'ul' );
				$div_tags = $dom_doc->getElementsByTagName( 'div' );

				// Loop through each ul, and add the data to it if it is a gallery block.
				foreach ( $ul_tags as $ul_tag ) {
					if ( false !== strpos( $ul_tag->getAttribute( 'class' ), 'wp-block-gallery' ) ) {
						$ul_tag->setAttribute(
							$data_key,
							wp_json_encode( $data_values )
						);
					}
				}

				/*
				 * Loop through each div and add the data, only when it's a gallery block div.
				 * We want to avoid adding data to divs like wp-block-columns.
				 * We do however want data on divs like wp-block-jetpack-tiled-gallery.
				 */
				foreach ( $div_tags as $div_tag ) {
					if (
						false === strpos( $div_tag->getAttribute( 'class' ), 'wp-block-' )
						|| false !== strpos( $div_tag->getAttribute( 'class' ), 'gallery' )
					) {
						$div_tag->setAttribute(
							$data_key,
							wp_json_encode( $data_values )
						);
					}
				}

				// Save our updated HTML.
				$html = $dom_doc->saveHTML();
			}
		}

		return $html;
	}