It is good idea have support for executing process on multiple records in Grid view. Same is requested in past https://groups.google.com/forum/#!topic/idempiere/YDnr8nCFGyA
I suggest to implement this as below
set Record_IDs when process invoked from toolbar in grid view and there are multiple records selected. Process designed to execute on multiple records will check this parameter and if presents then work on it other wise use Record_ID
Yes it works in info window but we have to create new info window for any multi processing requirement.
Mostly user don't prefer to work on two different window based on processing single record vs multiple records.
We have done without T_Selection but can do with T_Selection too.
, I mean, implement it in the window button, but using the same approach and table as is currently being used in info window.
That way we keep consistency, you just need to write one process and it can work for info window or window, both if needed.
Having a different approach for windows than for info windows sounds more confusing.
Yes, we are adding support in toolbar button on windows. We did without creating entry in T_selection but now we are creating entry in T_Selection too, So that process can be written to work in info window and from toolbar buttons on window.
Please find attached patch. Let me know if needs to do any correction.
Please note that previous patch was not considering detail view, now this patch has support for both detail view and header part.