Shiloh/brotherton-erpnext
1
1
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

feat: error log handling

Browse Source 
fix: toggle required accounts
This commit is contained in:
Gavin D'souza 2020-04-27 13:12:38 +05:30
parent 38c677689d
commit 82e7a9de28
1 changed files with 101 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

102
erpnext/erpnext_integrations/doctype/tally_migration/tally_migration.js
View File

@ -1,7 +1,7 @@
// Copyright (c) 2019, Frappe Technologies Pvt. Ltd. and contributors
// For license information, please see license.txt
frappe.ui.form.on('Tally Migration', {
frappe.ui.form.on("Tally Migration", {
onload: function (frm) {
let reload_status = true;
frappe.realtime.on("tally_migration_progress_update", function (data) {
@ -36,6 +36,10 @@ frappe.ui.form.on('Tally Migration', {
});
},
refresh: function (frm) {
frm.trigger("show_import_log");
["default_round_off_account", "default_warehouse", "default_cost_center"].forEach(account => {
frm.toggle_reqd(account, frm.doc.is_master_data_imported === 1)
})
if (frm.doc.master_data && !frm.doc.is_master_data_imported) {
if (frm.doc.is_master_data_processed) {
if (frm.doc.status != "Importing Master Data") {
@ -71,5 +75,101 @@ frappe.ui.form.on('Tally Migration', {
frm.reload_doc();
}
);
},
show_import_log(frm) {
let index = 0;
let import_log = JSON.parse(frm.doc.failed_import_log || "[]");
let logs = import_log.slice(0, 20);
let hidden_logs = import_log.slice(20);
frm.toggle_display("import_log_section", logs.length > 0);
const getError = (traceback) => {
let exc_error_idx = traceback.trim().lastIndexOf("\n") + 1
let error_line = traceback.substr(exc_error_idx)
let split_str_idx = (error_line.indexOf(':') > 0) ? error_line.indexOf(':') + 1 : 0;
return error_line.slice(split_str_idx).trim();
}
const cleanDoc = (obj) => {
let temp = obj;
$.each(temp, function(key, value){
if (value === "" || value === null){
delete obj[key];
} else if (Object.prototype.toString.call(value) === '[object Object]') {
cleanDoc(value);
} else if ($.isArray(value)) {
$.each(value, function (k,v) { cleanDoc(v); });
}
});
return temp;
};
let rows = logs
.map(({ doc, exc }) => {
let id = frappe.dom.get_unique_id();
let traceback = exc;
let error_message = getError(traceback);
index++;
let html = `
<button class="btn btn-default btn-xs m-3" type="button" data-toggle="collapse" data-target="#${id}-traceback" aria-expanded="false" aria-controls="${id}-traceback">
${__("Show Traceback")}
</button>
<div class="collapse margin-top" id="${id}-traceback">
<div class="well">
<pre style="font-size: smaller;">${traceback}</pre>
</div>
</div>`;
let show_doc = `
<button class='btn btn-default btn-xs m-3' type='button' data-toggle='collapse' data-target='#${id}-doc' aria-expanded='false' aria-controls='${id}-doc'>
${__("Show Document")}
</button>
<div class="collapse margin-top" id="${id}-doc">
<div class="well">
<pre style="font-size: smaller;">${JSON.stringify(cleanDoc(doc), null, 1)}</pre>
</div>
</div>`;
let create_button = `
<button class='btn btn-default btn-xs m-3' type='button' onclick='frappe.new_doc("${doc.doctype}", ${JSON.stringify(doc)})'>
${__("Create Document")}
</button>`
return `<tr>
<td>${index}</td>
<td>
<div>${doc.doctype}</div>
</td>
<td>
<div>${error_message}</div>
<div>${html}</div>
<div>${show_doc}</div>
</td>
<td>
<div>${create_button}</div>
</td>
</tr>`;
})
.join("");
frm.get_field("failed_import_preview").$wrapper.html(`
<table class="table table-bordered">
<tr class="text-muted">
<th width="5%">${__("#")}</th>
<th width="10%">${__("DocType")}</th>
<th width="75%">${__("Error Message")}</th>
<th width="10%">${__("Create")}</th>
</tr>
${rows}
<tr class="text-muted">
<td colspan="4">And ${hidden_logs.length} more others</td>
</tr>
</table>
`);
}
});