vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix(data_import): set payload count from existing importer

Browse source 
Don't create a new one, especially one that doesn't have all parameters (main issue here: console=False)

Signed-off-by: Akhil Narang <me@akhilnarang.dev>
This commit is contained in:
Akhil Narang 2025-09-03 17:56:41 +05:30
parent 7ad1378d00
commit 60b2d92af8
No known key found for this signature in database
GPG key ID: 9DCC61E211BF645F
1 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
frappe/core/doctype/data_import/data_import.py
View file

@ -77,10 +77,11 @@ class DataImport(Document):
return
validate_google_sheets_url(self.google_sheets_url)
def set_payload_count(self):
def set_payload_count(self, importer: Importer | None = None):
if self.import_file:
i = self.get_importer()
payloads = i.import_file.get_payloads_for_import()
if importer is None:
importer = self.get_importer()
payloads = importer.import_file.get_payloads_for_import()
self.payload_count = len(payloads)
@frappe.whitelist()
@ -268,9 +269,9 @@ def import_file(doctype, file_path, import_type, submit_after_import=False, cons
data_import.import_type = (
"Insert New Records" if import_type.lower() == "insert" else "Update Existing Records"
)
data_import.set_payload_count()
i = Importer(doctype=doctype, file_path=file_path, data_import=data_import, console=console)
data_import.set_payload_count(i)
i.import_data()