On 01/26/10 15:00, Padraig O'Briain wrote:


On 01/26/10 12:19, Padraig O'Briain wrote:


On 01/26/10 08:38, Padraig O'Briain wrote:


On 01/25/10 18:01, Shawn Walker wrote:
On 01/25/10 03:03 AM, Padraig O'Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-14003-v2/, fixes
14003 PM Publisher alias is not visible in the Publisher dropdown

This webrev changes Package Manager to use publisher alias instead of
publisher prefix.
It is used in the following places:

Publisher combobox
Search All Publishers page
Status bar when searching
Recent Searches list
Search results

If the alias name is not unique then <alias> (<prefix>) in the first 4
places listed above and <alias> is used in the Search results.

In the General info tab <alias> (<prefix>) is used for the Publisher if
there is an alias.

You should be able to assume that the alias is unique. The API does check for uniqueness when adding publishers. The bug is that it doesn't check for uniqueness when updating them.


I will leave our code as it is until 9218 is fixed.

I have respun the webrev, http://cr.opensolaris.org/~padraig/ips-14003-v3/, to change what is displayed for a publishers in Search All Publishers to <alias> (<prefix>) is used for the Publisher if there is an alias.

I have spun another webrev, http://cr.opensolaris.org/~padraig/ips-14003-v4/, to display alias if it exists in list of installed packages.

Another webrev, http://cr.opensolaris.org/~padraig/ips-14003-v5/, to display alias when reporting no search results found.

I have spun another webrev, http://cr.opensolaris.org/~padraig/ips-14003-v6/, to do some code refactoring after off-line review with John. The behavior of this webrev should be the same as the previous one.

Padraig


Padraig


Padraig
_______________________________________________
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

Reply via email to