loot/libloot
0
0
Fork 0
mirror of https://github.com/loot/libloot.git synced 2026-07-27 14:16:01 -07:00
Code Issues Packages Projects Releases Wiki Activity

Implemented validity checks when moving plugins (issue #61).

Browse Source
This commit is contained in:
WrinklyNinja
2013-09-19 10:13:47 +01:00
parent d9400ea57d
commit 157eca5c36
1 changed files with 33 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/gui/main.cpp
+33
View File
@@ -1064,6 +1064,23 @@ void LoadOrderPreview::OnMoveUp(wxCommandEvent& event) {
BOOST_LOG_TRIVIAL(trace) << "Moving plugin \"" << string(selectedText.ToUTF8());
//Check that move is OK.
list<boss::Plugin>::const_iterator selectedPlugin = find(_plugins.begin(), _plugins.end(), boss::Plugin(string(selectedText.ToUTF8())));
list<boss::Plugin>::const_iterator abovePlugin = find(_plugins.begin(), _plugins.end(), boss::Plugin(string(aboveText.ToUTF8())));
if (selectedPlugin != _plugins.end() && abovePlugin != _plugins.end()) {
if (selectedPlugin->MustLoadAfter(*abovePlugin)) {
BOOST_LOG_TRIVIAL(error) << "Cannot load \"" << selectedPlugin->Name() << "\" before \"" << abovePlugin->Name() << "\".";
wxMessageBox(
FromUTF8(format(loc::translate("Error: Cannot load \"%1%\" before \"%2%\".")) % selectedPlugin->Name() % abovePlugin->Name()),
translate("BOSS: Error"),
wxOK | wxICON_ERROR,
NULL);
selected = _loadOrder->GetNextSelected(selected);
continue;
}
}
_loadOrder->SetItemText(selected, aboveText);
_loadOrder->SetItemText(selected - 1, selectedText);
@@ -1092,6 +1109,22 @@ void LoadOrderPreview::OnMoveDown(wxCommandEvent& event) {
BOOST_LOG_TRIVIAL(trace) << "Moving plugin \"" << string(selectedText.ToUTF8());
//Check that move is OK.
list<boss::Plugin>::const_iterator selectedPlugin = find(_plugins.begin(), _plugins.end(), boss::Plugin(string(selectedText.ToUTF8())));
list<boss::Plugin>::const_iterator belowPlugin = find(_plugins.begin(), _plugins.end(), boss::Plugin(string(belowText.ToUTF8())));
if (selectedPlugin != _plugins.end() && belowPlugin != _plugins.end()) {
if (belowPlugin->MustLoadAfter(*selectedPlugin)) {
BOOST_LOG_TRIVIAL(error) << "Cannot load \"" << belowPlugin->Name() << "\" before \"" << selectedPlugin->Name() << "\".";
wxMessageBox(
FromUTF8(format(loc::translate("Error: Cannot load \"%1%\" before \"%2%\".")) % belowPlugin->Name() % selectedPlugin->Name()),
translate("BOSS: Error"),
wxOK | wxICON_ERROR,
NULL);
continue;
}
}
_loadOrder->SetItemText(i, belowText);
_loadOrder->SetItemText(i + 1, selectedText);