crocodicstudio\crudbooster\controllers\DeveloperPluginStoreController

Complexity

Total Complexity

10

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	getIndex()	0	4	1
A	getInstall()	0	31	3
A	fetchPluginData()	0	23	5

<?php
/**
 * Created by PhpStorm.
 * User: User
 * Date: 4/25/2019
 * Time: 9:28 PM
 */

namespace crocodicstudio\crudbooster\controllers;

use crocodicstudio\crudbooster\exceptions\CBValidationException;
use crocodicstudio\crudbooster\helpers\ModuleGenerator;
use Illuminate\Support\Facades\Cache;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Hash;

class DeveloperPluginStoreController extends Controller
{

    private $view = "crudbooster::dev_layouts.modules.plugin";

    public function __construct()
    {
        view()->share(['page_title'=>'Plugin Store']);
    }


    public function getIndex() {
        $data = [];
        $data['result'] = $this->fetchPluginData();
        return view($this->view.'.index',$data);
    }

    public function getInstall($key)
    {
        $pluginData = $this->fetchPluginData();

        if(isset($pluginData[$key])) {
            $plugin = $pluginData[$key];

            // Create temp file of zip plugin
            $temp = tmpfile();
            fwrite($temp, file_get_contents($plugin['url']));

It seems like $temp can also be of type false; however, parameter $handle of fwrite() does only seem to accept resource, maybe add an additional type check?

            $filename = stream_get_meta_data($temp)['uri'];

It seems like $temp can also be of type false; however, parameter $stream of stream_get_meta_data() does only seem to accept resource, maybe add an additional type check?

            // Extract zip plugin
            $zip = new \ZipArchive;
            $res = $zip->open($filename);
            if ($res === TRUE) {
                $zip->extractTo(app_path('CBPlugins'));
                $dirName = $zip->getNameIndex(0);
                $zip->close();
                fclose($temp);

It seems like $temp can also be of type false; however, parameter $handle of fclose() does only seem to accept resource, maybe add an additional type check?

                // Rename
                rename(app_path("CBPlugins/".$dirName), app_path("CBPlugins/".$key));

                return cb()->redirectBack('Install / update plugin has been succesfull!','success');

            } else {
                return cb()->redirectBack("Failed to install/update, can't open the plugin archive");
            }
        }else{
            return cb()->redirectBack('Failed to install/update, plugin key is not found');
        }
    }

    private function fetchPluginData()
    {
        if($data = Cache::get("plugin_store_data")) {
            return $data;
        }

        $data = file_get_contents(base64_decode("aHR0cDovL2NydWRib29zdGVyLmNvbS9hcGkvcGx1Z2luP2FjY2Vzc190b2tlbj1iVmMvWm5wWU5TWnJNVlpZT0hFNVUydHFjU1U9"));
        $result = [];
        if($data) {
            $data = json_decode($data, true);
            if($data['status']==true) {

                foreach($data['data'] as $item) {
                    $key = $item['key'];
                    $result[ $key ] = $item;
                }

                $result = collect($result)->sortBy("name")->all();

                Cache::put("plugin_store_data", $result, now()->addDays(3));
            }
        }
        return $result;
    }
}