[D8] File Delete

You need to resolved raise in PAReview. Important issue related to dependency namespace into info.yml

like :

drupal:node

Thank you for applying! I added the Git instructions for non-maintainer users.

Hi @jonnyeom,
Thanks for contributing!
I checked the code and found some minor remarks.

1. I think will be useful to specify that $entity is an instance of FileInterface for this form. You can add

  /**
   * The file entity being used by this form.
   *
   * @var \Drupal\file\FileInterface
   */
  protected $entity;

2. I don't sure if there is a reason to declare $fileUsage property as private. Maybe declare it as protected will be enough?

I hope my remarks have a sense.

Thanks for your contribution!

• FileDeleteForm::submitForm(): Performing validation should be done in the validate callback, not in the submit callback. Actually you should show the information that a file is in use directly on the form before any button is clicked. You could even disable the submit button in that case.

Did not see any security issues, looks good to me!

Thank you for your contribution! I am going to update your account.

These are some recommended readings to help with excellent maintainership:

• Dries' post on Responsible maintainers
• Best practices for creating and maintaining projects
• Maintaining a drupal.org project with Git
• Commit messages - providing history and credit
• Release naming conventions.
• Helping maintainers in the issue queues

You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, 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.

I thank all the dedicated reviewers as well.

@jonnyeom: you need to edit the project page and opt into security advisory coverage yourself.

@klausi is correct: We edit the user accounts to make users able to opt into security coverage for every project they create, but that requires they edit their projects. There is a Security advisory coverage setting, on https://www.drupal.org/node/3091257/edit.

Thank you! Please also note that unstable alpha and beta releases are not covered by the drupal security team, so best release a stable version next.