Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hello,
Show title is our first contribued Drupal (8.x) module and we wanna receive security advisory coverage.
Description
This module creates an Extrafield (pseudo-fields) of native node title field for rendering and use in display mode.
Project Url
https://www.drupal.org/project/show_title
To clone the repo
git clone --branch 8.x-1.x https://git.drupal.org/project/show_title.git
Thanks for your review.
Comments
Comment #2
Guietc CreditAttribution: Guietc commentedComment #3
ashish-deynap CreditAttribution: ashish-deynap at TO THE NEW commentedAutomated Review
The show_title.module does not implement hook_help(). See https://www.drupal.org/docs/develop/documenting-your-project/module-docu...
Manual Review
Comment #4
Guietc CreditAttribution: Guietc commentedThanks @ashish-deynap for your reply.
I added the hook_help() and the README.
Comment #5
RonanRBRReview of the 8.x-1.x branch (commit ac3f671):
This automated report was generated with PAReview.sh, your friendly project application review script.
Comment #6
apadernoComment #7
sleitner CreditAttribution: sleitner commentedThe name in the taxonomy terms is not shown in manage display list.
You have to use Drupal\taxonomy\Entity\Vocabulary (instead of Drupal\taxonomy\Entity\Term) to get the ids of the vocabularies.
Comment #8
apadernoIf you are still working on this application, you should fix all known problems and set the status to Needs review. (See also the project application workflow.)
Please don't change status of this application if you aren't sure you have time to dedicate to this application, or it will be closed again as won't fix.
I am closing this application due to lack of activity.
Comment #9
Guietc CreditAttribution: Guietc commentedThe #7 issue has been fixed.
Comment #10
sleitner CreditAttribution: sleitner commentedAutomated Review
Review of the 8.x-1.x branch (commit 916e04a):
This automated report was generated with PAReview.sh, your friendly project application review script.
Note that perfect adherence to Drupal Coding Standard is NOT a reason to block an application, except for total disregard of them. However, modules should follow them as closely as possible.
Manual Review
The starred items (*) are fairly big issues and warrant going back to Needs Work. Items marked with a plus sign (+) are important and should be addressed before a stable project release. The rest of the comments in the code walkthrough are recommendations.
If added, please don't remove the security tag, we keep that for statistics and to show examples of security problems.
This review uses the Project Application Review Template.
Comment #11
apadernoThank you for your contribution! I am going to update your account.
These are some recommended readings to help with excellent maintainership:
You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, 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.
I thank all the dedicated reviewers as well.