add flow_module to searchdefs AOW_Workflow

I thought this would have been a sensible thing to do and I’ve added it no problem.
Is there a reason why workflows are not filterable by module by default?

for those interested in the pretty-printed updated file:


<?php
/**
 * Advanced OpenWorkflow, Automating SugarCRM.
 * @package Advanced OpenWorkflow for SugarCRM
 * @copyright SalesAgility Ltd http://www.salesagility.com
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE as published by
 * the Free Software Foundation; either version 3 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
 * along with this program; if not, see http://www.gnu.org/licenses
 * or write to the Free Software Foundation,Inc., 51 Franklin Street,
 * Fifth Floor, Boston, MA 02110-1301  USA
 *
 * @author SalesAgility <info@salesagility.com>
 */

$module_name = 'AOW_WorkFlow';
  $searchdefs['AOW_WorkFlow'] =
  array (
    'templateMeta' =>
    array (
      'maxColumns' => '3',
      'maxColumnsBasic' => '4',
      'widths' => array ('label' => '10', 'field' => '30'),                     
    ),
    'layout' =>
    array (
      'basic_search' =>
      array (
        'name',
        'flow_module', // <--
        array (
          'name' => 'current_user_only',
          'label' => 'LBL_CURRENT_USER_FILTER',
          'type'=>'bool'
        ),
      ),
      'advanced_search' =>
      array (
        'name',
        'flow_module', // <-- 
        'status',
        array (
          'name' => 'assigned_user_id',
          'label' => 'LBL_ASSIGNED_TO',
          'type' => 'enum',
          'function' => array (
            'name' => 'get_user_array',
            'params' => array(false)
          )
        ),
      ),
    ),
  );

Sorry, can you explain a little better what you achieved? I don’t know what flow_module is…

flow_module is the field in the AOW_Workflows module which dictates which module the workflow is to run against

by default on list view for AOW_Workflows, there is no way to search for workflows by what module they are for which I think would be the most important field to filter against.

Ah, I see now. You’re right, this is useful.

Do you want to make a PR on GitHub and include this in the next version?

Thanks