Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
27 May 2013 at 14:21 UTC
Updated:
11 Dec 2013 at 23:26 UTC
Jump to comment: Most recent
Comments
Comment #1
Bojhan commented+1 Looking forward to this.
Comment #2
zterry95 commentedinfo file, the package change to user interface.
Comment #3
klausiYou need to set the status to "needs review" if you want to get a review, see https://drupal.org/node/532400
Comment #4
Bojhan commentedI am using this actively on my website, haven't encountered any problems.
Comment #5
PA robot commentedWe are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)
Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #6
peteruithoven commentedChanged package to user interface
http://drupalcode.org/sandbox/peteruithoven/2004852.git/blobdiff/f8edcae...
Comment #7
51Degrees.mobi commentedHi,
I had two issues when trying this module:
I thought it might need to use wikiheader_get_contributors but the comments suggest otherwise. It looks like the function is simply missing.
It's nice code that has no problems with the code sniffer, so I think with those issues sorted it should pass review.
Comment #8
peteruithoven commentedThanks 51Degrees.mobi, I fixed the error, updated the readme and project page description.
Comment #9
gabesulliceHi peteruithoven, this is a neat module. You've still got a call to an undefined function on line 66 of wikiheader.module though.
Comment #10
PA robot commentedClosing due to lack of activity. Feel free to reopen if you are still working on this application.
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #11
peteruithoven commented@gabesullice, is fixed.
I did quite some research looking into making it easier to use the module. I was hoping to find a way to add it as a field into the display page of content types. But there doesn't seem a way to do this without adding an extra field. So people will have to edit a node tpl.php for now.
Comment #12
kscheirerWhat about using a preprocess hook to and adding your header to the $content array. I think that should be enough to get it rendered with any node. But this module looks ok, I would RTBC if you can improve the documentation.
----
Top Shelf Modules - Crafted, Curated, Contributed.
Comment #13
PA robot commentedClosing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.