}
// save style
- if ($style !== null) {
- $style->update($styleData);
- }
- else {
+ if ($style === null) {
$styleData['packageID'] = $packageID;
$style = new StyleEditor(self::create($styleData));
- }
-
- // import preview image
- if (!empty($data['image'])) {
- $fileExtension = mb_substr($data['image'], mb_strrpos($data['image'], '.'));
- $index = $tar->getIndexByFilename($data['image']);
- if ($index !== false) {
- $filename = WCF_DIR.'images/stylePreview-'.$style->styleID.$fileExtension;
- $tar->extract($index, $filename);
- FileUtil::makeWritable($filename);
-
- if (file_exists($filename)) {
- try {
- if (($imageData = getimagesize($filename)) !== false) {
- switch ($imageData[2]) {
- case IMG_PNG:
- case IMG_JPEG:
- case IMG_JPG:
- case IMG_GIF:
- $style->update(array('image' => 'stylePreview-'.$style->styleID.$fileExtension));
+
+ // import preview image
+ if (!empty($data['image'])) {
+ $fileExtension = mb_substr($data['image'], mb_strrpos($data['image'], '.'));
+ $index = $tar->getIndexByFilename($data['image']);
+ if ($index !== false) {
+ $filename = WCF_DIR.'images/stylePreview-'.$style->styleID.$fileExtension;
+ $tar->extract($index, $filename);
+ FileUtil::makeWritable($filename);
+
+ if (file_exists($filename)) {
- $style->update(['image' => 'stylePreview-'.$style->styleID.$fileExtension]);
++ try {
++ if (($imageData = getimagesize($filename)) !== false) {
++ switch ($imageData[2]) {
++ case IMG_PNG:
++ case IMG_JPEG:
++ case IMG_JPG:
++ case IMG_GIF:
++ $style->update(['image' => 'stylePreview-'.$style->styleID.$fileExtension]);
++ }
+ }
+ }
- }
- catch (SystemException $e) {
- // broken image
++ catch (SystemException $e) {
++ // broken image
++ }
}
}
}