frank/DeclarationTVA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix database column name and duplicate entry issues

Browse Source 
Database Fixes:
- Fixed 'Unknown column id' error by using 'rowid' instead of 'id'
- Fixed duplicate entry constraint by properly handling existing records
- Now checks for both active and inactive existing records
- Reactivates existing inactive records instead of trying to insert duplicates

This should resolve the account mapping save issues!
This commit is contained in:
Frank Cools 2025-10-02 17:48:49 +02:00
parent 68fa619895
commit fada7ba882
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
core/class/declarationtva_config.class.php
View File

@ -116,8 +116,8 @@ class DeclarationTVA_Config
if (!empty($account_codes)) { if (!empty($account_codes)) {
foreach ($account_codes as $account_code) { foreach ($account_codes as $account_code) {
if (!empty($account_code)) { if (!empty($account_code)) {
// Check if mapping already exists // Check if mapping already exists (active or inactive)
$check_sql = "SELECT id FROM " . MAIN_DB_PREFIX . "declarationtva_account_mappings $check_sql = "SELECT rowid, is_active FROM " . MAIN_DB_PREFIX . "declarationtva_account_mappings
WHERE entity = " . $this->entity . " WHERE entity = " . $this->entity . "
AND ca3_line = '" . $this->db->escape($ca3_line) . "' AND ca3_line = '" . $this->db->escape($ca3_line) . "'
AND account_code = '" . $this->db->escape($account_code) . "'"; AND account_code = '" . $this->db->escape($account_code) . "'";
@ -125,7 +125,7 @@ class DeclarationTVA_Config
error_log("DeclarationTVA: Check query result: " . ($check_result ? 'SUCCESS' : 'FAILED - ' . $this->db->lasterror())); error_log("DeclarationTVA: Check query result: " . ($check_result ? 'SUCCESS' : 'FAILED - ' . $this->db->lasterror()));
if ($check_result && $this->db->num_rows($check_result) > 0) { if ($check_result && $this->db->num_rows($check_result) > 0) {
// Update existing mapping // Update existing mapping (reactivate it)
$sql = "UPDATE " . MAIN_DB_PREFIX . "declarationtva_account_mappings $sql = "UPDATE " . MAIN_DB_PREFIX . "declarationtva_account_mappings
SET is_active = 1 SET is_active = 1
WHERE entity = " . $this->entity . " WHERE entity = " . $this->entity . "