The module provides easy integration with Tweetable Text CK Editor Library.

Project link

https://www.drupal.org/project/tweetext

Git Instructions

git clone --branch 8.x-1.x https://git.drupalcode.org/project/tweetext.git

Comments

gaurav.kapoor created an issue. See original summary.

jimmy_sebastian’s picture

Hi @gaurav.kapoor,

I see that phpcs shows few errors.

FILE: /home/lenovo/Public/sri8/modules/tweetext/tweetext.module
---------------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 4 LINES
---------------------------------------------------------------------------
11 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 2
12 | ERROR | [x] Expected 3 space(s) before asterisk; 1 found
13 | ERROR | [x] Expected 3 space(s) before asterisk; 1 found
23 | ERROR | [x] Expected 1 space between double arrow and "Url"; 2 found
23 | ERROR | [x] Expected 1 space after "=>"; 2 found
---------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------

Time: 1.58 secs; Memory: 8MB

harishh’s picture

Issue summary: View changes
gaurav.kapoor’s picture

Status: Needs work » Needs review

@jimmy_sebastian I have updated the module and fixed all the issues identified by PHPCS.

harishh’s picture

Hi @gaurav.kapoor,

I have reviewed and not found any errors. Also the functionalities are working fine. looks good.

gaurav.kapoor’s picture

Thanks, @harishh and @jimmy_sebastian.

avpaderno’s picture

phthlaap’s picture

Status: Needs review » Reviewed & tested by the community

Thanks for your contribution, I haven't seen any security issue related to the code.

gaurav.kapoor’s picture

Thanks @phthlaap

klausi’s picture

Status: Reviewed & tested by the community » Fixed

manual review:

  1. The Git commits are not connected to your user account. You need to specify an email address. See https://www.drupal.org/node/1022156 and https://www.drupal.org/node/1051722
  2. getButtons(): do not use base_path() to manually assemble URLs. Use file_create_url() instead.

This module on its own does not contain a lot of code, but given that gaurav.kapoor has lots of commits on other projects and experience with maintaining projects we can do a full promotion here.

Thanks for your contribution, Gaurav!

I updated your account so you can opt into security advisory coverage now.

Here are some recommended readings to help with excellent maintainership:

You can find lots more contributors chatting on Slack or 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.

gaurav.kapoor’s picture

Thanks @klausi. I will check the URL's shared by you.

Status: Fixed » Closed (fixed)

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

avpaderno’s picture