d_s_bisht/devsofttech_hms_ci4
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: use db->getFieldNames() instead of hasField() in migration for reliable column existence check

Browse Source
This commit is contained in:
DevSoftTech Server 2026-04-25 09:11:15 +05:30
parent 846e21ce4b
commit 67b4006d39
1 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
app/Database/Migrations/2026-04-21-000050_AddBankSettlementStatementMatchFields.php
View File

@ -12,7 +12,9 @@ class AddBankSettlementStatementMatchFields extends Migration
return;
}
if (! $this->hasField('finance_bank_settlement_entries', 'reconciliation_status')) {
$existingFields = $this->db->getFieldNames('finance_bank_settlement_entries');
if (! in_array('reconciliation_status', $existingFields)) {
$this->forge->addColumn('finance_bank_settlement_entries', [
'reconciliation_status' => [
'type' => 'VARCHAR',
@ -24,7 +26,7 @@ class AddBankSettlementStatementMatchFields extends Migration
]);
}
if (! $this->hasField('finance_bank_settlement_entries', 'statement_matched_by')) {
if (! in_array('statement_matched_by', $existingFields)) {
$this->forge->addColumn('finance_bank_settlement_entries', [
'statement_matched_by' => [
'type' => 'VARCHAR',
@ -35,7 +37,7 @@ class AddBankSettlementStatementMatchFields extends Migration
]);
}
if (! $this->hasField('finance_bank_settlement_entries', 'statement_matched_at')) {
if (! in_array('statement_matched_at', $existingFields)) {
$this->forge->addColumn('finance_bank_settlement_entries', [
'statement_matched_at' => [
'type' => 'DATETIME',
@ -45,7 +47,7 @@ class AddBankSettlementStatementMatchFields extends Migration
]);
}
if (! $this->hasField('finance_bank_settlement_entries', 'statement_match_remarks')) {
if (! in_array('statement_match_remarks', $existingFields)) {
$this->forge->addColumn('finance_bank_settlement_entries', [
'statement_match_remarks' => [
'type' => 'TEXT',
@ -64,16 +66,18 @@ class AddBankSettlementStatementMatchFields extends Migration
return;
}
if ($this->hasField('finance_bank_settlement_entries', 'statement_match_remarks')) {
$existingFields = $this->db->getFieldNames('finance_bank_settlement_entries');
if (in_array('statement_match_remarks', $existingFields)) {
$this->forge->dropColumn('finance_bank_settlement_entries', 'statement_match_remarks');
}
if ($this->hasField('finance_bank_settlement_entries', 'statement_matched_at')) {
if (in_array('statement_matched_at', $existingFields)) {
$this->forge->dropColumn('finance_bank_settlement_entries', 'statement_matched_at');
}
if ($this->hasField('finance_bank_settlement_entries', 'statement_matched_by')) {
if (in_array('statement_matched_by', $existingFields)) {
$this->forge->dropColumn('finance_bank_settlement_entries', 'statement_matched_by');
}
if ($this->hasField('finance_bank_settlement_entries', 'reconciliation_status')) {
if (in_array('reconciliation_status', $existingFields)) {
$this->forge->dropColumn('finance_bank_settlement_entries', 'reconciliation_status');
}
}