Author: mzugec Date: Sun May 17 22:11:45 2009 New Revision: 57221 URL: http://svn.opensuse.org/viewcvs/yast?rev=57221&view=rev Log: code clean fixed: propose CHAN_IDS Modified: trunk/network/src/lan/complex.ycp trunk/network/src/lan/hardware.ycp trunk/network/src/modules/Lan.ycp Modified: trunk/network/src/lan/complex.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/network/src/lan/complex.ycp?rev=57221&r1=57220&r2=57221&view=diff ============================================================================== --- trunk/network/src/lan/complex.ycp (original) +++ trunk/network/src/lan/complex.ycp Sun May 17 22:11:45 2009 @@ -112,6 +112,7 @@ define boolean AddInterface () { Lan::Add (); + LanItems::operation = `add; LanItems::SelectHWMap(LanItems::getCurrentItem()["hwinfo"]:$[]); LanItems::Items[LanItems::current, "ifcfg"] = LanItems::getCurrentItem()["hwinfo", "dev_name"]:""; LanItems::Items[LanItems::current, "commited"] = false; Modified: trunk/network/src/lan/hardware.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/network/src/lan/hardware.ycp?rev=57221&r1=57220&r2=57221&view=diff ============================================================================== --- trunk/network/src/lan/hardware.ycp (original) +++ trunk/network/src/lan/hardware.ycp Sun May 17 22:11:45 2009 @@ -49,7 +49,7 @@ configured with the same module name, options will be merged while saving.</p> "); - if(!(LanItems::operation == `edit /* FIXME: unique || Lan::unique != "" */ || Arch::s390 () == true)) + if(!(LanItems::operation == `edit || Arch::s390 ())) { /* Manual dialog help 4/4 */ @@ -153,8 +153,7 @@ ); /* Disable PCMCIA and USB checkboxex on Edit and s390 */ - if(LanItems::operation == `edit /* FIXME: unique || Lan::unique != "" */ || Arch::s390 () == true) - CheckBoxes = `VSpacing(0); + if(LanItems::operation == `edit || Arch::s390 ()) CheckBoxes = `VSpacing(0); // #116211 - allow user to change modules from list /* Frame label */ @@ -191,13 +190,6 @@ ) ); - /* Add hardware -> don't allow some changes (card type) */ - // FIXME: devname if(Lan::operation == `add && Lan::unique != "") { - - /* Edit -> don't allow some changes */ - // FIXME: devname if(true /* FIXME: devname Lan::operation == `edit */) { - - term UdevWidget = // TODO: Ud ... Rules `Frame(_("Udev rules"), @@ -483,7 +475,7 @@ } void storeHW (string key, map event) { -if (LanItems::operation==`add){ + if (LanItems::operation==`add){ LanItems::type = (string) UI::QueryWidget(`id(`dev), `Value); string nm = sformat("%1%2", LanItems::type, (string) UI::QueryWidget(`id(`num), `Value)); Modified: trunk/network/src/modules/Lan.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/network/src/modules/Lan.ycp?rev=57221&r1=57220&r2=57221&view=diff ============================================================================== --- trunk/network/src/modules/Lan.ycp (original) +++ trunk/network/src/modules/Lan.ycp Sun May 17 22:11:45 2009 @@ -685,13 +685,7 @@ */ global define boolean Delete() { LanItems::DeleteItem(); -/* - operation = nil; - if(Select(name) != true) return false; - NetworkInterfaces::Delete(name); - operation = `delete; -*/ - return true; + return true; } -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org