alexmanno / trucker

src/Trucker/Bootstrapper.php

<?php

namespace Trucker;

use Illuminate\Container\Container;

/**
 * Finds configurations and paths.
 *
 * @author Alessandro Manno <alessandromanno96@gmail.com>
 */
class Bootstrapper
{
    /**
     * The Container.
     *
     * @var Container
     */
    protected $container;

    /**
     * Build a new Bootstrapper.
     *
     * @param Container $app
     */
    public function __construct(Container $app)
    {
        $this->app = $app;

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

    }

    /**
     * Bind paths to the container.
     */
    public function bindPaths()
    {
        $this->bindBase();
        $this->bindConfiguration();
    }

    /**
     * Get the path to the configuration folder.
     *
     * @return string
     */
    public function getConfigurationPath()
    {
        // Return path to Laravel configuration
        if ($this->app->bound('path')) {
            $laravel = $this->app['path'] . '/config/packages/alexmanno/trucker';
            if (file_exists($laravel)) {
                return $laravel;
            }
        }

        return $this->app['path.trucker.config'];
    }

    /**
     * Export the configuration files.
     */
    public function exportConfiguration()
    {
        $source = __DIR__ . '/../config';
        $destination = $this->getConfigurationPath();

        // Unzip configuration files
        $this->app['files']->copyDirectory($source, $destination);

        return $destination;
    }

    /**
     * Replace placeholders in configuration.
     *
     * @param string $folder
     * @param array  $values
     */
    public function updateConfiguration($folder, array $values = [])
    {
        // Replace stub values in files
        $files = $this->app['files']->files($folder);
        foreach ($files as $file) {
            foreach ($values as $name => $value) {
                $contents = str_replace('{' . $name . '}', $value, file_get_contents($file));
                $this->app['files']->put($file, $contents);
            }
        }
    }

    /**
     * Bind the base path to the Container.
     */
    protected function bindBase()
    {
        if ($this->app->bound('path.base')) {
            return;
        }

        $this->app->instance('path.base', getcwd());
    }

    /**
     * Bind paths to the configuration files.
     */
    protected function bindConfiguration()
    {
        $path = $this->app['path.base'] ? $this->app['path.base'] . '/' : '';
        $logs = $this->app->bound('path.storage') ? str_replace($this->unifySlashes($path), null, $this->unifySlashes($this->app['path.storage'])) : '.trucker';

        $paths = [
            'config' => '.trucker',
            'logs' => $logs . '/logs',
        ];

        foreach ($paths as $key => $file) {
            $filename = $path . $file;

            // Check whether we provided a file or folder
            if (!is_dir($filename) and file_exists($filename . '.php')) {
                $filename .= '.php';
            }

            // Use configuration in current folder if none found
            $realpath = realpath('.') . '/' . $file;

Are you sure realpath('.') of type false|string can be used in concatenation?

            if (!file_exists($filename) and file_exists($realpath)) {
                $filename = $realpath;
            }

            $this->app->instance('path.trucker.' . $key, $filename);
        }
    }

    /**
     * Unify the slashes to the UNIX mode (forward slashes).
     *
     * @param string $path
     *
     * @return string
     */
    protected function unifySlashes($path)
    {
        return str_replace('\\', '/', $path);
    }
}