- Issue created by @mably
- Merge request !18Issue #3422619: fix invalid base_field definition in maestro_all_flows view configuration → (Merged) created by mably
- Status changed to Needs review
10 months ago 11:32am 20 February 2024 - 🇫🇷France mably
Sadly, after a few updates on the view, it looks like the
base_field
value is set back toid
instead of the expectedprocess_id
.Hopefully, a simple export/import with the right
base_field
value fixes the problem.But will probably have to be done again every time the view is updated.
- 🇫🇷France mably
Ok, looks like the above problem can be fixed with a simple hook:
function vbo_maestro_views_data_alter(&$data) { if (isset($data['maestro_process'])) { $data['maestro_process']['table']['base']['field'] = 'process_id'; } }
Fix as been added to the VBO Maestro → module.
- Status changed to Closed: works as designed
10 months ago 6:25pm 20 February 2024 - 🇨🇦Canada _randy
Correct, you can override views config as you see fit.
Probably also best to not reuse core Maestro views just to be sure.
Closing.
- 🇫🇷France mably
I would not say that it works as designed as it doesn't work in some cases, but ok I'll keep my patch applied.
- Status changed to Needs review
10 months ago 5:56pm 21 February 2024 - 🇨🇦Canada _randy
Sorry, yes, you are correct. I had looked at this as if we were swapping the field in the Process entity. I'll commit this.
- Status changed to Fixed
10 months ago 6:08pm 21 February 2024 - 🇫🇷France mably
The Maestro workflow history view can now use Views Bulk Operations natively, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.