Should the module report errors if text files are missing, e.g. if the contrib modules' LICENSE.txt files were moved? It seems a little redundant to me, so maybe the Drush commands should have an option to ignore them?

CommentFileSizeAuthor
#4 hacked-ignore-txt-po-files-6173572-4.patch2.46 KBAnonymous (not verified)
#3 hacked-ignore-txt-po-files-6173572-3.patch2.29 KBAnonymous (not verified)
#2 hacked.patch2.29 KBAnonymous (not verified)

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Title: Option to remove missing text files? » Option to ignore missing text files?

Corrected the title.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Needs review
StatusFileSize
new hacked.patch2.29 KB

I have created a patch which adds two checkboxes to the settings page to be able to ignore the txt files, and also the po files.

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new hacked-ignore-txt-po-files-6173572-3.patch2.29 KB

sorry, I attach now the file with the right name

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new hacked-ignore-txt-po-files-6173572-4.patch2.46 KB

Ignore the two previous patches, I've upload now with the proper format

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Needs work

This needs some small improvements to match the Drupal coding standards.

steven jones’s picture

steven jones commented

I like the idea, it might be better to implement this as a list of file extensions people can ignore, or a regex that people can use to filter the lists.

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented
Version: 7.x-2.x-dev » 8.x-2.x-dev
Issue summary: View changes

Agreed, this is a much needed feature, but limiting to Text and Translation files is short sighted. One of the most commonly "Hacked" files in a site is the .htaccess file in a root project, and as such there needs to be a way to account for it and other miscellaneous files.

Moving to 8.x-2.x as new functionality should be added there and backported.

ivnish’s picture

ivnish
he/him
commented
Version: 8.x-2.x-dev » 7.x-2.x-dev
ivnish’s picture

ivnish
he/him
commented
Status: Needs work » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.