The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.I ran into a problem over at: #1284694: Tweak UI of issue following (bluecheese)
I wanted to override theme_project_issue_summary() so that we could stick the follow/unfollow link at the top of the page, above all the "Jump to" links. However, to invoke flag_create_link() we need the issue nid. Sadly, this theme function doesn't get that kind of context. I know we *could* do the horrible hack of calling arg() and node_load() and all that insanity, but I'd rather just fix this theme function to pass in an $issue node object so that implementors can get to useful properties of the issue when generating the markup for this header section.
I've got it working locally, stay tuned for a patch.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 1285798-1.theme_project_issue_summary-issue-context.patch | 1.6 KB | dww |
Comments
Comment #1
dwwComment #2
dwwCommitted to master and pushed.
Comment #3
dwwp.s. While I was at it, I just converted this into a theme template file which is a lot easier to deal with. Way too much business logic happening in the theme function itself. That's now a template_preprocess callback.
http://drupal.org/commitlog/commit/1894/28cdabaef80f46f2b9fba1dd72d30d90...