From 346480005ad3ebb02feb57ebf0367191dfad8dc1 Mon Sep 17 00:00:00 2001
From: Stone Li <stone.li@bambulab.com>
Date: Mon, 29 Aug 2022 11:28:06 +0800
Subject: [PATCH] NEW: add comments for print job

Change-Id: Ie7a65a76d535dc210268842704fdaf4a7475135a
Signed-off-by: Stone Li <stone.li@bambulab.com>
---
 src/slic3r/GUI/Jobs/PrintJob.cpp      | 17 +++++++++++++++++
 src/slic3r/Utils/bambu_networking.hpp |  1 +
 2 files changed, 18 insertions(+)

diff --git a/src/slic3r/GUI/Jobs/PrintJob.cpp b/src/slic3r/GUI/Jobs/PrintJob.cpp
index db760503a..715bbdf20 100644
--- a/src/slic3r/GUI/Jobs/PrintJob.cpp
+++ b/src/slic3r/GUI/Jobs/PrintJob.cpp
@@ -221,9 +221,19 @@ void PrintJob::process()
             return was_canceled();
         };
 
+
     NetworkAgent* m_agent = wxGetApp().getAgent();
 
     if (params.connection_type != "lan") {
+        if (params.dev_ip.empty())
+            params.comments = "no_ip";
+        else if (this->cloud_print_only)
+            params.comments = "low_version";
+        else if (!this->has_sdcard)
+            params.comments = "no_sdcard";
+        else if (params.password.empty())
+            params.comments = "no_password";
+
         if (!this->cloud_print_only
             && !params.password.empty() 
             && !params.dev_ip.empty()
@@ -232,6 +242,13 @@ void PrintJob::process()
             BOOST_LOG_TRIVIAL(info) << "print_job: try to start local print with record";
             this->update_status(curr_percent, _L("Sending print job over LAN"));
             result = m_agent->start_local_print_with_record(params, update_fn, cancel_fn);
+            if (result == BAMBU_NETWORK_ERR_FTP_LOGIN_DENIED) {
+                params.comments = "wrong_code";
+            } else if (result == BAMBU_NETWORK_ERR_FTP_UPLOAD_FAILED) {
+                params.comments = "upload_failed";
+            } else {
+                params.comments = (boost::format("failed(%1%)") % result).str();
+            }
             if (result < 0) {
                 // try to send with cloud
                 BOOST_LOG_TRIVIAL(warning) << "print_job: try to send with cloud";
diff --git a/src/slic3r/Utils/bambu_networking.hpp b/src/slic3r/Utils/bambu_networking.hpp
index da246a10b..0df84d203 100644
--- a/src/slic3r/Utils/bambu_networking.hpp
+++ b/src/slic3r/Utils/bambu_networking.hpp
@@ -113,6 +113,7 @@ struct PrintParams {
     std::string     ftp_file_md5;
     std::string     ams_mapping;
     std::string     connection_type;
+    std::string     comments;
 
     /* access options */
     std::string     dev_ip;