René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"MySQL Syntax Error in ActiveRelationResource (0.10.x)","articleBody":"## This issue is a (choose one):\r\n\r\n- [x] Problem/bug report.\r\n- [ ] Feature request.\r\n- [ ] Request for support. **Note: Please try to avoid submitting issues for support requests. Use [Gitter](https://gitter.im/cerebris/jsonapi-resources) instead.** \r\n\r\n## Checklist before submitting:\r\n\r\n- [x] I've searched for an existing issue.\r\n- [ ] I've asked my question on [Gitter](https://gitter.im/cerebris/jsonapi-resources) and have not received a satisfactory answer.\r\n- [ ] I've included a complete [bug report template](https://github.com/cerebris/jsonapi-resources/blob/master/lib/bug_report_templates/rails_5_master.rb). This step helps us and allows us to see the bug without trying to reproduce the problem from your description. It helps you because you will frequently detect if it's a problem specific to your project.\r\n- [ ] The feature I'm asking for is compliant with the [JSON:API](http://jsonapi.org/) spec.\r\n\r\n## Description\r\n\r\nChoose one section below and delete the other:\r\n\r\n### Bug reports:\r\n\r\nVersion `0.10.5`\r\n\r\n```\r\n#\u003cActiveRecord::StatementInvalid: Mysql2::Error: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '.\"id\" AS \"individuals_id\", individuals.id FROM `individuals` WHERE `individuals`' at line 1: SELECT  \"individuals\".\"id\" AS \"individuals_id\", individuals.id FROM `individuals` WHERE `individuals`.`deleted_at` IS NULL ORDER BY individuals.id asc LIMIT 10 OFFSET 0\u003e\r\n\u003e\u003e fragments = resource_klass.find_fragments(find_options[:filters], options)\r\n```\r\n\r\nThis is related to #1310, but is not a multi-tenancy issue. This a MySQL syntax issue that I believe it created by the [`ActiveRelationResource#quote`](https://github.com/cerebris/jsonapi-resources/blob/v0.10.5/lib/jsonapi/active_relation_resource.rb#L833-L835) method which is uses explicit double quotes for quoting table names in [`ActiveRelationResource#concat_table_field`](https://github.com/cerebris/jsonapi-resources/blob/v0.10.5/lib/jsonapi/active_relation_resource.rb#L789-L807) and [`ActiveRelationResource#alias_table_field`](https://github.com/cerebris/jsonapi-resources/blob/v0.10.5/lib/jsonapi/active_relation_resource.rb#L813-L831).\r\n\r\nBy adding MySQL to the CI workflow testing matrix, this issue is easily reproduced in CI testing. I will create a PR for adding MySQL to the testing matrix.\r\n","author":{"url":"https://github.com/karlwilbur","@type":"Person","name":"karlwilbur"},"datePublished":"2021-05-29T00:46:33.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":7},"url":"https://github.com/1369/jsonapi-resources/issues/1369"}