confusing latest_version_uri_prefix should be merged into active_versions

Bug #520542 reported by Leonard Richardson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
lazr.restful
Fix Released
Undecided
Unassigned

Bug Description

<leonardr> active_versions and latest_version_uri_prefix
<leonardr> active_versions is a list, the other is a stirng
<leonardr> lazr.restful serves web services for all of those strings
* danilos has quit (Ping timeout: 265 seconds)
<leonardr> so if active_versions is =['beta', '1.0'] and latest_version_uri_prefix is 'devel' there are three versions being served
<leonardr> if active_versions is [] and latest_version_uri_prefix is 'beta' then there is one version
<leonardr> though i don't recommend doing that
<noodles775> So if active_versions is =['beta', '1.0'] and latest_version_uri_prefix is 'devel', why are we evaluating earliest_version = 'beta', is that right?
<leonardr> noodles775: i'm having trouble parsing that sentence, but i can tell you that in that case earliest_version is 'beta'
<leonardr> latest_version_uri_prefix can only be the earliest version if active_versions is empty
<noodles775> OK, as the code implies, I'm just confused by the names.
<leonardr> noodles775: i'm kind of leaning towards treating the last item in active_versions as latest_version_uri_prefix

Do it. Get rid of latest_version_uri_prefix. Now that we have multiversion it's not just a URI prefix anymore.

Changed in lazr.restful:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.