mirror of
https://github.com/RGBCube/serenity
synced 2025-07-26 03:07:44 +00:00
Spreadsheet: Show the error (if any) in csv import dialog's preview
...and don't try to read from a CSV that has errors. Fixes #5942.
This commit is contained in:
parent
bbf6847d50
commit
102065a8a9
6 changed files with 43 additions and 14 deletions
|
@ -38,6 +38,7 @@
|
|||
#include <LibGUI/ComboBox.h>
|
||||
#include <LibGUI/ItemListModel.h>
|
||||
#include <LibGUI/RadioButton.h>
|
||||
#include <LibGUI/StackWidget.h>
|
||||
#include <LibGUI/TableView.h>
|
||||
#include <LibGUI/TextBox.h>
|
||||
#include <LibGUI/Wizards/AbstractWizardPage.h>
|
||||
|
@ -71,13 +72,10 @@ CSVImportDialogPage::CSVImportDialogPage(StringView csv)
|
|||
m_trim_leading_field_spaces_check_box = m_page->body_widget().find_descendant_of_type_named<GUI::CheckBox>("trim_leading_field_spaces_check_box");
|
||||
m_trim_trailing_field_spaces_check_box = m_page->body_widget().find_descendant_of_type_named<GUI::CheckBox>("trim_trailing_field_spaces_check_box");
|
||||
m_data_preview_table_view = m_page->body_widget().find_descendant_of_type_named<GUI::TableView>("data_preview_table_view");
|
||||
m_data_preview_error_label = m_page->body_widget().find_descendant_of_type_named<GUI::Label>("data_preview_error_label");
|
||||
m_data_preview_widget = m_page->body_widget().find_descendant_of_type_named<GUI::StackWidget>("data_preview_widget");
|
||||
|
||||
Vector<String> quote_escape_items {
|
||||
// Note: Keep in sync with Reader::ParserTraits::QuoteEscape.
|
||||
"Repeat",
|
||||
"Backslash",
|
||||
};
|
||||
m_quote_escape_combo_box->set_model(GUI::ItemListModel<String>::create(quote_escape_items));
|
||||
m_quote_escape_combo_box->set_model(GUI::ItemListModel<String>::create(m_quote_escape_items));
|
||||
|
||||
// By default, use commas, double quotes with repeat, and disable headers.
|
||||
m_delimiter_comma_radio->set_checked(true);
|
||||
|
@ -178,14 +176,24 @@ void CSVImportDialogPage::update_preview()
|
|||
m_previously_made_reader = make_reader();
|
||||
if (!m_previously_made_reader.has_value()) {
|
||||
m_data_preview_table_view->set_model(nullptr);
|
||||
m_data_preview_error_label->set_text("Could not read the given file");
|
||||
m_data_preview_widget->set_active_widget(m_data_preview_error_label);
|
||||
return;
|
||||
}
|
||||
|
||||
auto& reader = *m_previously_made_reader;
|
||||
if (reader.has_error()) {
|
||||
m_data_preview_table_view->set_model(nullptr);
|
||||
m_data_preview_error_label->set_text(String::formatted("XSV parse error:\n{}", reader.error_string()));
|
||||
m_data_preview_widget->set_active_widget(m_data_preview_error_label);
|
||||
return;
|
||||
}
|
||||
|
||||
auto headers = reader.headers();
|
||||
|
||||
m_data_preview_table_view->set_model(
|
||||
GUI::ItemListModel<Reader::XSV::Row, Reader::XSV, Vector<String>>::create(reader, headers, min(8ul, reader.size())));
|
||||
m_data_preview_widget->set_active_widget(m_data_preview_table_view);
|
||||
m_data_preview_table_view->update();
|
||||
}
|
||||
|
||||
|
@ -207,6 +215,9 @@ Result<NonnullRefPtrVector<Sheet>, String> ImportDialog::make_and_run_for(String
|
|||
NonnullRefPtrVector<Sheet> sheets;
|
||||
|
||||
if (reader.has_value()) {
|
||||
if (reader.value().has_error())
|
||||
return String::formatted("CSV Import failed: {}", reader.value().error_string());
|
||||
|
||||
auto sheet = Sheet::from_xsv(reader.value(), workbook);
|
||||
if (sheet)
|
||||
sheets.append(sheet.release_nonnull());
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue