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.
The Smiley Emoji project makes the Emoji from Github available to the Smiley module 7.x-2.x.
Links
https://www.drupal.org/sandbox/mparker17/2078481
git clone --branch 7.x-1.x http://git.drupal.org/sandbox/mparker17/2078481.git smiley_emoji
http://pareview.sh/pareview/httpgitdrupalorgsandboxmparker172078481git
Smooth Review: Quick tips checklist
- Application checklist: completed
- Review bonus: #1, #2, #3.
- Intended Drupal version: D7
- Avoid 3rd party code or images: Smiley Emoji uses the Libraries API to ensure all dependencies are not in the module.
- Similar modules:
- Textcomplete provides auto-completion for emoji, but does so by replacing the token with an image tag when the text is entered. This means you need permission to access a text format that allows
<img>
tags (which most site builders forbid for anonymous commenters), and it changes the data saved to the node, making the node dependant on the version of the Textcomplete library installed, and it's location (and, newer versions of the Textcomplete library don't include the emoji).
The Smiley module does not affect the text saved in the node: instead it performs the replacement using an input filter, meaning the text-to-image replacement happens when the text is rendered. This also removes the dependency on external libraries: if a library changes, the images will work after clearing caches. You can add the Smiley input filter to any text format, regardless of that format's restrictions on allowed tags / HTML (so long as the smiley replacement runs after the allowed-tag filter).
Smiley Emoji simply registers a list of replacement tokens with the Smiley module.
A brief read through the code of the Textcomplete module suggests that it would not currently be possible for the Smiley or Smiley Emoji modules to register themselves with Textcomplete to provide smiley previews when text is being entered.
- Textcomplete provides auto-completion for emoji, but does so by replacing the token with an image tag when the text is entered. This means you need permission to access a text format that allows
- XSS vulnerabilities: none found.
This module simply builds an array from file names in the library when asked to by the Smiley module. It filters non-alphanumeric characters from the keys to ensure that even a library with funky file names does not cause problems. - Translatable: done
- Uninstall variables: no variables used
- Background and experience:
- mparker17 taught himself Drupal in March 2009 in preparation for a job interview at PeaceWorks Technology Solutions in Waterloo, ON, Canada. He got the job, and worked first as a themer, then a site builder on over 43 Drupal sites before joining Myplanet Digital in Toronto, ON, Canada, where he worked on another 5 enterprise-level Drupal sites, built the Clean Markup module, Reviewed "Migrating to Drupal 7" by Trevor James, Was interviewed on the Modules Unravelled Podcast, episode 079, and attended DrupalCamp Toronto 2012, 2013, DrupalCon Portland 2013, Symfony Live Portland 2013, Palantir.net Drupal 8 Training, and the Drupal Midwest Developer Summit 2013. He later left Myplanet to join OpenConcept Consulting, where he's worked on another 7 Drupal sites, including two for major Canadian organizations, attended DrupalCamp Ottawa 2014, and started volunteering to write the Drupal Core Updates blog series. He's recently was asked to co-ordinate the Drupal Core Update blog series. At time-of-writing, he has 44 commits to Drupal 8 core, and 2 commits to Drupal 7 core. He also occasionally volunteers to maintain Pressflow/7, a friendly, shallow fork of Drupal with integrated performance, scalability, availability, and testing enhancements.
- Automated Project Application Review
Problem/Motivation
I've had a request to promote a project currently in my sandbox, Smiley Emoji, to a full project and make a release.
Proposed resolution
Give mparker17 permission to promote Smiley Emoji to a full project.
Comments
Comment #1
mparker17Comment #2
mparker17Comment #3
mparker17Comment #4
mparker17Comment #5
mparker17Added first review bonus: https://www.drupal.org/node/2457903#comment-9750767
Comment #6
guptas89 CreditAttribution: guptas89 commentedHi @mparker17
Thanks for contributing this module. Please change the project status from Active to needs review. So that drupal review process can be follow to make it Full project.
Please @see: https://www.drupal.org/node/894256
https://www.drupal.org/node/1975228
Comment #7
mparker17@guptas89, thanks for the tip! :) I was going to wait to put it into "Needs review" until I had completed the review bonuses first (so as not to waste other peoples' time), but I plan to do those review bonuses today.
Comment #8
mparker17Completed review bonus #2: https://www.drupal.org/node/2241231#comment-9904525
Comment #9
mparker17Completed review bonus #3: https://www.drupal.org/node/2378895#comment-9904697
Moving to Needs Review and adding the
PAReview: review bonus
tag.Comment #10
t0xicCode CreditAttribution: t0xicCode at OpenConcept Consulting Inc. commentedComment #11
klausimanual review:
Looks good to me. Removing review bonus tag, you can add it again if you have done another 3 reviews of other projects.
Assigning to er.pushpinderrana as he might have time to take a final look at this.
Comment #12
er.pushpinderrana CreditAttribution: er.pushpinderrana as a volunteer and at Publicis Sapient for Publicis Sapient commentedAutomated Review
Best practice issues identified by pareview.sh / drupalcs / coder. None
Review of the 7.x-1.x branch (commit 5dfba1b):
This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.
Manual Review
You should also really have a hook_help() with some basic info about the module.
None are blockers.
This project is too short to approve you as git vetted user but you already have depth understanding of Drupal Api's and highly involved with security fixes/contribution (196 commits), additionally if klausi is willing to give this a go, who am I to stop this any further :)
Thanks for your contribution, M Parker!
I updated your account so you can 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 stay 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.
Thanks to the dedicated reviewer(s) as well.
Comment #13
mparker17Awesome, thank you all very much!