Skip to content

event.add_organizer needs some TLC #524

@tobixen

Description

@tobixen

The test coverage report uncovered some obscure method add_organizer that was part of my efforts on scheduling some years ago. It has the following problems:

  • Function is not covered by tests
  • Function possibly does wrong things if the organizer field already exists
  • Function can only add the current principal to the organizer field - this contrasts with the name of the method. Should take optional parameter organizer (email string or Principal object)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions