mirror of
https://github.com/silverstripe/silverstripe-behat-extension
synced 2024-10-22 17:05:32 +02:00
Merge pull request #252 from creative-commoners/pulls/5.0/remove-todo
MNT Remove TODO comments
This commit is contained in:
commit
86c6c8148f
@ -299,7 +299,6 @@ class FixtureContext implements Context
|
||||
{
|
||||
|
||||
$class = $this->convertTypeToClass($type);
|
||||
// TODO Support more than one record
|
||||
$fields = $this->convertFields($class, $fieldsTable->getRowsHash());
|
||||
$fields = $this->prepareFixture($class, $id, $fields);
|
||||
|
||||
@ -522,7 +521,6 @@ class FixtureContext implements Context
|
||||
$yaml = implode("\n ", $yaml);
|
||||
|
||||
// Save fixtures into database
|
||||
// TODO Run prepareAsset() for each File and Folder record
|
||||
$yamlFixture = new YamlFixture($yaml);
|
||||
$yamlFixture->writeInto($this->getFixtureFactory());
|
||||
}
|
||||
|
@ -186,7 +186,7 @@ class ModuleSuiteLocator implements Controller
|
||||
// Resolve variables
|
||||
$resolvedConfig = $this->container->getParameterBag()->resolveValue($suiteConfig);
|
||||
return [
|
||||
'type' => null, // @todo figure out what this is for
|
||||
'type' => null,
|
||||
'settings' => $resolvedConfig,
|
||||
];
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user