Skip to content

Commit

Permalink
cleanups
Browse files Browse the repository at this point in the history
  • Loading branch information
oliwierptak committed Aug 8, 2021
1 parent 2260140 commit b55a61c
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 29 deletions.
2 changes: 0 additions & 2 deletions src/Popo/Model/Generate/GenerateModel.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@

use Popo\Builder\PopoBuilder;
use Popo\Builder\SchemaBuilder;
use Popo\Model\Generate\GenerateResult;
use Popo\PopoConfigurator;

class GenerateModel
Expand All @@ -20,7 +19,6 @@ public function __construct(
public function generate(PopoConfigurator $configurator): GenerateResult
{
$result = new GenerateResult;

$data = $this->schemaBuilder->build($configurator);

foreach ($data as $schemaName => $schemaCollection) {
Expand Down
50 changes: 23 additions & 27 deletions src/Popo/Model/Report/ReportModel.php
Original file line number Diff line number Diff line change
Expand Up @@ -37,24 +37,22 @@ protected function generateReport(array $data, SchemaFile $sharedSchemaFile): Re
$sharedSchemaFileConfig = [];

foreach ($sharedFileConfig as $dataItem) {
$result->add(
(new ReportResultItem())
->setName($dataItem['name'])
->setData($dataItem)
->setSchemaFilename($sharedSchemaFile->getFilename()->getPathname())
->markAsFileConfig()
$result->add((new ReportResultItem())
->setName($dataItem['name'])
->setData($dataItem)
->setSchemaFilename($sharedSchemaFile->getFilename()->getPathname())
->markAsFileConfig()
);
}

foreach ($data as $schemaFile) {
$fileConfig = $schemaFile->getFileConfig()['property'] ?? [];
foreach ($fileConfig as $dataItem) {
$result->add(
(new ReportResultItem())
->setName($dataItem['name'])
->setData($dataItem)
->setSchemaFilename($schemaFile->getFilename()->getPathname())
->markAsFileConfig()
$result->add((new ReportResultItem())
->setName($dataItem['name'])
->setData($dataItem)
->setSchemaFilename($schemaFile->getFilename()->getPathname())
->markAsFileConfig()
);
}

Expand All @@ -64,26 +62,24 @@ protected function generateReport(array $data, SchemaFile $sharedSchemaFile): Re
->getSchemaConfig()[$schemaName]['property'] ?? [];

foreach ($schemaConfig as $dataItem) {
$result->add(
(new ReportResultItem())
->setName($dataItem['name'])
->setData($dataItem)
->setSchemaName($schemaName)
->setSchemaFilename($schemaFile->getFilename()->getPathname())
->markAsSchemaConfig()
$result->add((new ReportResultItem())
->setName($dataItem['name'])
->setData($dataItem)
->setSchemaName($schemaName)
->setSchemaFilename($schemaFile->getFilename()->getPathname())
->markAsSchemaConfig()
);
}

foreach ($popoCollection as $popoName => $popoData) {
foreach ($popoData['property'] as $propertyData) {
$result->add(
(new ReportResultItem())
->setName($propertyData['name'])
->setData($propertyData)
->setPopoName($popoName)
->setSchemaName($schemaName)
->setSchemaFilename($schemaFile->getFilename()->getPathname())
->markAsPropertyConfig()
$result->add((new ReportResultItem())
->setName($propertyData['name'])
->setData($propertyData)
->setPopoName($popoName)
->setSchemaName($schemaName)
->setSchemaFilename($schemaFile->getFilename()->getPathname())
->markAsPropertyConfig()
);
}
}
Expand Down

0 comments on commit b55a61c

Please sign in to comment.