ResourcePacks: Fix various bugs

This commit is contained in:
spycrab 2019-02-07 23:02:44 +01:00
parent 2987e8313a
commit f86b34ceb9
4 changed files with 30 additions and 15 deletions

View File

@ -160,6 +160,12 @@ void ResourcePackManager::RepopulateTable()
SelectionChanged();
}
// Revert the indicies as to be more intuitive for users
int ResourcePackManager::GetResourcePackIndex(QTableWidgetItem* item) const
{
return m_table_widget->rowCount() - 1 - item->row();
}
void ResourcePackManager::Change()
{
auto items = m_table_widget->selectedItems();
@ -167,10 +173,14 @@ void ResourcePackManager::Change()
if (items.empty())
return;
if (ResourcePack::IsInstalled(ResourcePack::GetPacks()[items[0]->row()]))
if (ResourcePack::IsInstalled(ResourcePack::GetPacks()[GetResourcePackIndex(items[0])]))
{
Uninstall();
}
else
{
Install();
}
}
void ResourcePackManager::Install()
@ -180,7 +190,7 @@ void ResourcePackManager::Install()
if (items.empty())
return;
auto& item = ResourcePack::GetPacks()[m_table_widget->rowCount() - 1 - items[0]->row()];
auto& item = ResourcePack::GetPacks()[GetResourcePackIndex(items[0])];
bool success = item.Install(File::GetUserPath(D_USER_IDX));
@ -201,7 +211,7 @@ void ResourcePackManager::Uninstall()
if (items.empty())
return;
auto& item = ResourcePack::GetPacks()[m_table_widget->rowCount() - 1 - items[0]->row()];
auto& item = ResourcePack::GetPacks()[GetResourcePackIndex(items[0])];
bool success = item.Uninstall(File::GetUserPath(D_USER_IDX));
@ -232,8 +242,7 @@ void ResourcePackManager::Remove()
return;
Uninstall();
File::Delete(
ResourcePack::GetPacks()[m_table_widget->rowCount() - 1 - items[0]->row()].GetPath());
File::Delete(ResourcePack::GetPacks()[GetResourcePackIndex(items[0])].GetPath());
RepopulateTable();
}
@ -244,7 +253,7 @@ void ResourcePackManager::PriorityDown()
if (items.empty())
return;
int row = m_table_widget->rowCount() - 1 - items[0]->row();
auto row = GetResourcePackIndex(items[0]);
if (items[0]->row() >= m_table_widget->rowCount())
return;
@ -269,7 +278,7 @@ void ResourcePackManager::PriorityUp()
if (items.empty())
return;
int row = m_table_widget->rowCount() - 1 - items[0]->row();
auto row = GetResourcePackIndex(items[0]);
if (items[0]->row() == 0)
return;
@ -301,7 +310,8 @@ void ResourcePackManager::SelectionChanged()
if (has_selection)
{
m_change_button->setText(ResourcePack::IsInstalled(ResourcePack::GetPacks()[items[0]->row()]) ?
m_change_button->setText(
ResourcePack::IsInstalled(ResourcePack::GetPacks()[GetResourcePackIndex(items[0])]) ?
tr("Uninstall") :
tr("Install"));
}

View File

@ -31,6 +31,8 @@ private:
void SelectionChanged();
void ItemDoubleClicked(QTableWidgetItem* item);
int GetResourcePackIndex(QTableWidgetItem* item) const;
QPushButton* m_open_directory_button;
QPushButton* m_change_button;
QPushButton* m_remove_button;

View File

@ -60,6 +60,7 @@ bool Init()
continue;
}
if (i < packs.size())
order->Set(packs[i].GetManifest()->GetID(), static_cast<u64>(i));
}

View File

@ -265,7 +265,8 @@ bool ResourcePack::Uninstall(const std::string& path)
// Check if a higher priority pack already provides a given texture, don't delete it
for (const auto& pack : GetHigherPriorityPacks(*this))
{
if (std::find(pack->GetTextures().begin(), pack->GetTextures().end(), texture) !=
if (::ResourcePack::IsInstalled(*pack) &&
std::find(pack->GetTextures().begin(), pack->GetTextures().end(), texture) !=
pack->GetTextures().end())
{
provided_by_other_pack = true;
@ -279,7 +280,8 @@ bool ResourcePack::Uninstall(const std::string& path)
// Check if a lower priority pack provides a given texture - if so, install it.
for (auto& pack : lower)
{
if (std::find(pack->GetTextures().rbegin(), pack->GetTextures().rend(), texture) !=
if (::ResourcePack::IsInstalled(*pack) &&
std::find(pack->GetTextures().rbegin(), pack->GetTextures().rend(), texture) !=
pack->GetTextures().rend())
{
pack->Install(path);