Hi,
before we completely redesign the auth backend we should define a
stable state for the next release and postpone all other activity.
To step a bit forward I prepared a patch according my last suggestion
to move group handling into the user manager.
The attached patch changed the ModifyUser function of the user manager
only forward changed data to the backend. Entries the user didn't
change will be thrown into the bin. This feature I would like to see in
dokuwiki anyway.
I couldn't stop working so I added group handling too. The ModifyUser
function won't forward changed groups to the backend/modifyUser() any
longer. Instead it changes group membership itself with
backend/joinGroup and backend/leaveGroup. As side effect: If joinGroup
or leaveGroup is not supported, the group entry field will be disabled.
If this approach was accepted for the next release, some further
cleanups would be necessary:
1. The mysql/joinGroup won't create unknown groups by default. Adding
a user to a unknown group will result in an error message. This
should be changed (easy).
2. backend/modifyUser doesn't have to handle groups any longer and a
lot of dependencies are not longer necessary (Andi's first wish).
The code need some cleanup here.
3. The user manager uses backend/getUserCount in some places without
a support check. This should be fixed too.
Best Regards
Matthias
