There are all kinds of places in project* where we need to find a nid via a project_uri or vice versa. I've got a patch that adds two helper functions to consolidate all these queries in one place.

CommentFileSizeAuthor
#2 367577-2.project_uri_cleanup.patch3.88 KBdww
#1 367577-1.project_issue_uri_cleanup.patch1.73 KBdww
#1 367577-1.project_uri_cleanup.patch4.2 KBdww

Comments

dww’s picture

dww
we/he/they
commented
Status: Active » Needs review
StatusFileSize
new 367577-1.project_uri_cleanup.patch4.2 KB
new 367577-1.project_issue_uri_cleanup.patch1.73 KB

Any objections?

dww’s picture

dww
we/he/they
commented
StatusFileSize
new 367577-2.project_uri_cleanup.patch3.88 KB

Had a change to behavior in project_project_retrieve(), so I added a check for type == 'project_project'.
Also removed some stray newlines in the helpers.

dww’s picture

dww
we/he/they
commented
Status: Needs review » Fixed

After review from aclight in IRC, committed to HEAD of both modules.

Status: Fixed » Closed (fixed)

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