App\Console\Commands\PT::handle()

Complexity

Conditions	14
Paths	28

Size

Total Lines	98
Code Lines	46

Duplication

Lines	5
Ratio	5.1 %

Importance

Changes

0

<?php
    public function handle()
    {
        $this->info('Lade Gamefiles ohne index.json');

        //Get all game files
        $gamefiles = GamesFile::all();

        $counter = 0;

Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space

        $toindexed = [];

        //loop all gamefiles
        foreach ($gamefiles as $gamefile) {
            echo $gamefile->id.PHP_EOL;
            //Get the maker id
            $makerid = $gamefile->game()->first()->maker_id;

            //Run code only for rm2k(3)
            //Todo: Engine filter can be done with Query

Comments for TODO tasks are often forgotten in the code; it might be better to use a dedicated issue tracker.

            if ($makerid == 2 or $makerid == 3 or $makerid == 9) {

Using logical operators such as or instead of || is generally not recommended.

                //Get path to uploaded files
                $path = storage_path('app/public/'.$gamefile->filename);

                //use only zip fiels
                //Todo: ZIP filter can be done with Query

Comments for TODO tasks are often forgotten in the code; it might be better to use a dedicated issue tracker.

                if ($gamefile->extension == 'zip') {
                    //Open the ZIP file
                    $zip = new \ZipArchive();
                    $zip->open($path);

                    //Run through all files in ZIP
                    for ($i = 0; $i < $zip->numFiles; $i++) {
                        //Get the filename with fileindex
                        $filename = $zip->getNameIndex($i);

                        //Filter Directory and _MACOSX from index
                        if (! ends_with($filename, '/') and ! starts_with($filename, '_MACOSX')) {

Using logical operators such as and instead of && is generally not recommended.

                            //Get root path of the file
                            $phelper = new PlayerHelper();
                            $imp = $phelper->getZipRootPath($filename);

Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space

                            //if root path not ''
                            if (! $imp == '') {
                                $rel = new PlayerFileGamefileRel();

Equals sign not aligned with surrounding assignments; expected 14 spaces but found 1 space

                                $rel->gamefile_id = $gamefile->id;

                                if (! ends_with(strtolower($imp), ['.exe', '.lmu', '.ldb', 'ini', '.dll', 'lmt', 'lsd'])) {

This code seems to be duplicated across your project.

                                    $rel->orig_filename = preg_replace('/(\.\w+$)/', '', strtolower($imp));
                                } else {
                                    $rel->orig_filename = strtolower($imp);
                                }

                                //Decompress data
                                $filedata = $zip->getFromIndex($i);
                                //create hash of the file
                                $filehash = hash('sha1', $filedata);

                                //get storage path to hashed directory
                                $newfilepath = storage_path('app/public/games_hashed/'.substr($filehash, 0, 2).'/');

                                //check for directory existance
                                if (! file_exists($newfilepath)) {
                                    mkdir($newfilepath);
                                }

                                //write decompressed data to a new file to the storage path
                                file_put_contents($newfilepath.$filehash, $filedata);

                                //check for Database existance of this file
                                $check = PlayerFileHash::whereFilehash($filehash)->first();
                                if (! $check) {
                                    //create a new record to player_file_hash table
                                    $pfh = new PlayerFileHash();

Equals sign not aligned with surrounding assignments; expected 11 spaces but found 1 space

                                    $pfh->filehash = $filehash;
                                    $pfh->save();

                                    $rel->file_hash_id = $pfh->id;
                                } else {
                                    $rel->file_hash_id = $check->id;
                                }

                                $rel->save();
                            }
                        }
                    }
                    $zip->close();
                } else {
                    continue;
                }
            }
        }

        $this->info('Es wurden '.$counter.' Gamefiles gefunden.');

        $i = 0;

$i is not used, you could remove the assignment.

        foreach ($toindexed as $toindex) {

This foreach statement is empty and can be removed.

        }
    }
}