I would like to see a "port" category added to the project module..
alongside bug, task, feature etc. I believe this will make it easier
to ensure that fixes are backported/checked promptly and correctly.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dww’s picture

Version: 5.x-2.x-dev » 4.7.x-2.x-dev
Assigned: Unassigned » dww
Status: Active » Needs work
FileSize
1.22 KB

i'm not sure i love the name 'port' for this. i'd be happy to entertain alternate proposals. ;) but, here's a patch for now, pending any better suggestions...

dww’s picture

Status: Needs work » Needs review
FileSize
1.24 KB

"port request"? looks an awful lot like "support request" ;)

"backport request" is both pretty long and not always accurate (could be something that needs "forward" porting to HEAD or some such)...

still torn on what to call this thing.

moonray’s picture

How about plain "Porting"?

dww’s picture

Assigned: dww » Unassigned
Status: Needs review » Closed (won't fix)

spoke to dries about this and he's pretty strongly opposed. feature vs. bug is far more important than 'port'. if anything, this should be a complementary field, not a new category of issue. but a whole new field/property just for this is a ton of work. moreover, you can just bump the issue to the right target version.

if we were really slick, we'd let you assign the issue to someone else, so dries/drumm could bump the version to 4.7.x-dev and assign to killes when they wanted something backported. that's another issue entirely.

anyway, in spite of the simplicity of the code changes involved with this approach, it would clobber/mask useful information (you'd have to remember to reset the category back to 'bug' when done), and it's likely to confuse end users trying to submit new bug/feature issues.

therefore, "won't fix"...
sorry, zen. ;)
-derek

Zen’s picture

Status: Closed (won't fix) » Active

I actually agree.. it would be better off as a status option rather than a category.. "code (patch to be ported)" or some such.

-K

dww’s picture

Title: Add a 'port' category to the issue tracker » Add a 'port' status to the issue tracker
Project: Project issue tracking » Drupal.org site moderators
Version: 4.7.x-2.x-dev »
Component: User interface » web site
Status: Active » Needs review

if that's all we want, this is just a d.o infra request, since there's a settings page to add additional status values:

http://drupal.org/admin/settings/project_issue/status

no need to change project_issue.module for this, since the hard-coded default status values are already way too specific for our needs as it is. ;) i'd be happy to add this, so long as:

a) killes/dries/et al agree it's worth doing and
b) we agree on the name

there was a thread somewhere about renaming our existing status values to be shorter to help people w/ busted browsers. instead of adding to that problem, how about just calling it:

"patch (port needed)" ?

i'm still a little worried this extra choice will confuse people, but i guess that'd be true no matter what we do. we should consider where/how to document the new status in a way people will actually read it.

thanks,
-derek

Zen’s picture

patch (to be ported) is sufficient IMO.

As for easing the confusion to non-technical users, I suggest linking to pages like http://drupal.org/node/45111 (in a section dedicated to the issue tracker) in the issue form.

Thanks Derek.
-K

killes@www.drop.org’s picture

I think a new patch category would be ok, will add one, shouldn't I hear otherwise.

dww’s picture

Assigned: Unassigned » dww
Status: Needs review » Fixed

added the status, and gave the appropriate roles permission to set it.

Anonymous’s picture

Status: Fixed » Closed (fixed)