Conditions | 45 |
Paths | > 20000 |
Total Lines | 183 |
Code Lines | 99 |
Lines | 25 |
Ratio | 13.66 % |
Changes | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
1 | <?php |
||
3 | static function partner_provision( $access_token, $named_args ) { |
||
4 | // first, verify the token |
||
5 | $verify_response = self::verify_token( $access_token ); |
||
6 | |||
7 | if ( is_wp_error( $verify_response ) ) { |
||
8 | return $verify_response; |
||
9 | } |
||
10 | |||
11 | $url_args = array( |
||
12 | 'home_url' => 'WP_HOME', |
||
13 | 'site_url' => 'WP_SITEURL', |
||
14 | ); |
||
15 | |||
16 | foreach ( $url_args as $url_arg => $constant_name ) { |
||
17 | // Anonymous functions were introduced in 5.3.0. So, if we're running on |
||
18 | // >= 5.3.0, use an anonymous function to set the home/siteurl value%s. |
||
19 | // |
||
20 | // Otherwise, fallback to setting the home/siteurl value via the WP_HOME and |
||
21 | // WP_SITEURL constants if the constant hasn't already been defined. |
||
22 | if ( isset( $named_args[ $url_arg ] ) ) { |
||
23 | if ( version_compare( phpversion(), '5.3.0', '>=') ) { |
||
24 | add_filter( $url_arg, function( $url ) use ( $url_arg, $named_args ) { |
||
25 | return $named_args[ $url_arg ]; |
||
26 | }, 11 ); |
||
27 | } else if ( ! defined( $constant_name ) ) { |
||
28 | define( $constant_name, $named_args[ $url_arg ] ); |
||
29 | } |
||
30 | } |
||
31 | } |
||
32 | |||
33 | // If Jetpack is currently connected, and is not in Safe Mode already, kick off a sync of the current |
||
34 | // functions/callables so that we can test if this site is in IDC. |
||
35 | if ( Jetpack::is_active() && ! Jetpack::validate_sync_error_idc_option() && Jetpack_Sync_Actions::sync_allowed() ) { |
||
36 | Jetpack_Sync_Actions::do_full_sync( array( 'functions' => true ) ); |
||
37 | Jetpack_Sync_Actions::$sender->do_full_sync(); |
||
|
|||
38 | } |
||
39 | |||
40 | if ( Jetpack::validate_sync_error_idc_option() ) { |
||
41 | return new WP_Error( |
||
42 | 'site_in_safe_mode', |
||
43 | esc_html__( 'Can not provision a plan while in safe mode. See: https://jetpack.com/support/safe-mode/', 'jetpack' ) |
||
44 | ); |
||
45 | } |
||
46 | |||
47 | $blog_id = Jetpack_Options::get_option( 'id' ); |
||
48 | $blog_token = Jetpack_Options::get_option( 'blog_token' ); |
||
49 | |||
50 | if ( ! $blog_id || ! $blog_token || ( isset( $named_args['force_register'] ) && intval( $named_args['force_register'] ) ) ) { |
||
51 | // this code mostly copied from Jetpack::admin_page_load |
||
52 | Jetpack::maybe_set_version_option(); |
||
53 | $registered = Jetpack::try_registration(); |
||
54 | if ( is_wp_error( $registered ) ) { |
||
55 | return $registered; |
||
56 | } elseif ( ! $registered ) { |
||
57 | return new WP_Error( 'registration_error', __( 'There was an unspecified error registering the site', 'jetpack' ) ); |
||
58 | } |
||
59 | |||
60 | $blog_id = Jetpack_Options::get_option( 'id' ); |
||
61 | $blog_token = Jetpack_Options::get_option( 'blog_token' ); |
||
62 | } |
||
63 | |||
64 | // if the user isn't specified, but we have a current master user, then set that to current user |
||
65 | if ( ! get_current_user_id() && $master_user_id = Jetpack_Options::get_option( 'master_user' ) ) { |
||
66 | wp_set_current_user( $master_user_id ); |
||
67 | } |
||
68 | |||
69 | $site_icon = ( function_exists( 'has_site_icon') && has_site_icon() ) |
||
70 | ? get_site_icon_url() |
||
71 | : false; |
||
72 | |||
73 | $auto_enable_sso = ( ! Jetpack::is_active() || Jetpack::is_module_active( 'sso' ) ); |
||
74 | |||
75 | /** This filter is documented in class.jetpack-cli.php */ |
||
76 | if ( apply_filters( 'jetpack_start_enable_sso', $auto_enable_sso ) ) { |
||
77 | $redirect_uri = add_query_arg( |
||
78 | array( 'action' => 'jetpack-sso', 'redirect_to' => urlencode( admin_url() ) ), |
||
79 | wp_login_url() // TODO: come back to Jetpack dashboard? |
||
80 | ); |
||
81 | } else { |
||
82 | $redirect_uri = admin_url(); |
||
83 | } |
||
84 | |||
85 | $request_body = array( |
||
86 | 'jp_version' => JETPACK__VERSION, |
||
87 | 'redirect_uri' => $redirect_uri |
||
88 | ); |
||
89 | |||
90 | if ( $site_icon ) { |
||
91 | $request_body['site_icon'] = $site_icon; |
||
92 | } |
||
93 | |||
94 | if ( get_current_user_id() ) { |
||
95 | $user = wp_get_current_user(); |
||
96 | |||
97 | // role |
||
98 | $role = Jetpack::translate_current_user_to_role(); |
||
99 | $signed_role = Jetpack::sign_role( $role ); |
||
100 | |||
101 | $secrets = Jetpack::init()->generate_secrets( 'authorize' ); |
||
102 | |||
103 | // Jetpack auth stuff |
||
104 | $request_body['scope'] = $signed_role; |
||
105 | $request_body['secret'] = $secrets['secret_1']; |
||
106 | |||
107 | // User stuff |
||
108 | $request_body['user_id'] = $user->ID; |
||
109 | $request_body['user_email'] = $user->user_email; |
||
110 | $request_body['user_login'] = $user->user_login; |
||
111 | } |
||
112 | |||
113 | // optional additional params |
||
114 | View Code Duplication | if ( isset( $named_args['wpcom_user_id'] ) && ! empty( $named_args['wpcom_user_id'] ) ) { |
|
115 | $request_body['wpcom_user_id'] = $named_args['wpcom_user_id']; |
||
116 | } |
||
117 | |||
118 | // override email of selected user |
||
119 | View Code Duplication | if ( isset( $named_args['wpcom_user_email'] ) && ! empty( $named_args['wpcom_user_email'] ) ) { |
|
120 | $request_body['user_email'] = $named_args['wpcom_user_email']; |
||
121 | } |
||
122 | |||
123 | View Code Duplication | if ( isset( $named_args['plan'] ) && ! empty( $named_args['plan'] ) ) { |
|
124 | $request_body['plan'] = $named_args['plan']; |
||
125 | } |
||
126 | |||
127 | View Code Duplication | if ( isset( $named_args['onboarding'] ) && ! empty( $named_args['onboarding'] ) ) { |
|
128 | $request_body['onboarding'] = intval( $named_args['onboarding'] ); |
||
129 | } |
||
130 | |||
131 | View Code Duplication | if ( isset( $named_args['force_connect'] ) && ! empty( $named_args['force_connect'] ) ) { |
|
132 | $request_body['force_connect'] = intval( $named_args['force_connect'] ); |
||
133 | } |
||
134 | |||
135 | if ( isset( $request_body['onboarding'] ) && (bool) $request_body['onboarding'] ) { |
||
136 | Jetpack::create_onboarding_token(); |
||
137 | } |
||
138 | |||
139 | $request = array( |
||
140 | 'headers' => array( |
||
141 | 'Authorization' => "Bearer " . $access_token, |
||
142 | 'Host' => defined( 'JETPACK__WPCOM_JSON_API_HOST_HEADER' ) ? JETPACK__WPCOM_JSON_API_HOST_HEADER : 'public-api.wordpress.com', |
||
143 | ), |
||
144 | 'timeout' => 60, |
||
145 | 'method' => 'POST', |
||
146 | 'body' => json_encode( $request_body ) |
||
147 | ); |
||
148 | |||
149 | $url = sprintf( 'https://%s/rest/v1.3/jpphp/%d/partner-provision', self::get_api_host(), $blog_id ); |
||
150 | View Code Duplication | if ( ! empty( $named_args['partner-tracking-id'] ) ) { |
|
151 | $url = esc_url_raw( add_query_arg( 'partner_tracking_id', $named_args['partner-tracking-id'], $url ) ); |
||
152 | } |
||
153 | |||
154 | // add calypso env if set |
||
155 | if ( getenv( 'CALYPSO_ENV' ) ) { |
||
156 | $url = add_query_arg( array( 'calypso_env' => getenv( 'CALYPSO_ENV' ) ), $url ); |
||
157 | } |
||
158 | |||
159 | $result = Jetpack_Client::_wp_remote_request( $url, $request ); |
||
160 | |||
161 | if ( is_wp_error( $result ) ) { |
||
162 | return $result; |
||
163 | } |
||
164 | |||
165 | $response_code = wp_remote_retrieve_response_code( $result ); |
||
166 | $body_json = json_decode( wp_remote_retrieve_body( $result ) ); |
||
167 | |||
168 | View Code Duplication | if( 200 !== $response_code ) { |
|
169 | if ( isset( $body_json->error ) ) { |
||
170 | return new WP_Error( $body_json->error, $body_json->message ); |
||
171 | } else { |
||
172 | return new WP_Error( 'server_error', sprintf( __( "Request failed with code %s" ), $response_code ) ); |
||
173 | } |
||
174 | } |
||
175 | |||
176 | if ( isset( $body_json->access_token ) ) { |
||
177 | // check if this matches the existing token before replacing |
||
178 | $existing_token = Jetpack_Data::get_access_token( $user->ID ); |
||
179 | if ( empty( $existing_token ) || $existing_token->secret !== $body_json->access_token ) { |
||
180 | self::authorize_user( $user->ID, $body_json->access_token ); |
||
181 | } |
||
182 | } |
||
183 | |||
184 | return $body_json; |
||
185 | } |
||
186 | |||
247 |
This check looks for access to properties that are not accessible from the current context.
If you need to make a property accessible to another context you can either raise its visibility level or provide an accessible getter in the defining class.