Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
8 Oct 2012 at 03:32 UTC
Updated:
4 Jan 2014 at 02:24 UTC
Jump to comment: Most recent
Comments
Comment #1
d2ev commentedMake sure to review other project application and get a new review bonus to get this finished faster.
Drupal Code Sniffer has found some issues with your code.....
Comment #2
Andrey Inkin commentedThanks! Somehow my coder installation didn't detect most of thos, but all fixed now...
Comment #3
cubeinspire commentedEverything works as expected.
Syntax is ok (ventral.org)
Branch is ok.
The code size is just above the limit (120 lines 5 functions).
There is no security issue neither, check_plain() is applied on the output, addTag('term_access') also...
I guess this is RTBC.
Comment #4
cubeinspire commentedHi again !
I've found a compatibility issue with Entity module and created an issue here: http://drupal.org/node/1858622
But this is not a blocker issue.
As there have been no opposition to my last review I guess this is RTBC.
Thanks for your contribution, Andrey Inkin!
I updated your account to let you promote this to a full project and also create new projects as either a sandbox or a "full" project.
Here are some recommended readings to help with excellent maintainership:
You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and get involved!
Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.
Comment #5.0
(not verified) commentedrequired links added