Padraig, Will make those changes and spin new webrev. best Michal
Padraig O'Briain wrote: > Some of the dialogs are being displayed with type gtk.MESSAGE_QUESTION. > This looks wrong as we are just displaying information to the user, not > asking him to we should perform some action. > > Padraig > > On 11/13/08 18:12, Michal Pryc wrote: >> Hi, >> >> The webrev: >> http://cr.opensolaris.org/~migi/13_10_2008_bugs_4987_4990_v1 >> >> After adding each change I was testing if it works by raising this >> error or unplugging the network cable if I was testing network >> problems, so this should work fine. >> >> The bugs: >> >> 4990 Disable Repository rename >> URL: http://defect.opensolaris.org/bz/show_bug.cgi?id=4990 >> >> Lines in the code for this change (line numbers taken from the changed >> file): >> (93 - 95), (357-359) >> http://cr.opensolaris.org/~migi/13_10_2008_bugs_4987_4990_v1/src/gui/modules/repository.py.wdiff.html >> >> >> On the line 380 there is still "no defined..." string comparison, but >> we will never hit this as we disabled renaming of the repository and >> we will never go through the line (360) >> Similar situation is for line 400-403, we might hit those lines, but I >> didn't remove this check to not introduce too much risk. >> >> >> >> 4987 Improve error messages for PM >> URL: http://defect.opensolaris.org/bz/show_bug.cgi?id=4987 >> >> The resto of the changes: >> >> http://cr.opensolaris.org/~migi/13_10_2008_bugs_4987_4990_v1/src/packagemanager.py.wdiff.html >> >> >> line 1441-1445, because we are running with gksu this can only happen >> while user will start from the terminal without gksu and will try to >> edit repositories, then we are refreshing catalogs of course this will >> fail as the user doesn't have sufficient privilages but we already >> reported this while editing repositories. This is the only way of >> hitting those lines at the moment. Properly we shouldn't allow user to >> edit repositories in the first place, but such change will introduce >> too much risk in this stage. This is something to work on for the next >> release. >> >> line 1472 sometimes errors doesn't have data then this will fail so we >> need to check if data is in the error variables before checking if >> data contains any information. >> >> 1478-1479 we agreed to have nice info icon instead of error one for >> network problems. >> >> >> http://cr.opensolaris.org/~migi/13_10_2008_bugs_4987_4990_v1/src/gui/modules/installupdate.py.wdiff.html >> >> >> 42-44 and 515-516 I have checked to unplug the cable in each stage of >> the installation process and this needs to be added to show proper >> network problems dialog while we are downloading files. >> >> 368-369 If there is any question at the end of the error dialog we are >> showing nice blue "?" icon instead of red error sign. >> >> 339-340, 425-426, 457-458, If there is no question and the error is >> information to the user rather then error I've changed to show nice >> "i" icon rather then red error sign. >> >> 473-474 we need to check weather the args exists before we will take >> first argument - this failed when I was testing errors, that is why it >> is in that place. >> >> 495-507 I've splitted those two errors. The second one >> PlanMissingException will have detailed information in the text details. >> >> I was not changing the PlanCreationException for all actions, because >> they are fine, basically we wanted to have (example for Install/Update >> action) "Install/Update failure"with additional details view it is >> like this in the code. Somehow I've send wrong error message in >> yesterdays e-mail telling about network problem in the >> PlanCreationException: >> except api_errors.PlanCreationException, e: >> err_msg = self.parent._("Install/Update failure" \ >> " in plan creation.") >> err_text = str(e) >> gobject.idle_add(self.__error_with_details, \ >> err_msg, err_text) >> >> >> http://cr.opensolaris.org/~migi/13_10_2008_bugs_4987_4990_v1/src/gui/modules/repository.py.wdiff.html >> >> >> Most of the errors were changed to question icons or info icons the >> error ones are the real errors. >> > _______________________________________________ > pkg-discuss mailing list > [email protected] > http://mail.opensolaris.org/mailman/listinfo/pkg-discuss _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
