PackageSource.Name property, is it necessary?

Sep 15, 2010 at 11:57 PM

Currently the PackageSource class has a Name property, in addition to the Source property. My original intent is that Name represents a friendly, easy-to-remember name of the package source, e.g. CodePlex, Local, etc.

However, the Name property is currently not displayed anywhere. So I wonder if we should remove it. Thoughts?

Sep 16, 2010 at 12:00 AM

Remove it, add it back

Sep 16, 2010 at 12:02 AM

I think it would make the UI a bit nicer.  If we had that, the drop down would just display the name and not the URL.  e.g. it's nice to see just 'Codeplex' or 'MyCoolFeed' in there.

And if someone really wants to see URL's they can always set the name to be the same as the URL.

Sep 16, 2010 at 12:03 AM

But I should add that this is low priority.  We can certainly get by with just the URL for now.

Sep 16, 2010 at 12:05 AM

Particularly if we are going to go with an fwlink, the name would be nice. But like davidebb said, not high priority. I'd agree with erichexter; remove it for now, and we can add it back if or when make need it.