Complex classes like Jetpack_VideoPress often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use Jetpack_VideoPress, and based on these observations, apply Extract Interface, too.
1 | <?php |
||
7 | class Jetpack_VideoPress { |
||
8 | /** @var string */ |
||
9 | public $module = 'videopress'; |
||
10 | |||
11 | /** @var int */ |
||
12 | public $version = 5; |
||
13 | |||
14 | /** |
||
15 | * Singleton |
||
16 | */ |
||
17 | public static function init() { |
||
26 | |||
27 | /** |
||
28 | * Jetpack_VideoPress constructor. |
||
29 | * |
||
30 | * Sets up the initializer and makes sure that videopress activates and deactivates properly. |
||
31 | */ |
||
32 | private function __construct() { |
||
37 | |||
38 | /** |
||
39 | * Fires on init |
||
40 | */ |
||
41 | public function on_init() { |
||
60 | |||
61 | /** |
||
62 | * Runs when the VideoPress module is deactivated. |
||
63 | */ |
||
64 | public function jetpack_module_deactivated() { |
||
67 | |||
68 | /** |
||
69 | * A can of coke |
||
70 | * |
||
71 | * Similar to current_user_can, but internal to VideoPress. Returns |
||
72 | * true if the given VideoPress capability is allowed by the given user. |
||
73 | */ |
||
74 | public function can( $cap, $user_id = false ) { |
||
100 | |||
101 | /** |
||
102 | * Returns true if the provided user is the Jetpack connection owner. |
||
103 | */ |
||
104 | public function is_connection_owner( $user_id = false ) { |
||
113 | |||
114 | /** |
||
115 | * Register and enqueue VideoPress admin styles. |
||
116 | */ |
||
117 | public function enqueue_admin_styles() { |
||
121 | |||
122 | /** |
||
123 | * Register VideoPress admin scripts. |
||
124 | */ |
||
125 | public function enqueue_admin_scripts() { |
||
166 | |||
167 | /** |
||
168 | * An override for the attachment url, which returns back the WPCOM VideoPress processed url. |
||
169 | * |
||
170 | * This is an action proxy to the videopress_get_attachment_url() utility function. |
||
171 | * |
||
172 | * @param string $url |
||
173 | * @param int $post_id |
||
174 | * |
||
175 | * @return string |
||
176 | */ |
||
177 | public function update_attachment_url_for_videopress( $url, $post_id ) { |
||
184 | |||
185 | /** |
||
186 | * Modify the default plupload config to turn on videopress specific filters. |
||
187 | */ |
||
188 | public function videopress_pluploder_config( $config ) { |
||
201 | |||
202 | |||
203 | /** |
||
204 | * Helper function to determine if the media uploader should be overridden. |
||
205 | * |
||
206 | * The rules are simple, only try to load the script when on the edit post or new post pages. |
||
207 | * |
||
208 | * @return bool |
||
209 | */ |
||
210 | protected function should_override_media_uploader() { |
||
234 | |||
235 | /** |
||
236 | * A work-around / hack to make it possible to go to the media library with the add new box open. |
||
237 | * |
||
238 | * @return bool |
||
239 | */ |
||
240 | public function print_in_footer_open_media_add_new() { |
||
267 | |||
268 | /** |
||
269 | * Changes the add new menu location, so that VideoPress will be enabled |
||
270 | * when a user clicks that button. |
||
271 | */ |
||
272 | public function change_add_new_menu_location() { |
||
276 | |||
277 | /** |
||
278 | * Makes sure that all video mimes are added in, as multi site installs can remove them. |
||
279 | * |
||
280 | * @param array $existing_mimes |
||
281 | * @return array |
||
282 | */ |
||
283 | public function add_video_upload_mimes( $existing_mimes = array() ) { |
||
296 | |||
297 | /** |
||
298 | * Filter designed to get rid of non video mime types. |
||
299 | * |
||
300 | * @param string $value |
||
301 | * @return int |
||
302 | */ |
||
303 | public function filter_video_mimes( $value ) { |
||
306 | |||
307 | /** |
||
308 | * @param string $icon |
||
309 | * @param string $mime |
||
310 | * @param int $post_id |
||
311 | * |
||
312 | * @return string |
||
313 | */ |
||
314 | public function wp_mime_type_icon( $icon, $mime, $post_id ) { |
||
328 | |||
329 | /** |
||
330 | * @param array $extensions |
||
331 | * |
||
332 | * @return array |
||
333 | */ |
||
334 | public function add_videopress_extenstion( $extensions ) { |
||
339 | } |
||
340 | |||
343 |