yemenifree / wp-security

src/Helpers/WPGitHubUpdater.php

Braces +65 added lines, -44 removed lines

@@ -101,8 +101,9 @@ 
             'readme',
         );
         foreach ($required_config_params as $required_param) {
-            if (empty($this->config[$required_param]))
-                $this->missing_config[] = $required_param;
+            if (empty($this->config[$required_param])) {
+                            $this->missing_config[] = $required_param;
+            }
         }
         return (empty($this->missing_config));
     }
@@ -122,23 +123,31 @@ 
             $zip_url = add_query_arg(array('access_token' => $this->config['access_token']), $zip_url);
             $this->config['zip_url'] = $zip_url;
         }
-        if (!isset($this->config['new_version']))
-            $this->config['new_version'] = $this->get_new_version();
-        if (!isset($this->config['last_updated']))
-            $this->config['last_updated'] = $this->get_date();
-        if (!isset($this->config['description']))
-            $this->config['description'] = $this->get_description();
+        if (!isset($this->config['new_version'])) {
+                    $this->config['new_version'] = $this->get_new_version();
+        }
+        if (!isset($this->config['last_updated'])) {
+                    $this->config['last_updated'] = $this->get_date();
+        }
+        if (!isset($this->config['description'])) {
+                    $this->config['description'] = $this->get_description();
+        }
         $plugin_data = $this->get_plugin_data();
-        if (!isset($this->config['plugin_name']))
-            $this->config['plugin_name'] = $plugin_data['Name'];
-        if (!isset($this->config['version']))
-            $this->config['version'] = $plugin_data['Version'];
-        if (!isset($this->config['author']))
-            $this->config['author'] = $plugin_data['Author'];
-        if (!isset($this->config['homepage']))
-            $this->config['homepage'] = $plugin_data['PluginURI'];
-        if (!isset($this->config['readme']))
-            $this->config['readme'] = 'README.md';
+        if (!isset($this->config['plugin_name'])) {
+                    $this->config['plugin_name'] = $plugin_data['Name'];
+        }
+        if (!isset($this->config['version'])) {
+                    $this->config['version'] = $plugin_data['Version'];
+        }
+        if (!isset($this->config['author'])) {
+                    $this->config['author'] = $plugin_data['Author'];
+        }
+        if (!isset($this->config['homepage'])) {
+                    $this->config['homepage'] = $plugin_data['PluginURI'];
+        }
+        if (!isset($this->config['readme'])) {
+                    $this->config['readme'] = 'README.md';
+        }
     }
 
     /**
@@ -152,32 +161,38 @@ 
         $version = get_site_transient(md5($this->config['slug']) . '_new_version');
         if ($this->overrule_transients() || (!isset($version) || !$version || '' == $version)) {
             $raw_response = $this->remote_get(trailingslashit($this->config['raw_url']) . basename($this->config['slug']));
-            if (is_wp_error($raw_response))
-                $version = false;
+            if (is_wp_error($raw_response)) {
+                            $version = false;
+            }
             if (is_array($raw_response)) {
-                if (!empty($raw_response['body']))
-                    preg_match('/.*Version\:\s*(.*)$/mi', $raw_response['body'], $matches);
+                if (!empty($raw_response['body'])) {
+                                    preg_match('/.*Version\:\s*(.*)$/mi', $raw_response['body'], $matches);
+                }
+            }
+            if (empty($matches[1])) {
+                            $version = false;
+            } else {
+                            $version = $matches[1];
             }
-            if (empty($matches[1]))
-                $version = false;
-            else
-                $version = $matches[1];
             // back compat for older readme version handling
             // only done when there is no version found in file name
             if (false === $version) {
                 $raw_response = $this->remote_get(trailingslashit($this->config['raw_url']) . $this->config['readme']);
-                if (is_wp_error($raw_response))
-                    return $version;
+                if (is_wp_error($raw_response)) {
+                                    return $version;
+                }
                 preg_match('#^\s*`*~Current Version\:\s*([^~]*)~#im', $raw_response['body'], $__version);
                 if (isset($__version[1])) {
                     $version_readme = $__version[1];
-                    if (-1 == version_compare($version, $version_readme))
-                        $version = $version_readme;
+                    if (-1 == version_compare($version, $version_readme)) {
+                                            $version = $version_readme;
+                    }
                 }
             }
             // refresh every 6 hours
-            if (false !== $version)
-                set_site_transient(md5($this->config['slug']) . '_new_version', $version, 60 * 60 * 6);
+            if (false !== $version) {
+                            set_site_transient(md5($this->config['slug']) . '_new_version', $version, 60 * 60 * 6);
+            }
         }
         return $version;
     }
@@ -202,8 +217,9 @@ 
      */
     public function remote_get($query)
     {
-        if (!empty($this->config['access_token']))
-            $query = add_query_arg(array('access_token' => $this->config['access_token']), $query);
+        if (!empty($this->config['access_token'])) {
+                    $query = add_query_arg(array('access_token' => $this->config['access_token']), $query);
+        }
         $raw_response = wp_remote_get($query, array(
             'sslverify' => $this->config['sslverify']
         ));
@@ -236,8 +252,9 @@ 
             $github_data = get_site_transient(md5($this->config['slug']) . '_github_data');
             if ($this->overrule_transients() || (!isset($github_data) || !$github_data || '' == $github_data)) {
                 $github_data = $this->remote_get($this->config['api_url']);
-                if (is_wp_error($github_data))
-                    return false;
+                if (is_wp_error($github_data)) {
+                                    return false;
+                }
                 $github_data = json_decode($github_data['body']);
                 // refresh every 6 hours
                 set_site_transient(md5($this->config['slug']) . '_github_data', $github_data, 60 * 60 * 6);
@@ -294,8 +311,9 @@ 
      */
     public function http_request_sslverify($args, $url)
     {
-        if ($this->config['zip_url'] == $url)
-            $args['sslverify'] = $this->config['sslverify'];
+        if ($this->config['zip_url'] == $url) {
+                    $args['sslverify'] = $this->config['sslverify'];
+        }
         return $args;
     }
 
@@ -312,8 +330,9 @@ 
     {
         // Check if the transient contains the 'checked' information
         // If not, just return its value without hacking it
-        if (empty($transient->checked))
-            return $transient;
+        if (empty($transient->checked)) {
+                    return $transient;
+        }
         // check the version and decide if it's new
         $update = version_compare($this->config['new_version'], $this->config['version']);
         if (1 === $update) {
@@ -323,8 +342,9 @@ 
             $response->url = add_query_arg(array('access_token' => $this->config['access_token']), $this->config['github_url']);
             $response->package = $this->config['zip_url'];
             // If response is false, don't alter the transient
-            if (false !== $response)
-                $transient->response[$this->config['slug']] = $response;
+            if (false !== $response) {
+                            $transient->response[$this->config['slug']] = $response;
+            }
         }
         return $transient;
     }
@@ -343,8 +363,9 @@ 
     public function get_plugin_info($false, $action, $response)
     {
         // Check if this call API is for the right plugin
-        if (!isset($response->slug) || $response->slug != $this->config['slug'])
-            return false;
+        if (!isset($response->slug) || $response->slug != $this->config['slug']) {
+                    return false;
+        }
         $response->slug = $this->config['slug'];
         $response->plugin_name = $this->config['plugin_name'];
         $response->version = $this->config['new_version'];