aly/servatrice
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge pull request #1444 from woogerboy21/fix_blank_ban

Browse source 
Detect if a use is trying to place an invalid ban
This commit is contained in:
Zach 2015-09-01 22:20:05 -04:00
commit 3188ef4cde
1 changed files with 19 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
cockatrice/src/userlist.cpp
View file

@ -122,6 +122,25 @@ void BanDialog::okClicked()
QMessageBox::critical(this, tr("Error"), tr("You have to select a name-based, IP-based, clientId based, or some combination of the three to place a ban."));
return;
}
if (nameBanCheckBox->isChecked())
if (nameBanEdit->text() == ""){
QMessageBox::critical(this, tr("Error"), tr("You must have at value in the name ban when selecting the name ban checkbox."));
return;
}
if (ipBanCheckBox->isChecked())
if (ipBanEdit->text() == ""){
QMessageBox::critical(this, tr("Error"), tr("You must have at value in the ip ban when selecting the ip ban checkbox."));
return;
}
if (idBanCheckBox->isChecked())
if (idBanCheckBox->text() == ""){
QMessageBox::critical(this, tr("Error"), tr("You must have at value in the clientid ban when selecting the clientid ban checkbox."));
return;
}
accept();
}