mambax7 /
smartfaq
| 1 | <?php |
|||||||||||
| 2 | /* |
|||||||||||
| 3 | * You may not change or alter any portion of this comment or credits |
|||||||||||
| 4 | * of supporting developers from this source code or any supporting source code |
|||||||||||
| 5 | * which is considered copyrighted (c) material of the original comment or credit authors. |
|||||||||||
| 6 | * |
|||||||||||
| 7 | * This program is distributed in the hope that it will be useful, |
|||||||||||
| 8 | * but WITHOUT ANY WARRANTY; without even the implied warranty of |
|||||||||||
| 9 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. |
|||||||||||
| 10 | */ |
|||||||||||
| 11 | ||||||||||||
| 12 | /** |
|||||||||||
| 13 | * @copyright {@link https://xoops.org/ XOOPS Project} |
|||||||||||
| 14 | * @license {@link http://www.gnu.org/licenses/gpl-2.0.html GNU GPL 2 or later} |
|||||||||||
| 15 | * @package |
|||||||||||
| 16 | * @since |
|||||||||||
| 17 | * @author XOOPS Development Team, phppp (D.J., [email protected]) |
|||||||||||
| 18 | */ |
|||||||||||
| 19 | ||||||||||||
| 20 | use XoopsModules\Smartfaq; |
|||||||||||
| 21 | /** @var Smartfaq\Helper $helper */ |
|||||||||||
| 22 | $helper = Smartfaq\Helper::getInstance(); |
|||||||||||
| 23 | ||||||||||||
| 24 | if (!defined('NEWBB_FUNCTIONS_IMAGE')) : |
|||||||||||
| 25 | define('NEWBB_FUNCTIONS_IMAGE', true); |
|||||||||||
| 26 | ||||||||||||
| 27 | /** |
|||||||||||
| 28 | * @param $source |
|||||||||||
| 29 | * @return string |
|||||||||||
| 30 | */ |
|||||||||||
| 31 | function sf_attachmentImage($source) |
|||||||||||
| 32 | { |
|||||||||||
| 33 | /** @var Smartfaq\Helper $helper */ |
|||||||||||
| 34 | $helper = Smartfaq\Helper::getInstance(); |
|||||||||||
| 35 | ||||||||||||
| 36 | $img_path = XOOPS_ROOT_PATH . '/' . $helper->getConfig('dir_attachments'); |
|||||||||||
|
0 ignored issues
–
show
Bug
introduced
by
Loading history...
|
||||||||||||
| 37 | $img_url = XOOPS_URL . '/' . $helper->getConfig('dir_attachments'); |
|||||||||||
|
0 ignored issues
–
show
|
||||||||||||
| 38 | $thumb_path = $img_path . '/thumbs'; |
|||||||||||
| 39 | $thumb_url = $img_url . '/thumbs'; |
|||||||||||
| 40 | ||||||||||||
| 41 | $thumb = $thumb_path . '/' . $source; |
|||||||||||
| 42 | $image = $img_path . '/' . $source; |
|||||||||||
| 43 | $thumb_url = $thumb_url . '/' . $source; |
|||||||||||
| 44 | $image_url = $img_url . '/' . $source; |
|||||||||||
| 45 | ||||||||||||
| 46 | $imginfo = @getimagesize($image); |
|||||||||||
| 47 | $img_info = (count($imginfo) > 0) ? $imginfo[0] . 'X' . $imginfo[1] . ' px' : ''; |
|||||||||||
| 48 | ||||||||||||
| 49 | if ($helper->getConfig('max_image_width') > 0 && $helper->getConfig('max_image_height') > 0) { |
|||||||||||
| 50 | if ($imginfo[0] > $helper->getConfig('max_image_width') |
|||||||||||
| 51 | || $imginfo[1] > $helper->getConfig('max_image_height')) { |
|||||||||||
| 52 | //if (!file_exists($thumb_path.'/'.$source) && $imginfo[0] > $helper->getConfig('max_img_width')) { |
|||||||||||
| 53 | if (!file_exists($thumb_path . '/' . $source)) { |
|||||||||||
| 54 | sf_createThumbnail($source, $helper->getConfig('max_image_width')); |
|||||||||||
| 55 | } |
|||||||||||
| 56 | } |
|||||||||||
| 57 | ||||||||||||
| 58 | if ($imginfo[0] > $helper->getConfig('max_image_width') |
|||||||||||
| 59 | || $imginfo[1] > $helper->getConfig('max_image_height')) { |
|||||||||||
| 60 | $pseudo_width = $helper->getConfig('max_image_width'); |
|||||||||||
| 61 | $pseudo_height = $helper->getConfig('max_image_width') * ($imginfo[1] / $imginfo[0]); |
|||||||||||
| 62 | $pseudo_size = "width='" . $pseudo_width . "px' height='" . $pseudo_height . "px'"; |
|||||||||||
| 63 | } |
|||||||||||
| 64 | // irmtfan to fix Undefined variable: pseudo_height |
|||||||||||
| 65 | if (!empty($pseudo_height) && $helper->getConfig('max_image_height') > 0 |
|||||||||||
| 66 | && $pseudo_height > $helper->getConfig('max_image_height')) { |
|||||||||||
| 67 | $pseudo_height = $helper->getConfig('max_image_height'); |
|||||||||||
| 68 | $pseudo_width = $helper->getConfig('max_image_height') * ($imginfo[0] / $imginfo[1]); |
|||||||||||
| 69 | $pseudo_size = "width='" . $pseudo_width . "px' height='" . $pseudo_height . "px'"; |
|||||||||||
| 70 | } |
|||||||||||
| 71 | } |
|||||||||||
| 72 | ||||||||||||
| 73 | if (file_exists($thumb)) { |
|||||||||||
| 74 | $attachmentImage = '<a href="' . $image_url . '" title="' . $source . ' ' . $img_info . '" target="_blank">'; |
|||||||||||
| 75 | $attachmentImage .= '<img src="' . $thumb_url . '" alt="' . $source . ' ' . $img_info . '">'; |
|||||||||||
| 76 | $attachmentImage .= '</a>'; |
|||||||||||
| 77 | } elseif (!empty($pseudo_size)) { |
|||||||||||
| 78 | $attachmentImage = '<a href="' . $image_url . '" title="' . $source . ' ' . $img_info . '" target="_blank">'; |
|||||||||||
| 79 | $attachmentImage .= '<img src="' . $image_url . '" ' . $pseudo_size . ' alt="' . $source . ' ' . $img_info . '">'; |
|||||||||||
| 80 | $attachmentImage .= '</a>'; |
|||||||||||
| 81 | } elseif (file_exists($image)) { |
|||||||||||
| 82 | $attachmentImage = '<img src="' . $image_url . '" alt="' . $source . ' ' . $img_info . '">'; |
|||||||||||
| 83 | } else { |
|||||||||||
| 84 | $attachmentImage = ''; |
|||||||||||
| 85 | } |
|||||||||||
| 86 | ||||||||||||
| 87 | return $attachmentImage; |
|||||||||||
| 88 | } |
|||||||||||
| 89 | ||||||||||||
| 90 | /** |
|||||||||||
| 91 | * @param $source |
|||||||||||
| 92 | * @param $thumb_width |
|||||||||||
| 93 | * @return bool |
|||||||||||
| 94 | */ |
|||||||||||
| 95 | function sf_createThumbnail($source, $thumb_width) |
|||||||||||
| 96 | { |
|||||||||||
| 97 | /** @var Smartfaq\Helper $helper */ |
|||||||||||
| 98 | $helper = Smartfaq\Helper::getInstance(); |
|||||||||||
| 99 | ||||||||||||
| 100 | $img_path = XOOPS_ROOT_PATH . '/' . $helper->getConfig('dir_attachments'); |
|||||||||||
|
0 ignored issues
–
show
|
||||||||||||
| 101 | $thumb_path = $img_path . '/thumbs'; |
|||||||||||
| 102 | $src_file = $img_path . '/' . $source; |
|||||||||||
| 103 | $new_file = $thumb_path . '/' . $source; |
|||||||||||
| 104 | //$imageLibs = sf_getImageLibs(); |
|||||||||||
| 105 | ||||||||||||
| 106 | if (!filesize($src_file) || !is_readable($src_file)) { |
|||||||||||
| 107 | return false; |
|||||||||||
| 108 | } |
|||||||||||
| 109 | ||||||||||||
| 110 | if (!is_dir($thumb_path) || !is_writable($thumb_path)) { |
|||||||||||
| 111 | return false; |
|||||||||||
| 112 | } |
|||||||||||
| 113 | ||||||||||||
| 114 | $imginfo = @getimagesize($src_file); |
|||||||||||
| 115 | ||||||||||||
| 116 | if (null === $imginfo) { |
|||||||||||
| 117 | return false; |
|||||||||||
| 118 | } |
|||||||||||
| 119 | if ($imginfo[0] < $thumb_width) { |
|||||||||||
| 120 | return false; |
|||||||||||
| 121 | } |
|||||||||||
| 122 | ||||||||||||
| 123 | $newWidth = (int)min($imginfo[0], $thumb_width); |
|||||||||||
| 124 | $newHeight = (int)($imginfo[1] * $newWidth / $imginfo[0]); |
|||||||||||
| 125 | ||||||||||||
| 126 | if (1 == $helper->getConfig('image_lib') or 0 == $helper->getConfig('image_lib')) { |
|||||||||||
|
0 ignored issues
–
show
Comprehensibility
Best Practice
introduced
by
Using logical operators such as
or instead of || is generally not recommended.
PHP has two types of connecting operators (logical operators, and boolean operators):
The difference between these is the order in which they are executed. In most cases,
you would want to use a boolean operator like Let’s take a look at a few examples: // Logical operators have lower precedence:
$f = false or true;
// is executed like this:
($f = false) or true;
// Boolean operators have higher precedence:
$f = false || true;
// is executed like this:
$f = (false || true);
Logical Operators are used for Control-FlowOne case where you explicitly want to use logical operators is for control-flow such as this: $x === 5
or die('$x must be 5.');
// Instead of
if ($x !== 5) {
die('$x must be 5.');
}
Since // The following is currently a parse error.
$x === 5
or throw new RuntimeException('$x must be 5.');
These limitations lead to logical operators rarely being of use in current PHP code. Loading history...
|
||||||||||||
| 127 | if (preg_match("#[A-Z]:|\\\\#Ai", __FILE__)) { |
|||||||||||
| 128 | $cur_dir = __DIR__; |
|||||||||||
| 129 | $src_file_im = '"' . $cur_dir . '\\' . str_replace('/', '\\', $src_file) . '"'; |
|||||||||||
| 130 | $new_file_im = '"' . $cur_dir . '\\' . str_replace('/', '\\', $new_file) . '"'; |
|||||||||||
| 131 | } else { |
|||||||||||
| 132 | $src_file_im = @escapeshellarg($src_file); |
|||||||||||
| 133 | $new_file_im = @escapeshellarg($new_file); |
|||||||||||
| 134 | } |
|||||||||||
| 135 | $path = empty($helper->getConfig('path_magick')) ? '' : $helper->getConfig('path_magick') . '/'; |
|||||||||||
| 136 | $magick_command = $path . 'convert -quality 85 -antialias -sample ' . $newWidth . 'x' . $newHeight . ' ' . $src_file_im . ' +profile "*" ' . str_replace('\\', '/', $new_file_im) . ''; |
|||||||||||
| 137 | ||||||||||||
| 138 | @passthru($magick_command); |
|||||||||||
| 139 | if (file_exists($new_file)) { |
|||||||||||
| 140 | return true; |
|||||||||||
| 141 | } |
|||||||||||
| 142 | } |
|||||||||||
| 143 | ||||||||||||
| 144 | if (2 == $helper->getConfig('image_lib') or 0 == $helper->getConfig('image_lib')) { |
|||||||||||
|
0 ignored issues
–
show
Comprehensibility
Best Practice
introduced
by
Using logical operators such as
or instead of || is generally not recommended.
PHP has two types of connecting operators (logical operators, and boolean operators):
The difference between these is the order in which they are executed. In most cases,
you would want to use a boolean operator like Let’s take a look at a few examples: // Logical operators have lower precedence:
$f = false or true;
// is executed like this:
($f = false) or true;
// Boolean operators have higher precedence:
$f = false || true;
// is executed like this:
$f = (false || true);
Logical Operators are used for Control-FlowOne case where you explicitly want to use logical operators is for control-flow such as this: $x === 5
or die('$x must be 5.');
// Instead of
if ($x !== 5) {
die('$x must be 5.');
}
Since // The following is currently a parse error.
$x === 5
or throw new RuntimeException('$x must be 5.');
These limitations lead to logical operators rarely being of use in current PHP code. Loading history...
|
||||||||||||
| 145 | $path = empty($helper->getConfig('path_netpbm')) ? '' : $helper->getConfig('path_netpbm') . '/'; |
|||||||||||
| 146 | if (preg_match("/\.png/i", $source)) { |
|||||||||||
| 147 | $cmd = $path . "pngtopnm $src_file | " . $path . "pnmscale -xysize $newWidth $newHeight | " . $path . "pnmtopng > $new_file"; |
|||||||||||
| 148 | } elseif (preg_match("/\.(jpg|jpeg)/i", $source)) { |
|||||||||||
| 149 | $cmd = $path . "jpegtopnm $src_file | " . $path . "pnmscale -xysize $newWidth $newHeight | " . $path . "ppmtojpeg -quality=90 > $new_file"; |
|||||||||||
| 150 | } elseif (preg_match("/\.gif/i", $source)) { |
|||||||||||
| 151 | $cmd = $path . "giftopnm $src_file | " . $path . "pnmscale -xysize $newWidth $newHeight | ppmquant 256 | " . $path . "ppmtogif > $new_file"; |
|||||||||||
| 152 | } |
|||||||||||
| 153 | ||||||||||||
| 154 | @exec($cmd, $output, $retval); |
|||||||||||
| 155 | if (file_exists($new_file)) { |
|||||||||||
| 156 | return true; |
|||||||||||
| 157 | } |
|||||||||||
| 158 | } |
|||||||||||
| 159 | ||||||||||||
| 160 | $type = $imginfo[2]; |
|||||||||||
| 161 | $supported_types = []; |
|||||||||||
| 162 | ||||||||||||
| 163 | if (!extension_loaded('gd')) { |
|||||||||||
| 164 | return false; |
|||||||||||
| 165 | } |
|||||||||||
| 166 | if (function_exists('imagegif')) { |
|||||||||||
| 167 | $supported_types[] = 1; |
|||||||||||
| 168 | } |
|||||||||||
| 169 | if (function_exists('imagejpeg')) { |
|||||||||||
| 170 | $supported_types[] = 2; |
|||||||||||
| 171 | } |
|||||||||||
| 172 | if (function_exists('imagepng')) { |
|||||||||||
| 173 | $supported_types[] = 3; |
|||||||||||
| 174 | } |
|||||||||||
| 175 | ||||||||||||
| 176 | $imageCreateFunction = function_exists('imagecreatetruecolor') ? 'imagecreatetruecolor' : 'imagecreate'; |
|||||||||||
| 177 | ||||||||||||
| 178 | if (in_array($type, $supported_types)) { |
|||||||||||
| 179 | switch ($type) { |
|||||||||||
| 180 | case 1: |
|||||||||||
| 181 | if (!function_exists('imagecreatefromgif')) { |
|||||||||||
| 182 | return false; |
|||||||||||
| 183 | } |
|||||||||||
| 184 | $im = imagecreatefromgif($src_file); |
|||||||||||
| 185 | $new_im = imagecreate($newWidth, $newHeight); |
|||||||||||
| 186 | imagecopyresized($new_im, $im, 0, 0, 0, 0, $newWidth, $newHeight, $imginfo[0], $imginfo[1]); |
|||||||||||
| 187 | imagegif($new_im, $new_file); |
|||||||||||
| 188 | imagedestroy($im); |
|||||||||||
| 189 | imagedestroy($new_im); |
|||||||||||
| 190 | break; |
|||||||||||
| 191 | case 2: |
|||||||||||
| 192 | $im = imagecreatefromjpeg($src_file); |
|||||||||||
| 193 | $new_im = $imageCreateFunction($newWidth, $newHeight); |
|||||||||||
| 194 | imagecopyresized($new_im, $im, 0, 0, 0, 0, $newWidth, $newHeight, $imginfo[0], $imginfo[1]); |
|||||||||||
| 195 | imagejpeg($new_im, $new_file, 90); |
|||||||||||
| 196 | imagedestroy($im); |
|||||||||||
| 197 | imagedestroy($new_im); |
|||||||||||
| 198 | break; |
|||||||||||
| 199 | case 3: |
|||||||||||
| 200 | $im = imagecreatefrompng($src_file); |
|||||||||||
| 201 | $new_im = $imageCreateFunction($newWidth, $newHeight); |
|||||||||||
| 202 | imagecopyresized($new_im, $im, 0, 0, 0, 0, $newWidth, $newHeight, $imginfo[0], $imginfo[1]); |
|||||||||||
| 203 | imagepng($new_im, $new_file); |
|||||||||||
| 204 | imagedestroy($im); |
|||||||||||
| 205 | imagedestroy($new_im); |
|||||||||||
| 206 | break; |
|||||||||||
| 207 | } |
|||||||||||
| 208 | } |
|||||||||||
| 209 | ||||||||||||
| 210 | if (file_exists($new_file)) { |
|||||||||||
| 211 | return true; |
|||||||||||
| 212 | } else { |
|||||||||||
| 213 | return false; |
|||||||||||
| 214 | } |
|||||||||||
| 215 | } |
|||||||||||
| 216 | ||||||||||||
| 217 | endif; |
|||||||||||
| 218 |