filemanager::get_upload_destination() - Code Metrics - Inspection of "Remove Responsive filemanager" - blitze/phpBB-ext-sitemaker - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — develop ( 834258...491c3e )

by Daniel

created 2020-09-18 23:41 UTC

filemanager::get_upload_destination() A

↳ Parent: filemanager

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	1
nc	1
nop	0
dl	0
loc	3
rs	10
c	1
b	0
f	0

<?php

/**
 *
 * @package sitemaker
 * @copyright (c) 2017 Daniel A. (blitze)
 * @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2
 *
 */

namespace blitze\sitemaker\services;

/**
 * @package sitemaker
 */
class filemanager
{
    /** @var \phpbb\filesystem\filesystem */
    protected $filesystem;

    /** @var \phpbb\user */
    protected $user;

    /** @var string */
    protected $phpbb_root_path;

    /** @var string */
    protected $user_dir = false;

    /** @var string */
    protected $upload_dir = 'images/sitemaker_uploads/source/';

    /**
     * Constructor
     *
     * @param \phpbb\filesystem\filesystem	$filesystem			File system
     * @param \phpbb\user					$user				User object
     * @param string						$phpbb_root_path	phpBB root path
     */
    public function __construct(\phpbb\filesystem\filesystem $filesystem, \phpbb\user $user, $phpbb_root_path)
    {
        $this->filesystem = $filesystem;
        $this->user = $user;
        $this->phpbb_root_path = $phpbb_root_path;
    }

    /**
     * @return string
     */
    public function get_upload_dir()
    {
        return $this->upload_dir;
    }

    /**
     * @return string
     */
    public function get_upload_destination()
    {
        return $this->upload_dir . $this->get_user_dir();
    }

    /**
     * @return string
     */
    public function get_user_dir()
    {
        if (!$this->user_dir)
        {
            $this->set_user_dir();
        }

        return $this->user_dir;
    }

    /**
     * @return string
     */
    protected function set_user_dir()
    {
        // if user is not founder, they must have a directory
        $this->user_dir = ($this->user->data['user_type'] != USER_FOUNDER) ? 'users/' . $this->user->data['username'] . '/' : '';

        $destination = $this->phpbb_root_path . $this->upload_dir . $this->user_dir;

        if (!is_dir($destination))
        {
            $this->filesystem->mkdir($destination, 0755);
        }
    }
}


1			<?php
2
3			/**
4			*
5			* @package sitemaker
6			* @copyright (c) 2017 Daniel A. (blitze)
7			* @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2
8			*
9			*/
10
11			namespace blitze\sitemaker\services;
12
13			/**
14			* @package sitemaker
15			*/
16			class filemanager
17			{
18			/** @var \phpbb\filesystem\filesystem */
19			protected $filesystem;
20
21			/** @var \phpbb\user */
22			protected $user;
23
24			/** @var string */
25			protected $phpbb_root_path;
26
27			/** @var string */
28			protected $user_dir = false;
29
30			/** @var string */
31			protected $upload_dir = 'images/sitemaker_uploads/source/';
32
33			/**
34			* Constructor
35			*
36			* @param \phpbb\filesystem\filesystem $filesystem File system
37			* @param \phpbb\user $user User object
38			* @param string $phpbb_root_path phpBB root path
39			*/
40			public function __construct(\phpbb\filesystem\filesystem $filesystem, \phpbb\user $user, $phpbb_root_path)
41			{
42			$this->filesystem = $filesystem;
43			$this->user = $user;
44			$this->phpbb_root_path = $phpbb_root_path;
45			}
46
47			/**
48			* @return string
49			*/
50			public function get_upload_dir()
51			{
52			return $this->upload_dir;
53			}
54
55			/**
56			* @return string
57			*/
58			public function get_upload_destination()
59			{
60			return $this->upload_dir . $this->get_user_dir();
61			}
62
63			/**
64			* @return string
65			*/
66			public function get_user_dir()
67			{
68			if (!$this->user_dir)
69			{
70			$this->set_user_dir();
71			}
72
73			return $this->user_dir;
74			}
75
76			/**
77			* @return string
78			*/
79			protected function set_user_dir()
80			{
81			// if user is not founder, they must have a directory
82			$this->user_dir = ($this->user->data['user_type'] != USER_FOUNDER) ? 'users/' . $this->user->data['username'] . '/' : '';
83
84			$destination = $this->phpbb_root_path . $this->upload_dir . $this->user_dir;
85
86			if (!is_dir($destination))
87			{
88			$this->filesystem->mkdir($destination, 0755);
89			}
90			}
91			}
92

blitze / phpBB-ext-sitemaker

Push — develop ( 834258...491c3e )

filemanager::get_upload_destination() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like