Port Comment Alter to Drupal 8 [#2615210]

Any volunteers to port "Comment Alter" module from Drupal 7 to 8?

Drupal 8.0.0 will be released on November 19, 2015
News about "Porting modules and themes to Drupal 8 "
"Drupal Module Upgrader" module. Which can do some of this automatically for you, covering many API hooks.
Documentation about "Converting 7.x modules to 8.x"

We would be happy to contribute testing patch, quality assurance, documentation, and agile project management services if needed

Comments

Comment #1

14 November 2015 at 00:19

Francewhoa created an issue. See original summary.

Comment #2

Francewhoa

he/him

French

Sept-Îles, Québec, 🇨🇦

CreditAttribution: Francewhoa as a volunteer commented 14 November 2015 at 04:16

Comment #3

Boobaa

he/him

CreditAttribution: Boobaa at Pronovix for BrightCove commented 4 August 2016 at 08:20

Assigned:

Unassigned

» anchal29

Anchal Pandey is working on this as his GSoC-2016 project.

Comment #4

anchal29 CreditAttribution: anchal29 as a volunteer commented 21 August 2016 at 07:01

Version:	7.x-1.x-dev	» 8.x-1.x-dev
Status:	Active	» Needs work

Created a development release for this so changing the version here. Changing the status to Needs Work as some code cleanup is required as suggested by @boobaa and @czigor.

Comment #5

Thew CreditAttribution: Thew at Google Code-In commented 10 January 2017 at 14:56

Reviewed this project from a task in GCI.

1. This module produces an error on PHP7. After enabled User may alter this field from comment, the add new comment form become disappear. From the recent log messages...
Warning: Parameter 1 to comment_alter_form_comment_form_alter_after_build() expected to be a reference, value given in Drupal\Core\Form\FormBuilder->doBuildForm() (line 1052 of /core/lib/Drupal/Core/Form/FormBuilder.php) #0

2. In PHP5, after submitted comment form with making some changes to the field, the node becomes inaccessible and gives
Fatal error: Class 'Drupal\diff\EntityComparisonBase' not found in /modules/comment_alter/comment_alter.module on line 423

Comment #6

Francewhoa

he/him

French

Sept-Îles, Québec, 🇨🇦

CreditAttribution: Francewhoa at Ubertus.org commented 15 January 2017 at 04:28

Hi Boobaa, anchal29, Thew :) Thank you all for your contributions

Thew :) Welcome to the Drupal community. And thanks for reporting those two errors in your comment #5 above.

I hope the following helps. For a faster response time and facilitate communications I suggest to create one new ticket per bug report or per topic. In this case I suggest to create two new tickets. One per PHP version and its respective error. New tickets for the Comment Alter module can be create at https://www.drupal.org/node/add/project-issue/comment_alter

If you're not familiar with creating new tickets there is a documentation at https://www.drupal.org/node/73179

I just created the following two tickets. Feel free to edit them to your liking:

PHP7 error at https://www.drupal.org/node/2843584
PHP5 error at https://www.drupal.org/node/2843586

Comment #7

rattusrattus CreditAttribution: rattusrattus at Agile Collective commented 5 March 2018 at 11:54

Do folks still have time to work on this project. I've recently used this module for a client project and have discovered a few issues, patches for which are now in the issue queue, and I'm interested in contributing to the D8 porting where possible...