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.
  • 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:
  • 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

mparker17’s picture

Issue summary: View changes
mparker17’s picture

Issue summary: View changes
mparker17’s picture

Issue summary: View changes
mparker17’s picture

Issue summary: View changes
mparker17’s picture

Issue summary: View changes
guptas89’s picture

Hi @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

mparker17’s picture

@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.

mparker17’s picture

Issue summary: View changes
mparker17’s picture

Issue summary: View changes
Status: Active » Needs review
Issue tags: +PAreview: review bonus

Completed review bonus #3: https://www.drupal.org/node/2378895#comment-9904697

Moving to Needs Review and adding the PAReview: review bonus tag.

t0xicCode’s picture

Assigned: Unassigned » t0xicCode
klausi’s picture

Assigned: t0xicCode » er.pushpinderrana
Status: Needs review » Reviewed & tested by the community
Issue tags: -PAreview: review bonus

manual review:

  1. smiley_emoji_enable(): why isn't this stuff in hook_requirements()? Please add a comment or move it to hook_requirements() with the install $phase.
  2. This module alone is too short to give the git vetted user role away, but since mparker17 identified a security issue in another application and has contributed to quite a few other projects already I think we can proceed with the git vetted user role here.

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.

er.pushpinderrana’s picture

Assigned: er.pushpinderrana » Unassigned
Status: Reviewed & tested by the community » Fixed

Automated Review

Best practice issues identified by pareview.sh / drupalcs / coder. None

Review of the 7.x-1.x branch (commit 5dfba1b):

  • No automated test cases were found, did you consider writing Simpletests or PHPUnit tests? This is not a requirement but encouraged for professional software development.

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.

mparker17’s picture

Awesome, thank you all very much!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.