Author: mzugec Date: Wed Jun 11 14:02:29 2008 New Revision: 48228 URL: http://svn.opensuse.org/viewcvs/yast?rev=48228&view=rev Log: "change connection" - backend validation (bnc#388575) Modified: trunk/printer/VERSION trunk/printer/package/yast2-printer.changes trunk/printer/src/common/dialogs.ycp Modified: trunk/printer/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/printer/VERSION?rev=48228&r1=48227&r2=48228&view=diff ============================================================================== --- trunk/printer/VERSION (original) +++ trunk/printer/VERSION Wed Jun 11 14:02:29 2008 @@ -1 +1 @@ -2.17.0 +2.17.1 Modified: trunk/printer/package/yast2-printer.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/printer/package/yast2-printer.changes?rev=48228&r1=48227&r2=48228&view=diff ============================================================================== --- trunk/printer/package/yast2-printer.changes (original) +++ trunk/printer/package/yast2-printer.changes Wed Jun 11 14:02:29 2008 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Wed Jun 11 14:00:41 CEST 2008 - mzugec@suse.de + +- "change connection" - backend validation (bnc#388575) +- 2.17.1 + +------------------------------------------------------------------- Tue Jun 10 13:51:27 CEST 2008 - mzugec@suse.cz - automatic installation / not-fully supported printers (bnc#395397) Modified: trunk/printer/src/common/dialogs.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/printer/src/common/dialogs.ycp?rev=48228&r1=48227&r2=48228&view=diff ============================================================================== --- trunk/printer/src/common/dialogs.ycp (original) +++ trunk/printer/src/common/dialogs.ycp Wed Jun 11 14:02:29 2008 @@ -2628,8 +2628,17 @@ if (ret == `ok) { string uri = (string)UI::QueryWidget (`id (`connection_type), `Value); - y2milestone("changed uri %1 to %2", Printer::printer["uri"]:"", uri); - Printer::printer["uri"] = uri; + // backend validation (bnc#388575) + string cmd = "lpinfo -h localhost -v | cut -s -d ' ' -f2 | cut -d ':' -f1 | sort -u"; + map <string, any> exec = (map<string, any>)SCR::Execute(.target.bash_output, cmd); + if (exec["exit"]:-1==0){ + list<string> backends = filter(string key, splitstring(exec["stdout"]:"", "\n"), {return (size(key)>0);}); + if (!contains(backends, splitstring(uri, ":")[0]:"")) { + Popup::Warning(_("Selected backend is not valid!")); + } else Printer::printer["uri"] = uri; + } else y2warning("validation script failed : %1", exec["exit"]:-1); + y2milestone("new backend %1", splitstring(uri, ":")[0]:""); + y2milestone("changed uri %1 to %2", Printer::printer["uri"]:"", uri); } } -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org