Problem/Motivation

The tags in the version selector cause a couple of issues:

1. The select gets very long.

2. Especially core, but also the vast majority of contrib projects, never work on issues against tags - since the tag has already been tagged and can't be merged against. However I often see new users change the version of issues to the specific tag they happen to have installed.

3. A search on a branch won't show issues against the tag, so potentially some things get lost/duplicated due to this.

Proposed resolution

Only show branches and meta-branches in the version selector.

Related to this, it would be good to also remove unsupported branches as well - might open separate issue for that. Right now the version selector on this form goes all the way back to 4.7.x-1.0

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#17 Screen Shot 2017-01-10 at 6.48.51 PM.png141.1 KBdrumm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch created an issue. See original summary.

xjm’s picture

catch’s picture

catch’s picture

drumm’s picture

Title: Remove tags from the version selector » Add option to remove tagged releases from the version selector

I think this should be optional, probably an option for the whole site. (Not a per-project option.) I'm not sure if there is anyone out there using this in project module, but I don't want to take this out from under them. I do think we should keep this standard across all projects on Drupal.org.

For Drupal.org, we'll want to check the feasability:

  • How many issues are filed against tagged releases?
  • How many of those have a branch release to migrate to?
drumm’s picture

Project: Project issue tracking » Project
Component: Issues » Releases
Assigned: Unassigned » drumm

The issue version field is added by project_release, a submodule of project.

  • drumm committed 1b84ae2 on 2691889-version-options
    Issue #2691889: Add a field to track which types of releases should be...
drumm’s picture

To do:
- The stats from #5. Do we move issues on tagged releases or make them less-findable?
- Show issue counts/warning on edit, so contrib projects know the trade-offs?
- Hide the new field if releases are not enabled for the project node.
- The actual functionality.

This should also speed up issue page loads for logged in users.

drumm’s picture

All open core issues for non-dev versions do have corresponding dev versions, so we would not be making any issues less accessible. We’ll need to draft a message for the auto update to move the tagged issues.

There are various closed issues that would be affected, but I don’t think they are important.

  • drumm committed 059c17b on 2691889-version-options
    Issue #2691889: Hide issue version options if releases are not enabled
    

  • drumm committed 317c4af on 2691889-version-options
    Issue #2691889: Show issue counts for version options
    

  • drumm committed e38f05a on 2691889-version-options
    Issue #2691889: Add option to remove tagged releases from the version...
drumm’s picture

Status: Active » Needs review

This should be okay to deploy as-is, since it is a configuration. For core, we’ll want to write a script to sweep up all the tag version issues into their dev versions.

drumm’s picture

The versions list on issue listing pages needs a followup, #2842544: Move ProjectIssueHandlerFilterSearchApiVersion into release/views/handlers. So even if this is enabled right away for core, issues still filed under the tagged versions will still be accessible. If people update those issues, only the remaining options will be available.

  • drumm committed 059c17b on 7.x-2.x
    Issue #2691889: Hide issue version options if releases are not enabled
    
  • drumm committed 1b84ae2 on 7.x-2.x
    Issue #2691889: Add a field to track which types of releases should be...
  • drumm committed 317c4af on 7.x-2.x
    Issue #2691889: Show issue counts for version options
    
  • drumm committed e38f05a on 7.x-2.x
    Issue #2691889: Add option to remove tagged releases from the version...
drumm’s picture

Status: Needs review » Fixed
Issue tags: +needs drupal.org deployment
drumm’s picture

Issue tags: -needs drupal.org deployment
FileSize
141.1 KB

Deployed on Drupal.org and configured this project to use it:

Screenshot

Tagged releases in unchecked in the last form element.

  • drumm committed bef0f61 on 7.x-2.x
    Issue #2691889: Use LANGUAGE_NONE instead of node's language
    
dww’s picture

I guess this train already left the station, but I'm not necessarily a huge fan of this change. Part of the point of having official releases is to make it easier to reproduce bugs, since everyone can know what version of the code is showing the problem. Yes, feature requests should always be tied to a branch, not a release. However, I think there's still value in bug reports being reported against a specific version, not just a branch.

Also, point #3 from the summary is what the "meta version" options were supposed to help solve. People shouldn't be searching for issues via 8.x-1.x-dev, they should be using "8.x issues". The meta version options are already at the top of the list when filtering/searching for issues.

That said, I can see that having dozens of stable releases on each branch makes the version selector unwieldy.

If everyone thinks this is an improvement, maybe we could at least update the issue summary template to encourage people to include information about the specific version of the code they're using when reporting bugs?

Thanks,
-Derek

xjm’s picture

It is optional, so for projects where maintainers especially need this information, it can remain checked.

This is definitely going to make it easier to file and find core issues. Thanks @drumm!

xjm’s picture

@drumm, I tested this for core, and it looks like issues that were filed against a version (some 17k) will be moved to 9.x by default as soon as someone updates them. I don't think we want that. So probably we should do the bulk update, at least for open issues. Disabling again for now until we can do that.

xjm’s picture

Issue tags: +Needs followup

Should we file a separate issue for migrating open core issues to their branch?

drumm’s picture

Yes, since that's not something for project module. Either drupal or infrastructure project. Like the core minor version updates, main thing I need is a second draft of “Core issues are now filed against dev versions.” for the comment to go with the bulk version update.

drumm’s picture

Status: Fixed » Closed (fixed)

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