Author: mzugec Date: Sun Mar 16 14:32:06 2008 New Revision: 45498 URL: http://svn.opensuse.org/viewcvs/yast?rev=45498&view=rev Log: show confirm dialog only once Modified: trunk/network/src/lan/bridge.ycp Modified: trunk/network/src/lan/bridge.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/network/src/lan/bridge.ycp?rev=45498&r1=45497&r2=45498&view=diff ============================================================================== --- trunk/network/src/lan/bridge.ycp (original) +++ trunk/network/src/lan/bridge.ycp Sun Mar 16 14:32:06 2008 @@ -35,8 +35,6 @@ if (hasAnyValue(dev_name)) items = add(items, `item(`id(dev_name), sformat("%1 - %2", dev_name, description), contains(br_ports, dev_name))); } }); - - NetworkInterfaces::Select(old_name); UI::ChangeWidget(`id(key), `Items, items); } @@ -56,9 +54,12 @@ if (contains(sel, device) && hasAnyValue(device)) { if (contains(confs, device)){ if ((string)configurations[NetworkInterfaces::GetType(device), device, "IPADDR"]:"" != "0.0.0.0") { - if(!confirmed) confirmed = Popup::ContinueCancel(_("One or more selected devices are already configured. + if(!confirmed) { + valid = Popup::ContinueCancel(_("One or more selected devices are already configured. To work properly configuration must be removed.")); - if (confirmed) { + confirmed=true; + } + if (valid) { integer i = LanItems::current; if (LanItems::FindAndSelect(device)) { y2internal("delete %1", device); @@ -66,7 +67,7 @@ NetworkInterfaces::Add(); LanItems::current = i; } - } else valid=false; + } } } } -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org