Problem/Motivation
- The module shares the fs_authmap table with other modules and uses the fs_netforum_auth table. Uninstalling one module can break the other, and the names don't follow the Drupal convention of starting table names with the module name.
- Several global functions (to_xml(), uid_from_username(), netforum_*()) don't start with the module name and can clash with functions from other modules.
- README.txt still describes Drupal 7.

Proposed resolution
- Move data to netauth_user and netauth_authmap with an update hook.
- Rename global functions with the netauth_ prefix.
- Replace README.txt with an up-to-date README.md.

Comments

vinektiwari created an issue. See original summary.

  • vinektiwari committed e567c862 on 9.0.x
    Issue #3629983 by vinektiwari: Use own database tables and follow Drupal...
vinektiwari’s picture

Version: 9.0.1 » 9.0.2
Status: Needs work » Fixed

Released in 9.0.2

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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