Allow adding multiple scripts at once in the automation dialog

Originally committed to SVN as r5768.
This commit is contained in:
Thomas Goyne 2011-10-24 20:17:48 +00:00
parent 5cb7bdcbaf
commit 4e1e68c88b

View file

@ -181,27 +181,31 @@ static bool has_file(Container const& c, wxFileName const& fn)
void DialogAutomation::OnAdd(wxCommandEvent &) void DialogAutomation::OnAdd(wxCommandEvent &)
{ {
wxString fname = wxFileSelector( wxFileDialog diag(this,
_("Add Automation script"), _("Add Automation script"),
lagi_wxString(OPT_GET("Path/Last/Automation")->GetString()), lagi_wxString(OPT_GET("Path/Last/Automation")->GetString()),
"", "", "",
Automation4::ScriptFactory::GetWildcardStr(), Automation4::ScriptFactory::GetWildcardStr(),
wxFD_OPEN | wxFD_FILE_MUST_EXIST, wxFD_OPEN | wxFD_FILE_MUST_EXIST | wxFD_MULTIPLE);
this);
if (fname.empty()) return; if (diag.ShowModal() == wxID_CANCEL) return;
wxFileName fnpath(fname); wxArrayString fnames;
OPT_SET("Path/Last/Automation")->SetString(STD_STR(fnpath.GetPath())); diag.GetPaths(fnames);
if (has_file(local_manager->GetScripts(), fnpath) || has_file(global_manager->GetScripts(), fnpath)) { for (size_t i = 0; i < fnames.size(); ++i) {
wxLogError("Script '%s' is already loaded", fname); wxFileName fnpath(fnames[i]);
return; OPT_SET("Path/Last/Automation")->SetString(STD_STR(fnpath.GetPath()));
if (has_file(local_manager->GetScripts(), fnpath) || has_file(global_manager->GetScripts(), fnpath)) {
wxLogError("Script '%s' is already loaded", fnames[i]);
continue;
}
Automation4::Script *script = Automation4::ScriptFactory::CreateFromFile(fnames[i], true);
local_manager->Add(script);
AddScript(script, false);
} }
Automation4::Script *script = Automation4::ScriptFactory::CreateFromFile(fname, true);
local_manager->Add(script);
AddScript(script, false);
} }
void DialogAutomation::OnRemove(wxCommandEvent &) void DialogAutomation::OnRemove(wxCommandEvent &)