Divine\Engine\Library\Image

Complexity

Total Complexity

3

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	get()	0	3	1
A	__construct()	0	36	2

<?php
/* 	Divine CMS - Open source CMS for widespread use.
    Copyright (c) 2019 Mykola Burakov (burakov.work@gmail.com)

    See SOURCE.txt for other and additional information.

    This file is part of Divine CMS.

    This program is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
    the Free Software Foundation, either version 3 of the License, or
    (at your option) any later version.

    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
    GNU General Public License for more details.

    You should have received a copy of the GNU General Public License
    along with this program. If not, see <http://www.gnu.org/licenses/>. */

namespace Divine\Engine\Library;

// import the Intervention Image Manager Class
use Intervention\Image\ImageManagerStatic as IImage;

class Image
{
    public function __construct(string $file, int $img_width, int $img_height)

Expected 2 blank lines before function; 0 found

    {
        if (file_exists($_SERVER['DOCUMENT_ROOT'] . $file)) {

Blank line found at start of control structure

            // Prepare $file
            $this->file = $file;

The property file does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            // Prepare $img_width
            // self

            // Prepare $img_height
            // self

            // Prepare $watermark
            // if ($watermark) {
            //     $watermark = '/public_html/assets/images/watermark.png';
            // }

            // if (!file_exists($_SERVER['DOCUMENT_ROOT'] . '/temp')) {
            //     mkdir($_SERVER['DOCUMENT_ROOT'] . '/temp', 666, true);
            // }

            // Begin

            // open an image file
            $image = IImage::make($_SERVER['DOCUMENT_ROOT'] . $file)
                // now you are able to resize the instance
                ->resize($img_width, $img_height)
                // and insert a watermark for example
                // ->insert($watermark)
                // finally we save the image as a new file
                ->save($_SERVER['DOCUMENT_ROOT'] . '/temp' . $file);

            $this->image = $image;

The property image does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        } else {
            exit('Error: Could not load image ' . $file . '!');

Using exit here is not recommended.

        }
    }
    public static function get()
    {
        return $this->image;

Using $this inside a static method is generally not recommended and can lead to errors in newer PHP versions.

    }
}