Removed todo items (#29)

* Removed unused GroupProvider interface & implementations

* Removed @todo items

src/DependencyInjection/EzSystemsEzPlatformGraphQLExtension.php

@@ -58,8 +58,6 @@ public function prepend(ContainerBuilder $container)
     /**
      * Uses YamlConfigProvider to determinate what schema should be used.
      *
-     * @todo Make it extensible to possibly get rid of Yaml based dynamic schema.
-     *
      * @param string $configDir
      *
      * @return array

src/GraphQL/Mutation/UploadFiles.php

@@ -107,7 +107,6 @@ private function createContent(array $mapping, UploadedFile $file, $locationId,
             ])
         );
 
-        // @todo error handling
         $draft = $contentService->createContent($struct, [$locationService->newLocationCreateStruct($locationId)]);
         $content = $contentService->publishVersion($draft->getVersionInfo());
 

src/GraphQL/Resolver/DomainContentMutationResolver.php

@@ -195,8 +195,6 @@ public function deleteDomainContent(Argument $args)
             );
         }
 
-        // @todo check type of domain object
-
         try {
             $this->getContentService()->deleteContent($contentInfo);
         } catch (API\Exceptions\UnauthorizedException $e) {

src/Resources/config/default_settings.yml

@@ -50,7 +50,6 @@ parameters:
       definition_type: RelationListFieldDefinition
       value_type: RelationListFieldValue
     ezrichtext:
-      # @todo definition_type: RichTextFieldDefinition
       value_type: RichTextFieldValue
     ezselection:
       definition_type: SelectionFieldDefinition

src/Resources/config/graphql/DomainContent.types.yml

@@ -69,7 +69,6 @@ AbstractDomainContent:
                 type: String
                 resolve: "@=resolver('MainUrlAlias', [value])"
 
-# @todo interface ?
 DomainContentTypeGroup:
     type: object
     config:
@@ -105,6 +104,4 @@ DomainContentByIdentifierConnection:
             sliceSize:
                 type: Int!
             orderBy:
-                # @todo make an enum or object
-                # @todo generate
                 type: String

src/Resources/config/graphql/Field.types.yml

@@ -133,7 +133,6 @@ DateFieldValue:
             formatted:
                 type: "String"
                 description: "Formatted date"
-                # @todo: refactor with a datetime resolver
                 resolve: "@=resolver('DateTimeFormat', [args['format'], value.value])"
                 args:
                     format:
@@ -160,7 +159,6 @@ KeywordFieldValue:
             text:
                 type: "String"
                 description: "String representation of the value"
-                # @todo will fail, array to string conversion
                 resolve: "@=value"
 
 MapLocationFieldValue:

src/Resources/config/services/mutations.yml

@@ -154,11 +154,6 @@ services:
     tags:
       - { name: ezplatform_graphql.fieldtype_input_handler, fieldtype: 'ezstring', inputType: 'String' }
 
-  # @todo
-  #EzSystems\EzPlatformGraphQL\GraphQL\Mutation\InputHandler\FieldType\Time:
-  #  tags:
-  #      - { name: ezplatform_graphql.fieldtype_input_handler, fieldtype: 'eztime' }
-
   EzSystems\EzPlatformGraphQL\GraphQL\Mutation\InputHandler\FieldType\Url:
     class: 'EzSystems\EzPlatformGraphQL\GraphQL\Mutation\InputHandler\FieldType\FromHash'
     arguments:

src/Schema/Domain/Content/Worker/ContentType/DefineDomainContent.php

@@ -4,11 +4,10 @@
 use EzSystems\EzPlatformGraphQL\Schema\Domain\Content\Worker\BaseWorker;
 use EzSystems\EzPlatformGraphQL\Schema\Worker;
 use EzSystems\EzPlatformGraphQL\Schema\Builder\Input;
-use EzSystems\EzPlatformGraphQL\Schema\GroupProvider;
 use EzSystems\EzPlatformGraphQL\Schema\Builder;
 use eZ\Publish\API\Repository\Values\ContentType\ContentType;
 
-class DefineDomainContent extends BaseWorker implements Worker, GroupProvider
+class DefineDomainContent extends BaseWorker implements Worker
 {
     public function work(Builder $schema, array $args)
     {
@@ -33,9 +32,4 @@ protected function typeName(array $args): string
     {
         return $this->getNameHelper()->domainContentName($args['ContentType']);
     }
-
-    public function getGroups(array $args)
-    {
-        // @todo implement me
-    }
 }

src/Schema/Domain/Content/Worker/ContentType/DefineDomainContentType.php

@@ -4,11 +4,10 @@
 use EzSystems\EzPlatformGraphQL\Schema\Domain\Content\Worker\BaseWorker;
 use EzSystems\EzPlatformGraphQL\Schema\Worker;
 use EzSystems\EzPlatformGraphQL\Schema\Builder\Input;
-use EzSystems\EzPlatformGraphQL\Schema\GroupProvider;
 use EzSystems\EzPlatformGraphQL\Schema\Builder;
 use eZ\Publish\API\Repository\Values\ContentType\ContentType;
 
-class DefineDomainContentType extends BaseWorker implements Worker, GroupProvider
+class DefineDomainContentType extends BaseWorker implements Worker
 {
     public function work(Builder $schema, array $args)
     {
@@ -33,9 +32,4 @@ protected function typeName(array $args): string
     {
         return $this->getNameHelper()->domainContentTypeName($args['ContentType']);
     }
-
-    public function getGroups(array $args)
-    {
-        // @todo implement me
-    }
 }