laravel-workflow icon indicating copy to clipboard operation
laravel-workflow copied to clipboard

Add support for Laravel 7

Open willemo opened this issue 4 years ago • 1 comments

Based on the code in #73, I've created my own branch with some small fixes. This should be backwards compatible with the marking store config and also support a newer format.

Old:

'marking_store' => [
    'type' => 'single_state',
    'arguments' => ['currentPlace'],
],

New:

'marking_store' => [
    'arguments' => [true, 'currentPlace'],
],

You can use my fork by changing your composer.json and running composer update brexis/laravel-workflow:

"require": {
    "brexis/laravel-workflow": "dev-laravel-7-support"
},
"repositories": [
    {
        "type": "vcs",
        "url": "[email protected]:willemo/laravel-workflow.git"
    }
]

willemo avatar Aug 27 '20 10:08 willemo

A slight warning for people wanting to update to this version: the subjects for the workflows need to have getters and setters for the marking. So if you have a model that has a state property you use to store the current place of the workflow, you need to add the getState() and setState($state) methods to that model.

willemo avatar Aug 27 '20 10:08 willemo