[#GALAXY-54] Version qualifier is ignored, instead active version is always returned for an artifact

Galaxy

Version qualifier is ignored, instead active version is always returned for an artifact

Created: 25/Jan/08 02:24 PM Updated: 18/Feb/08 10:41 PM

Component/s:

Core, Web

Affects Version/s:

None

Fix Version/s:

1.0-beta-2

Time Tracking:

Not Specified

File Attachments:

problem_with_posting_new_versions_patch.patch (10 kB)

Labels:
User impact:	High

Description

« Hide

The behavior is the same with either rest calls or web ui.

E.g. if the currently active version is 1, then the following url will return it, ignoring v2: http://localhost:8080/api/registry/Andrew/test?version=2

For the UI, whatever version is clicked, it still returns only the active.

All

Comments

Work Log

Change History

Transitions

FishEye

Sort Order:

[ Permlink | « Hide ]

Dan Diephouse added a comment - 26/Jan/08 03:16 PM

The place this needs to be fixed is the RegistryServiceProvider class.

[ Permlink | « Hide ]

Andrew Perepelytsya added a comment - 26/Jan/08 05:12 PM

Dan, I'm attaching the patch for now, here's the deal. This patch fixes UI behavior, but I can't add new versions via a programmatic rest request. Check the ArtifactCollectionTest.testMultipleVersions()

Again, UI works fine with the patch.

[ Permlink | « Hide ]

Andrew Perepelytsya added a comment - 26/Jan/08 05:14 PM

P.S.: of course one should remove a thread.sleep at the end of the test, I've put it there to be able to manually query the test repo with external tools.

[ Permlink | « Hide ]

Dan Diephouse added a comment - 28/Jan/08 09:39 AM

I'll take over this issue then as I'm going to be updating the Abdera code today/tomorrow with the latest version from Abdera. This is going to change where the resolution happens anyway.

[ Permlink | « Hide ]

Andrew Perepelytsya added a comment - 28/Jan/08 09:42 AM

Dan, could you also look into uploading a source bundle for abdera as well so it can be linked to in IDE automatically?

[ Permlink | « Hide ]

Dan Diephouse added a comment - 28/Jan/08 09:44 AM

Sure thing. We're going to be moving to snapshots again and I'll be sure to publish the sources when I publish the new snapshots.

[ Permlink | « Hide ]

Dan Diephouse added a comment - 18/Feb/08 10:41 PM

This has been fixed for a while, I don't know the exact commit though.