Omit fields not defined on vardefs from listview definition api

- Check if if fields are defined on the vardefs and omit any that aren't
-- Log any found
- Clean moduleMatcher list view enable module list
- Add missing modules to use new list view
This commit is contained in:
Clemente Raposo 2020-08-27 17:15:48 +01:00 committed by Dillon-Brown
parent 63fe5948ae
commit 04d17f2631
3 changed files with 64 additions and 33 deletions

View file

@ -5,9 +5,13 @@ namespace App\Tests;
use App\Service\AclManagerInterface;
use App\Service\BulkActionDefinitionProvider;
use App\Service\ChartDefinitionProvider;
use App\Service\FilterDefinitionProvider;
use App\Service\FilterDefinitionProviderInterface;
use App\Service\LineActionDefinitionProvider;
use Codeception\Test\Unit;
use Exception;
use Monolog\Logger;
use Psr\Log\LoggerInterface;
use SuiteCRM\Core\Legacy\AclHandler;
use SuiteCRM\Core\Legacy\AppListStringsHandler;
use SuiteCRM\Core\Legacy\FieldDefinitionsHandler;
@ -168,6 +172,17 @@ final class ViewDefinitionsHandlerTest extends Unit
]
);

/** @var LoggerInterface $logger */
$logger = $this->make(
Logger::class,
[
'warning' => static function (
string $module
): void {
}
]
);

$this->viewDefinitionHandler = new ViewDefinitionsHandler(
$projectDir,
$legacyDir,
@ -179,7 +194,8 @@ final class ViewDefinitionsHandlerTest extends Unit
$bulkActionProvider,
$chartDefinitionProvider,
$lineActionDefinitionProvider,
$filterDefinitionHandler
$filterDefinitionHandler,
$logger
);

// Needed for aspect mock