Passed
Push — master ( 492cc7...e5a5fc )
by Michael
45s queued 11s
created
include/functions.language.php 1 patch
Indentation   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -15,22 +15,22 @@
 block discarded – undo
15 15
 define('NEWBB_FUNCTIONS_LANGUAGE_LOADED', true);
16 16
 
17 17
 if (!defined('NEWBB_FUNCTIONS_LANGUAGE')) {
18
-    define('NEWBB_FUNCTIONS_LANGUAGE', 1);
18
+	define('NEWBB_FUNCTIONS_LANGUAGE', 1);
19 19
 
20
-    /**
21
-     * @param         $page
22
-     * @param  string $dirname
23
-     * @return bool
24
-     */
25
-    function newbbLoadLanguage($page, $dirname = 'newbb')
26
-    {
27
-        $page = str_replace('..', '', $page);
28
-        if (!@require_once $GLOBALS['xoops']->path("modules/{$dirname}/{$GLOBALS['xoopsConfig']['language']}/{$language}.php")) {
29
-            if (!@require_once $GLOBALS['xoops']->path("modules/{$dirname}/language/{$language}.php")) {
30
-                return false;
31
-            }
32
-        }
20
+	/**
21
+	 * @param         $page
22
+	 * @param  string $dirname
23
+	 * @return bool
24
+	 */
25
+	function newbbLoadLanguage($page, $dirname = 'newbb')
26
+	{
27
+		$page = str_replace('..', '', $page);
28
+		if (!@require_once $GLOBALS['xoops']->path("modules/{$dirname}/{$GLOBALS['xoopsConfig']['language']}/{$language}.php")) {
29
+			if (!@require_once $GLOBALS['xoops']->path("modules/{$dirname}/language/{$language}.php")) {
30
+				return false;
31
+			}
32
+		}
33 33
 
34
-        return true;
35
-    }
34
+		return true;
35
+	}
36 36
 }
Please login to merge, or discard this patch.
include/display.php 1 patch
Indentation   +139 added lines, -139 removed lines patch added patch discarded remove patch
@@ -69,161 +69,161 @@
 block discarded – undo
69 69
 // uncomment to show text link instead of images (set to true)
70 70
 $displayText[''] = //$displayText['blank'] =
71 71
 
72
-    //$displayText['attachment'] =
73
-    //$displayText['whosonline'] =
74
-    //$displayText['statistik'] =
75
-    //$displayText['lastposticon'] =
76
-
77
-    //$displayText['plus'] =
78
-    //$displayText['minus'] =
79
-
80
-    //$displayText['forum'] =
81
-    //$displayText['forum_new'] =
82
-
83
-    //$displayText['topic'] =
84
-    //$displayText['topic_hot'] =
85
-    //$displayText['topic_sticky'] =
86
-    //$displayText['topic_digest'] =
87
-    //$displayText['topic_locked'] =
88
-    //$displayText['topic_new'] =
89
-    //$displayText['topic_hot_new'] =
90
-    //$displayText['topic_my'] =
91
-
92
-    //$displayText['post'] =
93
-
94
-    //$displayText['poll'] =
95
-    //$displayText['rss'] =
96
-    //$displayText['pdf'] =
97
-    //$displayText['subforum'] =
98
-
99
-    //$displayText['admin_move'] =
100
-    //$displayText['admin_merge'] =
101
-    //$displayText['admin_edit'] =
102
-    //$displayText['admin_delete'] =
103
-
104
-    //$displayText['document'] =
105
-
106
-    //$displayText['previous'] =
107
-    //$displayText['next'] =
108
-    //$displayText['right'] =
109
-    //$displayText['down'] =
110
-    //$displayText['up'] =
111
-    //$displayText['printer'] =
112
-    //$displayText['new_forum']  =
113
-
114
-    //$displayText['facebook'] =
115
-    //$displayText['twitter'] =
116
-    //$displayText['linkedin'] =
117
-    //$displayText['googleplus'] =
118
-    //$displayText['stumbleupon'] =
119
-    //$displayText['friendfeed'] =
120
-    //$displayText['digg'] =
121
-    //$displayText['reddit'] =
122
-    //$displayText['delicious'] =
123
-    //$displayText['technorati'] =
124
-    //$displayText['wong'] =
125
-    //$displayText['anonym'] =
126
-    //$displayText['more'] =
127
-    //$displayText['less'] =
72
+	//$displayText['attachment'] =
73
+	//$displayText['whosonline'] =
74
+	//$displayText['statistik'] =
75
+	//$displayText['lastposticon'] =
76
+
77
+	//$displayText['plus'] =
78
+	//$displayText['minus'] =
79
+
80
+	//$displayText['forum'] =
81
+	//$displayText['forum_new'] =
82
+
83
+	//$displayText['topic'] =
84
+	//$displayText['topic_hot'] =
85
+	//$displayText['topic_sticky'] =
86
+	//$displayText['topic_digest'] =
87
+	//$displayText['topic_locked'] =
88
+	//$displayText['topic_new'] =
89
+	//$displayText['topic_hot_new'] =
90
+	//$displayText['topic_my'] =
91
+
92
+	//$displayText['post'] =
93
+
94
+	//$displayText['poll'] =
95
+	//$displayText['rss'] =
96
+	//$displayText['pdf'] =
97
+	//$displayText['subforum'] =
98
+
99
+	//$displayText['admin_move'] =
100
+	//$displayText['admin_merge'] =
101
+	//$displayText['admin_edit'] =
102
+	//$displayText['admin_delete'] =
103
+
104
+	//$displayText['document'] =
105
+
106
+	//$displayText['previous'] =
107
+	//$displayText['next'] =
108
+	//$displayText['right'] =
109
+	//$displayText['down'] =
110
+	//$displayText['up'] =
111
+	//$displayText['printer'] =
112
+	//$displayText['new_forum']  =
113
+
114
+	//$displayText['facebook'] =
115
+	//$displayText['twitter'] =
116
+	//$displayText['linkedin'] =
117
+	//$displayText['googleplus'] =
118
+	//$displayText['stumbleupon'] =
119
+	//$displayText['friendfeed'] =
120
+	//$displayText['digg'] =
121
+	//$displayText['reddit'] =
122
+	//$displayText['delicious'] =
123
+	//$displayText['technorati'] =
124
+	//$displayText['wong'] =
125
+	//$displayText['anonym'] =
126
+	//$displayText['more'] =
127
+	//$displayText['less'] =
128 128
 
129 129
 $displayText['p_delete'] = $displayText['p_reply'] = $displayText['p_quote'] = $displayText['p_edit'] = $displayText['p_report'] = $displayText['t_new'] = $displayText['t_poll'] = $displayText['t_qr'] = $displayText['t_qr_expand'] = $displayText['t_reply'] =
130 130
 
131
-    //$displayText['online'] =
132
-    //$displayText['offline'] =
131
+	//$displayText['online'] =
132
+	//$displayText['offline'] =
133 133
 
134
-    //$displayText['new_subforum'] =
134
+	//$displayText['new_subforum'] =
135 135
 
136 136
 $displayText['p_bann'] =
137 137
 
138
-    true;
138
+	true;
139 139
 
140 140
 for ($i = 1; $i <= 5; ++$i) {
141
-    //$displayText['rate'.$i] = true;
141
+	//$displayText['rate'.$i] = true;
142 142
 }
143 143
 
144 144
 // uncomment to show images instead of text links (set to false)
145 145
 $displayText[''] = //$displayText['blank'] =
146 146
 
147
-    //$displayText['attachment'] =
148
-    //$displayText['whosonline'] =
149
-    //$displayText['statistik'] =
150
-    //$displayText['lastposticon'] =
151
-
152
-    //$displayText['plus'] =
153
-    //$displayText['minus'] =
154
-
155
-    //$displayText['forum'] =
156
-    //$displayText['forum_new'] =
157
-
158
-    //$displayText['topic'] =
159
-    //$displayText['topic_hot'] =
160
-    //$displayText['topic_sticky'] =
161
-    //$displayText['topic_digest'] =
162
-    //$displayText['topic_locked'] =
163
-    //$displayText['topic_new'] =
164
-    //$displayText['topic_hot_new'] =
165
-    //$displayText['topic_my'] =
166
-
167
-    //$displayText['post'] =
168
-
169
-    //$displayText['poll'] =
170
-    //$displayText['rss'] =
171
-    //$displayText['pdf'] =
172
-    //$displayText['subforum'] =
173
-
174
-    //$displayText['admin_move'] =
175
-    //$displayText['admin_merge'] =
176
-    //$displayText['admin_edit'] =
177
-    //$displayText['admin_delete'] =
178
-
179
-    //$displayText['document'] =
180
-
181
-    //$displayText['previous'] =
182
-    //$displayText['next'] =
183
-    //$displayText['right'] =
184
-    //$displayText['down'] =
185
-    //$displayText['up'] =
186
-    //$displayText['printer'] =
187
-    //$displayText['new_forum']  =
188
-
189
-    //$displayText['facebook'] =
190
-    //$displayText['twitter'] =
191
-    //$displayText['linkedin'] =
192
-    //$displayText['googleplus'] =
193
-    //$displayText['stumbleupon'] =
194
-    //$displayText['friendfeed'] =
195
-    //$displayText['digg'] =
196
-    //$displayText['reddit'] =
197
-    //$displayText['delicious'] =
198
-    //$displayText['technorati'] =
199
-    //$displayText['wong'] =
200
-    //$displayText['anonym'] =
201
-    //$displayText['more'] =
202
-    //$displayText['less'] =
203
-
204
-    //$displayText['p_delete'] =
205
-    //$displayText['p_reply'] =
206
-    //$displayText['p_quote'] =
207
-    //$displayText['p_edit'] =
208
-    //$displayText['p_report'] =
209
-
210
-    //$displayText['t_new'] =
211
-    //$displayText['t_poll'] =
212
-    //$displayText['t_qr'] =
213
-    //$displayText['t_qr_expand'] =
214
-    //$displayText['t_reply'] =
215
-
216
-    //$displayText['online'] =
217
-    //$displayText['offline'] =
218
-
219
-    //$displayText['new_subforum'] =
220
-
221
-    //$displayText['p_bann'] =
222
-
223
-    false;
147
+	//$displayText['attachment'] =
148
+	//$displayText['whosonline'] =
149
+	//$displayText['statistik'] =
150
+	//$displayText['lastposticon'] =
151
+
152
+	//$displayText['plus'] =
153
+	//$displayText['minus'] =
154
+
155
+	//$displayText['forum'] =
156
+	//$displayText['forum_new'] =
157
+
158
+	//$displayText['topic'] =
159
+	//$displayText['topic_hot'] =
160
+	//$displayText['topic_sticky'] =
161
+	//$displayText['topic_digest'] =
162
+	//$displayText['topic_locked'] =
163
+	//$displayText['topic_new'] =
164
+	//$displayText['topic_hot_new'] =
165
+	//$displayText['topic_my'] =
166
+
167
+	//$displayText['post'] =
168
+
169
+	//$displayText['poll'] =
170
+	//$displayText['rss'] =
171
+	//$displayText['pdf'] =
172
+	//$displayText['subforum'] =
173
+
174
+	//$displayText['admin_move'] =
175
+	//$displayText['admin_merge'] =
176
+	//$displayText['admin_edit'] =
177
+	//$displayText['admin_delete'] =
178
+
179
+	//$displayText['document'] =
180
+
181
+	//$displayText['previous'] =
182
+	//$displayText['next'] =
183
+	//$displayText['right'] =
184
+	//$displayText['down'] =
185
+	//$displayText['up'] =
186
+	//$displayText['printer'] =
187
+	//$displayText['new_forum']  =
188
+
189
+	//$displayText['facebook'] =
190
+	//$displayText['twitter'] =
191
+	//$displayText['linkedin'] =
192
+	//$displayText['googleplus'] =
193
+	//$displayText['stumbleupon'] =
194
+	//$displayText['friendfeed'] =
195
+	//$displayText['digg'] =
196
+	//$displayText['reddit'] =
197
+	//$displayText['delicious'] =
198
+	//$displayText['technorati'] =
199
+	//$displayText['wong'] =
200
+	//$displayText['anonym'] =
201
+	//$displayText['more'] =
202
+	//$displayText['less'] =
203
+
204
+	//$displayText['p_delete'] =
205
+	//$displayText['p_reply'] =
206
+	//$displayText['p_quote'] =
207
+	//$displayText['p_edit'] =
208
+	//$displayText['p_report'] =
209
+
210
+	//$displayText['t_new'] =
211
+	//$displayText['t_poll'] =
212
+	//$displayText['t_qr'] =
213
+	//$displayText['t_qr_expand'] =
214
+	//$displayText['t_reply'] =
215
+
216
+	//$displayText['online'] =
217
+	//$displayText['offline'] =
218
+
219
+	//$displayText['new_subforum'] =
220
+
221
+	//$displayText['p_bann'] =
222
+
223
+	false;
224 224
 
225 225
 for ($i = 1; $i <= 5; ++$i) {
226
-    //$displayText['rate'.$i] = false;
226
+	//$displayText['rate'.$i] = false;
227 227
 }
228 228
 
229 229
 return $displayText;
Please login to merge, or discard this patch.
include/functions.image.php 1 patch
Indentation   +188 added lines, -188 removed lines patch added patch discarded remove patch
@@ -30,192 +30,192 @@
 block discarded – undo
30 30
 //  ------------------------------------------------------------------------ //
31 31
 
32 32
 if (!defined('NEWBB_FUNCTIONS_IMAGE')) {
33
-    define('NEWBB_FUNCTIONS_IMAGE', true);
34
-
35
-    /**
36
-     * @param $source
37
-     * @return string
38
-     */
39
-    function newbbAttachmentImage($source)
40
-    {
41
-        $img_path   = $GLOBALS['xoops']->path($GLOBALS['xoopsModuleConfig']['dir_attachments']);
42
-        $img_url    = XOOPS_URL . '/' . $GLOBALS['xoopsModuleConfig']['dir_attachments'];
43
-        $thumb_path = $img_path . '/thumbs';
44
-        $thumb_url  = $img_url . '/thumbs';
45
-
46
-        $thumb     = $thumb_path . '/' . $source;
47
-        $image     = $img_path . '/' . $source;
48
-        $thumb_url = $thumb_url . '/' . $source;
49
-        $image_url = $img_url . '/' . $source;
50
-
51
-        $imginfo  = @getimagesize($image);
52
-        $img_info = (count($imginfo) > 0) ? $imginfo[0] . 'X' . $imginfo[1] . ' px' : '';
53
-
54
-        if ($GLOBALS['xoopsModuleConfig']['max_image_width'] > 0
55
-            && $GLOBALS['xoopsModuleConfig']['max_image_height'] > 0) {
56
-            if ($imginfo[0] > $GLOBALS['xoopsModuleConfig']['max_image_width']
57
-                || $imginfo[1] > $GLOBALS['xoopsModuleConfig']['max_image_height']) {
58
-                //if (!file_exists($thumb_path.'/'.$source) && $imginfo[0] > $GLOBALS['xoopsModuleConfig']['max_img_width']) {
59
-                if (!file_exists($thumb_path . '/' . $source)) {
60
-                    newbbCreateThumbnail($source, $GLOBALS['xoopsModuleConfig']['max_image_width']);
61
-                }
62
-            }
63
-
64
-            if ($imginfo[0] > $GLOBALS['xoopsModuleConfig']['max_image_width']
65
-                || $imginfo[1] > $GLOBALS['xoopsModuleConfig']['max_image_height']) {
66
-                $pseudo_width  = $GLOBALS['xoopsModuleConfig']['max_image_width'];
67
-                $pseudo_height = $GLOBALS['xoopsModuleConfig']['max_image_width'] * ($imginfo[1] / $imginfo[0]);
68
-                $pseudo_size   = "width='" . $pseudo_width . "px' height='" . $pseudo_height . "px'";
69
-            }
70
-            // irmtfan to fix Undefined variable: pseudo_height
71
-            if (!empty($pseudo_height) && $GLOBALS['xoopsModuleConfig']['max_image_height'] > 0
72
-                && $pseudo_height > $GLOBALS['xoopsModuleConfig']['max_image_height']) {
73
-                $pseudo_height = $GLOBALS['xoopsModuleConfig']['max_image_height'];
74
-                $pseudo_width  = $GLOBALS['xoopsModuleConfig']['max_image_height'] * ($imginfo[0] / $imginfo[1]);
75
-                $pseudo_size   = "width='" . $pseudo_width . "px' height='" . $pseudo_height . "px'";
76
-            }
77
-        }
78
-
79
-        if (file_exists($thumb)) {
80
-            $attachmentImage = '<a href="' . $image_url . '" title="' . $source . ' ' . $img_info . '" target="_blank">';
81
-            // $attachmentImage .= '<img src="' . $thumb_url . '" alt="' . $source . ' ' . $img_info . '" />';
82
-            $attachmentImage .= '<img src="' . $image_url . '" ' . $pseudo_size . ' alt="' . $source . ' ' . $img_info . '" />';
83
-            $attachmentImage .= '</a>';
84
-        } elseif (!empty($pseudo_size)) {
85
-            $attachmentImage = '<a href="' . $image_url . '" title="' . $source . ' ' . $img_info . '" target="_blank">';
86
-            $attachmentImage .= '<img src="' . $image_url . '" ' . $pseudo_size . ' alt="' . $source . ' ' . $img_info . '" />';
87
-            $attachmentImage .= '</a>';
88
-        } elseif (file_exists($image)) {
89
-            $attachmentImage = '<img src="' . $image_url . '" alt="' . $source . ' ' . $img_info . '" />';
90
-        } else {
91
-            $attachmentImage = '';
92
-        }
93
-
94
-        return $attachmentImage;
95
-    }
96
-
97
-    /**
98
-     * @param $source
99
-     * @param $thumb_width
100
-     * @return bool
101
-     */
102
-    function newbbCreateThumbnail($source, $thumb_width)
103
-    {
104
-        $cmd        = '';
105
-        $img_path   = $GLOBALS['xoops']->path($GLOBALS['xoopsModuleConfig']['dir_attachments']);
106
-        $thumb_path = $img_path . '/thumbs';
107
-        $src_file   = $img_path . '/' . $source;
108
-        $new_file   = $thumb_path . '/' . $source;
109
-        //$imageLibs = newbb_getImageLibs();
110
-
111
-        if (!filesize($src_file) || !is_readable($src_file)) {
112
-            return false;
113
-        }
114
-
115
-        if (!is_dir($thumb_path) || !is_writable($thumb_path)) {
116
-            return false;
117
-        }
118
-
119
-        $imginfo = @getimagesize($src_file);
120
-
121
-        if (null === $imginfo) {
122
-            return false;
123
-        }
124
-        if ($imginfo[0] < $thumb_width) {
125
-            return false;
126
-        }
127
-
128
-        $newWidth  = (int)min($imginfo[0], $thumb_width);
129
-        $newHeight = (int)($imginfo[1] * $newWidth / $imginfo[0]);
130
-
131
-        if (1 == $GLOBALS['xoopsModuleConfig']['image_lib'] || 0 == $GLOBALS['xoopsModuleConfig']['image_lib']) {
132
-            if (preg_match("#[A-Z]:|\\\\#Ai", __FILE__)) {
133
-                $cur_dir     = __DIR__;
134
-                $src_file_im = '"' . $cur_dir . '\\' . str_replace('/', '\\', $src_file) . '"';
135
-                $new_file_im = '"' . $cur_dir . '\\' . str_replace('/', '\\', $new_file) . '"';
136
-            } else {
137
-                $src_file_im = @escapeshellarg($src_file);
138
-                $new_file_im = @escapeshellarg($new_file);
139
-            }
140
-            $path           = empty($GLOBALS['xoopsModuleConfig']['path_magick']) ? '' : $GLOBALS['xoopsModuleConfig']['path_magick'] . '/';
141
-            $magick_command = $path . 'convert -quality 85 -antialias -sample ' . $newWidth . 'x' . $newHeight . ' ' . $src_file_im . ' +profile "*" ' . str_replace('\\', '/', $new_file_im) . '';
142
-
143
-            @passthru($magick_command);
144
-            if (file_exists($new_file)) {
145
-                return true;
146
-            }
147
-        }
148
-
149
-        if (2 == $GLOBALS['xoopsModuleConfig']['image_lib'] || 0 == $GLOBALS['xoopsModuleConfig']['image_lib']) {
150
-            $path = empty($GLOBALS['xoopsModuleConfig']['path_netpbm']) ? '' : $GLOBALS['xoopsModuleConfig']['path_netpbm'] . '/';
151
-            if (preg_match("/\.png/", $source)) {
152
-                $cmd = $path . "pngtopnm $src_file | " . $path . "pnmscale -xysize $newWidth $newHeight | " . $path . "pnmtopng > $new_file";
153
-            } elseif (preg_match("/\.(jpg|jpeg)/", $source)) {
154
-                $cmd = $path . "jpegtopnm $src_file | " . $path . "pnmscale -xysize $newWidth $newHeight | " . $path . "ppmtojpeg -quality=90 > $new_file";
155
-            } elseif (preg_match("/\.gif/", $source)) {
156
-                $cmd = $path . "giftopnm $src_file | " . $path . "pnmscale -xysize $newWidth $newHeight | ppmquant 256 | " . $path . "ppmtogif > $new_file";
157
-            }
158
-
159
-            @exec($cmd, $output, $retval);
160
-            if (file_exists($new_file)) {
161
-                return true;
162
-            }
163
-        }
164
-
165
-        $type            = $imginfo[2];
166
-        $supported_types = [];
167
-
168
-        if (!extension_loaded('gd')) {
169
-            return false;
170
-        }
171
-        if (function_exists('imagegif')) {
172
-            $supported_types[] = 1;
173
-        }
174
-        if (function_exists('imagejpeg')) {
175
-            $supported_types[] = 2;
176
-        }
177
-        if (function_exists('imagepng')) {
178
-            $supported_types[] = 3;
179
-        }
180
-
181
-        $imageCreateFunction = function_exists('imagecreatetruecolor') ? 'imagecreatetruecolor' : 'imagecreate';
182
-
183
-        if (in_array($type, $supported_types)) {
184
-            switch ($type) {
185
-                case 1:
186
-                    if (!function_exists('imagecreatefromgif')) {
187
-                        return false;
188
-                    }
189
-                    $im     = imagecreatefromgif($src_file);
190
-                    $new_im = imagecreate($newWidth, $newHeight);
191
-                    imagecopyresized($new_im, $im, 0, 0, 0, 0, $newWidth, $newHeight, $imginfo[0], $imginfo[1]);
192
-                    imagegif($new_im, $new_file);
193
-                    imagedestroy($im);
194
-                    imagedestroy($new_im);
195
-                    break;
196
-                case 2:
197
-                    $im     = imagecreatefromjpeg($src_file);
198
-                    $new_im = $imageCreateFunction($newWidth, $newHeight);
199
-                    imagecopyresized($new_im, $im, 0, 0, 0, 0, $newWidth, $newHeight, $imginfo[0], $imginfo[1]);
200
-                    imagejpeg($new_im, $new_file, 90);
201
-                    imagedestroy($im);
202
-                    imagedestroy($new_im);
203
-                    break;
204
-                case 3:
205
-                    $im     = imagecreatefrompng($src_file);
206
-                    $new_im = $imageCreateFunction($newWidth, $newHeight);
207
-                    imagecopyresized($new_im, $im, 0, 0, 0, 0, $newWidth, $newHeight, $imginfo[0], $imginfo[1]);
208
-                    imagepng($new_im, $new_file);
209
-                    imagedestroy($im);
210
-                    imagedestroy($new_im);
211
-                    break;
212
-            }
213
-        }
214
-
215
-        if (file_exists($new_file)) {
216
-            return true;
217
-        } else {
218
-            return false;
219
-        }
220
-    }
33
+	define('NEWBB_FUNCTIONS_IMAGE', true);
34
+
35
+	/**
36
+	 * @param $source
37
+	 * @return string
38
+	 */
39
+	function newbbAttachmentImage($source)
40
+	{
41
+		$img_path   = $GLOBALS['xoops']->path($GLOBALS['xoopsModuleConfig']['dir_attachments']);
42
+		$img_url    = XOOPS_URL . '/' . $GLOBALS['xoopsModuleConfig']['dir_attachments'];
43
+		$thumb_path = $img_path . '/thumbs';
44
+		$thumb_url  = $img_url . '/thumbs';
45
+
46
+		$thumb     = $thumb_path . '/' . $source;
47
+		$image     = $img_path . '/' . $source;
48
+		$thumb_url = $thumb_url . '/' . $source;
49
+		$image_url = $img_url . '/' . $source;
50
+
51
+		$imginfo  = @getimagesize($image);
52
+		$img_info = (count($imginfo) > 0) ? $imginfo[0] . 'X' . $imginfo[1] . ' px' : '';
53
+
54
+		if ($GLOBALS['xoopsModuleConfig']['max_image_width'] > 0
55
+			&& $GLOBALS['xoopsModuleConfig']['max_image_height'] > 0) {
56
+			if ($imginfo[0] > $GLOBALS['xoopsModuleConfig']['max_image_width']
57
+				|| $imginfo[1] > $GLOBALS['xoopsModuleConfig']['max_image_height']) {
58
+				//if (!file_exists($thumb_path.'/'.$source) && $imginfo[0] > $GLOBALS['xoopsModuleConfig']['max_img_width']) {
59
+				if (!file_exists($thumb_path . '/' . $source)) {
60
+					newbbCreateThumbnail($source, $GLOBALS['xoopsModuleConfig']['max_image_width']);
61
+				}
62
+			}
63
+
64
+			if ($imginfo[0] > $GLOBALS['xoopsModuleConfig']['max_image_width']
65
+				|| $imginfo[1] > $GLOBALS['xoopsModuleConfig']['max_image_height']) {
66
+				$pseudo_width  = $GLOBALS['xoopsModuleConfig']['max_image_width'];
67
+				$pseudo_height = $GLOBALS['xoopsModuleConfig']['max_image_width'] * ($imginfo[1] / $imginfo[0]);
68
+				$pseudo_size   = "width='" . $pseudo_width . "px' height='" . $pseudo_height . "px'";
69
+			}
70
+			// irmtfan to fix Undefined variable: pseudo_height
71
+			if (!empty($pseudo_height) && $GLOBALS['xoopsModuleConfig']['max_image_height'] > 0
72
+				&& $pseudo_height > $GLOBALS['xoopsModuleConfig']['max_image_height']) {
73
+				$pseudo_height = $GLOBALS['xoopsModuleConfig']['max_image_height'];
74
+				$pseudo_width  = $GLOBALS['xoopsModuleConfig']['max_image_height'] * ($imginfo[0] / $imginfo[1]);
75
+				$pseudo_size   = "width='" . $pseudo_width . "px' height='" . $pseudo_height . "px'";
76
+			}
77
+		}
78
+
79
+		if (file_exists($thumb)) {
80
+			$attachmentImage = '<a href="' . $image_url . '" title="' . $source . ' ' . $img_info . '" target="_blank">';
81
+			// $attachmentImage .= '<img src="' . $thumb_url . '" alt="' . $source . ' ' . $img_info . '" />';
82
+			$attachmentImage .= '<img src="' . $image_url . '" ' . $pseudo_size . ' alt="' . $source . ' ' . $img_info . '" />';
83
+			$attachmentImage .= '</a>';
84
+		} elseif (!empty($pseudo_size)) {
85
+			$attachmentImage = '<a href="' . $image_url . '" title="' . $source . ' ' . $img_info . '" target="_blank">';
86
+			$attachmentImage .= '<img src="' . $image_url . '" ' . $pseudo_size . ' alt="' . $source . ' ' . $img_info . '" />';
87
+			$attachmentImage .= '</a>';
88
+		} elseif (file_exists($image)) {
89
+			$attachmentImage = '<img src="' . $image_url . '" alt="' . $source . ' ' . $img_info . '" />';
90
+		} else {
91
+			$attachmentImage = '';
92
+		}
93
+
94
+		return $attachmentImage;
95
+	}
96
+
97
+	/**
98
+	 * @param $source
99
+	 * @param $thumb_width
100
+	 * @return bool
101
+	 */
102
+	function newbbCreateThumbnail($source, $thumb_width)
103
+	{
104
+		$cmd        = '';
105
+		$img_path   = $GLOBALS['xoops']->path($GLOBALS['xoopsModuleConfig']['dir_attachments']);
106
+		$thumb_path = $img_path . '/thumbs';
107
+		$src_file   = $img_path . '/' . $source;
108
+		$new_file   = $thumb_path . '/' . $source;
109
+		//$imageLibs = newbb_getImageLibs();
110
+
111
+		if (!filesize($src_file) || !is_readable($src_file)) {
112
+			return false;
113
+		}
114
+
115
+		if (!is_dir($thumb_path) || !is_writable($thumb_path)) {
116
+			return false;
117
+		}
118
+
119
+		$imginfo = @getimagesize($src_file);
120
+
121
+		if (null === $imginfo) {
122
+			return false;
123
+		}
124
+		if ($imginfo[0] < $thumb_width) {
125
+			return false;
126
+		}
127
+
128
+		$newWidth  = (int)min($imginfo[0], $thumb_width);
129
+		$newHeight = (int)($imginfo[1] * $newWidth / $imginfo[0]);
130
+
131
+		if (1 == $GLOBALS['xoopsModuleConfig']['image_lib'] || 0 == $GLOBALS['xoopsModuleConfig']['image_lib']) {
132
+			if (preg_match("#[A-Z]:|\\\\#Ai", __FILE__)) {
133
+				$cur_dir     = __DIR__;
134
+				$src_file_im = '"' . $cur_dir . '\\' . str_replace('/', '\\', $src_file) . '"';
135
+				$new_file_im = '"' . $cur_dir . '\\' . str_replace('/', '\\', $new_file) . '"';
136
+			} else {
137
+				$src_file_im = @escapeshellarg($src_file);
138
+				$new_file_im = @escapeshellarg($new_file);
139
+			}
140
+			$path           = empty($GLOBALS['xoopsModuleConfig']['path_magick']) ? '' : $GLOBALS['xoopsModuleConfig']['path_magick'] . '/';
141
+			$magick_command = $path . 'convert -quality 85 -antialias -sample ' . $newWidth . 'x' . $newHeight . ' ' . $src_file_im . ' +profile "*" ' . str_replace('\\', '/', $new_file_im) . '';
142
+
143
+			@passthru($magick_command);
144
+			if (file_exists($new_file)) {
145
+				return true;
146
+			}
147
+		}
148
+
149
+		if (2 == $GLOBALS['xoopsModuleConfig']['image_lib'] || 0 == $GLOBALS['xoopsModuleConfig']['image_lib']) {
150
+			$path = empty($GLOBALS['xoopsModuleConfig']['path_netpbm']) ? '' : $GLOBALS['xoopsModuleConfig']['path_netpbm'] . '/';
151
+			if (preg_match("/\.png/", $source)) {
152
+				$cmd = $path . "pngtopnm $src_file | " . $path . "pnmscale -xysize $newWidth $newHeight | " . $path . "pnmtopng > $new_file";
153
+			} elseif (preg_match("/\.(jpg|jpeg)/", $source)) {
154
+				$cmd = $path . "jpegtopnm $src_file | " . $path . "pnmscale -xysize $newWidth $newHeight | " . $path . "ppmtojpeg -quality=90 > $new_file";
155
+			} elseif (preg_match("/\.gif/", $source)) {
156
+				$cmd = $path . "giftopnm $src_file | " . $path . "pnmscale -xysize $newWidth $newHeight | ppmquant 256 | " . $path . "ppmtogif > $new_file";
157
+			}
158
+
159
+			@exec($cmd, $output, $retval);
160
+			if (file_exists($new_file)) {
161
+				return true;
162
+			}
163
+		}
164
+
165
+		$type            = $imginfo[2];
166
+		$supported_types = [];
167
+
168
+		if (!extension_loaded('gd')) {
169
+			return false;
170
+		}
171
+		if (function_exists('imagegif')) {
172
+			$supported_types[] = 1;
173
+		}
174
+		if (function_exists('imagejpeg')) {
175
+			$supported_types[] = 2;
176
+		}
177
+		if (function_exists('imagepng')) {
178
+			$supported_types[] = 3;
179
+		}
180
+
181
+		$imageCreateFunction = function_exists('imagecreatetruecolor') ? 'imagecreatetruecolor' : 'imagecreate';
182
+
183
+		if (in_array($type, $supported_types)) {
184
+			switch ($type) {
185
+				case 1:
186
+					if (!function_exists('imagecreatefromgif')) {
187
+						return false;
188
+					}
189
+					$im     = imagecreatefromgif($src_file);
190
+					$new_im = imagecreate($newWidth, $newHeight);
191
+					imagecopyresized($new_im, $im, 0, 0, 0, 0, $newWidth, $newHeight, $imginfo[0], $imginfo[1]);
192
+					imagegif($new_im, $new_file);
193
+					imagedestroy($im);
194
+					imagedestroy($new_im);
195
+					break;
196
+				case 2:
197
+					$im     = imagecreatefromjpeg($src_file);
198
+					$new_im = $imageCreateFunction($newWidth, $newHeight);
199
+					imagecopyresized($new_im, $im, 0, 0, 0, 0, $newWidth, $newHeight, $imginfo[0], $imginfo[1]);
200
+					imagejpeg($new_im, $new_file, 90);
201
+					imagedestroy($im);
202
+					imagedestroy($new_im);
203
+					break;
204
+				case 3:
205
+					$im     = imagecreatefrompng($src_file);
206
+					$new_im = $imageCreateFunction($newWidth, $newHeight);
207
+					imagecopyresized($new_im, $im, 0, 0, 0, 0, $newWidth, $newHeight, $imginfo[0], $imginfo[1]);
208
+					imagepng($new_im, $new_file);
209
+					imagedestroy($im);
210
+					imagedestroy($new_im);
211
+					break;
212
+			}
213
+		}
214
+
215
+		if (file_exists($new_file)) {
216
+			return true;
217
+		} else {
218
+			return false;
219
+		}
220
+	}
221 221
 }
Please login to merge, or discard this patch.
include/images.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 $forumImage['facebook'] = $forumImage['twitter'] = $forumImage['linkedin'] = $forumImage['googleplus'] = $forumImage['stumbleupon'] = $forumImage['friendfeed'] = $forumImage['digg'] = $forumImage['reddit'] = $forumImage['delicious'] = $forumImage['technorati'] = $forumImage['wong'] = $forumImage['anonym'] = $forumImage['more'] = $forumImage['less'] = 'icon';
49 49
 
50 50
 for ($i = 1; $i <= 5; ++$i) {
51
-    $forumImage['rate' . $i] = 'icon';
51
+	$forumImage['rate' . $i] = 'icon';
52 52
 }
53 53
 
54 54
 $forumImage['p_delete'] = $forumImage['p_reply'] = $forumImage['p_quote'] = $forumImage['p_edit'] = $forumImage['p_report'] =
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 
62 62
 $forumImage['p_bann'] =
63 63
 
64
-    'language';
64
+	'language';
65 65
 
66 66
 /*
67 67
 $forumImage[''] = $forumImage['blank'] = $forumImage['icon']."blank";
Please login to merge, or discard this patch.
include/functions.topic.php 1 patch
Indentation   +28 added lines, -28 removed lines patch added patch discarded remove patch
@@ -15,35 +15,35 @@
 block discarded – undo
15 15
 define('NEWBB_FUNCTIONS_TOPIC_LOADED', true);
16 16
 
17 17
 if (!defined('NEWBB_FUNCTIONS_TOPIC')) {
18
-    define('NEWBB_FUNCTIONS_TOPIC', 1);
18
+	define('NEWBB_FUNCTIONS_TOPIC', 1);
19 19
 
20
-    /**
21
-     * Create full title of a topic
22
-     *
23
-     * the title is composed of [type_name] if type_id is greater than 0 plus topic Title
24
-     * @param         $topicTitle
25
-     * @param  null   $prefixName
26
-     * @param  null   $prefixColor
27
-     * @return string
28
-     */
29
-    function newbbGetTopicTitle($topicTitle, $prefixName = null, $prefixColor = null)
30
-    {
31
-        return getTopicTitle($topicTitle, $prefixName = null, $prefixColor = null);
32
-    }
20
+	/**
21
+	 * Create full title of a topic
22
+	 *
23
+	 * the title is composed of [type_name] if type_id is greater than 0 plus topic Title
24
+	 * @param         $topicTitle
25
+	 * @param  null   $prefixName
26
+	 * @param  null   $prefixColor
27
+	 * @return string
28
+	 */
29
+	function newbbGetTopicTitle($topicTitle, $prefixName = null, $prefixColor = null)
30
+	{
31
+		return getTopicTitle($topicTitle, $prefixName = null, $prefixColor = null);
32
+	}
33 33
 
34
-    /**
35
-     * @param         $topicTitle
36
-     * @param  null   $prefixName
37
-     * @param  null   $prefixColor
38
-     * @return string
39
-     */
40
-    function getTopicTitle($topicTitle, $prefixName = null, $prefixColor = null)
41
-    {
42
-        if (empty($prefixName)) {
43
-            return $topicTitle;
44
-        }
45
-        $topicPrefix = $prefixColor ? '<em style="font-style: normal; color: ' . $prefixColor . ';">[' . $prefixName . ']</em> ' : '[' . $prefixName . '] ';
34
+	/**
35
+	 * @param         $topicTitle
36
+	 * @param  null   $prefixName
37
+	 * @param  null   $prefixColor
38
+	 * @return string
39
+	 */
40
+	function getTopicTitle($topicTitle, $prefixName = null, $prefixColor = null)
41
+	{
42
+		if (empty($prefixName)) {
43
+			return $topicTitle;
44
+		}
45
+		$topicPrefix = $prefixColor ? '<em style="font-style: normal; color: ' . $prefixColor . ';">[' . $prefixName . ']</em> ' : '[' . $prefixName . '] ';
46 46
 
47
-        return $topicPrefix . $topicTitle;
48
-    }
47
+		return $topicPrefix . $topicTitle;
48
+	}
49 49
 }
Please login to merge, or discard this patch.
include/functions.read.php 1 patch
Indentation   +61 added lines, -61 removed lines patch added patch discarded remove patch
@@ -17,75 +17,75 @@
 block discarded – undo
17 17
 define('NEWBB_FUNCTIONS_READ_LOADED', true);
18 18
 
19 19
 if (!defined('NEWBB_FUNCTIONS_READ')) {
20
-    define('NEWBB_FUNCTIONS_READ', 1);
20
+	define('NEWBB_FUNCTIONS_READ', 1);
21 21
 
22
-    /**
23
-     * @param        $type
24
-     * @param        $item_id
25
-     * @param        $post_id
26
-     * @param  null  $uid
27
-     * @return mixed
28
-     */
29
-    function newbbSetRead($type, $item_id, $post_id, $uid = null)
30
-    {
31
-        /** @var Newbb\ReadHandler $readHandler */
32
-        $readHandler = Newbb\Helper::getInstance()->getHandler('Read'.$type);
22
+	/**
23
+	 * @param        $type
24
+	 * @param        $item_id
25
+	 * @param        $post_id
26
+	 * @param  null  $uid
27
+	 * @return mixed
28
+	 */
29
+	function newbbSetRead($type, $item_id, $post_id, $uid = null)
30
+	{
31
+		/** @var Newbb\ReadHandler $readHandler */
32
+		$readHandler = Newbb\Helper::getInstance()->getHandler('Read'.$type);
33 33
 
34
-        return $readHandler->setRead($item_id, $post_id, $uid);
35
-    }
34
+		return $readHandler->setRead($item_id, $post_id, $uid);
35
+	}
36 36
 
37
-    /**
38
-     * @param        $type
39
-     * @param        $item_id
40
-     * @param  null  $uid
41
-     * @return mixed
42
-     */
43
-    function newbbGetRead($type, $item_id, $uid = null)
44
-    {
45
-        /** @var Newbb\ReadHandler $readHandler */
46
-        $readHandler = Newbb\Helper::getInstance()->getHandler('Read'.$type);
37
+	/**
38
+	 * @param        $type
39
+	 * @param        $item_id
40
+	 * @param  null  $uid
41
+	 * @return mixed
42
+	 */
43
+	function newbbGetRead($type, $item_id, $uid = null)
44
+	{
45
+		/** @var Newbb\ReadHandler $readHandler */
46
+		$readHandler = Newbb\Helper::getInstance()->getHandler('Read'.$type);
47 47
 
48
-        return $readHandler->getRead($item_id, $uid);
49
-    }
48
+		return $readHandler->getRead($item_id, $uid);
49
+	}
50 50
 
51
-    /**
52
-     * @param  int  $status
53
-     * @param  null $uid
54
-     * @return mixed
55
-     */
56
-    function newbbSetReadForum($status = 0, $uid = null)
57
-    {
58
-        /** @var Newbb\ReadForumHandler $readHandler */
59
-        $readForumHandler = Newbb\Helper::getInstance()->getHandler('Readforum');
51
+	/**
52
+	 * @param  int  $status
53
+	 * @param  null $uid
54
+	 * @return mixed
55
+	 */
56
+	function newbbSetReadForum($status = 0, $uid = null)
57
+	{
58
+		/** @var Newbb\ReadForumHandler $readHandler */
59
+		$readForumHandler = Newbb\Helper::getInstance()->getHandler('Readforum');
60 60
 
61
-        return $readForumHandler->setReadItems($status, $uid);
62
-    }
61
+		return $readForumHandler->setReadItems($status, $uid);
62
+	}
63 63
 
64
-    /**
65
-     * @param  int  $status
66
-     * @param  int  $forum_id
67
-     * @param  null $uid
68
-     * @return mixed
69
-     */
70
-    function newbbSetReadTopic($status = 0, $forum_id = 0, $uid = null)
71
-    {
72
-        /** @var Newbb\ReadtopicHandler $readHandler */
73
-        $readTopicHandler = Newbb\Helper::getInstance()->getHandler('Readtopic');
64
+	/**
65
+	 * @param  int  $status
66
+	 * @param  int  $forum_id
67
+	 * @param  null $uid
68
+	 * @return mixed
69
+	 */
70
+	function newbbSetReadTopic($status = 0, $forum_id = 0, $uid = null)
71
+	{
72
+		/** @var Newbb\ReadtopicHandler $readHandler */
73
+		$readTopicHandler = Newbb\Helper::getInstance()->getHandler('Readtopic');
74 74
 
75
-        return $readTopicHandler->setReadItems($status, $forum_id, $uid);
76
-    }
75
+		return $readTopicHandler->setReadItems($status, $forum_id, $uid);
76
+	}
77 77
 
78
-    /**
79
-     * @param        $type
80
-     * @param        $items
81
-     * @param  null  $uid
82
-     * @return mixed
83
-     */
84
-    function newbbIsRead($type, &$items, $uid = null)
85
-    {
86
-        /** @var Newbb\ReadHandler $readHandler */
87
-        $readHandler = Newbb\Helper::getInstance()->getHandler('Read'.$type);
78
+	/**
79
+	 * @param        $type
80
+	 * @param        $items
81
+	 * @param  null  $uid
82
+	 * @return mixed
83
+	 */
84
+	function newbbIsRead($type, &$items, $uid = null)
85
+	{
86
+		/** @var Newbb\ReadHandler $readHandler */
87
+		$readHandler = Newbb\Helper::getInstance()->getHandler('Read'.$type);
88 88
 
89
-        return $readHandler->isReadItems($items, $uid);
90
-    }
89
+		return $readHandler->isReadItems($items, $uid);
90
+	}
91 91
 }
Please login to merge, or discard this patch.
include/functions.ini.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@
 block discarded – undo
31 31
 // defined('XOOPS_ROOT_PATH') || die('Restricted access');
32 32
 
33 33
 if (defined('NEWBB_FUNCTIONS_INI')) {
34
-    return;
34
+	return;
35 35
 }
36 36
 define('NEWBB_FUNCTIONS_INI', 1);
37 37
 
Please login to merge, or discard this patch.
include/functions.php 1 patch
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -15,12 +15,12 @@
 block discarded – undo
15 15
 define('NEWBB_FUNCTIONS_LOADED', true);
16 16
 
17 17
 if (!defined('NEWBB_FUNCTIONS')) {
18
-    define('NEWBB_FUNCTIONS', 1);
18
+	define('NEWBB_FUNCTIONS', 1);
19 19
 
20
-    require_once __DIR__ . '/functions.image.php';
21
-    require_once __DIR__ . '/functions.user.php';
22
-    require_once __DIR__ . '/functions.render.php';
23
-    require_once __DIR__ . '/functions.forum.php';
24
-    require_once __DIR__ . '/functions.session.php';
25
-    require_once __DIR__ . '/functions.stats.php';
20
+	require_once __DIR__ . '/functions.image.php';
21
+	require_once __DIR__ . '/functions.user.php';
22
+	require_once __DIR__ . '/functions.render.php';
23
+	require_once __DIR__ . '/functions.forum.php';
24
+	require_once __DIR__ . '/functions.session.php';
25
+	require_once __DIR__ . '/functions.stats.php';
26 26
 }
Please login to merge, or discard this patch.
include/functions.stats.php 1 patch
Indentation   +78 added lines, -78 removed lines patch added patch discarded remove patch
@@ -17,97 +17,97 @@
 block discarded – undo
17 17
 define('NEWBB_FUNCTIONS_STATS_LOADED', true);
18 18
 
19 19
 if (!defined('NEWBB_FUNCTIONS_STATS')) {
20
-    define('NEWBB_FUNCTIONS_STATS', 1);
20
+	define('NEWBB_FUNCTIONS_STATS', 1);
21 21
 
22
-    /**
23
-     * @return mixed
24
-     */
25
-    function newbbGetStats()
26
-    {
27
-        /** @var Newbb\StatsHandler $statsHandler */
28
-        $statsHandler = Newbb\Helper::getInstance()->getHandler('Stats');
29
-        $stats        = $statsHandler->getStats();
22
+	/**
23
+	 * @return mixed
24
+	 */
25
+	function newbbGetStats()
26
+	{
27
+		/** @var Newbb\StatsHandler $statsHandler */
28
+		$statsHandler = Newbb\Helper::getInstance()->getHandler('Stats');
29
+		$stats        = $statsHandler->getStats();
30 30
 
31
-        return $stats;
32
-    }
31
+		return $stats;
32
+	}
33 33
 
34
-    /**
35
-     * @param        $id
36
-     * @param        $type
37
-     * @param  int   $increment
38
-     * @return mixed
39
-     */
40
-    function newbbUpdateStats($id, $type, $increment = 1)
41
-    {
42
-        /** @var Newbb\StatsHandler $statsHandler */
43
-        $statsHandler = Newbb\Helper::getInstance()->getHandler('Stats');
34
+	/**
35
+	 * @param        $id
36
+	 * @param        $type
37
+	 * @param  int   $increment
38
+	 * @return mixed
39
+	 */
40
+	function newbbUpdateStats($id, $type, $increment = 1)
41
+	{
42
+		/** @var Newbb\StatsHandler $statsHandler */
43
+		$statsHandler = Newbb\Helper::getInstance()->getHandler('Stats');
44 44
 
45
-        return $statsHandler->update($id, $type, $increment);
46
-    }
45
+		return $statsHandler->update($id, $type, $increment);
46
+	}
47 47
 
48
-    /*
48
+	/*
49 49
     * Gets the total number of topics in a form
50 50
     */
51
-    /**
52
-     * @param  string $forum_id
53
-     * @return mixed
54
-     */
55
-    function getTotalTopics($forum_id = '')
56
-    {
57
-        /** @var Newbb\TopicHandler $topicHandler */
58
-        $topicHandler = Newbb\Helper::getInstance()->getHandler('Topic');
59
-        $criteria     = new \CriteriaCompo(new \Criteria('approved', 0, '>'));
60
-        if ($forum_id) {
61
-            $criteria->add(new \Criteria('forum_id', (int)$forum_id));
62
-        }
51
+	/**
52
+	 * @param  string $forum_id
53
+	 * @return mixed
54
+	 */
55
+	function getTotalTopics($forum_id = '')
56
+	{
57
+		/** @var Newbb\TopicHandler $topicHandler */
58
+		$topicHandler = Newbb\Helper::getInstance()->getHandler('Topic');
59
+		$criteria     = new \CriteriaCompo(new \Criteria('approved', 0, '>'));
60
+		if ($forum_id) {
61
+			$criteria->add(new \Criteria('forum_id', (int)$forum_id));
62
+		}
63 63
 
64
-        return $topicHandler->getCount($criteria);
65
-    }
64
+		return $topicHandler->getCount($criteria);
65
+	}
66 66
 
67
-    /*
67
+	/*
68 68
     * Returns the total number of posts in the whole system, a forum, or a topic
69 69
     * Also can return the number of users on the system.
70 70
     */
71
-    /**
72
-     * @param  int    $id
73
-     * @param  string $type
74
-     * @return mixed
75
-     */
76
-    function getTotalPosts($id = 0, $type = 'all')
77
-    {
78
-        /** @var Newbb\PostHandler $postHandler */
79
-        $postHandler = Newbb\Helper::getInstance()->getHandler('Post');
80
-        $criteria    = new \CriteriaCompo(new \Criteria('approved', 0, '>'));
81
-        switch ($type) {
82
-            case 'forum':
83
-                if ($id > 0) {
84
-                    $criteria->add(new \Criteria('forum_id', (int)$id));
85
-                }
86
-                break;
87
-            case 'topic':
88
-                if ($id > 0) {
89
-                    $criteria->add(new \Criteria('topic_id', (int)$id));
90
-                }
91
-                break;
92
-            case 'all':
93
-            default:
94
-                break;
95
-        }
71
+	/**
72
+	 * @param  int    $id
73
+	 * @param  string $type
74
+	 * @return mixed
75
+	 */
76
+	function getTotalPosts($id = 0, $type = 'all')
77
+	{
78
+		/** @var Newbb\PostHandler $postHandler */
79
+		$postHandler = Newbb\Helper::getInstance()->getHandler('Post');
80
+		$criteria    = new \CriteriaCompo(new \Criteria('approved', 0, '>'));
81
+		switch ($type) {
82
+			case 'forum':
83
+				if ($id > 0) {
84
+					$criteria->add(new \Criteria('forum_id', (int)$id));
85
+				}
86
+				break;
87
+			case 'topic':
88
+				if ($id > 0) {
89
+					$criteria->add(new \Criteria('topic_id', (int)$id));
90
+				}
91
+				break;
92
+			case 'all':
93
+			default:
94
+				break;
95
+		}
96 96
 
97
-        return $postHandler->getCount($criteria);
98
-    }
97
+		return $postHandler->getCount($criteria);
98
+	}
99 99
 
100
-    /**
101
-     * @return null
102
-     */
103
-    function getTotalViews()
104
-    {
105
-        $sql = 'SELECT sum(topic_views) FROM ' . $GLOBALS['xoopsDB']->prefix('newbb_topics') . ' ';
106
-        if (!$result = $GLOBALS['xoopsDB']->query($sql)) {
107
-            return null;
108
-        }
109
-        list($total) = $GLOBALS['xoopsDB']->fetchRow($result);
100
+	/**
101
+	 * @return null
102
+	 */
103
+	function getTotalViews()
104
+	{
105
+		$sql = 'SELECT sum(topic_views) FROM ' . $GLOBALS['xoopsDB']->prefix('newbb_topics') . ' ';
106
+		if (!$result = $GLOBALS['xoopsDB']->query($sql)) {
107
+			return null;
108
+		}
109
+		list($total) = $GLOBALS['xoopsDB']->fetchRow($result);
110 110
 
111
-        return $total;
112
-    }
111
+		return $total;
112
+	}
113 113
 }
Please login to merge, or discard this patch.