Smarter double-click

launcher/modManager/cmodlistview_moc.cpp

@@ -893,29 +893,46 @@ void CModListView::on_allModsView_doubleClicked(const QModelIndex &index)
 {
 	if(!index.isValid())
 		return;
-
-	auto mod = modModel->getMod(index.data(ModRoles::ModNameRole).toString());
-
-	if(mod.isAvailable() && !mod.getName().contains('.'))
+	
+	auto modName = index.data(ModRoles::ModNameRole).toString();
+	auto mod = modModel->getMod(modName);
+	
+	bool hasInvalidDeps = !findInvalidDependencies(modName).empty();
+	bool hasBlockingMods = !findBlockingMods(modName).empty();
+	bool hasDependentMods = !findDependentMods(modName, true).empty();
+	
+	if(!hasInvalidDeps && mod.isAvailable() && !mod.getName().contains('.'))
 	{
 		on_installButton_clicked();
 		return;
 	}
 
-	if(mod.isUpdateable())
+	if(!hasInvalidDeps && !hasDependentMods && mod.isUpdateable() && index.column() == ModFields::STATUS_UPDATE)
 	{
 		on_updateButton_clicked();
+		return;
+	}
+	
+	if(index.column() == ModFields::NAME)
+	{
+		if(ui->allModsView->isExpanded(index))
+			ui->allModsView->collapse(index);
+		else
+			ui->allModsView->expand(index);
+		
+		return;
 	}
 
-	if(mod.isDisabled())
+	if(!hasBlockingMods && !hasInvalidDeps && mod.isDisabled())
 	{
 		on_enableButton_clicked();
 		return;
 	}
 
-	if(mod.isEnabled())
+	if(!hasDependentMods && !mod.isEssential() && mod.isEnabled())
 	{
 		on_disableButton_clicked();
+		return;
 	}
 }