Ballybran\Helpers\Log\Log

Complexity

Total Complexity

20

Size/Duplication

Total Lines	112
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	open()	0	23	6
A	Files()	0	3	1
A	__construct()	0	10	2
A	__destruct()	0	3	1
B	write()	0	25	10

<?php

/**
 * KNUT7 K7F (https://marciozebedeu.com/)
 * KNUT7 K7F (tm) : Rapid Development Framework (https://marciozebedeu.com/)
 *
 * Licensed under The MIT License
 * For full copyright and license information, please see the LICENSE.txt
 * Redistributions of files must retain the above copyright notice.
 *
 * @link      https://github.com/knut7/framework/ for the canonical source repository
 * @copyright (c) 2015.  KNUT7  Software Technologies AO Inc. (https://marciozebedeu.com/)
 * @license   https://marciozebedeu.com/license/new-bsd New BSD License
 * @author    Marcio Zebedeu - artphoweb@artphoweb.com
 * @version   1.0.2
 */

namespace Ballybran\Helpers\Log;


/**
 * Class Log
 * @package Ballybran\Helpers\Log
 */
class Log
{

    /**
     * @var
     */
    private $handle;
    /**
     * @var
     */
    private $filename;
    /**
     * @var
     */
    private $handles;

    /**
     *
     * @param type $filename

The type Ballybran\Helpers\Log\type was not found. Maybe you did not declare it correctly or list all dependencies?

     * @param null $dir

Are you sure the doc-type for parameter $dir is correct as it would always require null to be passed?

     */
    public function __construct($filename, $dir = null)
    {

        if ($dir != null) {

The condition $dir != null is always false.

            $this->filename = $dir . $filename;

        } else {

            $this->filename = DIR_LOGS . $filename;
        }
    }

    /**
     *
     * @param type $message
     * @param null $type

Are you sure the doc-type for parameter $type is correct as it would always require null to be passed?

     */
    public function write($message, $type = null)
    {
        if ($this->handle = fopen($this->filename, 'a')) {
            if (is_array($message) && $type === 'A') {

The condition is_array($message) is always false.

                foreach ($message as $key => $value) {
                    fwrite($this->handle, date('Y-m-d G:i:s') . ' - ' . print_r($key . " -> " . $value, true) . "\n");

                }
            }
            if (!empty($message)) {
                fwrite($this->handle, date('Y-m-d G:i:s') . ' - ' . print_r($message, true) . "\n");

Are you sure print_r($message, true) of type string|true can be used in concatenation?

            }
            // to use for get data

            if (is_array($message) && $type == 'F') {

The condition is_array($message) is always false.

                foreach ($message as $key => $value) {
                    fwrite($this->handle, date('Y-m-d G:i:s') . ';' . print_r($key . "=>" . $value, true) . "\n") . "<br>";

                }
            }
        }
        fclose($this->handle);
        if ($this->filename == true) {

It seems like you are loosely comparing $this->filename of type string to the boolean true. If you are specifically checking for a non-empty string, consider using the more explicit !== '' instead.

            chmod($this->filename, 0755);
        }
    }

    /**
     *
     */
    public function open()
    {
        if (file_exists($this->filename) && is_readable($this->filename) && $this->handles = fopen($this->filename, 'r')) {
            require_once DIR_FILE . 'View/header.phtml';
            ?>
            <div class="well">
                <ul>
                    <?php
                    while (!feof($this->handles)) {
                        $content = fgets($this->handles);

                        if (trim($content) != "") {
                            echo "<li style='color: blue'>$content</li>";
                        }
                    } ?>

                </ul>
            </div>
            <?php
            fclose($this->handles);

        } else {
            echo "Could not read from {$this->filename}";
        }
    }

    /**
     *
     */
    public function __destruct()
    {
        $this->filename;
    }

    /**
     *
     * @param type $files
     * @return type
     */
    public function Files($files)
    {
        return $this->files = $files;

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

    }

}