Contrib.social

BPMN: Array to string conversion warning - augmentor

Open on Drupal.org →
Created on 5 June 2024, over 1 year ago

Problem/Motivation

After installing Augmentor AI module, ECA it triggers the warning "Warning: Array to string conversion in /var/www/html/docroot/modules/contrib/eca/modules/modeller_bpmn/src/ModellerBpmnBase.php on line 776" when trying to visit the ECA workflow.

It seems like the value is being implicitly converted when casted as a string for source_fields that are provided by the Augmentor AI's action. 

case 'select':
    $fields[] = $this->optionsField($key, $label, $weight, $description, $definition['#options'], (string) $value, $definition['#required'] ?? FALSE);

Steps to reproduce

  1. Install ECA Module / Enable
  2. Install BPMN / Enable
  3. Install Augmentor AI / Enable
  4. Visit the workflow page and notice the warning.

Proposed resolution

Adding the below before the cast seems to cure the issue.
$value = is_array($value) ? '' : $value;

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Active

Version

1.0

Component

Code

Created by

🇺🇸United States j-barnes

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @j-barnes
  • Comment over 1 year ago →
    🇺🇸United States j-barnes
  • Comment over 1 year ago →
    🇺🇸United States j-barnes
  • Merge request !426Added array check validation for value. → (Closed) created by j-barnes
  • Comment over 1 year ago →
    🇺🇸United States j-barnes
  • Status changed to Needs review over 1 year ago
  • Comment over 1 year ago →
    🇺🇸United States j-barnes
  • Comment over 1 year ago →
    🇺🇸United States j-barnes
  • Pipeline finished with Success
    over 1 year ago
    Total: 469s
    #192245
  • Status changed to Closed: duplicate over 1 year ago
  • Comment over 1 year ago →
    🇩🇪Germany jurgenhaas Gottmadingen

    Hi @j-barnes thanks for reporting this and also contributing to the solution. However, this is a duplicate of ✨ Add support for select fields with multiple set to true Active where we've taken a different approach which keeps multi-values as an array and provide support in a textfield with values being separated by commas.

  • Pipeline finished with Success
    over 1 year ago
    Total: 184s
    #196250
  • Pipeline finished with Running
    over 1 year ago
    #197240
  • Pipeline finished with Success
    over 1 year ago
    Total: 212s
    #197628
  • Pipeline finished with Failed
    12 months ago
    Total: 186s
    #347884
  • Pipeline finished with Failed
    12 months ago
    Total: 383s
    #347888
  • Pipeline finished with Failed
    12 months ago
    Total: 556s
    #347889
  • Pipeline finished with Success
    11 months ago
    Total: 641s
    #350745
  • Pipeline finished with Skipped
    11 months ago
    #350811
  • Pipeline finished with Failed
    10 months ago
    Total: 190s
    #392307
  • Pipeline finished with Canceled
    10 months ago
    Total: 120s
    #392310
  • Pipeline finished with Success
    10 months ago
    Total: 823s
    #392311
  • Pipeline finished with Success
    9 months ago
    Total: 383s
    #404441
  • Pipeline finished with Failed
    7 months ago
    Total: 564s
    #459652
  • Pipeline finished with Canceled
    7 months ago
    Total: 416s
    #459668
  • Pipeline finished with Failed
    7 months ago
    Total: 683s
    #459671
  • Pipeline finished with Success
    7 months ago
    Total: 745s
    #459677
  • Comment 6 months ago →
    System Message
  • Pipeline finished with Failed
    2 months ago
    #588210
  • Pipeline finished with Failed
    2 months ago
    #588223
  • Pipeline finished with Success
    2 months ago
    #588777
  • Pipeline finished with Success
    about 1 month ago
    Total: 499s
    #608122
contrib.social Blog FAQ Discussions
Production build 0.71.5 2024