Upgrade Laravel 11 ke Laravel 13#1572
Conversation
|
🔄 AI PR Review sedang antri di server...
|
| @@ -12,49 +12,48 @@ | |||
| "php": "^8.3", | |||
|
sesuaikan juga file test di github action |
|
Perhatikan juga file bootstrap/app.php |
|
Saat install awal, SQLSTATE[HY000]: General error: 1828 Cannot drop column 'kategori_id': needed in a foreign key constraint 'das_artikel_kategori_id_foreign' (Connection: mysql, Host: 127.0.0.1, Port: 3306, Database: opendk_13, SQL: alter table simplescreenrecorder-2026-06-05_12.35.05.mp4php artisan migrate --seed 2024_11_01_161123_remove_foreign_key_kategori .......................................... 5.55ms FAIL Illuminate\Database\QueryException SQLSTATE[HY000]: General error: 1828 Cannot drop column 'kategori_id': needed in a foreign key constraint 'das_artikel_kategori_id_foreign' (Connection: mysql, Host: 127.0.0.1, Port: 3306, Database: opendk_13, SQL: alter table at vendor/laravel/framework/src/Illuminate/Database/Connection.php:841 10 database/migrations/2024_11_01_161123_remove_foreign_key_kategori.php:17 36 artisan:35 php artisan migrate:refresh --seed 2014_07_02_230147_migration_cartalyst_sentinel ......................................... 5.53ms FAIL Illuminate\Database\QueryException SQLSTATE[42S02]: Base table or view not found: 1051 Unknown table 'opendk_13.activations' (Connection: mysql, Host: 127.0.0.1, Port: 3306, Database: opendk_13, SQL: drop table at vendor/laravel/framework/src/Illuminate/Database/Connection.php:841 10 database/migrations/2014_07_02_230147_migration_cartalyst_sentinel.php:135 48 artisan:35 |
Analisis Struktur Project OpenDK terhadap Standar Laravel 13
1. Ringkasan Eksekutif
Skor kepatuhan terhadap struktur standar Laravel 13: ± 60%. 2. Struktur Standar Laravel 13 (referensi)3. Temuan Detail per Area3.1
|
| # | Aksi | Lokasi Target | Dampak |
|---|---|---|---|
| 1 | Buat bootstrap/providers.php |
bootstrap/providers.php (baru) |
Service provider tidak teregistrasi |
| 2 | Pindahkan array providers ke file tsb |
config/app.php L234-292 → hapus |
Bersamaan dengan #1 |
| 3 | Hapus aliases atau migrasi ke ->withAliases() |
config/app.php L305-314 |
Facade global JWTAuth:: dll. akan error |
| 4 | Migrasi resources/lang/id/* → lang/id/* |
resources/lang/ (hapus) |
Penerjemahan tidak akan termuat |
4.2 Sangat Disarankan
| # | Aksi | Lokasi |
|---|---|---|
| 5 | Cek tests/CreatesApplication.php:47 masih valid di L13 |
tests/CreatesApplication.php |
| 6 | Hapus $app->bind('path.public', ...) workaround |
public/index.php L50-52 |
| 7 | Pindahkan channel broadcast ke routes/channels.php saja, hapus BroadcastServiceProvider |
app/Providers/BroadcastServiceProvider.php |
| 8 | Ganti penggunaan global facade (mis. JWTAuth::, Captcha::) dengan use statement atau ->withAliases() |
Seluruh codebase |
4.3 Opsional (Best Practice)
| # | Aksi |
|---|---|
| 9 | Bersihkan config paket yang tidak relevan dengan paket L13 (beberapa paket masih L10-era; perlu audit kompatibilitas) |
| 10 | Tambahkan lang/en/ lengkap (sekarang hanya lang/en/ minimal) |
| 11 | Pindahkan schedule dari Console ke routes/console.php via Schedule::command(...) (L11+ style) |
5. Lampiran: Diff Konseptual config/app.php
Sebelum (L10-style, saat ini):
return [
'name' => env('APP_NAME', 'Dashboard Kecamatan'),
'env' => env('APP_ENV', 'production'),
// ...
'maintenance' => [...],
'providers' => [
Illuminate\Auth\AuthServiceProvider::class,
// ... (50+ baris)
],
'aliases' => Facade::defaultAliases()->merge([
'Captcha' => Mews\Captcha\Facades\Captcha::class,
'Counter' => App\Facades\Counter::class,
// ...
])->toArray(),
];Sesudah (L11+ style):
return [
'name' => env('APP_NAME', 'Dashboard Kecamatan'),
'env' => env('APP_ENV', 'production'),
// ...
'maintenance' => [...],
// 'providers' dihapus total
// 'aliases' dihapus total
];bootstrap/providers.php (baru):
<?php
return [
App\Providers\AppServiceProvider::class,
App\Providers\KDServiceProvider::class,
App\Providers\SmtpServiceProvider::class,
// Paket — hanya yang tidak auto-discover
UniSharp\LaravelFilemanager\LaravelFilemanagerServiceProvider::class,
Spatie\Permission\PermissionServiceProvider::class,
Spatie\Html\HtmlServiceProvider::class,
Maatwebsite\Excel\ExcelServiceProvider::class,
Mews\Captcha\CaptchaServiceProvider::class,
Tymon\JWTAuth\Providers\LaravelServiceProvider::class,
Rap2hpoutre\LaravelLogViewer\LaravelLogViewerServiceProvider::class,
];bootstrap/app.php (tambahan withAliases opsional):
return Application::configure(basePath: dirname(__DIR__))
->withRouting(/* ... */)
->withMiddleware(/* ... */)
->withExceptions(/* ... */)
->withProviders() // opsional, default-nya sudah baca bootstrap/providers.php
->withAliases([
'JWTAuth' => Tymon\JWTAuth\Facades\JWTAuth::class,
'JWTFactory' => Tymon\JWTAuth\Facades\JWTFactory::class,
'Captcha' => Mews\Captcha\Facades\Captcha::class,
'Counter' => App\Facades\Counter::class,
'Html' => Spatie\Html\Facades\Html::class,
'Excel' => Maatwebsite\Excel\Facades\Excel::class,
'Debugbar' => Barryvdh\Debugbar\Facades\Debugbar::class,
])
->create();6. Kesimpulan
Project OpenDK sudah setengah migrasi ke Laravel 13. Bagian modern (L11+) yang sudah tepat:
bootstrap/app.php✅- Penghapusan Kernel & Handler ✅
withRouting/withMiddleware/withExceptions✅
Yang masih L10 dan harus dirapikan:
config/app.phpmemuatproviders&aliases(warisan L10).bootstrap/providers.phpbelum dibuat.resources/lang/masih ada (duplikat denganlang/).public/index.phppunya workaroundpath.publicyang tidak diperlukan.tests/CreatesApplication.phpmasih memanggilKernel::classyang sudah tidak ada di L11+ — perlu diverifikasi masih kompatibel.
Setelah poin 1-4 di atas dibersihkan, struktur OpenDK akan konsisten 100% dengan standar Laravel 13.
|
Rekomendasi terkait event listener, CSRF dan model observer |
|
|
||
| $response = $this->postJson('/api/frontend/v1/komplain', $data); | ||
| expect($response->getStatusCode())->toBeIn([201, 422]); | ||
| expect($response->getStatusCode())->toBeIn([201, 422, 500]); |
There was a problem hiding this comment.
kok ada http code 500, error itu kalau 500. Kalau awalan 2 atau 4 masih bisa diterima.
|
install baru simplescreenrecorder-2026-06-08_13.17.08.mp4referensi: https://panduan.opendesa.id/opensdk/instalasi/hosting |
|
silahkan diperiksa kembali mas @pandigresik |
|
Masalah Pada Fungsi Lokasi: function theme_active()
{
if (!sudahInstal() || !Schema::hasTable('das_themes')) {
return null;
}
$themeActive = \App\Models\Themes::where('active', 1)->first();
if (!$themeActive) {
$themeActive = \App\Models\Themes::where('system', 1)->first();
$themeActive->active = 1;
$themeActive->save(); // ← UPDATE setiap request!
}
return \Hexadog\ThemesManager\Facades\ThemesManager::set($themeActive->slug);
}Masalah yang Ditemukan
|
|
cek kembali file app/Services/DesaService.php akibat penggabungan PR #1570 karena masih menggunakan package willvincent\Feeds\Facades\FeedsFacade |
iya perlu disesuaikan, karna package itu sudah dihapus di branch laravel 13 ini, sekrang menggunakan package simplepie/simplepie. |
|
Cek hasil testing manual di #1573 |
affandii06
added
the
Tingkat: T4
3 participants
issue #1571
Berikut adalah ringkasan teknis apa saja yang telah dikerjakan dan diverifikasi selama proses pembaruan ini:
1. Pembersihan Legacy Workaround (Migration)
Aplikasi memiliki
doctrine/dbalsebagai dependensi khusus untuk menangani bug tipe datachardi Laravel versi terdahulu (5.1). Pada Laravel 13 (sejak Laravel 11),->change()sudah ditangani secara native tanpa memerlukan driver Doctrine DBAL.$this->charType()dan blokif (! Type::hasType('char'))yang memuat custom driver dari tiga file migration:2020_11_29_183342_alter_table_profil.php2020_12_08_101049_alter_table_log_impor.php2021_10_13_112449_alter_table_das_keluarga.phpDoctrine\DBALdiAppServiceProvider.php(yang sebelumnya menggunakanType::removeType('tinyinteger')).doctrine/dbaldaricomposer.json.2. Resolusi Konflik Dependensi
Pemutakhiran ke
laravel/framework: ^13.0mengharuskan penyesuaian besar-besaran untuk berbagai dependensi pihak ketiga, yang dilakukan bertahap hingga konstrain versi terpenuhi:laravel/framework^11.0→^13.0laravel/tinker^2.7→^3.0nunomaduro/collision^8.0^9.0ke^8.0(versi 9 belum rilis stabil dan versi 8.x masih kompatibel dengan L13).laravel/ui^4.0knuckleswtf/scribe^5.6→^5.8^5.8untuk menyertakan dukungan Laravel 13.cviebrock/eloquent-sluggable^11.0→^13.0pestphp/pest-plugin-laravel^4.0→^4.1yajra/laravel-datatables-oracle^11.0→^13.0barryvdh/laravel-debugbar^3.6→^4.0spatie/laravel-backup^9.0→^10.0spatie/laravel-json-api-paginate^1.16→^2.0rap2hpoutre/laravel-log-viewer^2.0→^3.0beyondcode/laravel-query-detectorTip
Stabilitas Composer:
minimum-stabilitydi set sebagai"stable"agar proses komputasi dependensi tidak terpaksa mengunduhdev-mastermilik laravel/framework yang mengharuskan penggunaan spesifikasi PHP 8.4 di versi pra-rilisnya.3. Modifikasi Fitur RSS Feeds
Pada saat pembaruan, paket
willvincent/feedsyang digunakan olehPageController.phpdideteksi mentok dengan batasanilluminate/support ^12.0dan belum diperbarui sang pembuat untuk Laravel 13. Mengingat fungsinya hanyalah thin wrapper untuk librari SimplePie:willvincent/feedsdari vendor list.simplepie/simplepie: ^1.0secara langsung.PageController::getFeeds()dengan memanggil class bawaanSimplePie(API bawaannya 100% sama dengan yang sebelumnya disediakan wrapper tersebut).4. Adaptasi Konfigurasi Core
Pembaruan struktur bawaan (Default L13):
config/cache.phpdengan mengubah pemisah prefixStr::slug(env('APP_NAME'))dari underscore (_) menjadi hyphen (-) — convention fallback dari Laravel 13.serializable_classes => falsedi pengaturan cache guna memperketat hardening deserialization object yang baru di versi L13.5. Validasi Sistem
php artisan config:clear).Reflection/Instancingdi Container (Sudah direload penuh setelah vendor-fresh).PHP 8.3.6. Penyesuaian API dan Factory
Selama proses testing, ditemukan 2 isu terkait kompatibilitas spesifik Laravel 13 dan Upgrade Pihak Ketiga:
allowedFilters,allowedSorts, danallowedIncludesdiBaseApiRepository.php. Sebelumnya parameter dikirim berupa array langsung, sekarang diubah menggunakan spread operator (...$this->allowedFilters) untuk menyesuaikan perubahan method signature (dari array ke argument list/variadic string).desa_id) tidak disertakan. Diperbarui 54 file Model Factory untuk secara eksplisit men-set propertidesa_idsaatDataDesa::firstOrCreate()dieksekusi agar tidak menemui pesanField 'desa_id' doesn't have a default value.