This is very close, I know there should be one test failure, and it's a legit bug, but I want to get soem review on this, and see what the testbot says.

I'd like to look at one or two of the other maintainers bugs while I'm at it, and I need to grep through 6.x-1.x for all references to maintainers to make sure I'm not missing anything.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, project-maintainers-port.patch, failed testing.

mikey_p’s picture

Status: Needs work » Needs review
FileSize
35.19 KB

This should be passing now, and also fixes #1003674: Deleting a project does not delete maintainers by including hook_node_delete().

mikey_p’s picture

This also now includes updated project.api.php docs. Yay for docs.

mikey_p’s picture

Issue tags: +project, +drupal.org D7

Tagging for sprint

Status: Needs review » Needs work

The last submitted patch, 1549854-port_project_maintainers-2.patch, failed testing.

mikey_p’s picture

Status: Needs work » Needs review
FileSize
9.52 KB

Pretty sure the previous patch won't apply, since I hadn't pulled recently.

mikey_p’s picture

Whoops that patch was missing a large hunk.

mikey_p’s picture

Status: Needs review » Fixed

Committed.

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