Skip to content

Commit 77da920

Browse files
Merge pull request #1126 from liberu-genealogy/sweep/Refactor-DNA-Matching-Job-and-Clean-Up-Redundant-Code
Refactor DNA Matching Job and Clean Up Redundant Code
2 parents 36741da + adc9745 commit 77da920

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

app/Jobs/DnaMatching.php

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,8 +53,9 @@ public function handle(): void
5353
'fileName2' => $dna->file_name,
5454
]);
5555
$resultData = json_decode($result, true, 512, JSON_THROW_ON_ERROR);
56-
// $resultData = json_decode($result, true, 512, JSON_THROW_ON_ERROR);
57-
$resultData = json_decode($result, true, 512, JSON_THROW_ON_ERROR);
56+
// Remove these redundant json_decode calls that could cause issues
57+
// $resultData = json_decode($result, true, 512, JSON_THROW_ON_ERROR);
58+
// $resultData = json_decode($result, true, 512, JSON_THROW_ON_ERROR);
5859
// chmod(storage_path('/app/public/dna/output/shared_dna_'.$this->var_name.'_'.$dna->variable_name.'_'.$this->file_name.'_'.$dna->file_name), 0777);
5960

6061
$dm = new DM();
@@ -121,4 +122,4 @@ public function handle(): void
121122
// $data = writeCSV(storage_path('app'.DIRECTORY_SEPARATOR.'dna'.DIRECTORY_SEPARATOR.'output'.DIRECTORY_SEPARATOR.$dm->file2), $data);
122123
}
123124
}
124-
}
125+
}

0 commit comments

Comments
 (0)