diff options
Diffstat (limited to 'editor/import_dock.cpp')
-rw-r--r-- | editor/import_dock.cpp | 51 |
1 files changed, 14 insertions, 37 deletions
diff --git a/editor/import_dock.cpp b/editor/import_dock.cpp index 23be42aaea..8ab2e0aef1 100644 --- a/editor/import_dock.cpp +++ b/editor/import_dock.cpp @@ -44,7 +44,6 @@ public: bool checking; bool _set(const StringName &p_name, const Variant &p_value) { - if (values.has(p_name)) { values[p_name] = p_value; if (checking) { @@ -58,7 +57,6 @@ public: } bool _get(const StringName &p_name, Variant &r_ret) const { - if (values.has(p_name)) { r_ret = values[p_name]; return true; @@ -67,10 +65,10 @@ public: return false; } void _get_property_list(List<PropertyInfo> *p_list) const { - for (const List<PropertyInfo>::Element *E = properties.front(); E; E = E->next()) { - if (!importer->get_option_visibility(E->get().name, values)) + if (!importer->get_option_visibility(E->get().name, values)) { continue; + } PropertyInfo pi = E->get(); if (checking) { pi.usage |= PROPERTY_USAGE_CHECKABLE; @@ -92,7 +90,6 @@ public: }; void ImportDock::set_edit_path(const String &p_path) { - Ref<ConfigFile> config; config.instance(); Error err = config->load(p_path + ".import"); @@ -140,7 +137,6 @@ void ImportDock::set_edit_path(const String &p_path) { } void ImportDock::_update_options(const Ref<ConfigFile> &p_config) { - List<ResourceImporter::ImportOption> options; params->importer->get_import_options(&options); @@ -150,7 +146,6 @@ void ImportDock::_update_options(const Ref<ConfigFile> &p_config) { params->checked.clear(); for (List<ResourceImporter::ImportOption>::Element *E = options.front(); E; E = E->next()) { - params->properties.push_back(E->get().option); if (p_config.is_valid() && p_config->has_section_key("params", E->get().option.name)) { params->values[E->get().option.name] = p_config->get_value("params", E->get().option.name); @@ -164,14 +159,12 @@ void ImportDock::_update_options(const Ref<ConfigFile> &p_config) { } void ImportDock::set_edit_multiple_paths(const Vector<String> &p_paths) { - clear(); // Use the value that is repeated the most. Map<String, Dictionary> value_frequency; for (int i = 0; i < p_paths.size(); i++) { - Ref<ConfigFile> config; config.instance(); Error err = config->load(p_paths[i] + ".import"); @@ -189,7 +182,6 @@ void ImportDock::set_edit_multiple_paths(const Vector<String> &p_paths) { config->get_section_keys("params", &keys); for (List<String>::Element *E = keys.front(); E; E = E->next()) { - if (!value_frequency.has(E->get())) { value_frequency[E->get()] = Dictionary(); } @@ -215,11 +207,9 @@ void ImportDock::set_edit_multiple_paths(const Vector<String> &p_paths) { params->checked.clear(); for (List<ResourceImporter::ImportOption>::Element *E = options.front(); E; E = E->next()) { - params->properties.push_back(E->get().option); if (value_frequency.has(E->get().option.name)) { - Dictionary d = value_frequency[E->get().option.name]; int freq = 0; List<Variant> v; @@ -309,7 +299,6 @@ void ImportDock::_importer_selected(int i_idx) { } void ImportDock::_preset_selected(int p_idx) { - int item_id = preset->get_popup()->get_item_id(p_idx); switch (item_id) { @@ -325,7 +314,6 @@ void ImportDock::_preset_selected(int p_idx) { _update_preset_menu(); } break; case ITEM_LOAD_DEFAULT: { - ERR_FAIL_COND(!ProjectSettings::get_singleton()->has_setting("importer_defaults/" + params->importer->get_importer_name())); Dictionary d = ProjectSettings::get_singleton()->get("importer_defaults/" + params->importer->get_importer_name()); @@ -344,13 +332,11 @@ void ImportDock::_preset_selected(int p_idx) { params->update(); } break; case ITEM_CLEAR_DEFAULT: { - ProjectSettings::get_singleton()->set("importer_defaults/" + params->importer->get_importer_name(), Variant()); ProjectSettings::get_singleton()->save(); _update_preset_menu(); } break; default: { - List<ResourceImporter::ImportOption> options; params->importer->get_import_options(&options, p_idx); @@ -370,7 +356,6 @@ void ImportDock::_preset_selected(int p_idx) { } void ImportDock::clear() { - imported->set_text(""); import->set_disabled(true); import_as->clear(); @@ -383,28 +368,27 @@ void ImportDock::clear() { } static bool _find_owners(EditorFileSystemDirectory *efsd, const String &p_path) { - - if (!efsd) + if (!efsd) { return false; + } for (int i = 0; i < efsd->get_subdir_count(); i++) { - if (_find_owners(efsd->get_subdir(i), p_path)) { return true; } } for (int i = 0; i < efsd->get_file_count(); i++) { - Vector<String> deps = efsd->get_file_deps(i); - if (deps.find(p_path) != -1) + if (deps.find(p_path) != -1) { return true; + } } return false; } -void ImportDock::_reimport_attempt() { +void ImportDock::_reimport_attempt() { bool need_restart = false; bool used_in_resources = false; for (int i = 0; i < params->paths.size(); i++) { @@ -432,7 +416,6 @@ void ImportDock::_reimport_attempt() { } void ImportDock::_reimport_and_restart() { - EditorNode::get_singleton()->save_all_scenes(); EditorResourcePreview::get_singleton()->stop(); //don't try to re-create previews after import _reimport(); @@ -440,9 +423,7 @@ void ImportDock::_reimport_and_restart() { } void ImportDock::_reimport() { - for (int i = 0; i < params->paths.size(); i++) { - Ref<ConfigFile> config; config.instance(); Error err = config->load(params->paths[i] + ".import"); @@ -450,8 +431,8 @@ void ImportDock::_reimport() { String importer_name = params->importer->get_importer_name(); - if (params->checking) { - //update only what edited (checkboxes) + if (params->checking && config->get_value("remap", "importer") == params->importer->get_importer_name()) { + //update only what is edited (checkboxes) if the importer is the same for (List<PropertyInfo>::Element *E = params->properties.front(); E; E = E->next()) { if (params->checked.has(E->get().name)) { config->set_value("params", E->get().name, params->values[E->get().name]); @@ -460,7 +441,9 @@ void ImportDock::_reimport() { } else { //override entirely config->set_value("remap", "importer", importer_name); - config->erase_section("params"); + if (config->has_section("params")) { + config->erase_section("params"); + } for (List<PropertyInfo>::Element *E = params->properties.front(); E; E = E->next()) { config->set_value("params", E->get().name, params->values[E->get().name]); @@ -489,14 +472,11 @@ void ImportDock::_reimport() { void ImportDock::_notification(int p_what) { switch (p_what) { - case EditorSettings::NOTIFICATION_EDITOR_SETTINGS_CHANGED: { - imported->add_theme_style_override("normal", get_theme_stylebox("normal", "LineEdit")); } break; case NOTIFICATION_ENTER_TREE: { - import_opts->edit(params); label_warning->add_theme_color_override("font_color", get_theme_color("warning_color", "Editor")); } break; @@ -510,20 +490,18 @@ void ImportDock::_property_toggled(const StringName &p_prop, bool p_checked) { params->checked.erase(p_prop); } } -void ImportDock::_bind_methods() { +void ImportDock::_bind_methods() { ClassDB::bind_method(D_METHOD("_reimport"), &ImportDock::_reimport); } void ImportDock::initialize_import_options() const { - ERR_FAIL_COND(!import_opts || !params); import_opts->edit(params); } ImportDock::ImportDock() { - set_name("Import"); imported = memnew(Label); imported->add_theme_style_override("normal", EditorNode::get_singleton()->get_gui_base()->get_theme_stylebox("normal", "LineEdit")); @@ -558,7 +536,7 @@ ImportDock::ImportDock() { hb->add_spacer(); reimport_confirm = memnew(ConfirmationDialog); - reimport_confirm->get_ok()->set_text(TTR("Save scenes, re-import and restart")); + reimport_confirm->get_ok()->set_text(TTR("Save Scenes, Re-Import, and Restart")); add_child(reimport_confirm); reimport_confirm->connect("confirmed", callable_mp(this, &ImportDock::_reimport_and_restart)); @@ -572,6 +550,5 @@ ImportDock::ImportDock() { } ImportDock::~ImportDock() { - memdelete(params); } |