execut\import\components\source\File

Complexity

Total Complexity

8

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setContent()	0	4	2
A	__destruct()	0	5	3
A	getContent()	0	6	3

<?php
/**
 * Created by PhpStorm.
 * User: execut
 * Date: 9/27/16
 * Time: 2:36 PM
 */

namespace execut\import\components\source;


use yii\base\Component;

class File extends Component
{
    public $fileName = null;
    public $filePath = null;
    public $folder = null;
    protected $content = null;
    public function getContent() {
        if ($this->filePath !== null && $this->content === null) {
            return file_get_contents($this->filePath);
        }

        return $this->content;
    }

    public function setContent($content) {
        $this->content = $content;
        if ($this->filePath !== null) {
            file_put_contents($this->filePath, $this->content);
        }
    }

    public function __destruct()
    {
        if ($this->filePath !== null && file_exists($this->filePath)) {
            unlink($this->filePath);
            @rmdir(dirname($this->filePath));

It seems like you do not handle an error condition for rmdir(). This can introduce security issues, and is generally not recommended.

        }
    }
}