task/FOUR-26812 Dev Link functionality should include Service Task (Custom Packages) #8550
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This PR is being reviewed by Cursor Bugbot
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| $config = json_decode(Utils::getAttributeAtXPath($this->model, $path, 'pm:config'), true); | ||
|
|
||
| // Update resource ID in config | ||
| $config['resource']['id'] = $dependent->model->id; |
There was a problem hiding this comment.
Bug: Invalid JSON Handling in Resource Import
The importServiceTaskResources method attempts to update $config['resource']['id'] without validating the pm:config attribute. If json_decode returns null due to invalid JSON, or if the decoded config lacks the resource key, this causes PHP errors.
There was a problem hiding this comment.
@gustavobascope consider to include this defensive code in case the pm:config is not defined
|
caleeli
left a comment
caleeli
left a comment
There was a problem hiding this comment.
@gustavobascope consider to include this defensive code in case the pm:config is not defined
Issue & Reproduction Steps
Dev Link functionality should include Service Task (Custom Packages)
Solution
Added the new service task asset to the exportermap and processexporter
How to Test
As a Process Designer,
I want Dev Link to include packages such as Service Tasks,
So I can import all relarted assets from a Dev Link, including the packages like service tasks which are used by some processes.
Related Tickets & Packages
https://processmaker.atlassian.net/browse/FOUR-26812
https://github.com/ProcessMaker/package-service-task/pull/2