Author: kmachalkova Date: Wed Jun 11 14:36:50 2008 New Revision: 48230 URL: http://svn.opensuse.org/viewcvs/yast?rev=48230&view=rev Log: Do not screw the whole dialog up when user decides to cancel 'Browse' operation ( pass 'nil' filename) (#397851) Modified: trunk/apparmor/src/include/subdomain/reporting_dialogues.ycp Modified: trunk/apparmor/src/include/subdomain/reporting_dialogues.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/apparmor/src/include/subdomain/reporting_dialogues.ycp?rev=48230&r1=48229&r2=48230&view=diff ============================================================================== --- trunk/apparmor/src/include/subdomain/reporting_dialogues.ycp (original) +++ trunk/apparmor/src/include/subdomain/reporting_dialogues.ycp Wed Jun 11 14:36:50 2008 @@ -775,7 +775,7 @@ boolean audRep = archType["audRep"]:false; boolean essRep = archType["essRep"]:false; - if ( repPath == "" ) { + if ( repPath == "" || repPath == nil ) { repPath = "/var/log/apparmor/reports-archived/"; } @@ -1176,15 +1176,15 @@ if ( selectFile != nil ) { UI::ChangeWidget(`id(`repPath), `Value, selectFile); - } - - // set new reppath - repPath = selectFile; - Settings["repPath"] = repPath; - itemList = getArrayList(type,repPath); - Wizard::SetContentsButtons( _("AppArmor Security Event Report"), + // set new reppath + repPath = selectFile; + Settings["repPath"] = repPath; + itemList = getArrayList(type,repPath); + Wizard::SetContentsButtons( _("AppArmor Security Event Report"), viewForm(archType, itemList, repPath), archHelpText, Label::BackButton(), _("&Done") ); + } + } else if ( archId == `sirRep ) { formHelp = sirHelp; -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org