Author: lslezak Date: Mon Nov 23 17:27:24 2009 New Revision: 59722 URL: http://svn.opensuse.org/viewcvs/yast?rev=59722&view=rev Log: - repositories - added NFSv4 support (fate#306451), don't loose the current URL when switching between full and parsed view Modified: trunk/packager/package/yast2-packager.changes trunk/packager/src/modules/SourceDialogs.ycp Modified: trunk/packager/package/yast2-packager.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/package/yast2-packager.changes?rev=59722&r1=59721&r2=59722&view=diff ============================================================================== --- trunk/packager/package/yast2-packager.changes (original) +++ trunk/packager/package/yast2-packager.changes Mon Nov 23 17:27:24 2009 @@ -10,6 +10,8 @@ (bnc#549866) - repositories - improved ISO image file detection (use plain text instead of MIME type for 'file' output) (bnc#547683) +- repositories - added NFSv4 support (fate#306451), don't loose + the current URL when switching between full and parsed view ------------------------------------------------------------------- Fri Nov 20 15:57:12 CET 2009 - lslezak@suse.cz Modified: trunk/packager/src/modules/SourceDialogs.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/src/modules/SourceDialogs.ycp?rev=59722&r1=59721&r2=59722&view=diff ============================================================================== --- trunk/packager/src/modules/SourceDialogs.ycp (original) +++ trunk/packager/src/modules/SourceDialogs.ycp Mon Nov 23 17:27:24 2009 @@ -419,7 +419,9 @@ ) ), // checkbox label - `Left (`CheckBox ( `id (`ch_iso), _("&ISO Image"))) + `Left (`CheckBox ( `id (`ch_iso), _("&ISO Image"))), + // checkbox label + `Left (`CheckBox ( `id (`ch_nfs4), _("N&FS v4 Protocol"))) ); term nfs_complete_content = `InputField(`id(`complete_url), `opt(`hstretch), _("URL of the Repository")); @@ -449,6 +451,13 @@ UI::ChangeWidget (`id (`dir), `Value, parsed["path"]:""); UI::ChangeWidget (`id (`ch_iso), `Value, iso); UI::SetFocus (`server); + + boolean nfs4 = tolower(parsed["scheme"]:"nfs") == "nfs4" || + URL::MakeMapFromParams(parsed["query"]:"")["type"]:"" == "nfs4"; + + y2milestone("NFSv4: %1", nfs4); + + UI::ChangeWidget (`id (`ch_nfs4), `Value, nfs4); } else { @@ -456,6 +465,48 @@ } } +void NFSStoreParts() +{ + map parsed = $[ + "scheme" : "nfs", + "host" : NormalizeHost ( + (string)UI::QueryWidget (`id (`server), `Value)), + "path" : (string)UI::QueryWidget (`id (`dir), `Value), + ]; + + boolean nfs4 = (boolean)UI::QueryWidget (`id (`ch_nfs4), `Value); + if (nfs4) + { + // keep nfs4:// if it is used in the original URL + if (tolower(URL::Parse(_url)["scheme"]:"") == "nfs4") + { + parsed["scheme"] = "nfs4"; + } + else + { + parsed["query"] = "type=nfs4"; + } + } + + _url = URL::Build (parsed); + boolean iso = (boolean)UI::QueryWidget (`id (`ch_iso), `Value); + + // workaround: URL::Build does not accept numbers in scheme, + // for nfs4 scheme it returns URL with no scheme (like "://foo/bar") + if(!regexpmatch(_url, "^nfs")) + { + _url = "nfs4" + _url; + } + + if (iso) + _url = PosprocessISOURL (_url); +} + +void NFSStoreComplete() +{ + _url = (string)UI::QueryWidget(`id(`complete_url), `Value); +} + /** * Store function of a widget * @param key string widget key @@ -467,20 +518,11 @@ if (current_type == `edit_url_parts) { - map parsed = $[ - "scheme" : "nfs", - "host" : NormalizeHost ( - (string)UI::QueryWidget (`id (`server), `Value)), - "path" : (string)UI::QueryWidget (`id (`dir), `Value), - ]; - _url = URL::Build (parsed); - boolean iso = (boolean)UI::QueryWidget (`id (`ch_iso), `Value); - if (iso) - _url = PosprocessISOURL (_url); + NFSStoreParts(); } else { - _url = (string)UI::QueryWidget(`id(`complete_url), `Value); + NFSStoreComplete(); } } @@ -518,7 +560,17 @@ } else if ((event["ID"]:nil == `edit_url_parts || event["ID"]:nil == `edit_complete_url) && event["EventReason"]:"" == "ValueChanged") { - y2milestone("Changing dialog type"); + y2milestone("Changing dialog type: %1", event["ID"]:nil); + + // store the current settings + if (event["ID"]:nil == `edit_url_parts) + { + NFSStoreComplete(); + } + else + { + NFSStoreParts(); + } // reinitialize the dialog (set the current values) NFSInit(nil); @@ -1195,7 +1247,8 @@ `ReplacePoint (`id (`server_rp), `Empty ()) ); -term complete_content = `InputField(`id(`complete_url), `opt(`hstretch), _("URL of the Repository")); +// input field label +term complete_content = `InputField(`id(`complete_url), `opt(`hstretch), _("&URL of the Repository")); void ServerStoreParts() { @@ -1876,6 +1929,7 @@ else if ( _url == "https://" ) current = `https; else if ( _url == "smb://" ) current = `samba; else if ( _url == "nfs://" ) current = `nfs; + else if ( _url == "nfs4://" ) current = `nfs; else if ( _url == "cd:///" ) current = `cd; else if ( _url == "dvd:///" ) current = `dvd; else if ( _url == "hd://" ) current = `hd; -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org