mrward / monodevelop-nuget-extensions

Extends the MonoDevelop NuGet addin
MIT License
72 stars 9 forks source link

Package Console Extension pad id clashes with existing Package Console #16

Closed mrward closed 7 years ago

mrward commented 7 years ago

The pad id contains PackageConsole which clashes with this check in the main NuGet addin:

https://github.com/mono/monodevelop/blob/master/main/src/addins/MonoDevelop.PackageManagement/MonoDevelop.PackageManagement/PackageManagementProgressMonitorFactory.cs#L76

So sometimes the wrong pad is found. Changing the pad id in the NuGet extensions addin should fix this. Perhaps a better fix would be to change the progress monitor code to check the pad id starts with "OutputPad-PackageConsole" which should handle the case better.

mrward commented 7 years ago

This has been fixed in monodevelop.