Test Setup Failed
Push — master ( c2cd2f...3db12d )
by Sébastien
02:51
created
lib/Book.php 2 patches
Spacing   +58 added lines, -58 removed lines patch added patch discarded remove patch
@@ -7,55 +7,55 @@  discard block
 block discarded – undo
7 7
  */
8 8
 
9 9
 // Silly thing because PHP forbid string concatenation in class const
10
-define ('SQL_BOOKS_LEFT_JOIN', 'left outer join comments on comments.book = books.id
10
+define('SQL_BOOKS_LEFT_JOIN', 'left outer join comments on comments.book = books.id
11 11
                                 left outer join books_ratings_link on books_ratings_link.book = books.id
12 12
                                 left outer join ratings on books_ratings_link.rating = ratings.id ');
13
-define ('SQL_BOOKS_ALL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . ' order by books.sort ');
14
-define ('SQL_BOOKS_BY_PUBLISHER', 'select {0} from books_publishers_link, books ' . SQL_BOOKS_LEFT_JOIN . '
13
+define('SQL_BOOKS_ALL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . ' order by books.sort ');
14
+define('SQL_BOOKS_BY_PUBLISHER', 'select {0} from books_publishers_link, books ' . SQL_BOOKS_LEFT_JOIN . '
15 15
                                                     where books_publishers_link.book = books.id and publisher = ? {1} order by publisher');
16
-define ('SQL_BOOKS_BY_FIRST_LETTER', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
16
+define('SQL_BOOKS_BY_FIRST_LETTER', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
17 17
                                                     where upper (books.sort) like ? order by books.sort');
18
-define ('SQL_BOOKS_BY_AUTHOR', 'select {0} from books_authors_link, books ' . SQL_BOOKS_LEFT_JOIN . '
18
+define('SQL_BOOKS_BY_AUTHOR', 'select {0} from books_authors_link, books ' . SQL_BOOKS_LEFT_JOIN . '
19 19
                                                     left outer join books_series_link on books_series_link.book = books.id
20 20
                                                     where books_authors_link.book = books.id and author = ? {1} order by series desc, series_index asc, pubdate asc');
21
-define ('SQL_BOOKS_BY_SERIE', 'select {0} from books_series_link, books ' . SQL_BOOKS_LEFT_JOIN . '
21
+define('SQL_BOOKS_BY_SERIE', 'select {0} from books_series_link, books ' . SQL_BOOKS_LEFT_JOIN . '
22 22
                                                     where books_series_link.book = books.id and series = ? {1} order by series_index');
23
-define ('SQL_BOOKS_BY_TAG', 'select {0} from books_tags_link, books ' . SQL_BOOKS_LEFT_JOIN . '
23
+define('SQL_BOOKS_BY_TAG', 'select {0} from books_tags_link, books ' . SQL_BOOKS_LEFT_JOIN . '
24 24
                                                     where books_tags_link.book = books.id and tag = ? {1} order by sort');
25
-define ('SQL_BOOKS_BY_LANGUAGE', 'select {0} from books_languages_link, books ' . SQL_BOOKS_LEFT_JOIN . '
25
+define('SQL_BOOKS_BY_LANGUAGE', 'select {0} from books_languages_link, books ' . SQL_BOOKS_LEFT_JOIN . '
26 26
                                                     where books_languages_link.book = books.id and lang_code = ? {1} order by sort');
27
-define ('SQL_BOOKS_BY_CUSTOM', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
27
+define('SQL_BOOKS_BY_CUSTOM', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
28 28
                                                     where {2}.book = books.id and {2}.{3} = ? {1} order by sort');
29
-define ('SQL_BOOKS_BY_CUSTOM_BOOL_TRUE', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
29
+define('SQL_BOOKS_BY_CUSTOM_BOOL_TRUE', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
30 30
                                                     where {2}.book = books.id and {2}.value = 1 {1} order by sort');
31
-define ('SQL_BOOKS_BY_CUSTOM_BOOL_FALSE', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
31
+define('SQL_BOOKS_BY_CUSTOM_BOOL_FALSE', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
32 32
                                                     where {2}.book = books.id and {2}.value = 0 {1} order by sort');
33
-define ('SQL_BOOKS_BY_CUSTOM_BOOL_NULL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
33
+define('SQL_BOOKS_BY_CUSTOM_BOOL_NULL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
34 34
                                                     where books.id not in (select book from {2}) {1} order by sort');
35
-define ('SQL_BOOKS_BY_CUSTOM_RATING', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
35
+define('SQL_BOOKS_BY_CUSTOM_RATING', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
36 36
                                                     left join {2} on {2}.book = books.id
37 37
                                                     left join {3} on {3}.id = {2}.{4}
38 38
                                                     where {3}.value = ?  order by sort');
39
-define ('SQL_BOOKS_BY_CUSTOM_RATING_NULL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
39
+define('SQL_BOOKS_BY_CUSTOM_RATING_NULL', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
40 40
 								                    left join {2} on {2}.book = books.id
41 41
 								                    left join {3} on {3}.id = {2}.{4}
42 42
                                                     where ((books.id not in (select {2}.book from {2})) or ({3}.value = 0)) {1} order by sort');
43
-define ('SQL_BOOKS_BY_CUSTOM_DATE', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
43
+define('SQL_BOOKS_BY_CUSTOM_DATE', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
44 44
                                                     where {2}.book = books.id and date({2}.value) = ? {1} order by sort');
45
-define ('SQL_BOOKS_BY_CUSTOM_DIRECT', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
45
+define('SQL_BOOKS_BY_CUSTOM_DIRECT', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
46 46
                                                     where {2}.book = books.id and {2}.value = ? {1} order by sort');
47
-define ('SQL_BOOKS_BY_CUSTOM_DIRECT_ID', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
47
+define('SQL_BOOKS_BY_CUSTOM_DIRECT_ID', 'select {0} from {2}, books ' . SQL_BOOKS_LEFT_JOIN . '
48 48
                                                     where {2}.book = books.id and {2}.id = ? {1} order by sort');
49
-define ('SQL_BOOKS_QUERY', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
49
+define('SQL_BOOKS_QUERY', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
50 50
                                                     where (
51 51
                                                     exists (select null from authors, books_authors_link where book = books.id and author = authors.id and authors.name like ?) or
52 52
                                                     exists (select null from tags, books_tags_link where book = books.id and tag = tags.id and tags.name like ?) or
53 53
                                                     exists (select null from series, books_series_link on book = books.id and books_series_link.series = series.id and series.name like ?) or
54 54
                                                     exists (select null from publishers, books_publishers_link where book = books.id and books_publishers_link.publisher = publishers.id and publishers.name like ?) or
55 55
                                                     title like ?) {1} order by books.sort');
56
-define ('SQL_BOOKS_RECENT', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
56
+define('SQL_BOOKS_RECENT', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
57 57
                                                     where 1=1 {1} order by timestamp desc limit ');
58
-define ('SQL_BOOKS_BY_RATING', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
58
+define('SQL_BOOKS_BY_RATING', 'select {0} from books ' . SQL_BOOKS_LEFT_JOIN . '
59 59
                                                     where books_ratings_link.book = books.id and ratings.id = ? {1} order by sort');
60 60
 
61 61
 class Book extends Base
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
     public $serie = NULL;
106 106
     public $tags = NULL;
107 107
     public $languages = NULL;
108
-    public $format = array ();
108
+    public $format = array();
109 109
 
110 110
 
111 111
     public function __construct($line) {
@@ -127,24 +127,24 @@  discard block
 block discarded – undo
127 127
     }
128 128
 
129 129
     public function getEntryId() {
130
-        return self::ALL_BOOKS_UUID.':'.$this->uuid;
130
+        return self::ALL_BOOKS_UUID . ':' . $this->uuid;
131 131
     }
132 132
 
133
-    public static function getEntryIdByLetter ($startingLetter) {
134
-        return self::ALL_BOOKS_ID.':letter:'.$startingLetter;
133
+    public static function getEntryIdByLetter($startingLetter) {
134
+        return self::ALL_BOOKS_ID . ':letter:' . $startingLetter;
135 135
     }
136 136
 
137
-    public function getUri () {
138
-        return '?page='.parent::PAGE_BOOK_DETAIL.'&id=' . $this->id;
137
+    public function getUri() {
138
+        return '?page=' . parent::PAGE_BOOK_DETAIL . '&id=' . $this->id;
139 139
     }
140 140
 
141
-    public function getDetailUrl () {
141
+    public function getDetailUrl() {
142 142
         $urlParam = $this->getUri();
143
-        if (!is_null(GetUrlParam(DB))) $urlParam = addURLParameter($urlParam, DB, GetUrlParam (DB));
143
+        if (!is_null(GetUrlParam(DB))) $urlParam = addURLParameter($urlParam, DB, GetUrlParam(DB));
144 144
         return 'index.php' . $urlParam;
145 145
     }
146 146
 
147
-    public function getTitle () {
147
+    public function getTitle() {
148 148
         return $this->title;
149 149
     }
150 150
 
@@ -153,22 +153,22 @@  discard block
 block discarded – undo
153 153
     /**
154 154
      * @return Author[]
155 155
      */
156
-    public function getAuthors () {
156
+    public function getAuthors() {
157 157
         if (is_null($this->authors)) {
158 158
             $this->authors = Author::getAuthorByBookId($this->id);
159 159
         }
160 160
         return $this->authors;
161 161
     }
162 162
 
163
-    public function getAuthorsName () {
164
-        return implode(', ', array_map(function ($author) { return $author->name; }, $this->getAuthors()));
163
+    public function getAuthorsName() {
164
+        return implode(', ', array_map(function($author) { return $author->name; }, $this->getAuthors()));
165 165
     }
166 166
 
167
-    public function getAuthorsSort () {
168
-        return implode(', ', array_map(function ($author) { return $author->sort; }, $this->getAuthors()));
167
+    public function getAuthorsSort() {
168
+        return implode(', ', array_map(function($author) { return $author->sort; }, $this->getAuthors()));
169 169
     }
170 170
 
171
-    public function getPublisher () {
171
+    public function getPublisher() {
172 172
         if (is_null($this->publisher)) {
173 173
             $this->publisher = Publisher::getPublisherByBookId($this->id);
174 174
         }
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
      * @return Tag[]
208 208
      */
209 209
     public function getTags() {
210
-        if (is_null ($this->tags)) {
210
+        if (is_null($this->tags)) {
211 211
             $this->tags = array();
212 212
 
213 213
             $result = parent::getDb()->prepare('select tags.id as id, name
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
     }
226 226
 
227 227
     public function getTagsName() {
228
-        return implode(', ', array_map(function ($tag) { return $tag->name; }, $this->getTags()));
228
+        return implode(', ', array_map(function($tag) { return $tag->name; }, $this->getTags()));
229 229
     }
230 230
 
231 231
     /**
@@ -277,7 +277,7 @@  discard block
 block discarded – undo
277 277
 
278 278
     public function getDataById($idData)
279 279
     {
280
-        $reduced = array_filter($this->getDatas(), function ($data) use ($idData) {
280
+        $reduced = array_filter($this->getDatas(), function($data) use ($idData) {
281 281
             return $data->id == $idData;
282 282
         });
283 283
         return reset($reduced);
@@ -301,7 +301,7 @@  discard block
 block discarded – undo
301 301
         if (empty ($this->pubdate)) {
302 302
             return '';
303 303
         }
304
-        $dateY = (int) substr($this->pubdate, 0, 4);
304
+        $dateY = (int)substr($this->pubdate, 0, 4);
305 305
         if ($dateY > 102) {
306 306
             return str_pad($dateY, 4, '0', STR_PAD_LEFT);
307 307
         }
@@ -310,8 +310,8 @@  discard block
 block discarded – undo
310 310
 
311 311
     public function getComment($withSerie = true) {
312 312
         $addition = '';
313
-        $se = $this->getSerie ();
314
-        if (!is_null ($se) && $withSerie) {
313
+        $se = $this->getSerie();
314
+        if (!is_null($se) && $withSerie) {
315 315
             $addition = $addition . '<strong>' . localize('content.series') . '</strong>' . str_format(localize('content.series.data'), $this->seriesIndex, htmlspecialchars($se->name)) . "<br />\n";
316 316
         }
317 317
         if (preg_match('/<\/(div|p|a|span)>/', $this->comment))
@@ -325,7 +325,7 @@  discard block
 block discarded – undo
325 325
     }
326 326
 
327 327
     public function getDataFormat($format) {
328
-        $reduced = array_filter($this->getDatas(), function ($data) use ($format) {
328
+        $reduced = array_filter($this->getDatas(), function($data) use ($format) {
329 329
             return $data->format == $format;
330 330
         });
331 331
         return reset($reduced);
@@ -346,11 +346,11 @@  discard block
 block discarded – undo
346 346
 
347 347
         if ($relative)
348 348
         {
349
-            return $this->relativePath.'/'.$file;
349
+            return $this->relativePath . '/' . $file;
350 350
         }
351 351
         else
352 352
         {
353
-            return $this->path.'/'.$file;
353
+            return $this->path . '/' . $file;
354 354
         }
355 355
     }
356 356
 
@@ -370,7 +370,7 @@  discard block
 block discarded – undo
370 370
             }
371 371
             $epub->Authors($authorArray);
372 372
             $epub->Language($this->getLanguages());
373
-            $epub->Description ($this->getComment(false));
373
+            $epub->Description($this->getComment(false));
374 374
             $epub->Subjects($this->getTagsName());
375 375
             $epub->Cover2($this->getFilePath('jpg'), 'image/jpeg');
376 376
             $epub->Calibre($this->uuid);
@@ -380,7 +380,7 @@  discard block
 block discarded – undo
380 380
                 $epub->SerieIndex($this->seriesIndex);
381 381
             }
382 382
             $filename = $data->getUpdatedFilenameEpub();
383
-            if ($config['cops_provide_kepub'] == '1'  && preg_match('/Kobo/', $_SERVER['HTTP_USER_AGENT'])) {
383
+            if ($config['cops_provide_kepub'] == '1' && preg_match('/Kobo/', $_SERVER['HTTP_USER_AGENT'])) {
384 384
                 $epub->updateForKepub();
385 385
                 $filename = $data->getUpdatedFilenameKepub();
386 386
             }
@@ -406,11 +406,11 @@  discard block
 block discarded – undo
406 406
             if (!is_null($width)) {
407 407
                 $nw = $width;
408 408
                 if ($nw >= $w) { return false; }
409
-                $nh = ($nw*$h)/$w;
409
+                $nh = ($nw * $h) / $w;
410 410
             } else {
411 411
                 $nh = $height;
412 412
                 if ($nh >= $h) { return false; }
413
-                $nw = ($nh*$w)/$h;
413
+                $nw = ($nh * $w) / $h;
414 414
             }
415 415
         } else {
416 416
             return false;
@@ -418,16 +418,16 @@  discard block
 block discarded – undo
418 418
 
419 419
         //draw the image
420 420
         $src_img = imagecreatefromjpeg($file);
421
-        $dst_img = imagecreatetruecolor($nw,$nh);
422
-        imagecopyresampled($dst_img, $src_img, 0, 0, 0, 0, $nw, $nh, $w, $h);//resizing the image
423
-        imagejpeg($dst_img,$outputfile,80);
421
+        $dst_img = imagecreatetruecolor($nw, $nh);
422
+        imagecopyresampled($dst_img, $src_img, 0, 0, 0, 0, $nw, $nh, $w, $h); //resizing the image
423
+        imagejpeg($dst_img, $outputfile, 80);
424 424
         imagedestroy($src_img);
425 425
         imagedestroy($dst_img);
426 426
 
427 427
         return true;
428 428
     }
429 429
 
430
-    public function getLinkArray ()
430
+    public function getLinkArray()
431 431
     {
432 432
         $linkArray = array();
433 433
 
@@ -452,7 +452,7 @@  discard block
 block discarded – undo
452 452
         }
453 453
 
454 454
         $serie = $this->getSerie();
455
-        if (!is_null ($serie)) {
455
+        if (!is_null($serie)) {
456 456
             array_push($linkArray, new LinkNavigation($serie->getUri(), 'related', str_format(localize('content.series.data'), $this->seriesIndex, $serie->name)));
457 457
         }
458 458
 
@@ -477,13 +477,13 @@  discard block
 block discarded – undo
477 477
         $entry = new Entry(localize('allbooks.title'),
478 478
                           self::ALL_BOOKS_ID,
479 479
                           str_format(localize('allbooks.alphabetical', $nBooks), $nBooks), 'text',
480
-                          array(new LinkNavigation('?page='.parent::PAGE_ALL_BOOKS)), '', $nBooks);
480
+                          array(new LinkNavigation('?page=' . parent::PAGE_ALL_BOOKS)), '', $nBooks);
481 481
         array_push($result, $entry);
482 482
         if ($config['cops_recentbooks_limit'] > 0) {
483 483
             $entry = new Entry(localize('recent.title'),
484 484
                               self::ALL_RECENT_BOOKS_ID,
485 485
                               str_format(localize('recent.list'), $config['cops_recentbooks_limit']), 'text',
486
-                              array ( new LinkNavigation ('?page='.parent::PAGE_ALL_RECENT_BOOKS)), '', $config['cops_recentbooks_limit']);
486
+                              array(new LinkNavigation('?page=' . parent::PAGE_ALL_RECENT_BOOKS)), '', $config['cops_recentbooks_limit']);
487 487
             array_push($result, $entry);
488 488
         }
489 489
         return $result;
@@ -579,7 +579,7 @@  discard block
 block discarded – undo
579 579
     }
580 580
 
581 581
     public static function getBooks($n) {
582
-        list ($entryArray, $totalNumber) = self::getEntryArray(self::SQL_BOOKS_ALL , array (), $n);
582
+        list ($entryArray, $totalNumber) = self::getEntryArray(self::SQL_BOOKS_ALL, array(), $n);
583 583
         return array($entryArray, $totalNumber);
584 584
     }
585 585
 
@@ -596,7 +596,7 @@  discard block
 block discarded – undo
596 596
         {
597 597
             array_push($entryArray, new Entry($post->title, Book::getEntryIdByLetter($post->title),
598 598
                 str_format(localize('bookword', $post->count), $post->count), 'text',
599
-                array(new LinkNavigation('?page='.parent::PAGE_ALL_BOOKS_LETTER.'&id='. rawurlencode($post->title))), '', $post->count));
599
+                array(new LinkNavigation('?page=' . parent::PAGE_ALL_BOOKS_LETTER . '&id=' . rawurlencode($post->title))), '', $post->count));
600 600
         }
601 601
         return $entryArray;
602 602
     }
@@ -621,7 +621,7 @@  discard block
 block discarded – undo
621 621
 
622 622
     public static function getAllRecentBooks() {
623 623
         global $config;
624
-        list ($entryArray, ) = self::getEntryArray(self::SQL_BOOKS_RECENT . $config['cops_recentbooks_limit'], array(), -1);
624
+        list ($entryArray,) = self::getEntryArray(self::SQL_BOOKS_RECENT . $config['cops_recentbooks_limit'], array(), -1);
625 625
         return $entryArray;
626 626
     }
627 627
 
Please login to merge, or discard this patch.
Braces   +105 added lines, -82 removed lines patch added patch discarded remove patch
@@ -108,7 +108,8 @@  discard block
 block discarded – undo
108 108
     public $format = array ();
109 109
 
110 110
 
111
-    public function __construct($line) {
111
+    public function __construct($line)
112
+    {
112 113
         $this->id = $line->id;
113 114
         $this->title = $line->title;
114 115
         $this->timestamp = strtotime($line->timestamp);
@@ -126,25 +127,32 @@  discard block
 block discarded – undo
126 127
         $this->rating = $line->rating;
127 128
     }
128 129
 
129
-    public function getEntryId() {
130
+    public function getEntryId()
131
+    {
130 132
         return self::ALL_BOOKS_UUID.':'.$this->uuid;
131 133
     }
132 134
 
133
-    public static function getEntryIdByLetter ($startingLetter) {
135
+    public static function getEntryIdByLetter ($startingLetter)
136
+    {
134 137
         return self::ALL_BOOKS_ID.':letter:'.$startingLetter;
135 138
     }
136 139
 
137
-    public function getUri () {
140
+    public function getUri ()
141
+    {
138 142
         return '?page='.parent::PAGE_BOOK_DETAIL.'&id=' . $this->id;
139 143
     }
140 144
 
141
-    public function getDetailUrl () {
145
+    public function getDetailUrl ()
146
+    {
142 147
         $urlParam = $this->getUri();
143
-        if (!is_null(GetUrlParam(DB))) $urlParam = addURLParameter($urlParam, DB, GetUrlParam (DB));
148
+        if (!is_null(GetUrlParam(DB))) {
149
+            $urlParam = addURLParameter($urlParam, DB, GetUrlParam (DB));
150
+        }
144 151
         return 'index.php' . $urlParam;
145 152
     }
146 153
 
147
-    public function getTitle () {
154
+    public function getTitle ()
155
+    {
148 156
         return $this->title;
149 157
     }
150 158
 
@@ -153,22 +161,26 @@  discard block
 block discarded – undo
153 161
     /**
154 162
      * @return Author[]
155 163
      */
156
-    public function getAuthors () {
164
+    public function getAuthors ()
165
+    {
157 166
         if (is_null($this->authors)) {
158 167
             $this->authors = Author::getAuthorByBookId($this->id);
159 168
         }
160 169
         return $this->authors;
161 170
     }
162 171
 
163
-    public function getAuthorsName () {
172
+    public function getAuthorsName ()
173
+    {
164 174
         return implode(', ', array_map(function ($author) { return $author->name; }, $this->getAuthors()));
165 175
     }
166 176
 
167
-    public function getAuthorsSort () {
177
+    public function getAuthorsSort ()
178
+    {
168 179
         return implode(', ', array_map(function ($author) { return $author->sort; }, $this->getAuthors()));
169 180
     }
170 181
 
171
-    public function getPublisher () {
182
+    public function getPublisher ()
183
+    {
172 184
         if (is_null($this->publisher)) {
173 185
             $this->publisher = Publisher::getPublisherByBookId($this->id);
174 186
         }
@@ -178,7 +190,8 @@  discard block
 block discarded – undo
178 190
     /**
179 191
      * @return Serie
180 192
      */
181
-    public function getSerie() {
193
+    public function getSerie()
194
+    {
182 195
         if (is_null($this->serie)) {
183 196
             $this->serie = Serie::getSerieByBookId($this->id);
184 197
         }
@@ -188,7 +201,8 @@  discard block
 block discarded – undo
188 201
     /**
189 202
      * @return string
190 203
      */
191
-    public function getLanguages() {
204
+    public function getLanguages()
205
+    {
192 206
         $lang = array();
193 207
         $result = parent::getDb()->prepare('select languages.lang_code
194 208
                 from books_languages_link, languages
@@ -196,8 +210,7 @@  discard block
 block discarded – undo
196 210
                 and book = ?
197 211
                 order by item_order');
198 212
         $result->execute(array($this->id));
199
-        while ($post = $result->fetchObject())
200
-        {
213
+        while ($post = $result->fetchObject()) {
201 214
             array_push($lang, Language::getLanguageString($post->lang_code));
202 215
         }
203 216
         return implode(', ', $lang);
@@ -206,7 +219,8 @@  discard block
 block discarded – undo
206 219
     /**
207 220
      * @return Tag[]
208 221
      */
209
-    public function getTags() {
222
+    public function getTags()
223
+    {
210 224
         if (is_null ($this->tags)) {
211 225
             $this->tags = array();
212 226
 
@@ -216,15 +230,15 @@  discard block
 block discarded – undo
216 230
                 and book = ?
217 231
                 order by name');
218 232
             $result->execute(array($this->id));
219
-            while ($post = $result->fetchObject())
220
-            {
233
+            while ($post = $result->fetchObject()) {
221 234
                 array_push($this->tags, new Tag($post));
222 235
             }
223 236
         }
224 237
         return $this->tags;
225 238
     }
226 239
 
227
-    public function getTagsName() {
240
+    public function getTagsName()
241
+    {
228 242
         return implode(', ', array_map(function ($tag) { return $tag->name; }, $this->getTags()));
229 243
     }
230 244
 
@@ -241,9 +255,12 @@  discard block
 block discarded – undo
241 255
 
242 256
     /* End of other class (author, series, tag, ...) initialization and accessors */
243 257
 
244
-    public static function getFilterString() {
258
+    public static function getFilterString()
259
+    {
245 260
         $filter = getURLParam('tag', NULL);
246
-        if (empty($filter)) return '';
261
+        if (empty($filter)) {
262
+            return '';
263
+        }
247 264
 
248 265
         $exists = true;
249 266
         if (preg_match("/^!(.*)$/", $filter, $matches)) {
@@ -283,7 +300,8 @@  discard block
 block discarded – undo
283 300
         return reset($reduced);
284 301
     }
285 302
 
286
-    public function getRating() {
303
+    public function getRating()
304
+    {
287 305
         if (is_null($this->rating) || $this->rating == 0) {
288 306
             return '';
289 307
         }
@@ -297,7 +315,8 @@  discard block
 block discarded – undo
297 315
         return $retour;
298 316
     }
299 317
 
300
-    public function getPubDate() {
318
+    public function getPubDate()
319
+    {
301 320
         if (empty ($this->pubdate)) {
302 321
             return '';
303 322
         }
@@ -308,23 +327,22 @@  discard block
 block discarded – undo
308 327
         return '';
309 328
     }
310 329
 
311
-    public function getComment($withSerie = true) {
330
+    public function getComment($withSerie = true)
331
+    {
312 332
         $addition = '';
313 333
         $se = $this->getSerie ();
314 334
         if (!is_null ($se) && $withSerie) {
315 335
             $addition = $addition . '<strong>' . localize('content.series') . '</strong>' . str_format(localize('content.series.data'), $this->seriesIndex, htmlspecialchars($se->name)) . "<br />\n";
316 336
         }
317
-        if (preg_match('/<\/(div|p|a|span)>/', $this->comment))
318
-        {
337
+        if (preg_match('/<\/(div|p|a|span)>/', $this->comment)) {
319 338
             return $addition . html2xhtml($this->comment);
320
-        }
321
-        else
322
-        {
339
+        } else {
323 340
             return $addition . htmlspecialchars($this->comment);
324 341
         }
325 342
     }
326 343
 
327
-    public function getDataFormat($format) {
344
+    public function getDataFormat($format)
345
+    {
328 346
         $reduced = array_filter($this->getDatas(), function ($data) use ($format) {
329 347
             return $data->format == $format;
330 348
         });
@@ -333,23 +351,19 @@  discard block
 block discarded – undo
333 351
 
334 352
     public function getFilePath($extension, $idData = NULL, $relative = false)
335 353
     {
336
-        if ($extension == 'jpg')
337
-        {
354
+        if ($extension == 'jpg') {
338 355
             $file = 'cover.jpg';
339
-        }
340
-        else
341
-        {
356
+        } else {
342 357
             $data = $this->getDataById($idData);
343
-            if (!$data) return NULL;
358
+            if (!$data) {
359
+                return NULL;
360
+            }
344 361
             $file = $data->name . '.' . strtolower($data->format);
345 362
         }
346 363
 
347
-        if ($relative)
348
-        {
364
+        if ($relative) {
349 365
             return $this->relativePath.'/'.$file;
350
-        }
351
-        else
352
-        {
366
+        } else {
353 367
             return $this->path.'/'.$file;
354 368
         }
355 369
     }
@@ -359,8 +373,7 @@  discard block
 block discarded – undo
359 373
         global $config;
360 374
         $data = $this->getDataById($idData);
361 375
 
362
-        try
363
-        {
376
+        try {
364 377
             $epub = new EPub($data->getLocalPath());
365 378
 
366 379
             $epub->Title($this->title);
@@ -385,14 +398,13 @@  discard block
 block discarded – undo
385 398
                 $filename = $data->getUpdatedFilenameKepub();
386 399
             }
387 400
             $epub->download($filename);
388
-        }
389
-        catch (Exception $e)
390
-        {
401
+        } catch (Exception $e) {
391 402
             echo 'Exception : ' . $e->getMessage();
392 403
         }
393 404
     }
394 405
 
395
-    public function getThumbnail($width, $height, $outputfile = NULL) {
406
+    public function getThumbnail($width, $height, $outputfile = NULL)
407
+    {
396 408
         if (is_null($width) && is_null($height)) {
397 409
             return false;
398 410
         }
@@ -431,17 +443,14 @@  discard block
 block discarded – undo
431 443
     {
432 444
         $linkArray = array();
433 445
 
434
-        if ($this->hasCover)
435
-        {
446
+        if ($this->hasCover) {
436 447
             array_push($linkArray, Data::getLink($this, 'jpg', 'image/jpeg', Link::OPDS_IMAGE_TYPE, 'cover.jpg', NULL));
437 448
 
438 449
             array_push($linkArray, Data::getLink($this, 'jpg', 'image/jpeg', Link::OPDS_THUMBNAIL_TYPE, 'cover.jpg', NULL));
439 450
         }
440 451
 
441
-        foreach ($this->getDatas() as $data)
442
-        {
443
-            if ($data->isKnownType())
444
-            {
452
+        foreach ($this->getDatas() as $data) {
453
+            if ($data->isKnownType()) {
445 454
                 array_push($linkArray, $data->getDataLink(Link::OPDS_ACQUISITION_TYPE, $data->format));
446 455
             }
447 456
         }
@@ -460,17 +469,20 @@  discard block
 block discarded – undo
460 469
     }
461 470
 
462 471
 
463
-    public function getEntry() {
472
+    public function getEntry()
473
+    {
464 474
         return new EntryBook($this->getTitle(), $this->getEntryId(),
465 475
             $this->getComment(), 'text/html',
466 476
             $this->getLinkArray(), $this);
467 477
     }
468 478
 
469
-    public static function getBookCount($database = NULL) {
479
+    public static function getBookCount($database = NULL)
480
+    {
470 481
         return parent::executeQuerySingle('select count(*) from books', $database);
471 482
     }
472 483
 
473
-    public static function getCount() {
484
+    public static function getCount()
485
+    {
474 486
         global $config;
475 487
         $nBooks = parent::executeQuerySingle('select count(*) from books');
476 488
         $result = array();
@@ -489,27 +501,33 @@  discard block
 block discarded – undo
489 501
         return $result;
490 502
     }
491 503
 
492
-    public static function getBooksByAuthor($authorId, $n) {
504
+    public static function getBooksByAuthor($authorId, $n)
505
+    {
493 506
         return self::getEntryArray(self::SQL_BOOKS_BY_AUTHOR, array($authorId), $n);
494 507
     }
495 508
 
496
-    public static function getBooksByRating($ratingId, $n) {
509
+    public static function getBooksByRating($ratingId, $n)
510
+    {
497 511
         return self::getEntryArray(self::SQL_BOOKS_BY_RATING, array($ratingId), $n);
498 512
     }
499 513
 
500
-    public static function getBooksByPublisher($publisherId, $n) {
514
+    public static function getBooksByPublisher($publisherId, $n)
515
+    {
501 516
         return self::getEntryArray(self::SQL_BOOKS_BY_PUBLISHER, array($publisherId), $n);
502 517
     }
503 518
 
504
-    public static function getBooksBySeries($serieId, $n) {
519
+    public static function getBooksBySeries($serieId, $n)
520
+    {
505 521
         return self::getEntryArray(self::SQL_BOOKS_BY_SERIE, array($serieId), $n);
506 522
     }
507 523
 
508
-    public static function getBooksByTag($tagId, $n) {
524
+    public static function getBooksByTag($tagId, $n)
525
+    {
509 526
         return self::getEntryArray(self::SQL_BOOKS_BY_TAG, array($tagId), $n);
510 527
     }
511 528
 
512
-    public static function getBooksByLanguage($languageId, $n) {
529
+    public static function getBooksByLanguage($languageId, $n)
530
+    {
513 531
         return self::getEntryArray(self::SQL_BOOKS_BY_LANGUAGE, array($languageId), $n);
514 532
     }
515 533
 
@@ -519,32 +537,33 @@  discard block
 block discarded – undo
519 537
      * @param $n integer
520 538
      * @return array
521 539
      */
522
-    public static function getBooksByCustom($customColumn, $id, $n) {
540
+    public static function getBooksByCustom($customColumn, $id, $n)
541
+    {
523 542
         list($query, $params) = $customColumn->getQuery($id);
524 543
 
525 544
         return self::getEntryArray($query, $params, $n);
526 545
     }
527 546
 
528
-    public static function getBookById($bookId) {
547
+    public static function getBookById($bookId)
548
+    {
529 549
         $result = parent::getDb()->prepare('select ' . self::BOOK_COLUMNS . '
530 550
 from books ' . self::SQL_BOOKS_LEFT_JOIN . '
531 551
 where books.id = ?');
532 552
         $result->execute(array($bookId));
533
-        while ($post = $result->fetchObject())
534
-        {
553
+        while ($post = $result->fetchObject()) {
535 554
             $book = new Book($post);
536 555
             return $book;
537 556
         }
538 557
         return NULL;
539 558
     }
540 559
 
541
-    public static function getBookByDataId($dataId) {
560
+    public static function getBookByDataId($dataId)
561
+    {
542 562
         $result = parent::getDb()->prepare('select ' . self::BOOK_COLUMNS . ', data.name, data.format
543 563
 from data, books ' . self::SQL_BOOKS_LEFT_JOIN . '
544 564
 where data.book = books.id and data.id = ?');
545 565
         $result->execute(array($dataId));
546
-        while ($post = $result->fetchObject())
547
-        {
566
+        while ($post = $result->fetchObject()) {
548 567
             $book = new Book($post);
549 568
             $data = new Data($post, $book);
550 569
             $data->id = $dataId;
@@ -554,7 +573,8 @@  discard block
 block discarded – undo
554 573
         return NULL;
555 574
     }
556 575
 
557
-    public static function getBooksByQuery($query, $n, $database = NULL, $numberPerPage = NULL) {
576
+    public static function getBooksByQuery($query, $n, $database = NULL, $numberPerPage = NULL)
577
+    {
558 578
         $i = 0;
559 579
         $critArray = array();
560 580
         foreach (array(PageQueryResult::SCOPE_AUTHOR,
@@ -565,8 +585,7 @@  discard block
 block discarded – undo
565 585
             if (in_array($key, getCurrentOption('ignored_categories')) ||
566 586
                 (!array_key_exists($key, $query) && !array_key_exists('all', $query))) {
567 587
                 $critArray[$i] = self::BAD_SEARCH;
568
-            }
569
-            else {
588
+            } else {
570 589
                 if (array_key_exists($key, $query)) {
571 590
                     $critArray[$i] = $query[$key];
572 591
                 } else {
@@ -578,12 +597,14 @@  discard block
 block discarded – undo
578 597
         return self::getEntryArray(self::SQL_BOOKS_QUERY, $critArray, $n, $database, $numberPerPage);
579 598
     }
580 599
 
581
-    public static function getBooks($n) {
600
+    public static function getBooks($n)
601
+    {
582 602
         list ($entryArray, $totalNumber) = self::getEntryArray(self::SQL_BOOKS_ALL , array (), $n);
583 603
         return array($entryArray, $totalNumber);
584 604
     }
585 605
 
586
-    public static function getAllBooks() {
606
+    public static function getAllBooks()
607
+    {
587 608
         /* @var $result PDOStatement */
588 609
 
589 610
         list (, $result) = parent::executeQuery('select {0}
@@ -592,8 +613,7 @@  discard block
 block discarded – undo
592 613
 order by substr (upper (sort), 1, 1)', 'substr (upper (sort), 1, 1) as title, count(*) as count', self::getFilterString(), array(), -1);
593 614
 
594 615
         $entryArray = array();
595
-        while ($post = $result->fetchObject())
596
-        {
616
+        while ($post = $result->fetchObject()) {
597 617
             array_push($entryArray, new Entry($post->title, Book::getEntryIdByLetter($post->title),
598 618
                 str_format(localize('bookword', $post->count), $post->count), 'text',
599 619
                 array(new LinkNavigation('?page='.parent::PAGE_ALL_BOOKS_LETTER.'&id='. rawurlencode($post->title))), '', $post->count));
@@ -601,25 +621,27 @@  discard block
 block discarded – undo
601 621
         return $entryArray;
602 622
     }
603 623
 
604
-    public static function getBooksByStartingLetter($letter, $n, $database = NULL, $numberPerPage = NULL) {
624
+    public static function getBooksByStartingLetter($letter, $n, $database = NULL, $numberPerPage = NULL)
625
+    {
605 626
         return self::getEntryArray(self::SQL_BOOKS_BY_FIRST_LETTER, array($letter . '%'), $n, $database, $numberPerPage);
606 627
     }
607 628
 
608
-    public static function getEntryArray($query, $params, $n, $database = NULL, $numberPerPage = NULL) {
629
+    public static function getEntryArray($query, $params, $n, $database = NULL, $numberPerPage = NULL)
630
+    {
609 631
         /* @var $totalNumber integer */
610 632
         /* @var $result PDOStatement */
611 633
         list($totalNumber, $result) = parent::executeQuery($query, self::BOOK_COLUMNS, self::getFilterString(), $params, $n, $database, $numberPerPage);
612 634
 
613 635
         $entryArray = array();
614
-        while ($post = $result->fetchObject())
615
-        {
636
+        while ($post = $result->fetchObject()) {
616 637
             $book = new Book($post);
617 638
             array_push($entryArray, $book->getEntry());
618 639
         }
619 640
         return array($entryArray, $totalNumber);
620 641
     }
621 642
 
622
-    public static function getAllRecentBooks() {
643
+    public static function getAllRecentBooks()
644
+    {
623 645
         global $config;
624 646
         list ($entryArray, ) = self::getEntryArray(self::SQL_BOOKS_RECENT . $config['cops_recentbooks_limit'], array(), -1);
625 647
         return $entryArray;
@@ -631,7 +653,8 @@  discard block
 block discarded – undo
631 653
      * @param string[] $columns
632 654
      * @return CustomColumn[]
633 655
      */
634
-    public function getCustomColumnValues($columns, $asArray = false) {
656
+    public function getCustomColumnValues($columns, $asArray = false)
657
+    {
635 658
         $result = array();
636 659
 
637 660
         foreach ($columns as $lookup) {
Please login to merge, or discard this patch.