# Summary

Track the deletion of nodes, comments, files, users, vocabularies and terms. Whenever an entity is deleted, a record of the deletion is saved to the database, available in the Administrative UI under "Reports" section.

Currently, an initial port has been started by @malaynayak

# Project URL

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

# Where is the code?

#2922764: Port Entity Delete Log to Drupal 8
https://github.com/malaynayak/entity_delete_log_d8

# Estimated completion date

N/A

# Dependencies

N/A

# Who's doing the port?

@malaynayak

# What help do they need?

N/A

# D8 roadmap

#2922764: Port Entity Delete Log to Drupal 8

# Background and reference information

N/A

Comments

jwilson3 created an issue. See original summary.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Issue summary: View changes
malaynayak’s picture

malaynayak commented
Assigned: Unassigned » malaynayak

Hi @jwilson3,

I have started working on it. The code is at https://github.com/malaynayak/entity_delete_log_d8

Thanks

mmjvb’s picture

mmjvb commented
Status: Active » Needs work
jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Issue summary: View changes
malaynayak’s picture

malaynayak commented
Status: Needs work » Needs review

Hi @jwilson3,

Can you please review the code and suggest if any further changes required.

Thanks

mmjvb’s picture

mmjvb commented
Status: Needs review » Needs work

There is no alpha or beta version available, so Status back to Needs work.
See the project details for description on using Status for these issues!

malaynayak’s picture

malaynayak commented

Hi @mmjvb,

Once the code is reviewed we can go for a alpha/beta release. Correct me if I am wrong.

malaynayak’s picture

malaynayak commented
Status: Needs work » Needs review
mmjvb’s picture

mmjvb commented
Status: Needs review » Needs work

Already did, and again.
See the description of this project for correct usage of Status here.
This is not an issue of the module queue!

malaynayak’s picture

malaynayak commented

Hi @mmjvb,

I think the module maintainer has to do this as I don't have the access for it. Lets wait.

mmjvb’s picture

mmjvb commented

Unfortunately the module maintainer is not very active. Suggest to follow the procedure for abandoned module to get attention for this module.

malaynayak’s picture

malaynayak commented
Status: Needs work » Needs review

HI @mmjvb,

I have created a beta release at https://github.com/malaynayak/entity_delete_log_d8/releases/tag/v8.0-beta1. Please review.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented

Hi @mmjvb, I'm not in a position to review this atm. but I agree with mmjvb. There is nothing requiring the maintainer of the module to track this issue, because it is not their queue.

You need to create your code as a patch against 7.x (if there is no 8.x branch created yet) and attached to #2922764: Port Entity Delete Log to Drupal 8. And then we need to take the conversation over to that issue. If you believe the maintainer will not respond, then you can open a separate ticket in the issue queue requesting to take over the project:

Read https://www.drupal.org/node/251466 for details on how to do that.

mmjvb’s picture

mmjvb commented

No need to create a patch against D7. You can request the maintainer to assess your port and make it available on d.o.
He should be able to create the D8 branch and classify your work as dev, alpha, beta, rc or even stable. Consider making your port available as dev release on d.o. the minimum to facilitate those interested in working on a D8 version.

An active maintainer would not needed to be asked. Which is why I suggested to follow procedure for abandoned modules. Maybe you are interested to become co-maintainer, possibly only for D8 version. Maybe others are interested.

Me reviewing your port is of very little value, not being a maintainer of any module and not even a user of this module. When current maintainers don't show any interest, don't expect others to.
Despite this, I had a look at the code. Don't know whether it meets the coding standards, but don't like your use of comments. Comments should explain something that is not quite obvious from the code. Saw too many comments that have no added value.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Status: Needs review » Needs work

Based on feedback above, moving this back to needs work. I would also point out that your README.txt doesn't follow the typical Drupal standards.

https://www.drupal.org/docs/develop/documenting-your-project/readme-temp...

david.fcondezo’s picture

david.fcondezo commented

Have there been any updates on this port to D8?

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Issue tags: -Drupal 8 porting (duplicate)
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Assigned: malaynayak » Unassigned
malaynayak’s picture

malaynayak commented
Status: Needs work » Fixed

Closing this issue as the D8 port is complete and a new release has been published.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented

Status: Fixed » Closed (fixed)

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