Page MenuHomekolab.org

Mail enabled distribution groups
ClosedPublic

Authored by machniak on Tue, Dec 29, 3:10 PM.

Details

Reviewers
vanmeeuwen
Group Reviewers
Restricted Project
Commits
rK8d8ff46cd392: Mail enabled distribution groups
Summary

CLI tools for group management

TODO:

  • Model Tests,
  • Jobs tests,
  • LDAP backend and tests,
  • Checking Group::emailExists() when creating a user/alias
  • Suspending a group when suspending a user (idea abandoned for now)
  • Removing a group member when removing a user?
  • Deleting a user should delete groups
  • force-deleting a group should remove entitlement records
  • Limit "owner" of a "group" to be the same owner as the domain in which the group is to exist (could be automatic).
    • Limits groups to custom (non-public) domains.
Test Plan

./phpunit

Diff Detail

Repository
rK kolab
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

machniak requested review of this revision.Tue, Dec 29, 3:10 PM
machniak created this revision.
machniak edited the summary of this revision. (Show Details)Tue, Dec 29, 3:12 PM
machniak updated this revision to Diff 5524.Wed, Dec 30, 10:22 AM
  • Checking Group::emailExists() when creating a user/alias
machniak edited the summary of this revision. (Show Details)Wed, Dec 30, 10:23 AM
machniak edited the summary of this revision. (Show Details)Wed, Dec 30, 10:41 AM
machniak updated this revision to Diff 5536.Wed, Dec 30, 11:05 AM
  • Delete groups of user delete
machniak edited the summary of this revision. (Show Details)Wed, Dec 30, 11:07 AM
machniak edited the summary of this revision. (Show Details)
machniak edited the summary of this revision. (Show Details)
vanmeeuwen updated this revision to Diff 5542.Wed, Dec 30, 11:39 AM
vanmeeuwen added a subscriber: vanmeeuwen.
  • Update group->ldap interaction
vanmeeuwen edited the summary of this revision. (Show Details)Wed, Dec 30, 12:27 PM
machniak updated this revision to Diff 5548.Wed, Dec 30, 2:11 PM
  • Checking Group::emailExists() when creating a user/alias
  • Delete groups of user delete
  • Various group model fixes, added tests
  • Allow groups in non-public domains only
  • Fix group:info command tests
machniak edited the summary of this revision. (Show Details)Wed, Dec 30, 2:13 PM
vanmeeuwen updated this revision to Diff 5554.Wed, Dec 30, 2:29 PM
  • Update group->ldap interaction
  • Rename command classes
  • Remove owner command-line option, bill it to the same wallet as the domain itself
  • Fix reference to Create -> CreateCommand
machniak updated this revision to Diff 5566.Wed, Dec 30, 4:10 PM
  • Fix tests regressions
  • LDAP error handling, LDAP tests
  • Group jobs tests
machniak edited the summary of this revision. (Show Details)Wed, Dec 30, 4:10 PM
machniak updated this revision to Diff 5617.Thu, Dec 31, 2:08 PM
  • Fix group members update by the LDAP backend
  • Remove group member of user delete
machniak edited the summary of this revision. (Show Details)Thu, Dec 31, 2:08 PM
vanmeeuwen accepted this revision.Thu, Dec 31, 2:47 PM
This revision is now accepted and ready to land.Thu, Dec 31, 2:47 PM
This revision was automatically updated to reflect the committed changes.