As seen in the code, you can assing a task to multiple ids by placing the ids as a variable separated by coma characters. But when this string is detected, the variable used in maestro_engine_version1.class function assignTask($queueId,$assignment) is different from the one used in the forearch

          if (strpos($assignId, ':') !== false) {
            $userIds = explode(':', $assignId);
          }
          else {
            $assignIds = array($assignId);
          }
          foreach ($assignIds as $assignId) {

The change would be changing $userIds variable name and us $userIds.

I think this error is in the 7.x-1.1-dev version.
P.S. Sorry about the name of the patch. I'm quite new and don't know how to give it an appropiate name,.

CommentFileSizeAuthor
maestro-engine-version1.patch573 bytesbit_jammer
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

_randy’s picture

Status: Active » Needs review

Thanks for this. now in git.

bit_jammer’s picture

Status: Needs review » Fixed

Good!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit cce5841 on master, 8.x-1.x by _randy:
    Fix for issue #1284780 -- Incorrect multiple assignment variable
    
    
  • Commit c09598a on master, 8.x-1.x authored by bit_jammer, committed by _randy:
    Proper attribution to bit_jammer for fix #1284780