diff --git a/build_num b/build_num index 1b8483810..a41f864c0 100644 --- a/build_num +++ b/build_num @@ -1 +1 @@ -#define BUILD_NUM 1773 +#define BUILD_NUM 1776 diff --git a/src/gui/utils.cpp b/src/gui/utils.cpp index 4f6b8f169..1d09350fd 100644 --- a/src/gui/utils.cpp +++ b/src/gui/utils.cpp @@ -202,10 +202,13 @@ bool isTreeReadWrite(QWidget *parent, FWObject *obj) { if (obj->isReadOnly()) { - QMessageBox::warning(parent, "Firewall Builder", - QObject::tr("Impossible to apply changes because object is located in read-only\npart of the tee or data file was opened read-only"), - QObject::tr("&Continue"), 0, 0, - 0, 2 ); + QMessageBox::warning( + parent, "Firewall Builder", + QObject::tr("Impossible to apply changes because object is " + "located in read-only\npart of the tree or data " + "file was opened read-only"), + QObject::tr("&Continue"), 0, 0, + 0, 2 ); return false; } @@ -215,19 +218,30 @@ bool isTreeReadWrite(QWidget *parent, FWObject *obj) /* * compare names as QString objects to catch non-ascii names */ -bool validateName(QWidget *parent, - libfwbuilder::FWObject *obj,const QString &newname) +bool validateName(QWidget *parent, FWObject *obj, const QString &newname) { FWObject *p = obj->getParent(); for (FWObject::iterator i=p->begin(); i!=p->end(); ++i) { FWObject *o1= *i; - if (QString(o1->getName().c_str())==newname && o1!=obj) + // Another hack: we need to be able to create policy and nat + // ruleset objects with name "ftp-proxy/*" for PF. Allow + // objects of different type to have the same name. + + qDebug() << "o1=" << o1 << "obj=" << obj + << "o1->getTypeName()=" << o1->getTypeName().c_str() + << "obj->getTypeName()=" << obj->getTypeName().c_str(); + + if (o1 == obj || o1->getTypeName() != obj->getTypeName()) continue; + + if (QString(o1->getName().c_str()) == newname) { - QMessageBox::warning(parent, "Firewall Builder", - QObject::tr("Object with name '%1' already exists, please choose different name."). - arg(o1->getName().c_str()), - QObject::tr("&Continue editing"), NULL, NULL, 0, 2 ); + QMessageBox::warning( + parent, "Firewall Builder", + QObject::tr("Object with name '%1' already exists, " + "please choose different name."). + arg(o1->getName().c_str()), + QObject::tr("&Continue editing"), NULL, NULL, 0, 2 ); return false; } } diff --git a/test/pf/objects-for-regression-tests.fwb b/test/pf/objects-for-regression-tests.fwb index 2b0394166..0e719d556 100644 --- a/test/pf/objects-for-regression-tests.fwb +++ b/test/pf/objects-for-regression-tests.fwb @@ -1,6 +1,6 @@ - + @@ -94,11 +94,11 @@ - + - + @@ -109,7 +109,7 @@ - + @@ -748,16 +748,16 @@ - + - + - + @@ -956,15 +956,15 @@ - + - + - + @@ -1177,7 +1177,7 @@ - + @@ -1186,7 +1186,7 @@ - + @@ -1203,7 +1203,7 @@ - + @@ -1220,7 +1220,7 @@ - + @@ -1234,7 +1234,7 @@ - + @@ -1248,7 +1248,7 @@ - + @@ -1257,7 +1257,7 @@ - + @@ -1266,7 +1266,7 @@ - + @@ -1275,7 +1275,7 @@ - + @@ -1284,7 +1284,7 @@ - + @@ -1298,7 +1298,7 @@ - + @@ -1312,7 +1312,7 @@ - + @@ -1327,7 +1327,7 @@ - + @@ -1345,7 +1345,7 @@ - + @@ -1363,7 +1363,7 @@ - + @@ -1381,7 +1381,7 @@ - + @@ -1399,7 +1399,7 @@ - + @@ -1417,7 +1417,7 @@ - + @@ -1435,7 +1435,7 @@ - + @@ -1453,7 +1453,7 @@ - + @@ -1471,7 +1471,7 @@ - + @@ -1489,7 +1489,7 @@ - + @@ -1503,7 +1503,7 @@ - + @@ -1517,7 +1517,7 @@ - + @@ -1641,6 +1641,7 @@ + @@ -2342,15 +2343,15 @@ - + - + - + @@ -3238,23 +3239,23 @@ - + - + - + - + - + @@ -3508,11 +3509,11 @@ - + - + @@ -4585,23 +4586,23 @@ - + - + - + - + - + @@ -5117,23 +5118,23 @@ - + - + - + - + - + @@ -5367,15 +5368,15 @@ - + - + - + @@ -5496,23 +5497,23 @@ - + - + - + - + - + @@ -5629,23 +5630,23 @@ - + - + - + - + - + @@ -6028,20 +6029,20 @@ - + - + - + - + @@ -6228,14 +6229,14 @@ - + - + - + @@ -6634,14 +6635,14 @@ - + - + - + @@ -7140,14 +7141,14 @@ - + - + - + @@ -7527,18 +7528,18 @@ - + - + - + - + @@ -7858,16 +7859,16 @@ - + - + - + @@ -8360,18 +8361,18 @@ - + - + - + - + @@ -8730,19 +8731,19 @@ - + - + - + - + @@ -8915,11 +8916,11 @@ - + - + @@ -9132,14 +9133,14 @@ - + - + - + @@ -9428,14 +9429,14 @@ - + - + - + @@ -9728,14 +9729,14 @@ - + - + - + @@ -10024,14 +10025,14 @@ - + - + - + @@ -10355,14 +10356,14 @@ - + - + - + @@ -10651,14 +10652,14 @@ - + - + - + @@ -12215,19 +12216,19 @@ - + - + - + - + @@ -12629,12 +12630,12 @@ - + - + @@ -12976,15 +12977,15 @@ - + - + - + @@ -13284,15 +13285,15 @@ - + - + - + @@ -13679,11 +13680,11 @@ - + - + @@ -13974,15 +13975,15 @@ - + - + - + @@ -14260,20 +14261,20 @@ - + - + - + - + @@ -14616,12 +14617,12 @@ - + - + @@ -14872,19 +14873,19 @@ - + - + - + - + @@ -15309,15 +15310,15 @@ - + - + - + @@ -15703,23 +15704,23 @@ - + - + - + - + - + @@ -15978,23 +15979,23 @@ - + - + - + - + - + @@ -16124,7 +16125,7 @@ - + @@ -16170,7 +16171,51 @@ - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -16214,7 +16259,7 @@ - + @@ -16284,7 +16329,7 @@ - + @@ -16301,20 +16346,63 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - + - + - + @@ -16397,7 +16485,7 @@ - + @@ -16420,12 +16508,12 @@ - + - + @@ -16579,15 +16667,15 @@ - + - + - + @@ -17067,15 +17155,15 @@ - + - + - + @@ -17362,15 +17450,15 @@ - + - + - + @@ -17670,11 +17758,11 @@ - + - + @@ -17990,11 +18078,11 @@ - + - + @@ -18198,11 +18286,11 @@ - + - + @@ -18511,20 +18599,20 @@ - + - + - + - + @@ -18643,7 +18731,7 @@ - + @@ -18660,7 +18748,7 @@ - + @@ -18776,19 +18864,19 @@ - + - + - + - +