it'd be nice if the issue queue always let you leave the version alone, even if you're replying to an old issue pointing to a now-unpublished release. i.e. always insert the current value, whatever it is, into the list of valid choices for the version of an issue. the version string itself should be visisble, even if the release node is unpublished.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aclight’s picture

Version: 4.7.x-2.0 » 5.x-2.x-dev

Yep, this is still a problem, as I discovered last night when commenting on a lot of old issues in the project queue. I'm updating the version to reflect that this is still an active issue.

aclight’s picture

Assigned: dww » aclight
Status: Active » Needs review
FileSize
2.22 KB
2.46 KB

Here's a patch for both the project release module and the project issue tracking module that takes care of this.

I needed to make a change in the project release module so that the versions are still sorted in the right order.

Tested locally and works well for both admin and regular users.

hunmonk’s picture

Status: Needs review » Needs work

let's not put $rid directly into the query string -- it makes baby jesus cry... ;)

this can easily be worked around by passing the db_query() params as an array.

aclight’s picture

Status: Needs work » Needs review
FileSize
3.2 KB
2.29 KB

The baby jesus does not cry.

But this will ensure his tears are absent.

hunmonk’s picture

Status: Needs review » Fixed

looks good. tests out. committed to project release 5.x and HEAD, project issue 5.x-2.x and HEAD.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.