Error in detection of highest installed package version about chocolateygui HOT 9 CLOSED

there is a cver all -localonly but it seems slow. But does that do a better job?

I have an IComparer at work that could serve this purpose. Then it would become versionQuery.ToList().Sort(New VersionComparer).Last();

I'll if I can adapt the comparer to do what we need. And write the needed unittests.

from chocolateygui.

cver all -localonly works fine. But it's really not very fast. Yes, a VersionComparer would work fine I think.

from chocolateygui.

added failing tests for this.

from chocolateygui.

@chrissie1: Did you purposefully remove the file system service from the PackageManager during this commit? It looks like it was still there in your prior commit, but then it went away in this one and the dialog boxes during tests came back. I'd add it back in, but it looked like you might have done it on purpose and didn't want to get in a commit fight :)

from chocolateygui.

Nope thats was accidental, git messed up the merge, i guess, yeah it was git's fault

from chocolateygui.

FIGHT, FIGHT, FIGHT!!! :-)

from chocolateygui.

@cemrich Here is the gist to the comparer. Needs some more tests but I think it should work. Based on the full packagename. https://gist.github.com/chrissie1/4959638

from chocolateygui.

All fixed.

@chrissie1 are those 4 package name parsing tests added for this issue or a different one? I can fix them to properly parse out name, version, and pre but wouldn't want to step on any work being done to manage the sorting.

Also, should there have a been a link to the gist you just mentioned somewhere?

from chocolateygui.

The link is there. You need to look better.

I think they are unrelated. This one is about the fact that The .Last() call did not give the correct version because it does stringsorting. So I made the sorter to sort them correctly so that you can find the last version more easily. My sorter is based on the directoryname but it easily adapted to work on a Package

from chocolateygui.