Test Setup Failed
Pull Request — master (#491)
by
unknown
07:16 queued 37s
created
lib/Author.php 2 patches
Spacing   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -25,69 +25,69 @@
 block discarded – undo
25 25
         $this->sort = $post->sort;
26 26
     }
27 27
 
28
-    public function getUri () {
29
-        return "?page=".parent::PAGE_AUTHOR_DETAIL."&id=$this->id";
28
+    public function getUri() {
29
+        return "?page=" . parent::PAGE_AUTHOR_DETAIL . "&id=$this->id";
30 30
     }
31 31
 
32
-    public function getEntryId () {
33
-        return self::ALL_AUTHORS_ID.":".$this->id;
32
+    public function getEntryId() {
33
+        return self::ALL_AUTHORS_ID . ":" . $this->id;
34 34
     }
35 35
 
36
-    public static function getEntryIdByLetter ($startingLetter) {
37
-        return self::ALL_AUTHORS_ID.":letter:".$startingLetter;
36
+    public static function getEntryIdByLetter($startingLetter) {
37
+        return self::ALL_AUTHORS_ID . ":letter:" . $startingLetter;
38 38
     }
39 39
 
40 40
     public static function getCount() {
41 41
         // str_format (localize("authors.alphabetical", count(array))
42
-        return parent::getCountGeneric ("authors", self::ALL_AUTHORS_ID, parent::PAGE_ALL_AUTHORS);
42
+        return parent::getCountGeneric("authors", self::ALL_AUTHORS_ID, parent::PAGE_ALL_AUTHORS);
43 43
     }
44 44
 
45 45
     public static function getAllAuthorsByFirstLetter() {
46
-        list (, $result) = parent::executeQuery ("select {0}
46
+        list (, $result) = parent::executeQuery("select {0}
47 47
 from authors
48 48
 group by substr (upper (sort), 1, 1)
49
-order by substr (upper (sort), 1, 1)", "substr (upper (sort), 1, 1) as title, count(*) as count", "", array (), -1);
49
+order by substr (upper (sort), 1, 1)", "substr (upper (sort), 1, 1) as title, count(*) as count", "", array(), -1);
50 50
         $entryArray = array();
51
-        while ($post = $result->fetchObject ())
51
+        while ($post = $result->fetchObject())
52 52
         {
53
-            array_push ($entryArray, new Entry ($post->title, Author::getEntryIdByLetter ($post->title),
54
-                str_format (localize("authorword", $post->count), $post->count), "text",
55
-                array ( new LinkNavigation ("?page=".parent::PAGE_AUTHORS_FIRST_LETTER."&id=". rawurlencode ($post->title))), "", $post->count));
53
+            array_push($entryArray, new Entry($post->title, Author::getEntryIdByLetter($post->title),
54
+                str_format(localize("authorword", $post->count), $post->count), "text",
55
+                array(new LinkNavigation("?page=" . parent::PAGE_AUTHORS_FIRST_LETTER . "&id=" . rawurlencode($post->title))), "", $post->count));
56 56
         }
57 57
         return $entryArray;
58 58
     }
59 59
 
60 60
     public static function getAuthorsByStartingLetter($letter) {
61
-        return self::getEntryArray (self::SQL_AUTHORS_BY_FIRST_LETTER, array ($letter . "%"));
61
+        return self::getEntryArray(self::SQL_AUTHORS_BY_FIRST_LETTER, array($letter . "%"));
62 62
     }
63 63
 
64 64
     public static function getAuthorsForSearch($query) {
65
-        return self::getEntryArray (self::SQL_AUTHORS_FOR_SEARCH, array ($query . "%", $query . "%"));
65
+        return self::getEntryArray(self::SQL_AUTHORS_FOR_SEARCH, array($query . "%", $query . "%"));
66 66
     }
67 67
 
68 68
     public static function getAllAuthors() {
69
-        return self::getEntryArray (self::SQL_ALL_AUTHORS, array ());
69
+        return self::getEntryArray(self::SQL_ALL_AUTHORS, array());
70 70
     }
71 71
 
72
-    public static function getEntryArray ($query, $params) {
73
-        return Base::getEntryArrayWithBookNumber ($query, self::AUTHOR_COLUMNS, $params, "Author");
72
+    public static function getEntryArray($query, $params) {
73
+        return Base::getEntryArrayWithBookNumber($query, self::AUTHOR_COLUMNS, $params, "Author");
74 74
     }
75 75
 
76
-    public static function getAuthorById ($authorId) {
77
-        $result = parent::getDb ()->prepare('select ' . self::AUTHOR_COLUMNS . ' from authors where id = ?');
78
-        $result->execute (array ($authorId));
79
-        $post = $result->fetchObject ();
80
-        return new Author ($post);
76
+    public static function getAuthorById($authorId) {
77
+        $result = parent::getDb()->prepare('select ' . self::AUTHOR_COLUMNS . ' from authors where id = ?');
78
+        $result->execute(array($authorId));
79
+        $post = $result->fetchObject();
80
+        return new Author($post);
81 81
     }
82 82
 
83
-    public static function getAuthorByBookId ($bookId) {
84
-        $result = parent::getDb ()->prepare('select authors.id as id, authors.name as name, authors.sort as sort from authors, books_authors_link
83
+    public static function getAuthorByBookId($bookId) {
84
+        $result = parent::getDb()->prepare('select authors.id as id, authors.name as name, authors.sort as sort from authors, books_authors_link
85 85
 where author = authors.id
86 86
 and book = ? order by books_authors_link.id');
87
-        $result->execute (array ($bookId));
88
-        $authorArray = array ();
89
-        while ($post = $result->fetchObject ()) {
90
-            array_push ($authorArray, new Author ($post));
87
+        $result->execute(array($bookId));
88
+        $authorArray = array();
89
+        while ($post = $result->fetchObject()) {
90
+            array_push($authorArray, new Author($post));
91 91
         }
92 92
         return $authorArray;
93 93
     }
Please login to merge, or discard this patch.
Braces   +25 added lines, -14 removed lines patch added patch discarded remove patch
@@ -19,37 +19,42 @@  discard block
 block discarded – undo
19 19
     public $name;
20 20
     public $sort;
21 21
 
22
-    public function __construct($post) {
22
+    public function __construct($post)
23
+    {
23 24
         $this->id = $post->id;
24 25
         $this->name = str_replace("|", ",", $post->name);
25 26
         $this->sort = $post->sort;
26 27
     }
27 28
 
28
-    public function getUri () {
29
+    public function getUri ()
30
+    {
29 31
         return "?page=".parent::PAGE_AUTHOR_DETAIL."&id=$this->id";
30 32
     }
31 33
 
32
-    public function getEntryId () {
34
+    public function getEntryId ()
35
+    {
33 36
         return self::ALL_AUTHORS_ID.":".$this->id;
34 37
     }
35 38
 
36
-    public static function getEntryIdByLetter ($startingLetter) {
39
+    public static function getEntryIdByLetter ($startingLetter)
40
+    {
37 41
         return self::ALL_AUTHORS_ID.":letter:".$startingLetter;
38 42
     }
39 43
 
40
-    public static function getCount() {
44
+    public static function getCount()
45
+    {
41 46
         // str_format (localize("authors.alphabetical", count(array))
42 47
         return parent::getCountGeneric ("authors", self::ALL_AUTHORS_ID, parent::PAGE_ALL_AUTHORS);
43 48
     }
44 49
 
45
-    public static function getAllAuthorsByFirstLetter() {
50
+    public static function getAllAuthorsByFirstLetter()
51
+    {
46 52
         list (, $result) = parent::executeQuery ("select {0}
47 53
 from authors
48 54
 group by substr (upper (sort), 1, 1)
49 55
 order by substr (upper (sort), 1, 1)", "substr (upper (sort), 1, 1) as title, count(*) as count", "", array (), -1);
50 56
         $entryArray = array();
51
-        while ($post = $result->fetchObject ())
52
-        {
57
+        while ($post = $result->fetchObject ()) {
53 58
             array_push ($entryArray, new Entry ($post->title, Author::getEntryIdByLetter ($post->title),
54 59
                 str_format (localize("authorword", $post->count), $post->count), "text",
55 60
                 array ( new LinkNavigation ("?page=".parent::PAGE_AUTHORS_FIRST_LETTER."&id=". rawurlencode ($post->title))), "", $post->count));
@@ -57,30 +62,36 @@  discard block
 block discarded – undo
57 62
         return $entryArray;
58 63
     }
59 64
 
60
-    public static function getAuthorsByStartingLetter($letter) {
65
+    public static function getAuthorsByStartingLetter($letter)
66
+    {
61 67
         return self::getEntryArray (self::SQL_AUTHORS_BY_FIRST_LETTER, array ($letter . "%"));
62 68
     }
63 69
 
64
-    public static function getAuthorsForSearch($query) {
70
+    public static function getAuthorsForSearch($query)
71
+    {
65 72
         return self::getEntryArray (self::SQL_AUTHORS_FOR_SEARCH, array ($query . "%", $query . "%"));
66 73
     }
67 74
 
68
-    public static function getAllAuthors() {
75
+    public static function getAllAuthors()
76
+    {
69 77
         return self::getEntryArray (self::SQL_ALL_AUTHORS, array ());
70 78
     }
71 79
 
72
-    public static function getEntryArray ($query, $params) {
80
+    public static function getEntryArray ($query, $params)
81
+    {
73 82
         return Base::getEntryArrayWithBookNumber ($query, self::AUTHOR_COLUMNS, $params, "Author");
74 83
     }
75 84
 
76
-    public static function getAuthorById ($authorId) {
85
+    public static function getAuthorById ($authorId)
86
+    {
77 87
         $result = parent::getDb ()->prepare('select ' . self::AUTHOR_COLUMNS . ' from authors where id = ?');
78 88
         $result->execute (array ($authorId));
79 89
         $post = $result->fetchObject ();
80 90
         return new Author ($post);
81 91
     }
82 92
 
83
-    public static function getAuthorByBookId ($bookId) {
93
+    public static function getAuthorByBookId ($bookId)
94
+    {
84 95
         $result = parent::getDb ()->prepare('select authors.id as id, authors.name as name, authors.sort as sort from authors, books_authors_link
85 96
 where author = authors.id
86 97
 and book = ? order by books_authors_link.id');
Please login to merge, or discard this patch.
lib/PageCustomize.php 3 patches
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -55,11 +55,11 @@
 block discarded – undo
55 55
         $this->entryArray = array ();
56 56
 
57 57
         $ignoredBaseArray = array (PageQueryResult::SCOPE_AUTHOR,
58
-                                   PageQueryResult::SCOPE_TAG,
59
-                                   PageQueryResult::SCOPE_SERIES,
60
-                                   PageQueryResult::SCOPE_PUBLISHER,
61
-                                   PageQueryResult::SCOPE_RATING,
62
-                                   "language");
58
+                                    PageQueryResult::SCOPE_TAG,
59
+                                    PageQueryResult::SCOPE_SERIES,
60
+                                    PageQueryResult::SCOPE_PUBLISHER,
61
+                                    PageQueryResult::SCOPE_RATING,
62
+                                    "language");
63 63
 
64 64
         $content = "";
65 65
         if (!preg_match("/(Kobo|Kindle\/3.0|EBRD1101)/", $_SERVER['HTTP_USER_AGENT'])) {
Please login to merge, or discard this patch.
Spacing   +49 added lines, -49 removed lines patch added patch discarded remove patch
@@ -8,10 +8,10 @@  discard block
 block discarded – undo
8 8
 
9 9
 class PageCustomize extends Page
10 10
 {
11
-    private function isChecked ($key, $testedValue = 1) {
12
-        $value = getCurrentOption ($key);
13
-        if (is_array ($value)) {
14
-            if (in_array ($testedValue, $value)) {
11
+    private function isChecked($key, $testedValue = 1) {
12
+        $value = getCurrentOption($key);
13
+        if (is_array($value)) {
14
+            if (in_array($testedValue, $value)) {
15 15
                 return "checked='checked'";
16 16
             }
17 17
         } else {
@@ -22,39 +22,39 @@  discard block
 block discarded – undo
22 22
         return "";
23 23
     }
24 24
 
25
-    private function isSelected ($key, $value) {
26
-        if (getCurrentOption ($key) == $value) {
25
+    private function isSelected($key, $value) {
26
+        if (getCurrentOption($key) == $value) {
27 27
             return "selected='selected'";
28 28
         }
29 29
         return "";
30 30
     }
31 31
 
32
-    private function getTemplateList () {
33
-        $result = array ();
34
-        foreach (glob ("templates/*") as $filename) {
35
-            if (preg_match ('/templates\/(.*)/', $filename, $m)) {
36
-                array_push ($result, $m [1]);
32
+    private function getTemplateList() {
33
+        $result = array();
34
+        foreach (glob("templates/*") as $filename) {
35
+            if (preg_match('/templates\/(.*)/', $filename, $m)) {
36
+                array_push($result, $m [1]);
37 37
             }            
38 38
         }
39 39
         return $result;
40 40
     }    
41 41
 
42
-    private function getStyleList () {
43
-        $result = array ();
44
-        foreach (glob ("templates/" . getCurrentTemplate () . "/styles/style-*.css") as $filename) {
45
-            if (preg_match ('/styles\/style-(.*?)\.css/', $filename, $m)) {
46
-                array_push ($result, $m [1]);
42
+    private function getStyleList() {
43
+        $result = array();
44
+        foreach (glob("templates/" . getCurrentTemplate() . "/styles/style-*.css") as $filename) {
45
+            if (preg_match('/styles\/style-(.*?)\.css/', $filename, $m)) {
46
+                array_push($result, $m [1]);
47 47
             }
48 48
         }
49 49
         return $result;
50 50
     }
51 51
 
52
-    public function InitializeContent ()
52
+    public function InitializeContent()
53 53
     {
54
-        $this->title = localize ("customize.title");
55
-        $this->entryArray = array ();
54
+        $this->title = localize("customize.title");
55
+        $this->entryArray = array();
56 56
 
57
-        $ignoredBaseArray = array (PageQueryResult::SCOPE_AUTHOR,
57
+        $ignoredBaseArray = array(PageQueryResult::SCOPE_AUTHOR,
58 58
                                    PageQueryResult::SCOPE_TAG,
59 59
                                    PageQueryResult::SCOPE_SERIES,
60 60
                                    PageQueryResult::SCOPE_PUBLISHER,
@@ -65,60 +65,60 @@  discard block
 block discarded – undo
65 65
         if (!preg_match("/(Kobo|Kindle\/3.0|EBRD1101)/", $_SERVER['HTTP_USER_AGENT'])) {
66 66
             $content .= "<select id='template' onchange='updateCookie (this); window.location=$(\".headleft\").attr(\"href\");'>";
67 67
 
68
-            foreach ($this-> getTemplateList () as $filename) {
69
-                $content .= "<option value='{$filename}' " . $this->isSelected ("template", $filename) . ">{$filename}</option>";
68
+            foreach ($this-> getTemplateList() as $filename) {
69
+                $content .= "<option value='{$filename}' " . $this->isSelected("template", $filename) . ">{$filename}</option>";
70 70
             }
71 71
             $content .= '</select>';
72 72
         } else {
73
-            foreach ($this-> getTemplateList () as $filename) {
74
-                $content .= "<input type='radio' onchange='updateCookieFromCheckbox (this); window.location=$(\".headleft\").attr(\"href\");' id='template' name='template' value='{$filename}' " . $this->isChecked ("template", $filename) . " /><label for='template-{$filename}'> {$filename} </label>";
73
+            foreach ($this-> getTemplateList() as $filename) {
74
+                $content .= "<input type='radio' onchange='updateCookieFromCheckbox (this); window.location=$(\".headleft\").attr(\"href\");' id='template' name='template' value='{$filename}' " . $this->isChecked("template", $filename) . " /><label for='template-{$filename}'> {$filename} </label>";
75 75
             }
76 76
         }
77
-        array_push ($this->entryArray, new Entry ("Template", "",
77
+        array_push($this->entryArray, new Entry("Template", "",
78 78
                                         $content, "text",
79
-                                        array ()));
79
+                                        array()));
80 80
 
81 81
         $content = "";
82 82
         if (!preg_match("/(Kobo|Kindle\/3.0|EBRD1101)/", $_SERVER['HTTP_USER_AGENT'])) {
83 83
             $content .= '<select id="style" onchange="updateCookie (this);">';
84
-            foreach ($this-> getStyleList () as $filename) {
85
-                $content .= "<option value='{$filename}' " . $this->isSelected ("style", $filename) . ">{$filename}</option>";
84
+            foreach ($this-> getStyleList() as $filename) {
85
+                $content .= "<option value='{$filename}' " . $this->isSelected("style", $filename) . ">{$filename}</option>";
86 86
             }
87 87
             $content .= '</select>';
88 88
         } else {
89
-            foreach ($this-> getStyleList () as $filename) {
90
-                $content .= "<input type='radio' onchange='updateCookieFromCheckbox (this);' id='style-{$filename}' name='style' value='{$filename}' " . $this->isChecked ("style", $filename) . " /><label for='style-{$filename}'> {$filename} </label>";
89
+            foreach ($this-> getStyleList() as $filename) {
90
+                $content .= "<input type='radio' onchange='updateCookieFromCheckbox (this);' id='style-{$filename}' name='style' value='{$filename}' " . $this->isChecked("style", $filename) . " /><label for='style-{$filename}'> {$filename} </label>";
91 91
             }
92 92
         }
93
-        array_push ($this->entryArray, new Entry (localize ("customize.style"), "",
93
+        array_push($this->entryArray, new Entry(localize("customize.style"), "",
94 94
                                         $content, "text",
95
-                                        array ()));
96
-        if (!useServerSideRendering ()) {
97
-            $content = '<input type="checkbox" onchange="updateCookieFromCheckbox (this);" id="use_fancyapps" ' . $this->isChecked ("use_fancyapps") . ' />';
98
-            array_push ($this->entryArray, new Entry (localize ("customize.fancybox"), "",
95
+                                        array()));
96
+        if (!useServerSideRendering()) {
97
+            $content = '<input type="checkbox" onchange="updateCookieFromCheckbox (this);" id="use_fancyapps" ' . $this->isChecked("use_fancyapps") . ' />';
98
+            array_push($this->entryArray, new Entry(localize("customize.fancybox"), "",
99 99
                                             $content, "text",
100
-                                            array ()));
100
+                                            array()));
101 101
         }
102
-        $content = '<input type="number" onchange="updateCookie (this);" id="max_item_per_page" value="' . getCurrentOption ("max_item_per_page") . '" min="-1" max="1200" pattern="^[-+]?[0-9]+$" />';
103
-        array_push ($this->entryArray, new Entry (localize ("customize.paging"), "",
102
+        $content = '<input type="number" onchange="updateCookie (this);" id="max_item_per_page" value="' . getCurrentOption("max_item_per_page") . '" min="-1" max="1200" pattern="^[-+]?[0-9]+$" />';
103
+        array_push($this->entryArray, new Entry(localize("customize.paging"), "",
104 104
                                         $content, "text",
105
-                                        array ()));
106
-        $content = '<input type="text" onchange="updateCookie (this);" id="email" value="' . getCurrentOption ("email") . '" />';
107
-        array_push ($this->entryArray, new Entry (localize ("customize.email"), "",
105
+                                        array()));
106
+        $content = '<input type="text" onchange="updateCookie (this);" id="email" value="' . getCurrentOption("email") . '" />';
107
+        array_push($this->entryArray, new Entry(localize("customize.email"), "",
108 108
                                         $content, "text",
109
-                                        array ()));
110
-        $content = '<input type="checkbox" onchange="updateCookieFromCheckbox (this);" id="html_tag_filter" ' . $this->isChecked ("html_tag_filter") . ' />';
111
-        array_push ($this->entryArray, new Entry (localize ("customize.filter"), "",
109
+                                        array()));
110
+        $content = '<input type="checkbox" onchange="updateCookieFromCheckbox (this);" id="html_tag_filter" ' . $this->isChecked("html_tag_filter") . ' />';
111
+        array_push($this->entryArray, new Entry(localize("customize.filter"), "",
112 112
                                         $content, "text",
113
-                                        array ()));
113
+                                        array()));
114 114
         $content = "";
115 115
         foreach ($ignoredBaseArray as $key) {
116
-            $keyPlural = preg_replace ('/(ss)$/', 's', $key . "s");
117
-            $content .=  '<input type="checkbox" name="ignored_categories[]" onchange="updateCookieFromCheckboxGroup (this);" id="ignored_categories_' . $key . '" ' . $this->isChecked ("ignored_categories", $key) . ' > ' . localize ("{$keyPlural}.title") . '</input> ';
116
+            $keyPlural = preg_replace('/(ss)$/', 's', $key . "s");
117
+            $content .= '<input type="checkbox" name="ignored_categories[]" onchange="updateCookieFromCheckboxGroup (this);" id="ignored_categories_' . $key . '" ' . $this->isChecked("ignored_categories", $key) . ' > ' . localize("{$keyPlural}.title") . '</input> ';
118 118
         }
119 119
 
120
-        array_push ($this->entryArray, new Entry (localize ("customize.ignored"), "",
120
+        array_push($this->entryArray, new Entry(localize("customize.ignored"), "",
121 121
                                         $content, "text",
122
-                                        array ()));
122
+                                        array()));
123 123
     }
124 124
 }
Please login to merge, or discard this patch.
Braces   +8 added lines, -4 removed lines patch added patch discarded remove patch
@@ -8,7 +8,8 @@  discard block
 block discarded – undo
8 8
 
9 9
 class PageCustomize extends Page
10 10
 {
11
-    private function isChecked ($key, $testedValue = 1) {
11
+    private function isChecked ($key, $testedValue = 1)
12
+    {
12 13
         $value = getCurrentOption ($key);
13 14
         if (is_array ($value)) {
14 15
             if (in_array ($testedValue, $value)) {
@@ -22,14 +23,16 @@  discard block
 block discarded – undo
22 23
         return "";
23 24
     }
24 25
 
25
-    private function isSelected ($key, $value) {
26
+    private function isSelected ($key, $value)
27
+    {
26 28
         if (getCurrentOption ($key) == $value) {
27 29
             return "selected='selected'";
28 30
         }
29 31
         return "";
30 32
     }
31 33
 
32
-    private function getTemplateList () {
34
+    private function getTemplateList ()
35
+    {
33 36
         $result = array ();
34 37
         foreach (glob ("templates/*") as $filename) {
35 38
             if (preg_match ('/templates\/(.*)/', $filename, $m)) {
@@ -39,7 +42,8 @@  discard block
 block discarded – undo
39 42
         return $result;
40 43
     }    
41 44
 
42
-    private function getStyleList () {
45
+    private function getStyleList ()
46
+    {
43 47
         $result = array ();
44 48
         foreach (glob ("templates/" . getCurrentTemplate () . "/styles/style-*.css") as $filename) {
45 49
             if (preg_match ('/styles\/style-(.*?)\.css/', $filename, $m)) {
Please login to merge, or discard this patch.
lib/Data.php 3 patches
Doc Comments   +10 added lines patch added patch discarded remove patch
@@ -152,6 +152,9 @@  discard block
 block discarded – undo
152 152
         return new Link ($href, $this->getMimeType (), Link::OPDS_ACQUISITION_TYPE, $title);
153 153
     }
154 154
 
155
+    /**
156
+     * @param Book $book
157
+     */
155 158
     public static function getDataByBook ($book) {
156 159
         global $config;
157 160
 
@@ -176,6 +179,9 @@  discard block
 block discarded – undo
176 179
         return $out;
177 180
     }
178 181
 
182
+    /**
183
+     * @param string $urlParam
184
+     */
179 185
     public static function handleThumbnailLink ($urlParam, $height) {
180 186
         global $config;
181 187
 
@@ -195,6 +201,10 @@  discard block
 block discarded – undo
195 201
         return $urlParam;
196 202
     }
197 203
 
204
+    /**
205
+     * @param string $type
206
+     * @param string $filename
207
+     */
198 208
     public static function getLink ($book, $type, $mime, $rel, $filename, $idData, $title = NULL, $height = NULL, $view = false)
199 209
     {
200 210
         global $config;
Please login to merge, or discard this patch.
Spacing   +44 added lines, -44 removed lines patch added patch discarded remove patch
@@ -59,25 +59,25 @@  discard block
 block discarded – undo
59 59
         $this->id = $post->id;
60 60
         $this->name = $post->name;
61 61
         $this->format = $post->format;
62
-        $this->realFormat = str_replace ("ORIGINAL_", "", $post->format);
63
-        $this->extension = strtolower ($this->realFormat);
62
+        $this->realFormat = str_replace("ORIGINAL_", "", $post->format);
63
+        $this->extension = strtolower($this->realFormat);
64 64
         $this->book = $book;
65 65
     }
66 66
 
67
-    public function isKnownType () {
68
-        return array_key_exists ($this->extension, self::$mimetypes);
67
+    public function isKnownType() {
68
+        return array_key_exists($this->extension, self::$mimetypes);
69 69
     }
70 70
 
71
-    public function getMimeType () {
71
+    public function getMimeType() {
72 72
         $result = "application/octet-stream";
73
-        if ($this->isKnownType ()) {
73
+        if ($this->isKnownType()) {
74 74
             return self::$mimetypes [$this->extension];
75 75
         } elseif (function_exists('finfo_open') === true) {
76 76
             $finfo = finfo_open(FILEINFO_MIME_TYPE);
77 77
 
78 78
             if (is_resource($finfo) === true)
79 79
             {
80
-                $result = finfo_file($finfo, $this->getLocalPath ());
80
+                $result = finfo_file($finfo, $this->getLocalPath());
81 81
             }
82 82
 
83 83
             finfo_close($finfo);
@@ -86,55 +86,55 @@  discard block
 block discarded – undo
86 86
         return $result;
87 87
     }
88 88
 
89
-    public function isEpubValidOnKobo () {
89
+    public function isEpubValidOnKobo() {
90 90
         return $this->format == "EPUB" || $this->format == "KEPUB";
91 91
     }
92 92
 
93
-    public function getFilename () {
94
-        return $this->name . "." . strtolower ($this->format);
93
+    public function getFilename() {
94
+        return $this->name . "." . strtolower($this->format);
95 95
     }
96 96
 
97
-    public function getUpdatedFilename () {
98
-        return $this->book->getAuthorsSort () . " - " . $this->book->title;
97
+    public function getUpdatedFilename() {
98
+        return $this->book->getAuthorsSort() . " - " . $this->book->title;
99 99
     }
100 100
 
101
-    public function getUpdatedFilenameEpub () {
102
-        return $this->getUpdatedFilename () . ".epub";
101
+    public function getUpdatedFilenameEpub() {
102
+        return $this->getUpdatedFilename() . ".epub";
103 103
     }
104 104
 
105
-    public function getUpdatedFilenameKepub () {
106
-        $str = $this->getUpdatedFilename () . ".kepub.epub";
105
+    public function getUpdatedFilenameKepub() {
106
+        $str = $this->getUpdatedFilename() . ".kepub.epub";
107 107
         return str_replace(array(':', '#', '&'),
108
-                           array('-', '-', ' '), $str );
108
+                           array('-', '-', ' '), $str);
109 109
     }
110 110
 
111
-    public function getDataLink ($rel, $title = NULL, $view = false) {
111
+    public function getDataLink($rel, $title = NULL, $view = false) {
112 112
         global $config;
113 113
 
114 114
         if ($rel == Link::OPDS_ACQUISITION_TYPE && $config['cops_use_url_rewriting'] == "1") {
115 115
             return $this->getHtmlLinkWithRewriting($title, $view);
116 116
         }
117 117
 
118
-        return self::getLink ($this->book, $this->extension, $this->getMimeType (), $rel, $this->getFilename (), $this->id, $title, NULL, $view);
118
+        return self::getLink($this->book, $this->extension, $this->getMimeType(), $rel, $this->getFilename(), $this->id, $title, NULL, $view);
119 119
     }
120 120
 
121
-    public function getHtmlLink () {
121
+    public function getHtmlLink() {
122 122
         return $this->getDataLink(Link::OPDS_ACQUISITION_TYPE)->href;
123 123
     }
124 124
 
125
-    public function getViewHtmlLink () {
125
+    public function getViewHtmlLink() {
126 126
         return $this->getDataLink(Link::OPDS_ACQUISITION_TYPE, NULL, true)->href;
127 127
     }
128 128
 
129
-    public function getLocalPath () {
130
-        return $this->book->path . "/" . $this->getFilename ();
129
+    public function getLocalPath() {
130
+        return $this->book->path . "/" . $this->getFilename();
131 131
     }
132 132
 
133
-    public function getHtmlLinkWithRewriting ($title = NULL, $view = false) {
133
+    public function getHtmlLinkWithRewriting($title = NULL, $view = false) {
134 134
         global $config;
135 135
 
136 136
         $database = "";
137
-        if (!is_null (GetUrlParam (DB))) $database = GetUrlParam (DB) . "/";
137
+        if (!is_null(GetUrlParam(DB))) $database = GetUrlParam(DB) . "/";
138 138
 
139 139
         $prefix = "download";
140 140
         if ($view) {
@@ -143,19 +143,19 @@  discard block
 block discarded – undo
143 143
         $href = $prefix . "/" . $this->id . "/" . $database;
144 144
 
145 145
         if ($config['cops_provide_kepub'] == "1" &&
146
-            $this->isEpubValidOnKobo () &&
146
+            $this->isEpubValidOnKobo() &&
147 147
             preg_match("/Kobo/", $_SERVER['HTTP_USER_AGENT'])) {
148
-            $href .= rawurlencode ($this->getUpdatedFilenameKepub ());
148
+            $href .= rawurlencode($this->getUpdatedFilenameKepub());
149 149
         } else {
150
-            $href .= rawurlencode ($this->getFilename ());
150
+            $href .= rawurlencode($this->getFilename());
151 151
         }
152
-        return new Link ($href, $this->getMimeType (), Link::OPDS_ACQUISITION_TYPE, $title);
152
+        return new Link($href, $this->getMimeType(), Link::OPDS_ACQUISITION_TYPE, $title);
153 153
     }
154 154
 
155
-    public static function getDataByBook ($book) {
155
+    public static function getDataByBook($book) {
156 156
         global $config;
157 157
 
158
-        $out = array ();
158
+        $out = array();
159 159
 
160 160
         $sql = 'select id, format, name from data where book = ?';
161 161
         
@@ -166,21 +166,21 @@  discard block
 block discarded – undo
166 166
             . "')";
167 167
         }
168 168
 
169
-        $result = parent::getDb ()->prepare($sql);
170
-        $result->execute (array ($book->id));
169
+        $result = parent::getDb()->prepare($sql);
170
+        $result->execute(array($book->id));
171 171
 
172
-        while ($post = $result->fetchObject ())
172
+        while ($post = $result->fetchObject())
173 173
         {
174
-            array_push ($out, new Data ($post, $book));
174
+            array_push($out, new Data($post, $book));
175 175
         }
176 176
         return $out;
177 177
     }
178 178
 
179
-    public static function handleThumbnailLink ($urlParam, $height) {
179
+    public static function handleThumbnailLink($urlParam, $height) {
180 180
         global $config;
181 181
 
182
-        if (is_null ($height)) {
183
-            if (preg_match ('/feed.php/', $_SERVER["SCRIPT_NAME"])) {
182
+        if (is_null($height)) {
183
+            if (preg_match('/feed.php/', $_SERVER["SCRIPT_NAME"])) {
184 184
                 $height = $config['cops_opds_thumbnail_height'];
185 185
             }
186 186
             else
@@ -195,14 +195,14 @@  discard block
 block discarded – undo
195 195
         return $urlParam;
196 196
     }
197 197
 
198
-    public static function getLink ($book, $type, $mime, $rel, $filename, $idData, $title = NULL, $height = NULL, $view = false)
198
+    public static function getLink($book, $type, $mime, $rel, $filename, $idData, $title = NULL, $height = NULL, $view = false)
199 199
     {
200 200
         global $config;
201 201
 
202 202
         $urlParam = addURLParameter("", "data", $idData);
203 203
         if ($view) $urlParam = addURLParameter($urlParam, "view", 1);
204 204
 
205
-        if (Base::useAbsolutePath () ||
205
+        if (Base::useAbsolutePath() ||
206 206
             $rel == Link::OPDS_THUMBNAIL_TYPE ||
207 207
             ($type == "epub" && $config['cops_update_epub-metadata']))
208 208
         {
@@ -211,18 +211,18 @@  discard block
 block discarded – undo
211 211
                 $urlParam = self::handleThumbnailLink($urlParam, $height);
212 212
             }
213 213
             $urlParam = addURLParameter($urlParam, "id", $book->id);
214
-            if (!is_null (GetUrlParam (DB))) $urlParam = addURLParameter ($urlParam, DB, GetUrlParam (DB));
214
+            if (!is_null(GetUrlParam(DB))) $urlParam = addURLParameter($urlParam, DB, GetUrlParam(DB));
215 215
             if ($config['cops_thumbnail_handling'] != "1" &&
216 216
                 !empty ($config['cops_thumbnail_handling']) &&
217 217
                 $rel == Link::OPDS_THUMBNAIL_TYPE) {
218
-                return new Link ($config['cops_thumbnail_handling'], $mime, $rel, $title);
218
+                return new Link($config['cops_thumbnail_handling'], $mime, $rel, $title);
219 219
             } else {
220
-                return new Link ("fetch.php?" . $urlParam, $mime, $rel, $title);
220
+                return new Link("fetch.php?" . $urlParam, $mime, $rel, $title);
221 221
             }
222 222
         }
223 223
         else
224 224
         {
225
-            return new Link (str_replace('%2F','/',rawurlencode ($book->path."/".$filename)), $mime, $rel, $title);
225
+            return new Link(str_replace('%2F', '/', rawurlencode($book->path . "/" . $filename)), $mime, $rel, $title);
226 226
         }
227 227
     }
228 228
 }
Please login to merge, or discard this patch.
Braces   +47 added lines, -31 removed lines patch added patch discarded remove patch
@@ -55,7 +55,8 @@  discard block
 block discarded – undo
55 55
         'zip'   => 'application/zip'
56 56
     );
57 57
 
58
-    public function __construct($post, $book = null) {
58
+    public function __construct($post, $book = null)
59
+    {
59 60
         $this->id = $post->id;
60 61
         $this->name = $post->name;
61 62
         $this->format = $post->format;
@@ -64,19 +65,20 @@  discard block
 block discarded – undo
64 65
         $this->book = $book;
65 66
     }
66 67
 
67
-    public function isKnownType () {
68
+    public function isKnownType ()
69
+    {
68 70
         return array_key_exists ($this->extension, self::$mimetypes);
69 71
     }
70 72
 
71
-    public function getMimeType () {
73
+    public function getMimeType ()
74
+    {
72 75
         $result = "application/octet-stream";
73 76
         if ($this->isKnownType ()) {
74 77
             return self::$mimetypes [$this->extension];
75 78
         } elseif (function_exists('finfo_open') === true) {
76 79
             $finfo = finfo_open(FILEINFO_MIME_TYPE);
77 80
 
78
-            if (is_resource($finfo) === true)
79
-            {
81
+            if (is_resource($finfo) === true) {
80 82
                 $result = finfo_file($finfo, $this->getLocalPath ());
81 83
             }
82 84
 
@@ -86,29 +88,35 @@  discard block
 block discarded – undo
86 88
         return $result;
87 89
     }
88 90
 
89
-    public function isEpubValidOnKobo () {
91
+    public function isEpubValidOnKobo ()
92
+    {
90 93
         return $this->format == "EPUB" || $this->format == "KEPUB";
91 94
     }
92 95
 
93
-    public function getFilename () {
96
+    public function getFilename ()
97
+    {
94 98
         return $this->name . "." . strtolower ($this->format);
95 99
     }
96 100
 
97
-    public function getUpdatedFilename () {
101
+    public function getUpdatedFilename ()
102
+    {
98 103
         return $this->book->getAuthorsSort () . " - " . $this->book->title;
99 104
     }
100 105
 
101
-    public function getUpdatedFilenameEpub () {
106
+    public function getUpdatedFilenameEpub ()
107
+    {
102 108
         return $this->getUpdatedFilename () . ".epub";
103 109
     }
104 110
 
105
-    public function getUpdatedFilenameKepub () {
111
+    public function getUpdatedFilenameKepub ()
112
+    {
106 113
         $str = $this->getUpdatedFilename () . ".kepub.epub";
107 114
         return str_replace(array(':', '#', '&'),
108 115
                            array('-', '-', ' '), $str );
109 116
     }
110 117
 
111
-    public function getDataLink ($rel, $title = NULL, $view = false) {
118
+    public function getDataLink ($rel, $title = NULL, $view = false)
119
+    {
112 120
         global $config;
113 121
 
114 122
         if ($rel == Link::OPDS_ACQUISITION_TYPE && $config['cops_use_url_rewriting'] == "1") {
@@ -118,23 +126,29 @@  discard block
 block discarded – undo
118 126
         return self::getLink ($this->book, $this->extension, $this->getMimeType (), $rel, $this->getFilename (), $this->id, $title, NULL, $view);
119 127
     }
120 128
 
121
-    public function getHtmlLink () {
129
+    public function getHtmlLink ()
130
+    {
122 131
         return $this->getDataLink(Link::OPDS_ACQUISITION_TYPE)->href;
123 132
     }
124 133
 
125
-    public function getViewHtmlLink () {
134
+    public function getViewHtmlLink ()
135
+    {
126 136
         return $this->getDataLink(Link::OPDS_ACQUISITION_TYPE, NULL, true)->href;
127 137
     }
128 138
 
129
-    public function getLocalPath () {
139
+    public function getLocalPath ()
140
+    {
130 141
         return $this->book->path . "/" . $this->getFilename ();
131 142
     }
132 143
 
133
-    public function getHtmlLinkWithRewriting ($title = NULL, $view = false) {
144
+    public function getHtmlLinkWithRewriting ($title = NULL, $view = false)
145
+    {
134 146
         global $config;
135 147
 
136 148
         $database = "";
137
-        if (!is_null (GetUrlParam (DB))) $database = GetUrlParam (DB) . "/";
149
+        if (!is_null (GetUrlParam (DB))) {
150
+            $database = GetUrlParam (DB) . "/";
151
+        }
138 152
 
139 153
         $prefix = "download";
140 154
         if ($view) {
@@ -152,7 +166,8 @@  discard block
 block discarded – undo
152 166
         return new Link ($href, $this->getMimeType (), Link::OPDS_ACQUISITION_TYPE, $title);
153 167
     }
154 168
 
155
-    public static function getDataByBook ($book) {
169
+    public static function getDataByBook ($book)
170
+    {
156 171
         global $config;
157 172
 
158 173
         $out = array ();
@@ -169,22 +184,20 @@  discard block
 block discarded – undo
169 184
         $result = parent::getDb ()->prepare($sql);
170 185
         $result->execute (array ($book->id));
171 186
 
172
-        while ($post = $result->fetchObject ())
173
-        {
187
+        while ($post = $result->fetchObject ()) {
174 188
             array_push ($out, new Data ($post, $book));
175 189
         }
176 190
         return $out;
177 191
     }
178 192
 
179
-    public static function handleThumbnailLink ($urlParam, $height) {
193
+    public static function handleThumbnailLink ($urlParam, $height)
194
+    {
180 195
         global $config;
181 196
 
182 197
         if (is_null ($height)) {
183 198
             if (preg_match ('/feed.php/', $_SERVER["SCRIPT_NAME"])) {
184 199
                 $height = $config['cops_opds_thumbnail_height'];
185
-            }
186
-            else
187
-            {
200
+            } else {
188 201
                 $height = $config['cops_html_thumbnail_height'];
189 202
             }
190 203
         }
@@ -200,18 +213,23 @@  discard block
 block discarded – undo
200 213
         global $config;
201 214
 
202 215
         $urlParam = addURLParameter("", "data", $idData);
203
-        if ($view) $urlParam = addURLParameter($urlParam, "view", 1);
216
+        if ($view) {
217
+            $urlParam = addURLParameter($urlParam, "view", 1);
218
+        }
204 219
 
205 220
         if (Base::useAbsolutePath () ||
206 221
             $rel == Link::OPDS_THUMBNAIL_TYPE ||
207
-            ($type == "epub" && $config['cops_update_epub-metadata']))
208
-        {
209
-            if ($type != "jpg") $urlParam = addURLParameter($urlParam, "type", $type);
222
+            ($type == "epub" && $config['cops_update_epub-metadata'])) {
223
+            if ($type != "jpg") {
224
+                $urlParam = addURLParameter($urlParam, "type", $type);
225
+            }
210 226
             if ($rel == Link::OPDS_THUMBNAIL_TYPE) {
211 227
                 $urlParam = self::handleThumbnailLink($urlParam, $height);
212 228
             }
213 229
             $urlParam = addURLParameter($urlParam, "id", $book->id);
214
-            if (!is_null (GetUrlParam (DB))) $urlParam = addURLParameter ($urlParam, DB, GetUrlParam (DB));
230
+            if (!is_null (GetUrlParam (DB))) {
231
+                $urlParam = addURLParameter ($urlParam, DB, GetUrlParam (DB));
232
+            }
215 233
             if ($config['cops_thumbnail_handling'] != "1" &&
216 234
                 !empty ($config['cops_thumbnail_handling']) &&
217 235
                 $rel == Link::OPDS_THUMBNAIL_TYPE) {
@@ -219,9 +237,7 @@  discard block
 block discarded – undo
219 237
             } else {
220 238
                 return new Link ("fetch.php?" . $urlParam, $mime, $rel, $title);
221 239
             }
222
-        }
223
-        else
224
-        {
240
+        } else {
225 241
             return new Link (str_replace('%2F','/',rawurlencode ($book->path."/".$filename)), $mime, $rel, $title);
226 242
         }
227 243
     }
Please login to merge, or discard this patch.
lib/JSON_renderer.php 2 patches
Indentation   +50 added lines, -50 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
             if ($data = $book->getDataFormat ($format)) {
25 25
                 $i++;
26 26
                 array_push ($preferedData, array ("url" => $data->getHtmlLink (),
27
-                  "viewUrl" => $data->getViewHtmlLink (), "name" => $format));
27
+                    "viewUrl" => $data->getViewHtmlLink (), "name" => $format));
28 28
             }
29 29
         }
30 30
 
@@ -52,20 +52,20 @@  discard block
 block discarded – undo
52 52
         $cc = $book->getCustomColumnValues($config['cops_calibre_custom_column_list'], true);
53 53
 
54 54
         return array ("id" => $book->id,
55
-                      "hasCover" => $book->hasCover,
56
-                      "preferedData" => $preferedData,
57
-                      "rating" => $book->getRating (),
58
-                      "publisherName" => $pn,
59
-                      "publisherurl" => $pu,
60
-                      "pubDate" => $book->getPubDate (),
61
-                      "languagesName" => $book->getLanguages (),
62
-                      "authorsName" => $book->getAuthorsName (),
63
-                      "tagsName" => $book->getTagsName (),
64
-                      "seriesName" => $sn,
65
-                      "seriesIndex" => $book->seriesIndex,
66
-                      "seriesCompleteName" => $scn,
67
-                      "seriesurl" => $su,
68
-                      "customcolumns_list" => $cc);
55
+                        "hasCover" => $book->hasCover,
56
+                        "preferedData" => $preferedData,
57
+                        "rating" => $book->getRating (),
58
+                        "publisherName" => $pn,
59
+                        "publisherurl" => $pu,
60
+                        "pubDate" => $book->getPubDate (),
61
+                        "languagesName" => $book->getLanguages (),
62
+                        "authorsName" => $book->getAuthorsName (),
63
+                        "tagsName" => $book->getTagsName (),
64
+                        "seriesName" => $sn,
65
+                        "seriesIndex" => $book->seriesIndex,
66
+                        "seriesCompleteName" => $scn,
67
+                        "seriesurl" => $su,
68
+                        "customcolumns_list" => $cc);
69 69
     }
70 70
 
71 71
     /**
@@ -148,46 +148,46 @@  discard block
 block discarded – undo
148 148
         $out = $in;
149 149
 
150 150
         $out ["c"] = array ("version" => VERSION, "i18n" => array (
151
-                           "coverAlt" => localize("i18n.coversection"),
152
-                           "authorsTitle" => localize("authors.title"),
153
-                           "bookwordTitle" => localize("bookword.title"),
154
-                           "tagsTitle" => localize("tags.title"),
155
-                           "linksTitle" => localize("links.title"),
156
-                           "seriesTitle" => localize("series.title"),
157
-                           "customizeTitle" => localize ("customize.title"),
158
-                           "aboutTitle" => localize ("about.title"),
159
-                           "previousAlt" => localize ("paging.previous.alternate"),
160
-                           "nextAlt" => localize ("paging.next.alternate"),
161
-                           "searchAlt" => localize ("search.alternate"),
162
-                           "sortAlt" => localize ("sort.alternate"),
163
-                           "homeAlt" => localize ("home.alternate"),
164
-                           "cogAlt" => localize ("cog.alternate"),
165
-                           "permalinkAlt" => localize ("permalink.alternate"),
166
-                           "publisherName" => localize("publisher.name"),
167
-                           "pubdateTitle" => localize("pubdate.title"),
168
-                           "languagesTitle" => localize("language.title"),
169
-                           "contentTitle" => localize("content.summary"),
170
-                           "filterClearAll" => localize("filter.clearall"),
171
-                           "sortorderAsc" => localize("search.sortorder.asc"),
172
-                           "sortorderDesc" => localize("search.sortorder.desc"),
173
-                           "customizeEmail" => localize("customize.email")),
174
-                       "url" => array (
175
-                           "detailUrl" => "index.php?page=13&id={0}&db={1}",
176
-                           "coverUrl" => "fetch.php?id={0}&db={1}",
177
-                           "thumbnailUrl" => "fetch.php?height=" . $config['cops_html_thumbnail_height'] . "&id={0}&db={1}"),
178
-                       "config" => array (
179
-                           "use_fancyapps" => $config ["cops_use_fancyapps"],
180
-                           "max_item_per_page" => $config['cops_max_item_per_page'],
181
-                           "kindleHack"        => "",
182
-                           "server_side_rendering" => useServerSideRendering (),
183
-                           "html_tag_filter" => $config['cops_html_tag_filter']));
151
+                            "coverAlt" => localize("i18n.coversection"),
152
+                            "authorsTitle" => localize("authors.title"),
153
+                            "bookwordTitle" => localize("bookword.title"),
154
+                            "tagsTitle" => localize("tags.title"),
155
+                            "linksTitle" => localize("links.title"),
156
+                            "seriesTitle" => localize("series.title"),
157
+                            "customizeTitle" => localize ("customize.title"),
158
+                            "aboutTitle" => localize ("about.title"),
159
+                            "previousAlt" => localize ("paging.previous.alternate"),
160
+                            "nextAlt" => localize ("paging.next.alternate"),
161
+                            "searchAlt" => localize ("search.alternate"),
162
+                            "sortAlt" => localize ("sort.alternate"),
163
+                            "homeAlt" => localize ("home.alternate"),
164
+                            "cogAlt" => localize ("cog.alternate"),
165
+                            "permalinkAlt" => localize ("permalink.alternate"),
166
+                            "publisherName" => localize("publisher.name"),
167
+                            "pubdateTitle" => localize("pubdate.title"),
168
+                            "languagesTitle" => localize("language.title"),
169
+                            "contentTitle" => localize("content.summary"),
170
+                            "filterClearAll" => localize("filter.clearall"),
171
+                            "sortorderAsc" => localize("search.sortorder.asc"),
172
+                            "sortorderDesc" => localize("search.sortorder.desc"),
173
+                            "customizeEmail" => localize("customize.email")),
174
+                        "url" => array (
175
+                            "detailUrl" => "index.php?page=13&id={0}&db={1}",
176
+                            "coverUrl" => "fetch.php?id={0}&db={1}",
177
+                            "thumbnailUrl" => "fetch.php?height=" . $config['cops_html_thumbnail_height'] . "&id={0}&db={1}"),
178
+                        "config" => array (
179
+                            "use_fancyapps" => $config ["cops_use_fancyapps"],
180
+                            "max_item_per_page" => $config['cops_max_item_per_page'],
181
+                            "kindleHack"        => "",
182
+                            "server_side_rendering" => useServerSideRendering (),
183
+                            "html_tag_filter" => $config['cops_html_tag_filter']));
184 184
         if ($config['cops_thumbnail_handling'] == "1") {
185 185
             $out ["c"]["url"]["thumbnailUrl"] = $out ["c"]["url"]["coverUrl"];
186 186
         } else if (!empty ($config['cops_thumbnail_handling'])) {
187 187
             $out ["c"]["url"]["thumbnailUrl"] = $config['cops_thumbnail_handling'];
188 188
         }
189 189
         if (preg_match("/./", $_SERVER['HTTP_USER_AGENT'])) {
190
-          $out ["c"]["config"]["kindleHack"] = 'style="text-decoration: none !important;"';
190
+            $out ["c"]["config"]["kindleHack"] = 'style="text-decoration: none !important;"';
191 191
         }
192 192
         return $out;
193 193
     }
@@ -246,7 +246,7 @@  discard block
 block discarded – undo
246 246
         }
247 247
         if (!is_null (getURLParam ("complete")) || $complete) {
248 248
             $out = self::addCompleteArray ($out);
249
-       }
249
+        }
250 250
 
251 251
         $out ["containsBook"] = 0;
252 252
         if ($currentPage->containsBook ()) {
Please login to merge, or discard this patch.
Spacing   +99 added lines, -99 removed lines patch added patch discarded remove patch
@@ -14,53 +14,53 @@  discard block
 block discarded – undo
14 14
      * @param Book $book
15 15
      * @return array
16 16
      */
17
-    public static function getBookContentArray ($book) {
17
+    public static function getBookContentArray($book) {
18 18
         global $config;
19 19
         $i = 0;
20
-        $preferedData = array ();
20
+        $preferedData = array();
21 21
         foreach ($config['cops_prefered_format'] as $format)
22 22
         {
23 23
             if ($i == 2) { break; }
24
-            if ($data = $book->getDataFormat ($format)) {
24
+            if ($data = $book->getDataFormat($format)) {
25 25
                 $i++;
26
-                array_push ($preferedData, array ("url" => $data->getHtmlLink (),
27
-                  "viewUrl" => $data->getViewHtmlLink (), "name" => $format));
26
+                array_push($preferedData, array("url" => $data->getHtmlLink(),
27
+                  "viewUrl" => $data->getViewHtmlLink(), "name" => $format));
28 28
             }
29 29
         }
30 30
 
31 31
         $publisher = $book->getPublisher();
32
-        if (is_null ($publisher)) {
32
+        if (is_null($publisher)) {
33 33
             $pn = "";
34 34
             $pu = "";
35 35
         } else {
36 36
             $pn = $publisher->name;
37
-            $link = new LinkNavigation ($publisher->getUri ());
38
-            $pu = $link->hrefXhtml ();
37
+            $link = new LinkNavigation($publisher->getUri());
38
+            $pu = $link->hrefXhtml();
39 39
         }
40 40
 
41
-        $serie = $book->getSerie ();
42
-        if (is_null ($serie)) {
41
+        $serie = $book->getSerie();
42
+        if (is_null($serie)) {
43 43
             $sn = "";
44 44
             $scn = "";
45 45
             $su = "";
46 46
         } else {
47 47
             $sn = $serie->name;
48
-            $scn = str_format (localize ("content.series.data"), $book->seriesIndex, $serie->name);
49
-            $link = new LinkNavigation ($serie->getUri ());
50
-            $su = $link->hrefXhtml ();
48
+            $scn = str_format(localize("content.series.data"), $book->seriesIndex, $serie->name);
49
+            $link = new LinkNavigation($serie->getUri());
50
+            $su = $link->hrefXhtml();
51 51
         }
52 52
         $cc = $book->getCustomColumnValues($config['cops_calibre_custom_column_list'], true);
53 53
 
54
-        return array ("id" => $book->id,
54
+        return array("id" => $book->id,
55 55
                       "hasCover" => $book->hasCover,
56 56
                       "preferedData" => $preferedData,
57
-                      "rating" => $book->getRating (),
57
+                      "rating" => $book->getRating(),
58 58
                       "publisherName" => $pn,
59 59
                       "publisherurl" => $pu,
60
-                      "pubDate" => $book->getPubDate (),
61
-                      "languagesName" => $book->getLanguages (),
62
-                      "authorsName" => $book->getAuthorsName (),
63
-                      "tagsName" => $book->getTagsName (),
60
+                      "pubDate" => $book->getPubDate(),
61
+                      "languagesName" => $book->getLanguages(),
62
+                      "authorsName" => $book->getAuthorsName(),
63
+                      "tagsName" => $book->getTagsName(),
64 64
                       "seriesName" => $sn,
65 65
                       "seriesIndex" => $book->seriesIndex,
66 66
                       "seriesCompleteName" => $scn,
@@ -72,45 +72,45 @@  discard block
 block discarded – undo
72 72
      * @param Book $book
73 73
      * @return array
74 74
      */
75
-    public static function getFullBookContentArray ($book) {
75
+    public static function getFullBookContentArray($book) {
76 76
         global $config;
77
-        $out = self::getBookContentArray ($book);
78
-        $database = GetUrlParam (DB);
77
+        $out = self::getBookContentArray($book);
78
+        $database = GetUrlParam(DB);
79 79
 
80
-        $out ["coverurl"] = Data::getLink ($book, "jpg", "image/jpeg", Link::OPDS_IMAGE_TYPE, "cover.jpg", NULL)->hrefXhtml ();
81
-        $out ["thumbnailurl"] = Data::getLink ($book, "jpg", "image/jpeg", Link::OPDS_THUMBNAIL_TYPE, "cover.jpg", NULL, NULL, $config['cops_html_thumbnail_height'] * 2)->hrefXhtml ();
82
-        $out ["content"] = $book->getComment (false);
83
-        $out ["datas"] = array ();
84
-        $dataKindle = $book->GetMostInterestingDataToSendToKindle ();
80
+        $out ["coverurl"] = Data::getLink($book, "jpg", "image/jpeg", Link::OPDS_IMAGE_TYPE, "cover.jpg", NULL)->hrefXhtml();
81
+        $out ["thumbnailurl"] = Data::getLink($book, "jpg", "image/jpeg", Link::OPDS_THUMBNAIL_TYPE, "cover.jpg", NULL, NULL, $config['cops_html_thumbnail_height'] * 2)->hrefXhtml();
82
+        $out ["content"] = $book->getComment(false);
83
+        $out ["datas"] = array();
84
+        $dataKindle = $book->GetMostInterestingDataToSendToKindle();
85 85
         foreach ($book->getDatas() as $data) {
86
-            $tab = array ("id" => $data->id,
86
+            $tab = array("id" => $data->id,
87 87
                 "format" => $data->format, 
88
-                "url" => $data->getHtmlLink (),
89
-                "viewUrl" => $data->getViewHtmlLink (),
88
+                "url" => $data->getHtmlLink(),
89
+                "viewUrl" => $data->getViewHtmlLink(),
90 90
                 "mail" => 0, 
91 91
                 "readerUrl" => "");
92
-            if (!empty ($config['cops_mail_configuration']) && !is_null ($dataKindle) && $data->id == $dataKindle->id) {
92
+            if (!empty ($config['cops_mail_configuration']) && !is_null($dataKindle) && $data->id == $dataKindle->id) {
93 93
                 $tab ["mail"] = 1;
94 94
             }
95 95
             if ($data->format == "EPUB") {
96 96
                 $tab ["readerUrl"] = "epubreader.php?data={$data->id}&db={$database}";
97 97
             }
98
-            array_push ($out ["datas"], $tab);
98
+            array_push($out ["datas"], $tab);
99 99
         }
100
-        $out ["authors"] = array ();
101
-        foreach ($book->getAuthors () as $author) {
102
-            $link = new LinkNavigation ($author->getUri ());
103
-            array_push ($out ["authors"], array ("name" => $author->name, "url" => $link->hrefXhtml ()));
100
+        $out ["authors"] = array();
101
+        foreach ($book->getAuthors() as $author) {
102
+            $link = new LinkNavigation($author->getUri());
103
+            array_push($out ["authors"], array("name" => $author->name, "url" => $link->hrefXhtml()));
104 104
         }
105
-        $out ["tags"] = array ();
106
-        foreach ($book->getTags () as $tag) {
107
-            $link = new LinkNavigation ($tag->getUri ());
108
-            array_push ($out ["tags"], array ("name" => $tag->name, "url" => $link->hrefXhtml ()));
105
+        $out ["tags"] = array();
106
+        foreach ($book->getTags() as $tag) {
107
+            $link = new LinkNavigation($tag->getUri());
108
+            array_push($out ["tags"], array("name" => $tag->name, "url" => $link->hrefXhtml()));
109 109
         }
110 110
 
111
-        $out ["identifiers"] = array ();
112
-        foreach ($book->getIdentifiers () as $ident) {
113
-            array_push ($out ["identifiers"], array ("name" => $ident->formattedType, "url" => $ident->getUri ()));
111
+        $out ["identifiers"] = array();
112
+        foreach ($book->getIdentifiers() as $ident) {
113
+            array_push($out ["identifiers"], array("name" => $ident->formattedType, "url" => $ident->getUri()));
114 114
         }
115 115
 
116 116
         $out ["customcolumns_preview"] = $book->getCustomColumnValues($config['cops_calibre_custom_column_preview'], true);
@@ -118,51 +118,51 @@  discard block
 block discarded – undo
118 118
         return $out;
119 119
     }
120 120
 
121
-    public static function getContentArray ($entry) {
121
+    public static function getContentArray($entry) {
122 122
         if ($entry instanceof EntryBook) {
123
-            $out = array ( "title" => $entry->title);
124
-            $out ["book"] = self::getBookContentArray ($entry->book);
123
+            $out = array("title" => $entry->title);
124
+            $out ["book"] = self::getBookContentArray($entry->book);
125 125
             return $out;
126 126
         }
127
-        return array ( "title" => $entry->title, "content" => $entry->content, "navlink" => $entry->getNavLink (), "number" => $entry->numberOfElement );
127
+        return array("title" => $entry->title, "content" => $entry->content, "navlink" => $entry->getNavLink(), "number" => $entry->numberOfElement);
128 128
     }
129 129
 
130
-    public static function getContentArrayTypeahead ($page) {
131
-        $out = array ();
130
+    public static function getContentArrayTypeahead($page) {
131
+        $out = array();
132 132
         foreach ($page->entryArray as $entry) {
133 133
             if ($entry instanceof EntryBook) {
134
-                array_push ($out, array ("class" => $entry->className, "title" => $entry->title, "navlink" => $entry->book->getDetailUrl ()));
134
+                array_push($out, array("class" => $entry->className, "title" => $entry->title, "navlink" => $entry->book->getDetailUrl()));
135 135
             } else {
136
-                if (empty ($entry->className) xor Base::noDatabaseSelected ()) {
137
-                    array_push ($out, array ("class" => $entry->className, "title" => $entry->title, "navlink" => $entry->getNavLink ()));
136
+                if (empty ($entry->className) xor Base::noDatabaseSelected()) {
137
+                    array_push($out, array("class" => $entry->className, "title" => $entry->title, "navlink" => $entry->getNavLink()));
138 138
                 } else {
139
-                    array_push ($out, array ("class" => $entry->className, "title" => $entry->content, "navlink" => $entry->getNavLink ()));
139
+                    array_push($out, array("class" => $entry->className, "title" => $entry->content, "navlink" => $entry->getNavLink()));
140 140
                 }
141 141
             }
142 142
         }
143 143
         return $out;
144 144
     }
145 145
 
146
-    public static function addCompleteArray ($in) {
146
+    public static function addCompleteArray($in) {
147 147
         global $config;
148 148
         $out = $in;
149 149
 
150
-        $out ["c"] = array ("version" => VERSION, "i18n" => array (
150
+        $out ["c"] = array("version" => VERSION, "i18n" => array(
151 151
                            "coverAlt" => localize("i18n.coversection"),
152 152
                            "authorsTitle" => localize("authors.title"),
153 153
                            "bookwordTitle" => localize("bookword.title"),
154 154
                            "tagsTitle" => localize("tags.title"),
155 155
                            "linksTitle" => localize("links.title"),
156 156
                            "seriesTitle" => localize("series.title"),
157
-                           "customizeTitle" => localize ("customize.title"),
158
-                           "aboutTitle" => localize ("about.title"),
159
-                           "previousAlt" => localize ("paging.previous.alternate"),
160
-                           "nextAlt" => localize ("paging.next.alternate"),
161
-                           "searchAlt" => localize ("search.alternate"),
162
-                           "sortAlt" => localize ("sort.alternate"),
163
-                           "homeAlt" => localize ("home.alternate"),
164
-                           "cogAlt" => localize ("cog.alternate"),
165
-                           "permalinkAlt" => localize ("permalink.alternate"),
157
+                           "customizeTitle" => localize("customize.title"),
158
+                           "aboutTitle" => localize("about.title"),
159
+                           "previousAlt" => localize("paging.previous.alternate"),
160
+                           "nextAlt" => localize("paging.next.alternate"),
161
+                           "searchAlt" => localize("search.alternate"),
162
+                           "sortAlt" => localize("sort.alternate"),
163
+                           "homeAlt" => localize("home.alternate"),
164
+                           "cogAlt" => localize("cog.alternate"),
165
+                           "permalinkAlt" => localize("permalink.alternate"),
166 166
                            "publisherName" => localize("publisher.name"),
167 167
                            "pubdateTitle" => localize("pubdate.title"),
168 168
                            "languagesTitle" => localize("language.title"),
@@ -171,15 +171,15 @@  discard block
 block discarded – undo
171 171
                            "sortorderAsc" => localize("search.sortorder.asc"),
172 172
                            "sortorderDesc" => localize("search.sortorder.desc"),
173 173
                            "customizeEmail" => localize("customize.email")),
174
-                       "url" => array (
174
+                       "url" => array(
175 175
                            "detailUrl" => "index.php?page=13&id={0}&db={1}",
176 176
                            "coverUrl" => "fetch.php?id={0}&db={1}",
177 177
                            "thumbnailUrl" => "fetch.php?height=" . $config['cops_html_thumbnail_height'] . "&id={0}&db={1}"),
178
-                       "config" => array (
178
+                       "config" => array(
179 179
                            "use_fancyapps" => $config ["cops_use_fancyapps"],
180 180
                            "max_item_per_page" => $config['cops_max_item_per_page'],
181 181
                            "kindleHack"        => "",
182
-                           "server_side_rendering" => useServerSideRendering (),
182
+                           "server_side_rendering" => useServerSideRendering(),
183 183
                            "html_tag_filter" => $config['cops_html_tag_filter']));
184 184
         if ($config['cops_thumbnail_handling'] == "1") {
185 185
             $out ["c"]["url"]["thumbnailUrl"] = $out ["c"]["url"]["coverUrl"];
@@ -192,32 +192,32 @@  discard block
 block discarded – undo
192 192
         return $out;
193 193
     }
194 194
 
195
-    public static function getJson ($complete = false) {
195
+    public static function getJson($complete = false) {
196 196
         global $config;
197
-        $page = getURLParam ("page", Base::PAGE_INDEX);
198
-        $query = getURLParam ("query");
199
-        $search = getURLParam ("search");
200
-        $qid = getURLParam ("id");
201
-        $n = getURLParam ("n", "1");
202
-        $database = GetUrlParam (DB);
197
+        $page = getURLParam("page", Base::PAGE_INDEX);
198
+        $query = getURLParam("query");
199
+        $search = getURLParam("search");
200
+        $qid = getURLParam("id");
201
+        $n = getURLParam("n", "1");
202
+        $database = GetUrlParam(DB);
203 203
 
204
-        $currentPage = Page::getPage ($page, $qid, $query, $n);
205
-        $currentPage->InitializeContent ();
204
+        $currentPage = Page::getPage($page, $qid, $query, $n);
205
+        $currentPage->InitializeContent();
206 206
 
207 207
         if ($search) {
208
-            return self::getContentArrayTypeahead ($currentPage);
208
+            return self::getContentArrayTypeahead($currentPage);
209 209
         }
210 210
 
211
-        $out = array ( "title" => $currentPage->title);
212
-        $entries = array ();
211
+        $out = array("title" => $currentPage->title);
212
+        $entries = array();
213 213
         foreach ($currentPage->entryArray as $entry) {
214
-            array_push ($entries, self::getContentArray ($entry));
214
+            array_push($entries, self::getContentArray($entry));
215 215
         }
216
-        if (!is_null ($currentPage->book)) {
217
-            $out ["book"] = self::getFullBookContentArray ($currentPage->book);
216
+        if (!is_null($currentPage->book)) {
217
+            $out ["book"] = self::getFullBookContentArray($currentPage->book);
218 218
         }
219
-        $out ["databaseId"] = GetUrlParam (DB, "");
220
-        $out ["databaseName"] = Base::getDbName ();
219
+        $out ["databaseId"] = GetUrlParam(DB, "");
220
+        $out ["databaseName"] = Base::getDbName();
221 221
         if ($out ["databaseId"] == "") {
222 222
             $out ["databaseName"] = "";
223 223
         }
@@ -226,42 +226,42 @@  discard block
 block discarded – undo
226 226
             $out ["fullTitle"] = $out ["databaseName"] . " > " . $out ["fullTitle"];
227 227
         }
228 228
         $out ["page"] = $page;
229
-        $out ["multipleDatabase"] = Base::isMultipleDatabaseEnabled () ? 1 : 0;
229
+        $out ["multipleDatabase"] = Base::isMultipleDatabaseEnabled()?1:0;
230 230
         $out ["entries"] = $entries;
231 231
         $out ["isPaginated"] = 0;
232
-        if ($currentPage->isPaginated ()) {
233
-            $prevLink = $currentPage->getPrevLink ();
234
-            $nextLink = $currentPage->getNextLink ();
232
+        if ($currentPage->isPaginated()) {
233
+            $prevLink = $currentPage->getPrevLink();
234
+            $nextLink = $currentPage->getNextLink();
235 235
             $out ["isPaginated"] = 1;
236 236
             $out ["prevLink"] = "";
237
-            if (!is_null ($prevLink)) {
238
-                $out ["prevLink"] = $prevLink->hrefXhtml ();
237
+            if (!is_null($prevLink)) {
238
+                $out ["prevLink"] = $prevLink->hrefXhtml();
239 239
             }
240 240
             $out ["nextLink"] = "";
241
-            if (!is_null ($nextLink)) {
242
-                $out ["nextLink"] = $nextLink->hrefXhtml ();
241
+            if (!is_null($nextLink)) {
242
+                $out ["nextLink"] = $nextLink->hrefXhtml();
243 243
             }
244
-            $out ["maxPage"] = $currentPage->getMaxPage ();
244
+            $out ["maxPage"] = $currentPage->getMaxPage();
245 245
             $out ["currentPage"] = $currentPage->n;
246 246
         }
247
-        if (!is_null (getURLParam ("complete")) || $complete) {
248
-            $out = self::addCompleteArray ($out);
247
+        if (!is_null(getURLParam("complete")) || $complete) {
248
+            $out = self::addCompleteArray($out);
249 249
        }
250 250
 
251 251
         $out ["containsBook"] = 0;
252
-        if ($currentPage->containsBook ()) {
252
+        if ($currentPage->containsBook()) {
253 253
             $out ["containsBook"] = 1;
254 254
         }
255 255
 
256
-        $out["abouturl"] = "index.php" . addURLParameter ("?page=" . Base::PAGE_ABOUT, DB, $database);
256
+        $out["abouturl"] = "index.php" . addURLParameter("?page=" . Base::PAGE_ABOUT, DB, $database);
257 257
 
258 258
         if ($page == Base::PAGE_ABOUT) {
259
-            $temp = preg_replace ("/\<h1\>About COPS\<\/h1\>/", "<h1>About COPS " . VERSION . "</h1>", file_get_contents('about.html'));
259
+            $temp = preg_replace("/\<h1\>About COPS\<\/h1\>/", "<h1>About COPS " . VERSION . "</h1>", file_get_contents('about.html'));
260 260
             $out ["fullhtml"] = $temp;
261 261
         }
262 262
 
263 263
         $out ["homeurl"] = "index.php";
264
-        if ($page != Base::PAGE_INDEX && !is_null ($database)) $out ["homeurl"] = $out ["homeurl"] .  "?" . addURLParameter ("", DB, $database);
264
+        if ($page != Base::PAGE_INDEX && !is_null($database)) $out ["homeurl"] = $out ["homeurl"] . "?" . addURLParameter("", DB, $database);
265 265
 
266 266
         return $out;
267 267
     }
Please login to merge, or discard this patch.
lib/Book.php 3 patches
Upper-Lower-Casing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -101,13 +101,13 @@  discard block
 block discarded – undo
101 101
     public $seriesIndex;
102 102
     public $comment;
103 103
     public $rating;
104
-    public $datas = NULL;
105
-    public $authors = NULL;
106
-    public $publisher = NULL;
107
-    public $serie = NULL;
108
-    public $tags = NULL;
109
-    public $identifiers = NULL;
110
-    public $languages = NULL;
104
+    public $datas = null;
105
+    public $authors = null;
106
+    public $publisher = null;
107
+    public $serie = null;
108
+    public $tags = null;
109
+    public $identifiers = null;
110
+    public $languages = null;
111 111
     public $format = array ();
112 112
 
113 113
 
@@ -267,7 +267,7 @@  discard block
 block discarded – undo
267 267
     /* End of other class (author, series, tag, ...) initialization and accessors */
268 268
 
269 269
     public static function getFilterString() {
270
-        $filter = getURLParam('tag', NULL);
270
+        $filter = getURLParam('tag', null);
271 271
         if (empty($filter)) return '';
272 272
 
273 273
         $exists = true;
@@ -289,7 +289,7 @@  discard block
 block discarded – undo
289 289
     {
290 290
         $bestFormatForKindle = array('EPUB', 'PDF', 'AZW3', 'MOBI');
291 291
         $bestRank = -1;
292
-        $bestData = NULL;
292
+        $bestData = null;
293 293
         foreach ($this->getDatas() as $data) {
294 294
             $key = array_search($data->format, $bestFormatForKindle);
295 295
             if ($key !== false && $key > $bestRank) {
@@ -356,7 +356,7 @@  discard block
 block discarded – undo
356 356
         return reset($reduced);
357 357
     }
358 358
 
359
-    public function getFilePath($extension, $idData = NULL, $relative = false)
359
+    public function getFilePath($extension, $idData = null, $relative = false)
360 360
     {
361 361
         if ($extension == 'jpg')
362 362
         {
@@ -365,7 +365,7 @@  discard block
 block discarded – undo
365 365
         else
366 366
         {
367 367
             $data = $this->getDataById($idData);
368
-            if (!$data) return NULL;
368
+            if (!$data) return null;
369 369
             $file = $data->name . '.' . strtolower($data->format);
370 370
         }
371 371
 
@@ -417,7 +417,7 @@  discard block
 block discarded – undo
417 417
         }
418 418
     }
419 419
 
420
-    public function getThumbnail($width, $height, $outputfile = NULL) {
420
+    public function getThumbnail($width, $height, $outputfile = null) {
421 421
         if (is_null($width) && is_null($height)) {
422 422
             return false;
423 423
         }
@@ -458,9 +458,9 @@  discard block
 block discarded – undo
458 458
 
459 459
         if ($this->hasCover)
460 460
         {
461
-            array_push($linkArray, Data::getLink($this, 'jpg', 'image/jpeg', Link::OPDS_IMAGE_TYPE, 'cover.jpg', NULL));
461
+            array_push($linkArray, Data::getLink($this, 'jpg', 'image/jpeg', Link::OPDS_IMAGE_TYPE, 'cover.jpg', null));
462 462
 
463
-            array_push($linkArray, Data::getLink($this, 'jpg', 'image/jpeg', Link::OPDS_THUMBNAIL_TYPE, 'cover.jpg', NULL));
463
+            array_push($linkArray, Data::getLink($this, 'jpg', 'image/jpeg', Link::OPDS_THUMBNAIL_TYPE, 'cover.jpg', null));
464 464
         }
465 465
 
466 466
         foreach ($this->getDatas() as $data)
@@ -491,7 +491,7 @@  discard block
 block discarded – undo
491 491
             $this->getLinkArray(), $this);
492 492
     }
493 493
 
494
-    public static function getBookCount($database = NULL) {
494
+    public static function getBookCount($database = null) {
495 495
         return parent::executeQuerySingle('select count(*) from books', $database);
496 496
     }
497 497
 
@@ -560,7 +560,7 @@  discard block
 block discarded – undo
560 560
             $book = new Book($post);
561 561
             return $book;
562 562
         }
563
-        return NULL;
563
+        return null;
564 564
     }
565 565
 
566 566
     public static function getBookByDataId($dataId) {
@@ -576,10 +576,10 @@  discard block
 block discarded – undo
576 576
             $book->datas = array($data);
577 577
             return $book;
578 578
         }
579
-        return NULL;
579
+        return null;
580 580
     }
581 581
 
582
-    public static function getBooksByQuery($query, $n, $database = NULL, $numberPerPage = NULL) {
582
+    public static function getBooksByQuery($query, $n, $database = null, $numberPerPage = null) {
583 583
         $i = 0;
584 584
         $critArray = array();
585 585
         foreach (array(PageQueryResult::SCOPE_AUTHOR,
@@ -626,11 +626,11 @@  discard block
 block discarded – undo
626 626
         return $entryArray;
627 627
     }
628 628
 
629
-    public static function getBooksByStartingLetter($letter, $n, $database = NULL, $numberPerPage = NULL) {
629
+    public static function getBooksByStartingLetter($letter, $n, $database = null, $numberPerPage = null) {
630 630
         return self::getEntryArray(self::SQL_BOOKS_BY_FIRST_LETTER, array($letter . '%'), $n, $database, $numberPerPage);
631 631
     }
632 632
 
633
-    public static function getEntryArray($query, $params, $n, $database = NULL, $numberPerPage = NULL) {
633
+    public static function getEntryArray($query, $params, $n, $database = null, $numberPerPage = null) {
634 634
         /* @var $totalNumber integer */
635 635
         /* @var $result PDOStatement */
636 636
         list($totalNumber, $result) = parent::executeQuery($query, self::BOOK_COLUMNS, self::getFilterString(), $params, $n, $database, $numberPerPage);
Please login to merge, or discard this patch.
Spacing   +59 added lines, -59 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
 require('Identifier.php');
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
     public $tags = NULL;
109 109
     public $identifiers = NULL;
110 110
     public $languages = NULL;
111
-    public $format = array ();
111
+    public $format = array();
112 112
 
113 113
 
114 114
     public function __construct($line) {
@@ -130,24 +130,24 @@  discard block
 block discarded – undo
130 130
     }
131 131
 
132 132
     public function getEntryId() {
133
-        return self::ALL_BOOKS_UUID.':'.$this->uuid;
133
+        return self::ALL_BOOKS_UUID . ':' . $this->uuid;
134 134
     }
135 135
 
136
-    public static function getEntryIdByLetter ($startingLetter) {
137
-        return self::ALL_BOOKS_ID.':letter:'.$startingLetter;
136
+    public static function getEntryIdByLetter($startingLetter) {
137
+        return self::ALL_BOOKS_ID . ':letter:' . $startingLetter;
138 138
     }
139 139
 
140
-    public function getUri () {
141
-        return '?page='.parent::PAGE_BOOK_DETAIL.'&id=' . $this->id;
140
+    public function getUri() {
141
+        return '?page=' . parent::PAGE_BOOK_DETAIL . '&id=' . $this->id;
142 142
     }
143 143
 
144
-    public function getDetailUrl () {
144
+    public function getDetailUrl() {
145 145
         $urlParam = $this->getUri();
146
-        if (!is_null(GetUrlParam(DB))) $urlParam = addURLParameter($urlParam, DB, GetUrlParam (DB));
146
+        if (!is_null(GetUrlParam(DB))) $urlParam = addURLParameter($urlParam, DB, GetUrlParam(DB));
147 147
         return 'index.php' . $urlParam;
148 148
     }
149 149
 
150
-    public function getTitle () {
150
+    public function getTitle() {
151 151
         return $this->title;
152 152
     }
153 153
 
@@ -156,22 +156,22 @@  discard block
 block discarded – undo
156 156
     /**
157 157
      * @return Author[]
158 158
      */
159
-    public function getAuthors () {
159
+    public function getAuthors() {
160 160
         if (is_null($this->authors)) {
161 161
             $this->authors = Author::getAuthorByBookId($this->id);
162 162
         }
163 163
         return $this->authors;
164 164
     }
165 165
 
166
-    public function getAuthorsName () {
167
-        return implode(', ', array_map(function ($author) { return $author->name; }, $this->getAuthors()));
166
+    public function getAuthorsName() {
167
+        return implode(', ', array_map(function($author) { return $author->name; }, $this->getAuthors()));
168 168
     }
169 169
 
170
-    public function getAuthorsSort () {
171
-        return implode(', ', array_map(function ($author) { return $author->sort; }, $this->getAuthors()));
170
+    public function getAuthorsSort() {
171
+        return implode(', ', array_map(function($author) { return $author->sort; }, $this->getAuthors()));
172 172
     }
173 173
 
174
-    public function getPublisher () {
174
+    public function getPublisher() {
175 175
         if (is_null($this->publisher)) {
176 176
             $this->publisher = Publisher::getPublisherByBookId($this->id);
177 177
         }
@@ -210,7 +210,7 @@  discard block
 block discarded – undo
210 210
      * @return Tag[]
211 211
      */
212 212
     public function getTags() {
213
-        if (is_null ($this->tags)) {
213
+        if (is_null($this->tags)) {
214 214
             $this->tags = array();
215 215
 
216 216
             $result = parent::getDb()->prepare('select tags.id as id, name
@@ -228,7 +228,7 @@  discard block
 block discarded – undo
228 228
     }
229 229
 
230 230
     public function getTagsName() {
231
-        return implode(', ', array_map(function ($tag) { return $tag->name; }, $this->getTags()));
231
+        return implode(', ', array_map(function($tag) { return $tag->name; }, $this->getTags()));
232 232
     }
233 233
 
234 234
 
@@ -236,7 +236,7 @@  discard block
 block discarded – undo
236 236
      * @return Identifiers[]
237 237
      */
238 238
     public function getIdentifiers() {
239
-        if (is_null ($this->identifiers)) {
239
+        if (is_null($this->identifiers)) {
240 240
             $this->identifiers = array();
241 241
 
242 242
             $result = parent::getDb()->prepare('select type, val, id
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
 
303 303
     public function getDataById($idData)
304 304
     {
305
-        $reduced = array_filter($this->getDatas(), function ($data) use ($idData) {
305
+        $reduced = array_filter($this->getDatas(), function($data) use ($idData) {
306 306
             return $data->id == $idData;
307 307
         });
308 308
         return reset($reduced);
@@ -326,7 +326,7 @@  discard block
 block discarded – undo
326 326
         if (empty ($this->pubdate)) {
327 327
             return '';
328 328
         }
329
-        $dateY = (int) substr($this->pubdate, 0, 4);
329
+        $dateY = (int)substr($this->pubdate, 0, 4);
330 330
         if ($dateY > 102) {
331 331
             return str_pad($dateY, 4, '0', STR_PAD_LEFT);
332 332
         }
@@ -335,15 +335,15 @@  discard block
 block discarded – undo
335 335
 
336 336
     public function getComment($withSerie = true) {
337 337
         $addition = '';
338
-        $se = $this->getSerie ();
339
-        if (!is_null ($se) && $withSerie) {
338
+        $se = $this->getSerie();
339
+        if (!is_null($se) && $withSerie) {
340 340
             $addition = $addition . '<strong>' . localize('content.series') . '</strong>' . str_format(localize('content.series.data'), $this->seriesIndex, htmlspecialchars($se->name)) . "<br />\n";
341 341
         }
342 342
         return $addition . html2xhtml($this->comment);
343 343
     }
344 344
 
345 345
     public function getDataFormat($format) {
346
-        $reduced = array_filter($this->getDatas(), function ($data) use ($format) {
346
+        $reduced = array_filter($this->getDatas(), function($data) use ($format) {
347 347
             return $data->format == $format;
348 348
         });
349 349
         return reset($reduced);
@@ -364,11 +364,11 @@  discard block
 block discarded – undo
364 364
 
365 365
         if ($relative)
366 366
         {
367
-            return $this->relativePath.'/'.$file;
367
+            return $this->relativePath . '/' . $file;
368 368
         }
369 369
         else
370 370
         {
371
-            return $this->path.'/'.$file;
371
+            return $this->path . '/' . $file;
372 372
         }
373 373
     }
374 374
 
@@ -388,7 +388,7 @@  discard block
 block discarded – undo
388 388
             }
389 389
             $epub->Authors($authorArray);
390 390
             $epub->Language($this->getLanguages());
391
-            $epub->Description ($this->getComment(false));
391
+            $epub->Description($this->getComment(false));
392 392
             $epub->Subjects($this->getTagsName());
393 393
             $epub->Cover2($this->getFilePath('jpg'), 'image/jpeg');
394 394
             $epub->Calibre($this->uuid);
@@ -398,7 +398,7 @@  discard block
 block discarded – undo
398 398
                 $epub->SerieIndex($this->seriesIndex);
399 399
             }
400 400
             $filename = $data->getUpdatedFilenameEpub();
401
-            if ($config['cops_provide_kepub'] == '1'  && preg_match('/Kobo/', $_SERVER['HTTP_USER_AGENT'])) {
401
+            if ($config['cops_provide_kepub'] == '1' && preg_match('/Kobo/', $_SERVER['HTTP_USER_AGENT'])) {
402 402
                 $epub->updateForKepub();
403 403
                 $filename = $data->getUpdatedFilenameKepub();
404 404
             }
@@ -424,11 +424,11 @@  discard block
 block discarded – undo
424 424
             if (!is_null($width)) {
425 425
                 $nw = $width;
426 426
                 if ($nw >= $w) { return false; }
427
-                $nh = ($nw*$h)/$w;
427
+                $nh = ($nw * $h) / $w;
428 428
             } else {
429 429
                 $nh = $height;
430 430
                 if ($nh >= $h) { return false; }
431
-                $nw = ($nh*$w)/$h;
431
+                $nw = ($nh * $w) / $h;
432 432
             }
433 433
         } else {
434 434
             return false;
@@ -436,16 +436,16 @@  discard block
 block discarded – undo
436 436
 
437 437
         //draw the image
438 438
         $src_img = imagecreatefromjpeg($file);
439
-        $dst_img = imagecreatetruecolor($nw,$nh);
440
-        imagecopyresampled($dst_img, $src_img, 0, 0, 0, 0, $nw, $nh, $w, $h);//resizing the image
441
-        imagejpeg($dst_img,$outputfile,80);
439
+        $dst_img = imagecreatetruecolor($nw, $nh);
440
+        imagecopyresampled($dst_img, $src_img, 0, 0, 0, 0, $nw, $nh, $w, $h); //resizing the image
441
+        imagejpeg($dst_img, $outputfile, 80);
442 442
         imagedestroy($src_img);
443 443
         imagedestroy($dst_img);
444 444
 
445 445
         return true;
446 446
     }
447 447
 
448
-    public function getLinkArray ()
448
+    public function getLinkArray()
449 449
     {
450 450
         $linkArray = array();
451 451
 
@@ -470,7 +470,7 @@  discard block
 block discarded – undo
470 470
         }
471 471
 
472 472
         $serie = $this->getSerie();
473
-        if (!is_null ($serie)) {
473
+        if (!is_null($serie)) {
474 474
             array_push($linkArray, new LinkNavigation($serie->getUri(), 'related', str_format(localize('content.series.data'), $this->seriesIndex, $serie->name)));
475 475
         }
476 476
 
@@ -495,13 +495,13 @@  discard block
 block discarded – undo
495 495
         $entry = new Entry(localize('allbooks.title'),
496 496
                           self::ALL_BOOKS_ID,
497 497
                           str_format(localize('allbooks.alphabetical', $nBooks), $nBooks), 'text',
498
-                          array(new LinkNavigation('?page='.parent::PAGE_ALL_BOOKS)), '', $nBooks);
498
+                          array(new LinkNavigation('?page=' . parent::PAGE_ALL_BOOKS)), '', $nBooks);
499 499
         array_push($result, $entry);
500 500
         if ($config['cops_recentbooks_limit'] > 0) {
501 501
             $entry = new Entry(localize('recent.title'),
502 502
                               self::ALL_RECENT_BOOKS_ID,
503 503
                               str_format(localize('recent.list'), $config['cops_recentbooks_limit']), 'text',
504
-                              array ( new LinkNavigation ('?page='.parent::PAGE_ALL_RECENT_BOOKS)), '', $config['cops_recentbooks_limit']);
504
+                              array(new LinkNavigation('?page=' . parent::PAGE_ALL_RECENT_BOOKS)), '', $config['cops_recentbooks_limit']);
505 505
             array_push($result, $entry);
506 506
         }
507 507
         return $result;
@@ -597,7 +597,7 @@  discard block
 block discarded – undo
597 597
     }
598 598
 
599 599
     public static function getBooks($n) {
600
-        list ($entryArray, $totalNumber) = self::getEntryArray(self::SQL_BOOKS_ALL , array (), $n);
600
+        list ($entryArray, $totalNumber) = self::getEntryArray(self::SQL_BOOKS_ALL, array(), $n);
601 601
         return array($entryArray, $totalNumber);
602 602
     }
603 603
 
@@ -614,7 +614,7 @@  discard block
 block discarded – undo
614 614
         {
615 615
             array_push($entryArray, new Entry($post->title, Book::getEntryIdByLetter($post->title),
616 616
                 str_format(localize('bookword', $post->count), $post->count), 'text',
617
-                array(new LinkNavigation('?page='.parent::PAGE_ALL_BOOKS_LETTER.'&id='. rawurlencode($post->title))), '', $post->count));
617
+                array(new LinkNavigation('?page=' . parent::PAGE_ALL_BOOKS_LETTER . '&id=' . rawurlencode($post->title))), '', $post->count));
618 618
         }
619 619
         return $entryArray;
620 620
     }
@@ -639,7 +639,7 @@  discard block
 block discarded – undo
639 639
 
640 640
     public static function getAllRecentBooks() {
641 641
         global $config;
642
-        list ($entryArray, ) = self::getEntryArray(self::SQL_BOOKS_RECENT . $config['cops_recentbooks_limit'], array(), -1);
642
+        list ($entryArray,) = self::getEntryArray(self::SQL_BOOKS_RECENT . $config['cops_recentbooks_limit'], array(), -1);
643 643
         return $entryArray;
644 644
     }
645 645
 
Please login to merge, or discard this patch.
Braces   +106 added lines, -80 removed lines patch added patch discarded remove patch
@@ -111,7 +111,8 @@  discard block
 block discarded – undo
111 111
     public $format = array ();
112 112
 
113 113
 
114
-    public function __construct($line) {
114
+    public function __construct($line)
115
+    {
115 116
         $this->id = $line->id;
116 117
         $this->title = $line->title;
117 118
         $this->timestamp = strtotime($line->timestamp);
@@ -129,25 +130,32 @@  discard block
 block discarded – undo
129 130
         $this->rating = $line->rating;
130 131
     }
131 132
 
132
-    public function getEntryId() {
133
+    public function getEntryId()
134
+    {
133 135
         return self::ALL_BOOKS_UUID.':'.$this->uuid;
134 136
     }
135 137
 
136
-    public static function getEntryIdByLetter ($startingLetter) {
138
+    public static function getEntryIdByLetter ($startingLetter)
139
+    {
137 140
         return self::ALL_BOOKS_ID.':letter:'.$startingLetter;
138 141
     }
139 142
 
140
-    public function getUri () {
143
+    public function getUri ()
144
+    {
141 145
         return '?page='.parent::PAGE_BOOK_DETAIL.'&id=' . $this->id;
142 146
     }
143 147
 
144
-    public function getDetailUrl () {
148
+    public function getDetailUrl ()
149
+    {
145 150
         $urlParam = $this->getUri();
146
-        if (!is_null(GetUrlParam(DB))) $urlParam = addURLParameter($urlParam, DB, GetUrlParam (DB));
151
+        if (!is_null(GetUrlParam(DB))) {
152
+            $urlParam = addURLParameter($urlParam, DB, GetUrlParam (DB));
153
+        }
147 154
         return 'index.php' . $urlParam;
148 155
     }
149 156
 
150
-    public function getTitle () {
157
+    public function getTitle ()
158
+    {
151 159
         return $this->title;
152 160
     }
153 161
 
@@ -156,22 +164,26 @@  discard block
 block discarded – undo
156 164
     /**
157 165
      * @return Author[]
158 166
      */
159
-    public function getAuthors () {
167
+    public function getAuthors ()
168
+    {
160 169
         if (is_null($this->authors)) {
161 170
             $this->authors = Author::getAuthorByBookId($this->id);
162 171
         }
163 172
         return $this->authors;
164 173
     }
165 174
 
166
-    public function getAuthorsName () {
175
+    public function getAuthorsName ()
176
+    {
167 177
         return implode(', ', array_map(function ($author) { return $author->name; }, $this->getAuthors()));
168 178
     }
169 179
 
170
-    public function getAuthorsSort () {
180
+    public function getAuthorsSort ()
181
+    {
171 182
         return implode(', ', array_map(function ($author) { return $author->sort; }, $this->getAuthors()));
172 183
     }
173 184
 
174
-    public function getPublisher () {
185
+    public function getPublisher ()
186
+    {
175 187
         if (is_null($this->publisher)) {
176 188
             $this->publisher = Publisher::getPublisherByBookId($this->id);
177 189
         }
@@ -181,7 +193,8 @@  discard block
 block discarded – undo
181 193
     /**
182 194
      * @return Serie
183 195
      */
184
-    public function getSerie() {
196
+    public function getSerie()
197
+    {
185 198
         if (is_null($this->serie)) {
186 199
             $this->serie = Serie::getSerieByBookId($this->id);
187 200
         }
@@ -191,7 +204,8 @@  discard block
 block discarded – undo
191 204
     /**
192 205
      * @return string
193 206
      */
194
-    public function getLanguages() {
207
+    public function getLanguages()
208
+    {
195 209
         $lang = array();
196 210
         $result = parent::getDb()->prepare('select languages.lang_code
197 211
                 from books_languages_link, languages
@@ -199,8 +213,7 @@  discard block
 block discarded – undo
199 213
                 and book = ?
200 214
                 order by item_order');
201 215
         $result->execute(array($this->id));
202
-        while ($post = $result->fetchObject())
203
-        {
216
+        while ($post = $result->fetchObject()) {
204 217
             array_push($lang, Language::getLanguageString($post->lang_code));
205 218
         }
206 219
         return implode(', ', $lang);
@@ -209,7 +222,8 @@  discard block
 block discarded – undo
209 222
     /**
210 223
      * @return Tag[]
211 224
      */
212
-    public function getTags() {
225
+    public function getTags()
226
+    {
213 227
         if (is_null ($this->tags)) {
214 228
             $this->tags = array();
215 229
 
@@ -219,15 +233,15 @@  discard block
 block discarded – undo
219 233
                 and book = ?
220 234
                 order by name');
221 235
             $result->execute(array($this->id));
222
-            while ($post = $result->fetchObject())
223
-            {
236
+            while ($post = $result->fetchObject()) {
224 237
                 array_push($this->tags, new Tag($post));
225 238
             }
226 239
         }
227 240
         return $this->tags;
228 241
     }
229 242
 
230
-    public function getTagsName() {
243
+    public function getTagsName()
244
+    {
231 245
         return implode(', ', array_map(function ($tag) { return $tag->name; }, $this->getTags()));
232 246
     }
233 247
 
@@ -235,7 +249,8 @@  discard block
 block discarded – undo
235 249
     /**
236 250
      * @return Identifiers[]
237 251
      */
238
-    public function getIdentifiers() {
252
+    public function getIdentifiers()
253
+    {
239 254
         if (is_null ($this->identifiers)) {
240 255
             $this->identifiers = array();
241 256
 
@@ -244,8 +259,7 @@  discard block
 block discarded – undo
244 259
                 where book = ?
245 260
                 order by type');
246 261
             $result->execute(array($this->id));
247
-            while ($post = $result->fetchObject())
248
-            {
262
+            while ($post = $result->fetchObject()) {
249 263
                 array_push($this->identifiers, new Identifier($post));
250 264
             }
251 265
             
@@ -266,9 +280,12 @@  discard block
 block discarded – undo
266 280
 
267 281
     /* End of other class (author, series, tag, ...) initialization and accessors */
268 282
 
269
-    public static function getFilterString() {
283
+    public static function getFilterString()
284
+    {
270 285
         $filter = getURLParam('tag', NULL);
271
-        if (empty($filter)) return '';
286
+        if (empty($filter)) {
287
+            return '';
288
+        }
272 289
 
273 290
         $exists = true;
274 291
         if (preg_match("/^!(.*)$/", $filter, $matches)) {
@@ -308,7 +325,8 @@  discard block
 block discarded – undo
308 325
         return reset($reduced);
309 326
     }
310 327
 
311
-    public function getRating() {
328
+    public function getRating()
329
+    {
312 330
         if (is_null($this->rating) || $this->rating == 0) {
313 331
             return '';
314 332
         }
@@ -322,7 +340,8 @@  discard block
 block discarded – undo
322 340
         return $retour;
323 341
     }
324 342
 
325
-    public function getPubDate() {
343
+    public function getPubDate()
344
+    {
326 345
         if (empty ($this->pubdate)) {
327 346
             return '';
328 347
         }
@@ -333,7 +352,8 @@  discard block
 block discarded – undo
333 352
         return '';
334 353
     }
335 354
 
336
-    public function getComment($withSerie = true) {
355
+    public function getComment($withSerie = true)
356
+    {
337 357
         $addition = '';
338 358
         $se = $this->getSerie ();
339 359
         if (!is_null ($se) && $withSerie) {
@@ -342,7 +362,8 @@  discard block
 block discarded – undo
342 362
         return $addition . html2xhtml($this->comment);
343 363
     }
344 364
 
345
-    public function getDataFormat($format) {
365
+    public function getDataFormat($format)
366
+    {
346 367
         $reduced = array_filter($this->getDatas(), function ($data) use ($format) {
347 368
             return $data->format == $format;
348 369
         });
@@ -351,23 +372,19 @@  discard block
 block discarded – undo
351 372
 
352 373
     public function getFilePath($extension, $idData = NULL, $relative = false)
353 374
     {
354
-        if ($extension == 'jpg')
355
-        {
375
+        if ($extension == 'jpg') {
356 376
             $file = 'cover.jpg';
357
-        }
358
-        else
359
-        {
377
+        } else {
360 378
             $data = $this->getDataById($idData);
361
-            if (!$data) return NULL;
379
+            if (!$data) {
380
+                return NULL;
381
+            }
362 382
             $file = $data->name . '.' . strtolower($data->format);
363 383
         }
364 384
 
365
-        if ($relative)
366
-        {
385
+        if ($relative) {
367 386
             return $this->relativePath.'/'.$file;
368
-        }
369
-        else
370
-        {
387
+        } else {
371 388
             return $this->path.'/'.$file;
372 389
         }
373 390
     }
@@ -377,8 +394,7 @@  discard block
 block discarded – undo
377 394
         global $config;
378 395
         $data = $this->getDataById($idData);
379 396
 
380
-        try
381
-        {
397
+        try {
382 398
             $epub = new EPub($data->getLocalPath());
383 399
 
384 400
             $epub->Title($this->title);
@@ -403,14 +419,13 @@  discard block
 block discarded – undo
403 419
                 $filename = $data->getUpdatedFilenameKepub();
404 420
             }
405 421
             $epub->download($filename);
406
-        }
407
-        catch (Exception $e)
408
-        {
422
+        } catch (Exception $e) {
409 423
             echo 'Exception : ' . $e->getMessage();
410 424
         }
411 425
     }
412 426
 
413
-    public function getThumbnail($width, $height, $outputfile = NULL) {
427
+    public function getThumbnail($width, $height, $outputfile = NULL)
428
+    {
414 429
         if (is_null($width) && is_null($height)) {
415 430
             return false;
416 431
         }
@@ -449,17 +464,14 @@  discard block
 block discarded – undo
449 464
     {
450 465
         $linkArray = array();
451 466
 
452
-        if ($this->hasCover)
453
-        {
467
+        if ($this->hasCover) {
454 468
             array_push($linkArray, Data::getLink($this, 'jpg', 'image/jpeg', Link::OPDS_IMAGE_TYPE, 'cover.jpg', NULL));
455 469
 
456 470
             array_push($linkArray, Data::getLink($this, 'jpg', 'image/jpeg', Link::OPDS_THUMBNAIL_TYPE, 'cover.jpg', NULL));
457 471
         }
458 472
 
459
-        foreach ($this->getDatas() as $data)
460
-        {
461
-            if ($data->isKnownType())
462
-            {
473
+        foreach ($this->getDatas() as $data) {
474
+            if ($data->isKnownType()) {
463 475
                 array_push($linkArray, $data->getDataLink(Link::OPDS_ACQUISITION_TYPE, $data->format));
464 476
             }
465 477
         }
@@ -478,17 +490,20 @@  discard block
 block discarded – undo
478 490
     }
479 491
 
480 492
 
481
-    public function getEntry() {
493
+    public function getEntry()
494
+    {
482 495
         return new EntryBook($this->getTitle(), $this->getEntryId(),
483 496
             $this->getComment(), 'text/html',
484 497
             $this->getLinkArray(), $this);
485 498
     }
486 499
 
487
-    public static function getBookCount($database = NULL) {
500
+    public static function getBookCount($database = NULL)
501
+    {
488 502
         return parent::executeQuerySingle('select count(*) from books', $database);
489 503
     }
490 504
 
491
-    public static function getCount() {
505
+    public static function getCount()
506
+    {
492 507
         global $config;
493 508
         $nBooks = parent::executeQuerySingle('select count(*) from books');
494 509
         $result = array();
@@ -507,27 +522,33 @@  discard block
 block discarded – undo
507 522
         return $result;
508 523
     }
509 524
 
510
-    public static function getBooksByAuthor($authorId, $n) {
525
+    public static function getBooksByAuthor($authorId, $n)
526
+    {
511 527
         return self::getEntryArray(self::SQL_BOOKS_BY_AUTHOR, array($authorId), $n);
512 528
     }
513 529
 
514
-    public static function getBooksByRating($ratingId, $n) {
530
+    public static function getBooksByRating($ratingId, $n)
531
+    {
515 532
         return self::getEntryArray(self::SQL_BOOKS_BY_RATING, array($ratingId), $n);
516 533
     }
517 534
 
518
-    public static function getBooksByPublisher($publisherId, $n) {
535
+    public static function getBooksByPublisher($publisherId, $n)
536
+    {
519 537
         return self::getEntryArray(self::SQL_BOOKS_BY_PUBLISHER, array($publisherId), $n);
520 538
     }
521 539
 
522
-    public static function getBooksBySeries($serieId, $n) {
540
+    public static function getBooksBySeries($serieId, $n)
541
+    {
523 542
         return self::getEntryArray(self::SQL_BOOKS_BY_SERIE, array($serieId), $n);
524 543
     }
525 544
 
526
-    public static function getBooksByTag($tagId, $n) {
545
+    public static function getBooksByTag($tagId, $n)
546
+    {
527 547
         return self::getEntryArray(self::SQL_BOOKS_BY_TAG, array($tagId), $n);
528 548
     }
529 549
 
530
-    public static function getBooksByLanguage($languageId, $n) {
550
+    public static function getBooksByLanguage($languageId, $n)
551
+    {
531 552
         return self::getEntryArray(self::SQL_BOOKS_BY_LANGUAGE, array($languageId), $n);
532 553
     }
533 554
 
@@ -537,32 +558,33 @@  discard block
 block discarded – undo
537 558
      * @param $n integer
538 559
      * @return array
539 560
      */
540
-    public static function getBooksByCustom($customColumn, $id, $n) {
561
+    public static function getBooksByCustom($customColumn, $id, $n)
562
+    {
541 563
         list($query, $params) = $customColumn->getQuery($id);
542 564
 
543 565
         return self::getEntryArray($query, $params, $n);
544 566
     }
545 567
 
546
-    public static function getBookById($bookId) {
568
+    public static function getBookById($bookId)
569
+    {
547 570
         $result = parent::getDb()->prepare('select ' . self::BOOK_COLUMNS . '
548 571
 from books ' . self::SQL_BOOKS_LEFT_JOIN . '
549 572
 where books.id = ?');
550 573
         $result->execute(array($bookId));
551
-        while ($post = $result->fetchObject())
552
-        {
574
+        while ($post = $result->fetchObject()) {
553 575
             $book = new Book($post);
554 576
             return $book;
555 577
         }
556 578
         return NULL;
557 579
     }
558 580
 
559
-    public static function getBookByDataId($dataId) {
581
+    public static function getBookByDataId($dataId)
582
+    {
560 583
         $result = parent::getDb()->prepare('select ' . self::BOOK_COLUMNS . ', data.name, data.format
561 584
 from data, books ' . self::SQL_BOOKS_LEFT_JOIN . '
562 585
 where data.book = books.id and data.id = ?');
563 586
         $result->execute(array($dataId));
564
-        while ($post = $result->fetchObject())
565
-        {
587
+        while ($post = $result->fetchObject()) {
566 588
             $book = new Book($post);
567 589
             $data = new Data($post, $book);
568 590
             $data->id = $dataId;
@@ -572,7 +594,8 @@  discard block
 block discarded – undo
572 594
         return NULL;
573 595
     }
574 596
 
575
-    public static function getBooksByQuery($query, $n, $database = NULL, $numberPerPage = NULL) {
597
+    public static function getBooksByQuery($query, $n, $database = NULL, $numberPerPage = NULL)
598
+    {
576 599
         $i = 0;
577 600
         $critArray = array();
578 601
         foreach (array(PageQueryResult::SCOPE_AUTHOR,
@@ -583,8 +606,7 @@  discard block
 block discarded – undo
583 606
             if (in_array($key, getCurrentOption('ignored_categories')) ||
584 607
                 (!array_key_exists($key, $query) && !array_key_exists('all', $query))) {
585 608
                 $critArray[$i] = self::BAD_SEARCH;
586
-            }
587
-            else {
609
+            } else {
588 610
                 if (array_key_exists($key, $query)) {
589 611
                     $critArray[$i] = $query[$key];
590 612
                 } else {
@@ -596,12 +618,14 @@  discard block
 block discarded – undo
596 618
         return self::getEntryArray(self::SQL_BOOKS_QUERY, $critArray, $n, $database, $numberPerPage);
597 619
     }
598 620
 
599
-    public static function getBooks($n) {
621
+    public static function getBooks($n)
622
+    {
600 623
         list ($entryArray, $totalNumber) = self::getEntryArray(self::SQL_BOOKS_ALL , array (), $n);
601 624
         return array($entryArray, $totalNumber);
602 625
     }
603 626
 
604
-    public static function getAllBooks() {
627
+    public static function getAllBooks()
628
+    {
605 629
         /* @var $result PDOStatement */
606 630
 
607 631
         list (, $result) = parent::executeQuery('select {0}
@@ -610,8 +634,7 @@  discard block
 block discarded – undo
610 634
 order by substr (upper (sort), 1, 1)', 'substr (upper (sort), 1, 1) as title, count(*) as count', self::getFilterString(), array(), -1);
611 635
 
612 636
         $entryArray = array();
613
-        while ($post = $result->fetchObject())
614
-        {
637
+        while ($post = $result->fetchObject()) {
615 638
             array_push($entryArray, new Entry($post->title, Book::getEntryIdByLetter($post->title),
616 639
                 str_format(localize('bookword', $post->count), $post->count), 'text',
617 640
                 array(new LinkNavigation('?page='.parent::PAGE_ALL_BOOKS_LETTER.'&id='. rawurlencode($post->title))), '', $post->count));
@@ -619,25 +642,27 @@  discard block
 block discarded – undo
619 642
         return $entryArray;
620 643
     }
621 644
 
622
-    public static function getBooksByStartingLetter($letter, $n, $database = NULL, $numberPerPage = NULL) {
645
+    public static function getBooksByStartingLetter($letter, $n, $database = NULL, $numberPerPage = NULL)
646
+    {
623 647
         return self::getEntryArray(self::SQL_BOOKS_BY_FIRST_LETTER, array($letter . '%'), $n, $database, $numberPerPage);
624 648
     }
625 649
 
626
-    public static function getEntryArray($query, $params, $n, $database = NULL, $numberPerPage = NULL) {
650
+    public static function getEntryArray($query, $params, $n, $database = NULL, $numberPerPage = NULL)
651
+    {
627 652
         /* @var $totalNumber integer */
628 653
         /* @var $result PDOStatement */
629 654
         list($totalNumber, $result) = parent::executeQuery($query, self::BOOK_COLUMNS, self::getFilterString(), $params, $n, $database, $numberPerPage);
630 655
 
631 656
         $entryArray = array();
632
-        while ($post = $result->fetchObject())
633
-        {
657
+        while ($post = $result->fetchObject()) {
634 658
             $book = new Book($post);
635 659
             array_push($entryArray, $book->getEntry());
636 660
         }
637 661
         return array($entryArray, $totalNumber);
638 662
     }
639 663
 
640
-    public static function getAllRecentBooks() {
664
+    public static function getAllRecentBooks()
665
+    {
641 666
         global $config;
642 667
         list ($entryArray, ) = self::getEntryArray(self::SQL_BOOKS_RECENT . $config['cops_recentbooks_limit'], array(), -1);
643 668
         return $entryArray;
@@ -649,7 +674,8 @@  discard block
 block discarded – undo
649 674
      * @param string[] $columns
650 675
      * @return CustomColumn[]
651 676
      */
652
-    public function getCustomColumnValues($columns, $asArray = false) {
677
+    public function getCustomColumnValues($columns, $asArray = false)
678
+    {
653 679
         $result = array();
654 680
 
655 681
         foreach ($columns as $lookup) {
Please login to merge, or discard this patch.
lib/Identifier.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@
 block discarded – undo
17 17
     public function __construct($post)
18 18
     {
19 19
         $this->id = $post->id;
20
-        $this->type = strtolower($post->type);;
20
+        $this->type = strtolower($post->type); ;
21 21
         $this->val = $post->val;
22 22
         $this->formatType();
23 23
     }
Please login to merge, or discard this patch.