I had the same problem with uninstalling this module. It's because you add a reaction field, and then add permissions, and then you can't remove the field because of the permissions. For anyone else who wandered here because of Google, here's the fix:
- Get your environment back to a "sane" state:
drush cim - Remove reaction permissions using the Drupal GUI
- Remove the field using the Drupal GUI
- Uninstall the module:
drush pm:uninstall votingapi_reaction
Thanks @kristen-pol, I've gone through one more time to find any remaining references to README.txt in the code base, and I think I've got them all. Please test.
Whoops, @skessler's latest version still deletes modules/README.txt. Here's a new version that renames instead of deletes. @kristen-pol I'll let you create the MR since I've never done that in the context of d.org. :-)
I'm uploading my own version of the patch with only one change: rename modules/README.txt to modules/README.md, instead of deleting it. Please test.
The patch applies cleanly! However, I noticed one remaining issue: modules/README.txt gets deleted. Was that intentional?
I'm keeping this marked as "needs work" until this last issue gets resolved.
I'm jumping on this issue as part of the mentored contribution at Drupalcon 24!
Rather than apply this patch, I made the same updates manually, and rolled a new stable version to encourage people to use this module in newer versions of Drupal.
Thanks for the patch, and thanks for testing!
Development version: 8.x-1.x-dev updated 14 Feb 2024 at 02:45 UTC