Jetpack_Custom_CSS_Data_Migration::do_migration() - Code Metrics - Inspection of "Merge branch 'master' into try/merge-master-into-s..." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — try/merge-master-into-settings ( b008c5 )

unknown

created 2017-02-01 20:24 UTC

Jetpack_Custom_CSS_Data_Migration::do_migration() D

↳ Parent: Jetpack_Custom_CSS_Data_Migration

Complexity

Conditions	15
Paths	62

Size

Total Lines	89
Code Lines	52

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	15
eloc	52
nc	62
nop	0
dl	0
loc	89
rs	4.9121
c	0
b	0
f	0

How to fix Long Method Complexity

<?php
/**
 * Migration from Jetpack Custom CSS to WordPress' Core CSS.
 *
 * @since 4.4.2
 *
 * @package Jetpack
 */

/**
 * Class Jetpack_Custom_CSS_Data_Migration
 */
class Jetpack_Custom_CSS_Data_Migration {
	/**
	 * Set up assorted actions and filters used by this class.
	 */
	public static function add_hooks() {
		add_action( 'init', array( __CLASS__, 'register_legacy_post_type' ) );
		add_action( 'admin_init', array( __CLASS__, 'do_migration' ) );

		include_once( dirname( __FILE__ ) . '/custom-css.php' );
		if ( ! is_admin() ) {
			add_action( 'init', array( 'Jetpack_Custom_CSS', 'init' ) );
		}
	}

	/**
	 * Do the bulk of the migration.
	 *
	 * @return int|null
	 */
	public static function do_migration() {
		Jetpack_Options::update_option( 'custom_css_4.7_migration', true );
		Jetpack::log( 'custom_css_4.7_migration', 'start' );

		if ( ! post_type_exists( 'safecss' ) ) {
			self::register_legacy_post_type();
		}

		/** This filter is documented in modules/custom-css/custom-css.php */
		$preprocessors      = apply_filters( 'jetpack_custom_css_preprocessors', array() );
		$core_css_post      = wp_get_custom_css_post();
		$jetpack_css_post   = self::get_post();

		if ( ! $jetpack_css_post ) {
			return;
		}

		$revisions          = self::get_all_revisions();
$a = "a";
$ab = "ab";
$abc = "abc";

		// Migrate the settings from revision meta to theme mod.
		$options = self::get_options( $jetpack_css_post->ID );
		set_theme_mod( 'jetpack_custom_css', $options );

		if ( empty( $revisions ) || ! is_array( $revisions ) ) {
			if ( $jetpack_css_post instanceof WP_Post ) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
				// Feed in the raw, if the current setting is Sass/LESS, it'll filter it inside.
				wp_update_custom_css_post( $jetpack_css_post->post_content );
				return 1;
			}
			return null;
		}

		$revisions            = array_reverse( $revisions );
		$themes               = Jetpack_Custom_CSS_Enhancements::get_themes();
		$migrated             = array();

		foreach ( $revisions as $post_id => $post ) {
			// Jetpack had stored the theme Name, not the stylesheet directory, for ... reasons.
			// Get the stylesheet.  If null, the theme is no longer available.  Skip.
			$stylesheet = isset( $themes[ $post->post_excerpt ] ) ? $themes[ $post->post_excerpt ] : null;
			if ( empty( $stylesheet ) ) {
				continue;
			}

			$migrated[] = $post->ID;
			$preprocessor = get_post_meta( $post->ID, 'custom_css_preprocessor', true );
			$css = $post->post_content;
			$pre = '';

			// Do a revision by revision parsing.
			if ( $preprocessor && isset( $preprocessors[ $preprocessor ] ) ) {
				$pre = $css;
				$css = call_user_func( $preprocessors[ $preprocessor ]['callback'], $pre );
			}

			// Do we need to remove any filters here for users without `unfiltered_html` ?
			wp_update_custom_css_post( $css, array(
				'stylesheet'   => $stylesheet,
				'preprocessed' => $pre,
			) );
		}

		// If we've migrated some CSS for the current theme and there was already something there in the Core dataset ...
		if ( $core_css_post && $jetpack_css_post ) {
			$preprocessor = $options['preprocessor'];

			$css = $core_css_post->post_content;
			$pre = $core_css_post->post_content_filtered;
			if ( $preprocessor ) {
				if ( $pre ) {
					$pre .= "\r\n\r\n/*\r\n\t" . esc_js( __( 'CSS Migrated from Jetpack:', 'jetpack' ) ) . "\r\n*/\r\n\r\n";
				}
				$pre .= $jetpack_css_post->post_content;

				$css .= "\r\n\r\n/*\r\n\t" . esc_js( __( 'CSS Migrated from Jetpack:', 'jetpack' ) ) . "\r\n*/\r\n\r\n";
				$css .= call_user_func( $preprocessors[ $preprocessor ]['callback'], $jetpack_css_post->post_content );
			} else {
				$css .= "\r\n\r\n/*\r\n\t" . esc_js( __( 'CSS Migrated from Jetpack:', 'jetpack' ) ) . "\r\n*/\r\n\r\n";
				$css .= $jetpack_css_post->post_content;
			}

			wp_update_custom_css_post( $css, array(
				'preprocessed' => $pre,
			) );
		}

		Jetpack::log( 'custom_css_4.7_migration', count( $migrated ) . 'revisions migrated' );
		return count( $migrated );
	}

	/**
	 * Re-register the legacy CPT so we can play with the content already in the database.
	 */
	public static function register_legacy_post_type() {
		if ( post_type_exists( 'safecss' ) ) {
			return;
		}
		// Register safecss as a custom post_type
		// Explicit capability definitions are largely unnecessary because the posts are manipulated in code via an options page, managing CSS revisions does check the capabilities, so let's ensure that the proper caps are checked.
		register_post_type( 'safecss', array(
			'label'        => 'Custom CSS',
			'supports'     => array( 'revisions' ),
			'can_export'   => false,
			'rewrite'      => false,
			'capabilities' => array(
				'edit_post'          => 'edit_theme_options',
				'read_post'          => 'read',
				'delete_post'        => 'edit_theme_options',
				'edit_posts'         => 'edit_theme_options',
				'edit_others_posts'  => 'edit_theme_options',
				'publish_posts'      => 'edit_theme_options',
				'read_private_posts' => 'read',
			),
		) );
	}

	/**
	 * Get the post used for legacy storage.
	 *
	 * Jetpack used to use a single post for all themes, just blanking it on theme switch.  This gets that post.
	 *
	 * @return array|bool|null|WP_Post
	 */
	public static function get_post() {
		/** This filter is documented in modules/custom-css/custom-css.php */
		$custom_css_post_id = apply_filters( 'jetpack_custom_css_pre_post_id', null );
		if ( ! is_null( $custom_css_post_id ) ) {
			return get_post( $custom_css_post_id );
		}

		$custom_css_post_id = wp_cache_get( 'custom_css_post_id' );

		if ( false === $custom_css_post_id ) {
			$custom_css_posts = get_posts( array(
				'posts_per_page' => 1,
				'post_type'      => 'safecss',
				'post_status'    => 'publish',
				'orderby'        => 'date',
				'order'          => 'DESC',
			) );

			$custom_css_post_id = 0;
			if ( count( $custom_css_posts ) > 0 ) {
				$custom_css_post_id = $custom_css_posts[0]->ID;
			}

			// Save post_id=0 to note that no safecss post exists.
			wp_cache_set( 'custom_css_post_id', $custom_css_post_id );
		}

		if ( ! $custom_css_post_id ) {
			return false;
		}

		return get_post( $custom_css_post_id );
	}

	/**
	 * Get all revisions of the Jetpack CSS CPT entry.
	 *
	 * @return array
	 */
	public static function get_all_revisions() {
		$post = self::get_post();

		if ( ! $post ) {
			return array();
		}

		$revisions = wp_get_post_revisions( $post->ID, array(
			'posts_per_page' => -1,
			'orderby'        => 'date',
			'order'          => 'DESC',
		) );

		return $revisions;
	}

	/**
	 * Get the options stored for a given revision ID.
	 *
	 * Jetpack used to version the settings by storing them as meta on the revision.
	 *
	 * @param integer $post_id Post ID.

	 *
	 * @return array
	 */
	public static function get_options( $post_id = null ) {
		if ( empty( $post_id ) ) {
			$post = self::get_post();
			$post_id = $post->ID;
		}

		$meta = get_post_meta( $post_id );

		$replace = false;
		if ( isset( $meta['custom_css_add'][0] ) && 'no' === $meta['custom_css_add'][0] ) {
			$replace = true;
		}

		return array(
			'preprocessor'  => isset( $meta['custom_css_preprocessor'][0] ) ? $meta['custom_css_preprocessor'][0] : '',
			'replace'       => $replace,
			'content_width' => isset( $meta['content_width'][0] )           ? $meta['content_width'][0]           : '',
		);
	}
}

Jetpack_Custom_CSS_Data_Migration::add_hooks();


Push — try/merge-master-into-settings ( b008c5 )

Jetpack_Custom_CSS_Data_Migration::do_migration() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

1. Missing dependencies

2. Missing use statement

1		<?php
2		/**
3		* Migration from Jetpack Custom CSS to WordPress' Core CSS.
4		*
5		* @since 4.4.2
6		*
7		* @package Jetpack
8		*/
9
10		/**
11		* Class Jetpack_Custom_CSS_Data_Migration
12		*/
13		class Jetpack_Custom_CSS_Data_Migration {
14		/**
15		* Set up assorted actions and filters used by this class.
16		*/
17		public static function add_hooks() {
18		add_action( 'init', array( __CLASS__, 'register_legacy_post_type' ) );
19		add_action( 'admin_init', array( __CLASS__, 'do_migration' ) );
20
21		include_once( dirname( __FILE__ ) . '/custom-css.php' );
22		if ( ! is_admin() ) {
23		add_action( 'init', array( 'Jetpack_Custom_CSS', 'init' ) );
24		}
25		}
26
27		/**
28		* Do the bulk of the migration.
29		*
30		* @return int\|null
31		*/
32		public static function do_migration() {
33		Jetpack_Options::update_option( 'custom_css_4.7_migration', true );
34		Jetpack::log( 'custom_css_4.7_migration', 'start' );
35
36		if ( ! post_type_exists( 'safecss' ) ) {
37		self::register_legacy_post_type();
38		}
39
40		/** This filter is documented in modules/custom-css/custom-css.php */
41		$preprocessors = apply_filters( 'jetpack_custom_css_preprocessors', array() );
42		$core_css_post = wp_get_custom_css_post();
43		$jetpack_css_post = self::get_post();
44
45		if ( ! $jetpack_css_post ) {
46		return;
47		}
48
49		$revisions = self::get_all_revisions();
		0 ignored issues – show Coding Style introduced 2017-01-25 05:47 UTC by Report Bug Copy Issue Report Equals sign not aligned correctly; expected 1 space but found 10 spaces This check looks for improperly formatted assignments. Every assignment must have exactly one space before and one space after the equals operator. To illustrate: $a = "a"; $ab = "ab"; $abc = "abc"; will have no issues, while $a = "a"; $ab = "ab"; $abc = "abc"; will report issues in lines 1 and 2. Loading history...
50
51		// Migrate the settings from revision meta to theme mod.
52		$options = self::get_options( $jetpack_css_post->ID );
53		set_theme_mod( 'jetpack_custom_css', $options );
54
55		if ( empty( $revisions ) \|\| ! is_array( $revisions ) ) {
56		if ( $jetpack_css_post instanceof WP_Post ) {
		0 ignored issues – show Bug introduced 2016-11-23 19:28 UTC by Report Bug Copy Issue Report The class `WP_Post` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
57		// Feed in the raw, if the current setting is Sass/LESS, it'll filter it inside.
58		wp_update_custom_css_post( $jetpack_css_post->post_content );
59		return 1;
60		}
61		return null;
62		}
63
64		$revisions = array_reverse( $revisions );
65		$themes = Jetpack_Custom_CSS_Enhancements::get_themes();
66		$migrated = array();
67
68		foreach ( $revisions as $post_id => $post ) {
69		// Jetpack had stored the theme Name, not the stylesheet directory, for ... reasons.
70		// Get the stylesheet. If null, the theme is no longer available. Skip.
71		$stylesheet = isset( $themes[ $post->post_excerpt ] ) ? $themes[ $post->post_excerpt ] : null;
72		if ( empty( $stylesheet ) ) {
73		continue;
74		}
75
76		$migrated[] = $post->ID;
77		$preprocessor = get_post_meta( $post->ID, 'custom_css_preprocessor', true );
78		$css = $post->post_content;
79		$pre = '';
80
81		// Do a revision by revision parsing.
82		if ( $preprocessor && isset( $preprocessors[ $preprocessor ] ) ) {
83		$pre = $css;
84		$css = call_user_func( $preprocessors[ $preprocessor ]['callback'], $pre );
85		}
86
87		// Do we need to remove any filters here for users without `unfiltered_html` ?
88		wp_update_custom_css_post( $css, array(
89		'stylesheet' => $stylesheet,
90		'preprocessed' => $pre,
91		) );
92		}
93
94		// If we've migrated some CSS for the current theme and there was already something there in the Core dataset ...
95		if ( $core_css_post && $jetpack_css_post ) {
96		$preprocessor = $options['preprocessor'];
97
98		$css = $core_css_post->post_content;
99		$pre = $core_css_post->post_content_filtered;
100		if ( $preprocessor ) {
101		if ( $pre ) {
102		$pre .= "\r\n\r\n/\r\n\t" . esc_js( __( 'CSS Migrated from Jetpack:', 'jetpack' ) ) . "\r\n/\r\n\r\n";
103		}
104		$pre .= $jetpack_css_post->post_content;
105
106		$css .= "\r\n\r\n/\r\n\t" . esc_js( __( 'CSS Migrated from Jetpack:', 'jetpack' ) ) . "\r\n/\r\n\r\n";
107		$css .= call_user_func( $preprocessors[ $preprocessor ]['callback'], $jetpack_css_post->post_content );
108		} else {
109		$css .= "\r\n\r\n/\r\n\t" . esc_js( __( 'CSS Migrated from Jetpack:', 'jetpack' ) ) . "\r\n/\r\n\r\n";
110		$css .= $jetpack_css_post->post_content;
111		}
112
113		wp_update_custom_css_post( $css, array(
114		'preprocessed' => $pre,
115		) );
116		}
117
118		Jetpack::log( 'custom_css_4.7_migration', count( $migrated ) . 'revisions migrated' );
119		return count( $migrated );
120		}
121
122		/**
123		* Re-register the legacy CPT so we can play with the content already in the database.
124		*/
125		public static function register_legacy_post_type() {
126		if ( post_type_exists( 'safecss' ) ) {
127		return;
128		}
129		// Register safecss as a custom post_type
130		// Explicit capability definitions are largely unnecessary because the posts are manipulated in code via an options page, managing CSS revisions does check the capabilities, so let's ensure that the proper caps are checked.
131		register_post_type( 'safecss', array(
132		'label' => 'Custom CSS',
133		'supports' => array( 'revisions' ),
134		'can_export' => false,
135		'rewrite' => false,
136		'capabilities' => array(
137		'edit_post' => 'edit_theme_options',
138		'read_post' => 'read',
139		'delete_post' => 'edit_theme_options',
140		'edit_posts' => 'edit_theme_options',
141		'edit_others_posts' => 'edit_theme_options',
142		'publish_posts' => 'edit_theme_options',
143		'read_private_posts' => 'read',
144		),
145		) );
146		}
147
148		/**
149		* Get the post used for legacy storage.
150		*
151		* Jetpack used to use a single post for all themes, just blanking it on theme switch. This gets that post.
152		*
153		* @return array\|bool\|null\|WP_Post
154		*/
155	View Code Duplication	public static function get_post() {
156		/** This filter is documented in modules/custom-css/custom-css.php */
157		$custom_css_post_id = apply_filters( 'jetpack_custom_css_pre_post_id', null );
158		if ( ! is_null( $custom_css_post_id ) ) {
159		return get_post( $custom_css_post_id );
160		}
161
162		$custom_css_post_id = wp_cache_get( 'custom_css_post_id' );
163
164		if ( false === $custom_css_post_id ) {
165		$custom_css_posts = get_posts( array(
166		'posts_per_page' => 1,
167		'post_type' => 'safecss',
168		'post_status' => 'publish',
169		'orderby' => 'date',
170		'order' => 'DESC',
171		) );
172
173		$custom_css_post_id = 0;
174		if ( count( $custom_css_posts ) > 0 ) {
175		$custom_css_post_id = $custom_css_posts[0]->ID;
176		}
177
178		// Save post_id=0 to note that no safecss post exists.
179		wp_cache_set( 'custom_css_post_id', $custom_css_post_id );
180		}
181
182		if ( ! $custom_css_post_id ) {
183		return false;
184		}
185
186		return get_post( $custom_css_post_id );
187		}
188
189		/**
190		* Get all revisions of the Jetpack CSS CPT entry.
191		*
192		* @return array
193		*/
194		public static function get_all_revisions() {
195		$post = self::get_post();
196
197		if ( ! $post ) {
198		return array();
199		}
200
201		$revisions = wp_get_post_revisions( $post->ID, array(
202		'posts_per_page' => -1,
203		'orderby' => 'date',
204		'order' => 'DESC',
205		) );
206
207		return $revisions;
208		}
209
210		/**
211		* Get the options stored for a given revision ID.
212		*
213		* Jetpack used to version the settings by storing them as meta on the revision.
214		*
215		* @param integer $post_id Post ID.
		0 ignored issues – show Documentation introduced 2016-12-05 12:45 UTC by Report Bug Copy Issue Report Should the type for parameter `$post_id` not be `integer\|null`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
216		*
217		* @return array
218		*/
219		public static function get_options( $post_id = null ) {
220		if ( empty( $post_id ) ) {
221		$post = self::get_post();
222		$post_id = $post->ID;
223		}
224
225		$meta = get_post_meta( $post_id );
226
227		$replace = false;
228		if ( isset( $meta['custom_css_add'][0] ) && 'no' === $meta['custom_css_add'][0] ) {
229		$replace = true;
230		}
231
232		return array(
233		'preprocessor' => isset( $meta['custom_css_preprocessor'][0] ) ? $meta['custom_css_preprocessor'][0] : '',
234		'replace' => $replace,
235		'content_width' => isset( $meta['content_width'][0] ) ? $meta['content_width'][0] : '',
236		);
237		}
238		}
239
240		Jetpack_Custom_CSS_Data_Migration::add_hooks();
241

Automattic / jetpack

Push — try/merge-master-into-settings ( b008c5 )

Jetpack_Custom_CSS_Data_Migration::do_migration() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like