refactor(Lpj): optimalkan kode untuk peningkatan efisiensi
- Ganti `isNumeric` dengan `ctype_digit` dan hapus fungsi `isNumeric` yang tidak diperlukan. - Simplifikasi fungsi `onRomawi` dengan memanfaatkan fungsi `convertToRoman`. - Hapus fungsi `holidays` karena tidak digunakan lagi dalam kode. Signed-off-by: Daeng Deni Mardaeni <ddeni05@gmail.com>
This commit is contained in:
@@ -134,7 +134,7 @@
|
|||||||
if ($penawaran) {
|
if ($penawaran) {
|
||||||
$isNum = substr($maxCode, 2); // memastikan string ke 3 s/d 8 adalan numiric
|
$isNum = substr($maxCode, 2); // memastikan string ke 3 s/d 8 adalan numiric
|
||||||
$isNP = substr($maxCode, 0, 2);
|
$isNP = substr($maxCode, 0, 2);
|
||||||
if ((8 == strlen($maxCode)) && ("NP" == $isNP) && (isNumeric($isNum))) {
|
if ((8 == strlen($maxCode)) && ("NP" == $isNP) && (ctype_digit($isNum))) {
|
||||||
$code_penawaran_last = substr($maxCode, -4);
|
$code_penawaran_last = substr($maxCode, -4);
|
||||||
$year_penawaran_last = Carbon::parse($penawaran->created_at)->year;
|
$year_penawaran_last = Carbon::parse($penawaran->created_at)->year;
|
||||||
$year_now = Carbon::now()->year;
|
$year_now = Carbon::now()->year;
|
||||||
@@ -149,11 +149,6 @@
|
|||||||
return 'NP' . Carbon::now()->format('y') . $noUrutAkhirString;
|
return 'NP' . Carbon::now()->format('y') . $noUrutAkhirString;
|
||||||
}
|
}
|
||||||
|
|
||||||
function isNumeric($str)
|
|
||||||
{
|
|
||||||
return ctype_digit($str);
|
|
||||||
}
|
|
||||||
|
|
||||||
// generate last penawaran.no_spk
|
// generate last penawaran.no_spk
|
||||||
function onLastnumberCodePenawaranSPK($jenis_laporan_code)
|
function onLastnumberCodePenawaranSPK($jenis_laporan_code)
|
||||||
: string
|
: string
|
||||||
@@ -188,46 +183,7 @@
|
|||||||
function onRomawi(int $bln)
|
function onRomawi(int $bln)
|
||||||
: string
|
: string
|
||||||
{
|
{
|
||||||
switch ($bln) {
|
return convertToRoman($bln);
|
||||||
case 1:
|
|
||||||
return "I";
|
|
||||||
break;
|
|
||||||
case 2:
|
|
||||||
return "II";
|
|
||||||
break;
|
|
||||||
case 3:
|
|
||||||
return "III";
|
|
||||||
break;
|
|
||||||
case 4:
|
|
||||||
return "IV";
|
|
||||||
break;
|
|
||||||
case 5:
|
|
||||||
return "V";
|
|
||||||
break;
|
|
||||||
case 6:
|
|
||||||
return "VI";
|
|
||||||
break;
|
|
||||||
case 7:
|
|
||||||
return "VII";
|
|
||||||
break;
|
|
||||||
case 8:
|
|
||||||
return "VIII";
|
|
||||||
break;
|
|
||||||
case 9:
|
|
||||||
return "IX";
|
|
||||||
break;
|
|
||||||
case 10:
|
|
||||||
return "X";
|
|
||||||
break;
|
|
||||||
case 11:
|
|
||||||
return "XI";
|
|
||||||
break;
|
|
||||||
case 12:
|
|
||||||
return "XII";
|
|
||||||
break;
|
|
||||||
|
|
||||||
}
|
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
function penyebut($nilai)
|
function penyebut($nilai)
|
||||||
@@ -304,15 +260,6 @@
|
|||||||
return $hariKerja;
|
return $hariKerja;
|
||||||
}
|
}
|
||||||
|
|
||||||
function holidays()
|
|
||||||
{
|
|
||||||
return HolidayCalendar::pluck('date')->map(
|
|
||||||
function ($item) {
|
|
||||||
return Carbon::parse($item)->format('Y-m-d');
|
|
||||||
},
|
|
||||||
)->toArray();
|
|
||||||
}
|
|
||||||
|
|
||||||
function countPermohonanForUser($userId)
|
function countPermohonanForUser($userId)
|
||||||
{
|
{
|
||||||
$validStatuses = [
|
$validStatuses = [
|
||||||
|
|||||||
Reference in New Issue
Block a user