e9ac8ff9a236e0ff1ad9a5c711d3617ceff44c62

Committed 16 Apr 2024 11:54AM UTC coverage: 52.267% (+0.03%) from 52.242%

Build # e9ac8ff9a236e0ff1ad9a5c711d3617ceff44c62

Build Type

push

github

Committed by

web-flow

Commit Message

Merge pull request #21267 from Yoast/21186-refactor-integration-enabledisables

Refactor integration enable disabled for script data.

Run Details

7551 of 13895 branches covered (54.34%)

Branch coverage included in aggregate %.

57 of 85 new or added lines in 8 files covered. (67.06%)

779 existing lines in 1 file now uncovered.

28618 of 55305 relevant lines covered (51.75%)

40616.81 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

71.43

/src/editors/framework/integrations/jetpack-markdown.php

<?php

// @phpcs:disable Yoast.NamingConventions.NamespaceName.TooLong -- This namespace should reflect the namespace of the original class.
namespace Yoast\WP\SEO\Editors\Framework\Integrations;

use Jetpack;
use Yoast\WP\SEO\Editors\Domain\Integrations\Integration_Data_Provider_Interface;

/**
 * Describes if the Jetpack markdown integration is enabled.
 */
class Jetpack_Markdown implements Integration_Data_Provider_Interface {

        /**
         * If the integration is activated.
         *
         * @return bool If the integration is activated.
         */
        public function is_enabled(): bool {
                return $this->is_markdown_enabled();
        }

        /**
         * Return this object represented by a key value array.
         *
         * @return array<string,bool> Returns the name and if the feature is enabled.
         */
        public function to_array(): array {
                return [
                        'markdownEnabled' => $this->is_enabled(),
                ];
        }

        /**
         * Returns this object represented by a key value structure that is compliant with the script data array.
         *
         * @return array<string,bool> Returns the legacy key and if the feature is enabled.
         */
        public function to_legacy_array(): array {
                return [
                        'markdownEnabled' => $this->is_enabled(),
                ];
        }

        /**
         * Checks if Jetpack's markdown module is enabled.
         * Can be extended to work with other plugins that parse markdown in the content.
         *
         * @return bool
         */
        private function is_markdown_enabled() {
                $is_markdown = false;

                if ( \class_exists( 'Jetpack' ) && \method_exists( 'Jetpack', 'get_active_modules' ) ) {
                        $active_modules = Jetpack::get_active_modules();

                        // First at all, check if Jetpack's markdown module is active.
                        $is_markdown = \in_array( 'markdown', $active_modules, true );
                }

                /**
                 * Filters whether markdown support is active in the readability- and seo-analysis.
                 *
                 * @since 11.3
                 *
                 * @param array $is_markdown Is markdown support for Yoast SEO active.
                 */
                return \apply_filters( 'wpseo_is_markdown_enabled', $is_markdown );
        }
}

1	<?php
2
3	// @phpcs:disable Yoast.NamingConventions.NamespaceName.TooLong -- This namespace should reflect the namespace of the original class.
4	namespace Yoast\WP\SEO\Editors\Framework\Integrations;
5
6	use Jetpack;
7	use Yoast\WP\SEO\Editors\Domain\Integrations\Integration_Data_Provider_Interface;
8
9	/**
10	* Describes if the Jetpack markdown integration is enabled.
11	*/
12	class Jetpack_Markdown implements Integration_Data_Provider_Interface {
13
14	/**
15	* If the integration is activated.
16	*
17	* @return bool If the integration is activated.
18	*/
19	public function is_enabled(): bool {	4✔
20	return $this->is_markdown_enabled();	4✔
21	}
22
23	/**
24	* Return this object represented by a key value array.
25	*
26	* @return array<string,bool> Returns the name and if the feature is enabled.
27	*/
NEW 28	public function to_array(): array {	×
29	return [
NEW 30	'markdownEnabled' => $this->is_enabled(),	×
31	];
32	}
33
34	/**
35	* Returns this object represented by a key value structure that is compliant with the script data array.
36	*
37	* @return array<string,bool> Returns the legacy key and if the feature is enabled.
38	*/
39	public function to_legacy_array(): array {	4✔
40	return [	2✔
41	'markdownEnabled' => $this->is_enabled(),	4✔
42	];	2✔
43	}
44
45	/**
46	* Checks if Jetpack's markdown module is enabled.
47	* Can be extended to work with other plugins that parse markdown in the content.
48	*
49	* @return bool
50	*/
51	private function is_markdown_enabled() {	4✔
52	$is_markdown = false;	4✔
53
54	if ( \class_exists( 'Jetpack' ) && \method_exists( 'Jetpack', 'get_active_modules' ) ) {	4✔
NEW 55	$active_modules = Jetpack::get_active_modules();	×
56
57	// First at all, check if Jetpack's markdown module is active.
NEW 58	$is_markdown = \in_array( 'markdown', $active_modules, true );	×
59	}
60
61	/**
62	* Filters whether markdown support is active in the readability- and seo-analysis.
63	*
64	* @since 11.3
65	*
66	* @param array $is_markdown Is markdown support for Yoast SEO active.
67	*/
68	return \apply_filters( 'wpseo_is_markdown_enabled', $is_markdown );	4✔
69	}
70	}

Yoast / wordpress-seo / e9ac8ff9a236e0ff1ad9a5c711d3617ceff44c62

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous