Status settings for an issue

Last modified: April 10, 2009 - 04:57
Active
No patch is attached to the issue. This is the default state of a new issue, but may also sometimes be used for 'resetting' an issue that has gotten off-course even if the issue has old patches attached.
Needs Work ["CNW"]
The patch needs additional work before it should be reviewed. The author of the patch may indicate that it is incomplete, or the patch has gone through the reviewing process and has received feedback about areas where it needs improvement.
Needs Review ["CNR"]
A patch has been created and needs review and testing. The Testing Bot uses this status to trigger an automated review of the patch, and reports the results. Patch Reviewers also use this status to find patches that need to be reviewed and evaluated. With sufficient positive feedback, the patch may be promoted to the status "patch (reviewed & tested by the community)" [see next definition]. If a reviewer, either human or Bot, finds a problem with the patch, it will then be marked as "patch (code needs work)". Note that if a patch is not required in order to complete the idea described in the issue, then it is the idea itself that needs review.
Reviewed & Tested by the Community ["RTBC"]
The patch has received a thorough review and test by one or more experienced developers, and they have deemed it as ready to be committed to the project.

One should not set their own patch to this status. Patches need to go through the review process. Some exceptions may be made if another developer voices their support, or when the patch is utterly trivial (e.g. fixing a spelling mistake in a comment).

At the end of the day, setting an issue to this status is a judgment call: if you have thoroughly tested and reviewed the patch and believe it is ready, you may change the status; if you are unsure, the status should not be changed. Simply add the findings of your review to the issue.

Remember that even if one or two developers believe a patch is ready, doesn't necessarily mean that it will be committed. Patches at this status may still require additional reviews to help ensure that the patch is truly ready. If it is not, the status can be reverted to an earlier status. The better the reviews, the more likely the code will actually get committed.

If a patch remains at this status for a long time, it should be reevaluated to determine if the status is appropriate.

If a patch is not needed to complete the idea described in the issue, then it is the idea that has been reviewed and tested.

Patch (to be ported)
The patch has been successfully committed to a branch of the project, and still needs to be committed to another, but the current patch doesn't apply to the target branch and needs to be modified in order to do so.
Fixed
The issue has been resolved (usually by committing a patch). Issues that remain in this state without any additional comments will automatically be set to "closed" after two weeks. This gives interested parties time to reactivate the issue if they see a problem with the fix and also allows time to see that a change has been made.
Postponed
The issue seems like a good idea, but other (often related) issues need to be dealt with first. The intention is to come back to these issues at a later date.
Postponed (maintainer needs more info)
There is insufficient information in the issue to proceed. Should no additional information about the issue be provided, the issue may be marked either "closed" or "won't fix" at the discretion of the reviewer.
Duplicate
An identical, or strikingly similar issue has already been created. The issue being marked as a duplicate issue should provide a link back to the earlier version of the issue.
Won't Fix
It has been decided that this issue will not be fixed. This includes feature requests that are deemed to be outside the scope of the project, and bug reports that cannot be reproduced or are unsupported.
By Design
The reported issue has been deemed not to be an issue. It is either an intentional part of the project, or the issue is caused by manually applied customizations of a user (wrong configuration, code alterations, theme override functions).
Closed
This status is used exclusively by the Project Engine to close "fixed" automatically after two weeks of inactivity. You should not need to set this status yourself. The issue is no longer current. Issues that have reached this status should typically not be reopened, but instead a new issue should be opened, providing a link to the closed issue. Closed issues do not appear in the default view of the issue queue. This provides a cleaner queue, while still maintaining the issues for historical reasons.

NOTE: An overview of all the drop-down menus on the "Submit issue" form can be found on the Issue Submission Form Description page.

Statuses that define closure of Issues

Even though closed is the status that says that the issue is closed, the statuses - won't fix, duplicate, by design - are also effectively statuses that define some kind of closure of the Issue. Issues marked any of these statuses will not be listed when you click on the "View open issues or create one" link from the project pages.

Is "closed" also appropriate

leenwebb - April 8, 2009 - 16:41

Is "closed" also appropriate to use for "resolved"? My issues are hardly ever bugs; usually they are support requests. So when the maintainer has helped me out and solved my problem it is "fixed" though there is no patch involved, and generally the issue ends up "closed" but I'd have to set that manually and this page indicates I shouldn't do that. These status definitions work well for bugs but are a little weird for support requests.

"Fixed" is appropriate

Freso - May 10, 2009 - 07:54

"Fixed" would be what you're looking for. Using "fixed" will ensure that the issue are also still visible in your issue queue for some time (2 weeks) after they stop getting activity, which means people are less likely to post a duplicate issue.

Is there a "proper" way to

naught101 - May 11, 2009 - 01:14

Is there a "proper" way to mark a bug as a duplicate? Is changing status to "duplicate" and just providing a link appropriate?

Yes, that's how people

Xano - May 27, 2009 - 18:22

Yes, that's how people generally do it :)

 
 

Drupal is a registered trademark of Dries Buytaert.