Passed
Push — ci ( c9c5cb...fdc038 )
by litefeel
02:28
created
lib/import.php 1 patch
Spacing   +71 added lines, -71 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
      *
23 23
      * @param Writing_On_GitHub $app Application container.
24 24
      */
25
-    public function __construct( Writing_On_GitHub $app ) {
25
+    public function __construct(Writing_On_GitHub $app) {
26 26
         $this->app = $app;
27 27
     }
28 28
 
@@ -32,24 +32,24 @@  discard block
 block discarded – undo
32 32
      *
33 33
      * @return string|WP_Error
34 34
      */
35
-    public function payload( Writing_On_GitHub_Payload $payload ) {
35
+    public function payload(Writing_On_GitHub_Payload $payload) {
36 36
 
37
-        $result = $this->app->api()->fetch()->compare( $payload->get_before_commit_id() );
37
+        $result = $this->app->api()->fetch()->compare($payload->get_before_commit_id());
38 38
 
39
-        if ( is_wp_error( $result ) ) {
39
+        if (is_wp_error($result)) {
40 40
             /* @var WP_Error $result */
41 41
             return $result;
42 42
         }
43 43
 
44
-        if ( is_array( $result ) ) {
45
-            $result = $this->import_files( $result );
44
+        if (is_array($result)) {
45
+            $result = $this->import_files($result);
46 46
         }
47 47
 
48
-        if ( is_wp_error( $result ) ) {
48
+        if (is_wp_error($result)) {
49 49
             return $files;
50 50
         }
51 51
 
52
-        return __( 'Payload processed', 'writing-on-github' );
52
+        return __('Payload processed', 'writing-on-github');
53 53
     }
54 54
 
55 55
     /**
@@ -58,33 +58,33 @@  discard block
 block discarded – undo
58 58
      *
59 59
      * @return true|WP_Error
60 60
      */
61
-    protected function import_files( $files ) {
61
+    protected function import_files($files) {
62 62
 
63 63
         $error = true;
64 64
 
65
-        foreach ( $files as $file ) {
66
-            if ( ! $this->importable_file( $file ) ) {
65
+        foreach ($files as $file) {
66
+            if ( ! $this->importable_file($file)) {
67 67
                 continue;
68 68
             }
69 69
 
70
-            $blob = $this->app->api()->fetch()->blob( $file );
70
+            $blob = $this->app->api()->fetch()->blob($file);
71 71
             // network error ?
72
-            if ( ! $blob instanceof Writing_On_GitHub_Blob ) {
72
+            if ( ! $blob instanceof Writing_On_GitHub_Blob) {
73 73
                 continue;
74 74
             }
75 75
 
76 76
             $is_remove = 'removed' == $file->status;
77 77
 
78 78
             $result = false;
79
-            if ( $this->importable_raw_file( $blob ) ) {
80
-                $result = $this->import_raw_file( $blob, $is_remove );
81
-            } elseif ( $this->importable_post( $blob ) ) {
82
-                $result = $this->import_post( $blob, $is_remove );
79
+            if ($this->importable_raw_file($blob)) {
80
+                $result = $this->import_raw_file($blob, $is_remove);
81
+            } elseif ($this->importable_post($blob)) {
82
+                $result = $this->import_post($blob, $is_remove);
83 83
             }
84 84
 
85
-            if ( is_wp_error( $result ) ) {
85
+            if (is_wp_error($result)) {
86 86
                 /* @var WP_Error $result */
87
-                $error = wogh_append_error( $error, $result );
87
+                $error = wogh_append_error($error, $result);
88 88
             }
89 89
         }
90 90
 
@@ -99,21 +99,21 @@  discard block
 block discarded – undo
99 99
     public function master() {
100 100
         $result = $this->app->api()->fetch()->tree_recursive();
101 101
 
102
-        if ( is_wp_error( $result ) ) {
102
+        if (is_wp_error($result)) {
103 103
             /* @var WP_Error $result */
104 104
             return $result;
105 105
         }
106 106
 
107
-        if ( is_array( $result ) ) {
108
-            $result = $this->import_files( $result );
107
+        if (is_array($result)) {
108
+            $result = $this->import_files($result);
109 109
         }
110 110
 
111
-        if ( is_wp_error( $result ) ) {
111
+        if (is_wp_error($result)) {
112 112
             /* @var WP_Error $result */
113 113
             return $result;
114 114
         }
115 115
 
116
-        return __( 'Payload processed', 'writing-on-github' );
116
+        return __('Payload processed', 'writing-on-github');
117 117
     }
118 118
 
119 119
     /**
@@ -123,14 +123,14 @@  discard block
 block discarded – undo
123 123
      *
124 124
      * @return bool
125 125
      */
126
-    protected function importable_file( Writing_On_GitHub_File_Info $file ) {
126
+    protected function importable_file(Writing_On_GitHub_File_Info $file) {
127 127
 
128 128
         $path = $file->path;
129 129
 
130 130
         // only _pages, _posts and images
131
-        $prefixs = array( '_pages/', '_posts/', 'images/');
131
+        $prefixs = array('_pages/', '_posts/', 'images/');
132 132
         foreach ($prefixs as $prefix) {
133
-            if ( ! strncasecmp($path, $prefix, strlen( $prefix ) ) ) {
133
+            if ( ! strncasecmp($path, $prefix, strlen($prefix))) {
134 134
                 return true;
135 135
             }
136 136
         }
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
      *
145 145
      * @return bool
146 146
      */
147
-    protected function importable_post( Writing_On_GitHub_Blob $blob ) {
147
+    protected function importable_post(Writing_On_GitHub_Blob $blob) {
148 148
         // global $wpdb;
149 149
 
150 150
         // // Skip the repo's readme.
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
157 157
         //  return false;
158 158
         // }
159 159
 
160
-        if ( ! $blob->has_frontmatter() ) {
160
+        if ( ! $blob->has_frontmatter()) {
161 161
             return false;
162 162
         }
163 163
 
@@ -170,39 +170,39 @@  discard block
 block discarded – undo
170 170
      * @param  bool                   $is_remove
171 171
      * @return WP_Error|bool
172 172
      */
173
-    protected function import_post( Writing_On_GitHub_Blob $blob, $is_remove ) {
173
+    protected function import_post(Writing_On_GitHub_Blob $blob, $is_remove) {
174 174
 
175
-        if ( $is_remove ) {
175
+        if ($is_remove) {
176 176
             $id = $blob->id();
177
-            if ( empty( $id ) ) {
177
+            if (empty($id)) {
178 178
                 return false;
179 179
             }
180
-            $result = $this->app->database()->delete_post( $id );
181
-            if ( is_wp_error( $result ) ) {
180
+            $result = $this->app->database()->delete_post($id);
181
+            if (is_wp_error($result)) {
182 182
                 /* @var WP_Error $result */
183 183
                 return $result;
184 184
             }
185 185
             return true;
186 186
         }
187 187
 
188
-        $post = $this->blob_to_post( $blob );
188
+        $post = $this->blob_to_post($blob);
189 189
 
190
-        if ( ! $post instanceof Writing_On_GitHub_Post ) {
190
+        if ( ! $post instanceof Writing_On_GitHub_Post) {
191 191
             return false;
192 192
         }
193 193
 
194
-        $result = $this->app->database()->save_post( $post );
195
-        if ( is_wp_error( $result ) ) {
194
+        $result = $this->app->database()->save_post($post);
195
+        if (is_wp_error($result)) {
196 196
             /** @var WP_Error $result */
197 197
             return $result;
198 198
         }
199 199
 
200
-        if ( $post->is_new() ||
201
-                ! wogh_equal_front_matter( $post, $blob ) ) {
200
+        if ($post->is_new() ||
201
+                ! wogh_equal_front_matter($post, $blob)) {
202 202
 
203
-            $result = $this->app->export()->new_posts( array( $post ) );
203
+            $result = $this->app->export()->new_posts(array($post));
204 204
 
205
-            if ( is_wp_error( $result ) ) {
205
+            if (is_wp_error($result)) {
206 206
                 /** @var WP_Error $result */
207 207
                 return $result;
208 208
             }
@@ -216,13 +216,13 @@  discard block
 block discarded – undo
216 216
      * @param  Writing_On_GitHub_Blob $blob
217 217
      * @return bool
218 218
      */
219
-    protected function importable_raw_file( Writing_On_GitHub_Blob $blob ) {
220
-        if ( $blob->has_frontmatter() ) {
219
+    protected function importable_raw_file(Writing_On_GitHub_Blob $blob) {
220
+        if ($blob->has_frontmatter()) {
221 221
             return false;
222 222
         }
223 223
 
224 224
         // only images
225
-        if ( strncasecmp($blob->path(), 'images/', strlen('images/') ) != 0) {
225
+        if (strncasecmp($blob->path(), 'images/', strlen('images/')) != 0) {
226 226
             return false;
227 227
         }
228 228
 
@@ -234,16 +234,16 @@  discard block
 block discarded – undo
234 234
      * @param  Writing_On_GitHub_Blob $blob
235 235
      * @param  bool                   $is_remove
236 236
      */
237
-    protected function import_raw_file( Writing_On_GitHub_Blob $blob, $is_remove ) {
237
+    protected function import_raw_file(Writing_On_GitHub_Blob $blob, $is_remove) {
238 238
         $arr = wp_upload_dir();
239 239
         $path = $arr['basedir'] . '/writing-on-github/' . $blob->path();
240
-        if ( $is_remove ) {
241
-            if ( file_exists($path) ) {
240
+        if ($is_remove) {
241
+            if (file_exists($path)) {
242 242
                 unlink($path);
243 243
             }
244 244
         } else {
245 245
             $dirname = dirname($path);
246
-            if ( ! file_exists($dirname) ) {
246
+            if ( ! file_exists($dirname)) {
247 247
                 wp_mkdir_p($dirname);
248 248
             }
249 249
 
@@ -259,57 +259,57 @@  discard block
 block discarded – undo
259 259
      *
260 260
      * @return Writing_On_GitHub_Post|false
261 261
      */
262
-    protected function blob_to_post( Writing_On_GitHub_Blob $blob ) {
263
-        $args = array( 'post_content' => $blob->content_import() );
262
+    protected function blob_to_post(Writing_On_GitHub_Blob $blob) {
263
+        $args = array('post_content' => $blob->content_import());
264 264
         $meta = $blob->meta();
265 265
 
266 266
         $id = false;
267 267
 
268
-        if ( ! empty( $meta ) ) {
269
-            if ( array_key_exists( 'layout', $meta ) ) {
268
+        if ( ! empty($meta)) {
269
+            if (array_key_exists('layout', $meta)) {
270 270
                 $args['post_type'] = $meta['layout'];
271
-                unset( $meta['layout'] );
271
+                unset($meta['layout']);
272 272
             }
273 273
 
274
-            if ( array_key_exists( 'published', $meta ) ) {
274
+            if (array_key_exists('published', $meta)) {
275 275
                 $args['post_status'] = true === $meta['published'] ? 'publish' : 'draft';
276
-                unset( $meta['published'] );
276
+                unset($meta['published']);
277 277
             }
278 278
 
279
-            if ( array_key_exists( 'post_title', $meta ) ) {
279
+            if (array_key_exists('post_title', $meta)) {
280 280
                 $args['post_title'] = $meta['post_title'];
281
-                unset( $meta['post_title'] );
281
+                unset($meta['post_title']);
282 282
             }
283 283
 
284
-            if ( array_key_exists( 'post_name', $meta ) ) {
284
+            if (array_key_exists('post_name', $meta)) {
285 285
                 $args['post_name'] = $meta['post_name'];
286
-                unset( $meta['post_name'] );
286
+                unset($meta['post_name']);
287 287
             }
288 288
 
289
-            if ( array_key_exists( 'ID', $meta ) ) {
289
+            if (array_key_exists('ID', $meta)) {
290 290
                 $id = $args['ID'] = $meta['ID'];
291 291
                 $blob->set_id($id);
292
-                unset( $meta['ID'] );
292
+                unset($meta['ID']);
293 293
             }
294 294
         }
295 295
 
296 296
         $meta['_wogh_sha'] = $blob->sha();
297 297
 
298
-        if ( $id ) {
299
-            $old_sha = get_post_meta( $id, '_wogh_sha', true );
300
-            $old_github_path = get_post_meta( $id, '_wogh_github_path', true );
298
+        if ($id) {
299
+            $old_sha = get_post_meta($id, '_wogh_sha', true);
300
+            $old_github_path = get_post_meta($id, '_wogh_github_path', true);
301 301
 
302 302
             // dont save post when has same sha
303
-            if ( $old_sha  && $old_sha == $meta['_wogh_sha'] &&
304
-                 $old_github_path && $old_github_path == $blob->path() ) {
303
+            if ($old_sha && $old_sha == $meta['_wogh_sha'] &&
304
+                 $old_github_path && $old_github_path == $blob->path()) {
305 305
                 return false;
306 306
             }
307 307
         }
308 308
 
309
-        $post = new Writing_On_GitHub_Post( $args, $this->app->api() );
310
-        $post->set_old_github_path( $blob->path() );
311
-        $post->set_meta( $meta );
312
-        $blob->set_id( $post->id() );
309
+        $post = new Writing_On_GitHub_Post($args, $this->app->api());
310
+        $post->set_old_github_path($blob->path());
311
+        $post->set_meta($meta);
312
+        $blob->set_id($post->id());
313 313
 
314 314
         return $post;
315 315
     }
Please login to merge, or discard this patch.
lib/blob.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -55,8 +55,8 @@  discard block
 block discarded – undo
55 55
      *
56 56
      * @param stdClass $data Raw blob data.
57 57
      */
58
-    public function __construct( stdClass $data ) {
59
-        $this->interpret_data( $data );
58
+    public function __construct(stdClass $data) {
59
+        $this->interpret_data($data);
60 60
     }
61 61
 
62 62
     public function id() {
@@ -84,12 +84,12 @@  discard block
 block discarded – undo
84 84
      *
85 85
      * @return $this
86 86
      */
87
-    public function set_content( $content, $base64 = false ) {
88
-        if ( $base64 ) {
89
-            $content = base64_decode( $content );
87
+    public function set_content($content, $base64 = false) {
88
+        if ($base64) {
89
+            $content = base64_decode($content);
90 90
         }
91 91
 
92
-        $this->frontmatter = '---' === substr( $this->content = $content, 0, 3 );
92
+        $this->frontmatter = '---' === substr($this->content = $content, 0, 3);
93 93
 
94 94
         return $this;
95 95
     }
@@ -136,21 +136,21 @@  discard block
 block discarded – undo
136 136
     public function content_import() {
137 137
         $content = $this->content();
138 138
 
139
-        if ( $this->has_frontmatter() ) {
139
+        if ($this->has_frontmatter()) {
140 140
             // Break out content.
141
-            preg_match( '/(^---(.*?)---$)?(.*)/ms', $content, $matches );
141
+            preg_match('/(^---(.*?)---$)?(.*)/ms', $content, $matches);
142 142
             $this->front_matter = $matches[1];
143
-            $content = array_pop( $matches );
143
+            $content = array_pop($matches);
144 144
         }
145 145
 
146
-        if ( function_exists( 'wpmarkdown_markdown_to_html' ) ) {
147
-            $content = wpmarkdown_markdown_to_html( $content );
146
+        if (function_exists('wpmarkdown_markdown_to_html')) {
147
+            $content = wpmarkdown_markdown_to_html($content);
148 148
         }
149 149
 
150 150
         /**
151 151
          * Filters the content for import.
152 152
          */
153
-        return apply_filters( 'wogh_content_import', trim( $content ) );
153
+        return apply_filters('wogh_content_import', trim($content));
154 154
     }
155 155
 
156 156
     /**
@@ -161,13 +161,13 @@  discard block
 block discarded – undo
161 161
     public function meta() {
162 162
         $meta = array();
163 163
 
164
-        if ( $this->has_frontmatter() ) {
164
+        if ($this->has_frontmatter()) {
165 165
             // Break out meta, if present.
166
-            preg_match( '/(^---(.*?)---$)?(.*)/ms', $this->content(), $matches );
167
-            array_pop( $matches );
166
+            preg_match('/(^---(.*?)---$)?(.*)/ms', $this->content(), $matches);
167
+            array_pop($matches);
168 168
 
169
-            $meta = spyc_load( $matches[2] );
170
-            if ( 'yes' == get_option('wogh_ignore_author') ) {
169
+            $meta = spyc_load($matches[2]);
170
+            if ('yes' == get_option('wogh_ignore_author')) {
171 171
                 unset($meta['author']);
172 172
             }
173 173
             // if ( isset( $meta['link'] ) ) {
@@ -213,7 +213,7 @@  discard block
 block discarded – undo
213 213
         // $data->type = 'blob';
214 214
 
215 215
         $data->path = $this->path();
216
-        $data->content = base64_encode( $this->content() );
216
+        $data->content = base64_encode($this->content());
217 217
         $data->sha = $this->sha;
218 218
 
219 219
         return $data;
@@ -222,13 +222,13 @@  discard block
 block discarded – undo
222 222
     /**
223 223
      * Interprets the blob's data into properties.
224 224
      */
225
-    protected function interpret_data( $data ) {
226
-        $this->sha  = isset( $data->sha  ) ? $data->sha  : '';
227
-        $this->path = isset( $data->path ) ? $data->path : '';
225
+    protected function interpret_data($data) {
226
+        $this->sha  = isset($data->sha) ? $data->sha : '';
227
+        $this->path = isset($data->path) ? $data->path : '';
228 228
 
229 229
         $this->set_content(
230
-            isset( $data->content ) ? trim( $data->content ) : '',
231
-            isset( $data->encoding ) && 'base64' === $data->encoding ? true : false
230
+            isset($data->content) ? trim($data->content) : '',
231
+            isset($data->encoding) && 'base64' === $data->encoding ? true : false
232 232
         );
233 233
     }
234 234
 }
Please login to merge, or discard this patch.
lib/function.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -7,9 +7,9 @@  discard block
 block discarded – undo
7 7
  * @param  WP_Error   $error2
8 8
  * @return WP_Error
9 9
  */
10
-function wogh_append_error( $error, $error2 ) {
11
-    if ( is_wp_error( $error ) ) {
12
-        $error->add( $error2->get_error_code(), $error2->get_error_message() );
10
+function wogh_append_error($error, $error2) {
11
+    if (is_wp_error($error)) {
12
+        $error->add($error2->get_error_code(), $error2->get_error_message());
13 13
     }
14 14
     return $error2;
15 15
 }
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
  * @param  Writing_On_GitHub_Blob $blob
21 21
  * @return bool
22 22
  */
23
-function wogh_equal_front_matter( $post, $blob ) {
23
+function wogh_equal_front_matter($post, $blob) {
24 24
     $str1 = $post->front_matter();
25 25
     $str2 = $blob->front_matter();
26 26
     return trim($str1) === trim($str2);
Please login to merge, or discard this patch.