Bug #15013
openExcel Import does not work with multiple importRegion
100%
Description
Aus einem Excel sollen die Spalten A,B und D importiert werden.
A soll in die Spalte 'klasse', B in die Spalte 'stufe' und D in die Spalte 'grundlohn'.
Formelement.parameter wurde wie folgt gesetzt:
importToTable = lohn
importToColumns = klasse, stufe, grundlohn
importRegion = 1,A,6,B,904|1,D,6,D,904
importMode = append
importType = auto
accept = *
fileDestination = fileadmin/protected/imports/pay_regulations.xlsx
fileReplace = always
Spalten A und B funktionieren korrekt. Die Spalte D, welche in einer zweiten 'importRegion' angegeben wurde, wird jedoch nach 'klasse' importiert, anstatt nach 'grundlohn'.
Wenn eine 2. importRegion angegeben wird, dann sollte nicht auf die 1. angegebene DB-Spalte zurückgesprungen werden!
Updated by Enis Nuredini over 1 year ago
- Status changed from In Progress to Ready to sync (develop)
Bei bisheriger Funktionalität beschränkte sich der Import pro Region auf das Insert Statement.
Mit der neuen Implementierung ist folgendes möglich:
Parameter
importToTable = lohn
importToColumns = klasse, stufe, grundlohn, versicherterLohn
importRegion = 1,A,6,B,904|1,D,6,E,904|1,N,6,O,904,append|1,Q,6,Q,904
importMode = replace
importType = auto
accept = *
Ergebnis
1. Region -> Insert mit new rows für klasse und stufe
2. Region -> Update der Rows von Region 1 mit den Werten für grundlohn und versicherterLohn.
3. Region -> Durch das Keyword noSameRow wird wieder beim ersten Column angefangen mit dem Insert in klasse und stufe
4. Region -> Update der Rows von Region 3 mit dem Wert für grundlohn wird ausgeführt.
Updated by Carsten Rose about 1 year ago
- Status changed from Ready to sync (develop) to Feedback
Ich finde keinen Commit und keinen Merge Request - wenn es Aenderungen am Code gab: wann wurden die gemerged?
Updated by Enis Nuredini about 1 year ago
Wurde im Ticket #15014 implementiert.
https://git.math.uzh.ch/typo3/qfq/-/merge_requests/479
Updated by Enis Nuredini about 1 year ago
- Assignee changed from Enis Nuredini to Carsten Rose