Avoid deeply nested control flow statements.

Open

                        if (opts.initSelection === undefined) {
                            opts.initSelection = function (element, callback) {
                                var data = [];
                                $(splitVal(element.val(), opts.separator, opts.transformVal)).each(function () {
                                    var obj = { id: this, text: this },

Method __construct has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

        PmServiceInterface $pmService,
        NewConversationForm $newConversationForm,
        NewMessageForm $newMessageForm,
        DeleteConversationsForm $deleteConversationsForm,
        ModuleOptionsInterface $options,

Avoid deeply nested control flow statements.

Open

                        if (equal(opts.id(token), opts.id(selection[i]))) {
                            dupe = true; break;
                        }

Avoid deeply nested control flow statements.

Open

                        if (opts.createSearchChoice === undefined) {
                            opts.createSearchChoice = function (term) { return {id: $.trim(term), text: $.trim(term)}; };
                        }

Consider simplifying this complex logical expression.

Open

                if (selected.length &&
                    (e.which == KEY.LEFT || e.which == KEY.RIGHT || e.which == KEY.BACKSPACE || e.which == KEY.DELETE || e.which == KEY.ENTER)) {
                    var selectedChoice = selected;
                    if (e.which == KEY.LEFT && prev.length) {
                        selectedChoice = prev;

Function addReceivers has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public function addReceivers(ConversationInterface $conversation, array $receivers, ?UserInterface $author = null)
    {
        $receivers = array_unique($receivers);
        
        $existingReceivers = $this->pmMapper->getReceiversByConversation($conversation);

• Read up
  Read up

Avoid too many return statements within this function.

Open

                    return null;

Avoid too many return statements within this function.

Open

        return false;

Avoid too many return statements within this function.

Open

                    return this.opts.width();

Avoid too many return statements within this function.

Open

                        return;

Avoid too many return statements within this function.

Open

                return;

Avoid too many return statements within this function.

Open

                        return;

Avoid too many return statements within this function.

Open

                        return;

Avoid too many return statements within this method.

Open

        return $this->redirect()->toRoute('eye4web/zfc-user/pm/list');

Avoid too many return statements within this method.

Open

        return $this->redirect()->toRoute('eye4web/zfc-user/pm/read-conversation', ['conversationId' => $conversation->getId()]);

Avoid too many return statements within this function.

Open

                        return;

Avoid too many return statements within this function.

Open

                        return;

Avoid too many return statements within this function.

Open

                    return;

Avoid too many return statements within this function.

Open

                        return (this.opts.element.outerWidth(false) === 0 ? 'auto' : this.opts.element.outerWidth(false) + 'px');

Avoid too many return statements within this method.

Open

        return $this->redirect()->toRoute('eye4web/zfc-user/pm/list');