Add confirmation on customised components
Signed-off-by: TheKodeToad <TheKodeToad@proton.me>
This commit is contained in:
parent
127b094c41
commit
ee003cd9ee
@ -318,14 +318,30 @@ void VersionPage::on_actionReload_triggered()
|
||||
|
||||
void VersionPage::on_actionRemove_triggered()
|
||||
{
|
||||
if (ui->packageView->currentIndex().isValid())
|
||||
if (!ui->packageView->currentIndex().isValid())
|
||||
{
|
||||
return;
|
||||
}
|
||||
int index = ui->packageView->currentIndex().row();
|
||||
auto component = m_profile->getComponent(index);
|
||||
if (component->isCustom())
|
||||
{
|
||||
auto response = CustomMessageBox::selectable(this, tr("CAREFUL!"),
|
||||
tr("About to remove: %1\n"
|
||||
"This is permanent and will completely remove the custom component.\n\n"
|
||||
"Are you sure?")
|
||||
.arg(component->getName()),
|
||||
QMessageBox::Warning, QMessageBox::Yes | QMessageBox::No, QMessageBox::No)
|
||||
->exec();
|
||||
|
||||
if (response != QMessageBox::Yes)
|
||||
return;
|
||||
}
|
||||
// FIXME: use actual model, not reloading.
|
||||
if (!m_profile->remove(ui->packageView->currentIndex().row()))
|
||||
if (!m_profile->remove(index))
|
||||
{
|
||||
QMessageBox::critical(this, tr("Error"), tr("Couldn't remove file"));
|
||||
}
|
||||
}
|
||||
updateButtons();
|
||||
reloadPackProfile();
|
||||
m_container->refreshContainer();
|
||||
@ -707,6 +723,19 @@ void VersionPage::on_actionRevert_triggered()
|
||||
{
|
||||
return;
|
||||
}
|
||||
auto component = m_profile->getComponent(version);
|
||||
|
||||
auto response = CustomMessageBox::selectable(this, tr("CAREFUL!"),
|
||||
tr("About to revert: %1\n"
|
||||
"This is permanent and will completely revert your customizations.\n\n"
|
||||
"Are you sure?")
|
||||
.arg(component->getName()),
|
||||
QMessageBox::Warning, QMessageBox::Yes | QMessageBox::No, QMessageBox::No)
|
||||
->exec();
|
||||
|
||||
if (response != QMessageBox::Yes)
|
||||
return;
|
||||
|
||||
if(!m_profile->revertToBase(version))
|
||||
{
|
||||
// TODO: some error box here
|
||||
|
Loading…
Reference in New Issue
Block a user