Skip to content

fix(FOUR-32849): send task action-by-email to self-service pool recip… - #9108

Open
gproly wants to merge 1 commit into
developfrom
bugfix/FOUR-32849
Open

gproly wants to merge 1 commit into
developfrom
bugfix/FOUR-32849

Conversation

@gproly

@gproly gproly commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

…ients

Resolve ABE recipients from self_service_groups when the task has no assignee (self-service with user_group/group). Previously email was only sent when a single user was assigned, so self-service tasks never triggered ABE.

https://processmaker.atlassian.net/browse/FOUR-32849

…ients

Resolve ABE recipients from self_service_groups when the task has no
assignee (self-service with user_group/group). Previously email was
only sent when a single user was assigned, so self-service tasks never
triggered ABE.

https://processmaker.atlassian.net/browse/FOUR-32849
@cursor

cursor Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

PR Summary

Medium Risk
Changes who receives ABE on task activation for unassigned self-service tasks, which can broaden email distribution to entire pools; recipient filtering reuses assignable-user rules.

Overview
Task action-by-email (ABE) now runs when a task activates even if there is no single assignee, fixing self-service tasks that use user/group pools.

When the assignee has no email, validateAndSendActionByEmail builds the to list from self_service_groups: direct user IDs plus members resolved from assigned groups via getConsolidatedUsers, then loads active assignable users with non-empty emails and sends to a comma-separated recipient list (same path as before once to is set).

A feature test with a self-service user_group BPMN fixture asserts an unassigned self-service task still creates a ProcessAbeRequestToken.

Reviewed by Cursor Bugbot for commit c546429. Bugbot is set up for automated code reviews on this repo. Configure here.

@decisions-sonarqube

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c546429. Configure here.

Comment thread ProcessMaker/Repositories/TokenRepository.php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants