Select * is preventing from groupping by for Interface types

[EdwinDayot]

Versions:

• graphql-laravel Version: 5.1.4
• Laravel Version: 8.0
• PHP Version: 7.4

Description:

The generated SQL for a group by on an interface would raise an error as * is selected instead of the group by fields

Steps To Reproduce:

<?php

class MyQuery extends Query {

    public function type()
    {
        return Type::listOf(GraphQL::type('anInterface'));
    }

    public function resolve($root, $args, $context, $resolveInfo, $getSelectFields)
    {
        $fields = $getSelectFields();
        $select = $fields->getSelect();
        $with = $fields->getRelations();

        $myResults = MyModel::select($select)->with($with)->groupBy('my_field')->get();
    }
}

<?php

class MyInterface extends InterfaceType {
    public function fields(): array
    {
        return [
            'my_field' => ['type' => Type::string()]
        ]
    }
}

<?php

class MyType extends Type {
    public function fields(): array
    {
        return $interface->getFields();
    }
}

With this kind of configuration, the SQL request would result in:

select * from my_table group by my_field;

And of course, it would raise an SQL error: Syntax error or access violation: 1055 Expression #1 of SELECT list is not in GROUP BY clause and contains nonaggregated column ...

Expectations

Only the required fields should be selected

[illambo]

Maybe the problem is this

graphql-laravel/src/Support/SelectFields.php

Line 256 in 19ea3ed

if (is_a($parentType, UnionType::class) || is_a($parentType, \GraphQL\Type\Definition\InterfaceType::class)) {

[EdwinDayot]

I think it's just a part of a larger problem. Each Interface or Union should assume that every field defined in it are available in the database, but should use the alias and query defined in the type implementing the Interface or Union.

I have been working on it with the method

graphql-laravel/src/Support/SelectFields.php

Line 449 in 19ea3ed

protected static function handleInterfaceFields(

But it does not support aliases. (apparently)

[illambo]

Great! May the force be with you :)

[mfn]

Added the SelectFields label, seems like another shortcoming with this part.

[crissi]

I am taking a look at this, it seems that relationships are not working well either...

[crissi]

Tried to tackle this by using webonyx QueryPlan helper, which could ease this problem quite a bit, but since it still have unresolved issues, like this: webonyx/graphql-php#831, It will have to wait for now

[mfn]

Does https://github.com/nuwave/lighthouse support? Not sure, might be worth checking out how they do it.