pimcore / data-hub

Data delivery & consumption platform for Pimcore.
Other
127 stars 108 forks source link

[Bug]: modificationDate #849

Open andrewa17github opened 6 months ago

andrewa17github commented 6 months ago

Expected behavior

Previous versions of datahub allowed modicationDate to be passed as an input in the mutation.

Actual behavior

The latest version does not allow this with error on attempting a mutation with input of modificationDate using a timestamp, this is a problem to perform third party system delta updates with larger datasets to make sure no data is missed and also not repeat processing the same object due to a newer modification date.

` )

        [0] => stdClass Object

            (

                [message] => Field "modificationDate" is not defined by type "UpdateProductInput".

                [locations] => Array

                    (

                        [0] => stdClass Object

                            (

                                [line] => 1

                                [column] => 53

                            )

                    )

                [extensions] => stdClass Object

                    (

                        [file] => /.../vendor/webonyx/graphql-php/src/Validator/Rules/ValuesOfCorrectType.php

                        [line] => 117

                    )

            )

    )`

Steps to reproduce

$mutation = 'updateProduct(id: '.$productMaster->id.', input: {modificationDate: ' . $startDtm . '}) {success,message,output{id,key,published} }';

github-actions[bot] commented 5 months ago

Thanks a lot for reporting the issue. We did not consider the issue as "Pimcore:Priority", "Pimcore:ToDo" or "Pimcore:Backlog", so we're not going to work on that anytime soon. Please create a pull request to fix the issue if this is a bug report. We'll then review it as quickly as possible. If you're interested in contributing a feature, please contact us first here before creating a pull request. We'll then decide whether we'd accept it or not. Thanks for your understanding.