PackageInstaller needs a "Yes to all" button in dialog box

[scottmc]

When reinstalling from a pkg file sometimes the files already exist, so you get a dialog box where it asks if you want to overwrite it, and your options are yes, no, and abort. It would be nice to have an option to answer "Yes to all".

[scottmc]

Not much fun clicking yes 54 times...

[sil2100]

Indeed, I will take care of it ASAP.

[sil2100]

Possible fix of the problem

[sil2100]

I am sorry this took so long. I made this patch a few days ago, but the tracker was down, and I had to leave for a trip later that day. But I finally found a place to connect to.

This patch does the thing in mention a bit differently. Since the maximum number of buttons in a BAlarm is 3, I couldn't add a 'Yes to all' button - it's not really a good idea anyway, since the number of buttons would make the alarm less readable. Instead, when a conflict appears, a second dialog box appears asking the user if the PackageInstaller should remember the previous choice for all conflicts in the future. Right now the installer will ask only once, but a 'Ask later' button is possible - although I can't find as much use to such thing, it would probably just unnecessarily bloat the alert.

I also exported the user asking mechanism to the View instead of the inner structures. This way seems much better. Tested on r31688.

[stippi]

The patch looks pretty nice to me, although I have not actually looked for any bugs. You have commit access, right? ;-) My suggestions for improvements would be two things:

1) You should never use wording in the UI that makes the computer into a person ("Should I remove the file..."). Instead, use passive language like "Should the file be removed...". That sounds much more professional.

2) Maybe "fCurrentPolicy" is not the best name. Policy for what? Perhaps "fCurrentItemExistsPolicy" would be better.

[sil2100]

Thanks for reviewing the patch, stippi. I do not have commit access, so it will be necessary for someone to apply the patch for me. I have fixed the issues you have highlighted and attached the patch here as well. Thanks!

[sil2100]

Second revision of the patch

[axeld]

Applied in r32161 with some minor style changes (most notably, removed some extraneous parenthesis in combination with the '&' operator). Thanks!