graphql-rails-api 0.8.0 → 0.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 529885203fa8337d28c94ac45e07cedc82c9393dc671c1dca5ab2880c3904013
-  data.tar.gz: 214d0d3c85fb42b6d1a7e50cbe3214f169a71240cd3df9fde43120a71c9d826e
+  metadata.gz: 2e7801279a80cf36fceb00b2c734cda2ff89e0c695fa77efa10e706e10fb708b
+  data.tar.gz: e0fa94077d38b530f56617ccf14825f330d427c510bf44259f43baf2d28ffc8a
 SHA512:
-  metadata.gz: a5d2d9873c382edee93417c8dbb63265c08594b98f1461f3b35de18518c12584f568d5c6fad6d30f0e4f067d2e119a66d588d13b99302043bc460c7a22d8236e
-  data.tar.gz: 8804df58777ba615ece39a245a72457ea0a52db7d22a99411e7c02abfa3de97fd63f60f1d9ecf6f1d3d9fdcf8e464e7c5e6e0e415c3de1ad010cf149687a93e1
+  metadata.gz: 45940578253a9645e325860234d793dc8c00c7fdb69c5758748f45fe66d09c32f08e43d60b3180537953dead3f2b5edf470fe2516cf40a5ed0a237124cd3df8e
+  data.tar.gz: 3f830d57e8ae131cae74735ba29ea24e3a8b2391899aea329b18aec18fc3b75647e3d085201211c514fd6d397dba1739ae0471f0ea906e0650c6214fbcf429ee

data/README.md

@@ -1,48 +1,87 @@
-# GraphqlRailsApi
+# Graphql Rails Api
+`graphql-rails-api` is a wrapper around [graphql-ruby](https://graphql-ruby.org/) for rails application. It describes easily your graphql API in a domain driven design way.
 
-`graphql-rails-api` is a gem that provides generators to describe easily your graphql API in a domain driven design way.
+The main purpose of this gem is to earn time providing:
+- A normalized code architecture by mapping graphql resources to the active record models
+- A global graphql service to directly perform crud mutations
+- A set of generators to create or modify graphql resource and active record model at the same time
 
-Need any help or wanna talk with me about it on slack ? Don't hesitate, 
-https://bit.ly/2KvV8Pk
+### Notes
+Only postgresql adapter is maintained.
+Only model using uuid as identifier are compatible with generated migrations right now.
+A model User will be created during installation.
 
 ## Installation
 
-Create a rails app via
-```
-rails new project-name --api --database=postgresql
-cd project-name
-bundle
-rails db:create
-```
-
-Add these lines to your application's Gemfile:
+Add the gem to your application's Gemfile:
 ```ruby
-gem 'graphql'
 gem 'graphql-rails-api'
 ```
 
-And then execute:
+Download and install the gem:
 ```bash
 $ bundle
 $ rails generate graphql_rails_api:install
 ```
 
+### Options
+The following options to the `graphql_rails_api:install` command are available:
+
+
 To disable PostgreSQL uuid extension, add the option `--no-pg-uuid`
 
 To disable ActionCable websocket subscriptions, add the option `--no-action-cable-subs`
 
 To disable Apollo compatibility, add the option `--no-apollo-compatibility`
 
-Automatically, `post '/graphql', to: 'graphql#execute'` will be added to your `config/route.rb`
-To avoid this, you can just add the option `--no-generate-graphql-route`
+To avoid the addition of a new post '/graphql' route , add the option `--no-generate-graphql-route`
 
-# Usage
+## Get Started
 
-## Resources generation
+Generate a new active record model with its graphql type and input type.
+```bash
+$ rails generate graphql_resource city name:string
+```
+Reboot the rails server, and you're good to go!
 
+Now You can perform crud mutation on resources:
 ```bash
-$ rails generate graphql_resource resource_name field1:string field2:float belongs_to:other_resource_name has_many:other_resources_name many_to_many:other_resources_name
+curl -X POST http://localhost:3000/graphql \
+  -H "content-type: application/json"      \
+  -d '{ "query":"mutation($name: String) { create_city(city: { name: $name }) { name } }", "variables": {"name":"Paris"} }'
+
+=> {"data":{"create_city":{"name":"Paris"}}}
 ```
+You can perform queries as well:
+```bash
+curl -X POST http://localhost:3000/graphql \
+  -H "content-type: application/json"       \
+  -d '{ "query": "{ cities { name } }" }'
+
+=> {"data":{"cities":[ {"name":"Paris"} ] } }
+```
+
+## Generators
+
+```bash
+$ rails generate graphql_resource computer code:string price:integer power_bench:float belongs_to:user has_many:hard_drives many_to_many:tags
+```
+
+
+This line will create the data migration, the model and the graphql type of the Computer resource.
+
+It will automatically add `has_many :computers` to the User model
+
+It will add a `computer_id` to the `HardDrive` model, and
+
+respectively the `has_many :hard_drives` and `belongs_to :computer` to the `Computer` and `HardDrive` models.
+
+The `many_to_many` option will make the `has_many through` association and create the join table between tag and computer.
+
+All of these relations will be propagated to the graphql types.
+
+### Options
+
 
 To disable migration generation, add the option `--no-migration`
 
@@ -56,40 +95,236 @@ To disable graphql-type generation, add the option `--no-graphql-type`
 
 To disable graphql-input-type generation, add the option `--no-graphql-input-type`
 
-To disable propagation (has_many creating the id in the other table, many to many creating the join table, and apply to the graphql types), add the option `--no-propagation`
-
-I made the choice of migrate automatically after generating a resource, to avoid doing it each time.
-You can of course disable the automatic migrate by adding the option `--no-migrate`
+To disable propagation (has_many creating the id in the other table, many to many creating the join table and apply to the graphql types), add the option `--no-propagation`
+To avoid running migrations after a resource generation, add the option `--no-migrate`
 
-## On generating resources
 
+### Note on enum
+The library handle enum with an integer column in the model table. Enum is defined into the active record model.
+Example:
 ```bash
-$ rails generate graphql_resource computer code:string price:integer power_bench:float belongs_to:user has_many:hard_drives many_to_many:tags
+$ rails generate graphql_resource house energy_grade:integer belongs_to:user belongs_to:city
+```
+house.rb
+```ruby
+class House < ApplicationRecord
+  belongs_to :city
+  belongs_to :user
+  enum energy_grade: {
+    good: 0,
+    average: 1,
+    bad: 2,
+  }
+end
 ```
 
-This line will create the data migration, the model and the graphql type of the Computer resource.
+## About queries
+3 types of queries are available.
 
-It will automatically add `has_many :computers` to the User model
+show query:
+```gql
+query($id: String!) {
+  city(id: $id) {
+    id
+  }
+}
+```
+index query:
+A max number of 1000 results are return.
+```gql
+query($page: String, per_page: String, $filter: String, $order_by: String) {
+  cities {
+    id
+  }
+}
+```
+index query with pagination: Add the suffix `paginated_` to any index query to use pagination
+```gql
+query($page: String, per_page: String, $filter: String, $order_by: String) {
+  paginated_cities {
+    id
+  }
+}
+```
 
-It will add a `computer_id` to the `HardDrive` model, and
-respectively the `has_many :hard_drives` and `belongs_to :computer` to the `Computer` and `HardDrive` models.
+### Filter query results
+`filter` is a not required string argument used to filter data based on conditions made on active record model fields.
+You can use :
+- parenthesis : `()`
+- Logical operators : `&&` , `||`
+- Comparaisons operators : `==`, `!=`, `===`, `!==`, `>`, `<`, `>=`, `<=`
 
-The `many_to_many` option will make the `has_many through` association and create the join table between tag and computer.
+The operators `===` and `!==` are used to perform case sensitive comparaisons on string fields
+Example:
+The following model is generated
+```
+rails generate graphql_resource house \
+  street:string        \
+  number:integer       \
+  price:float          \
+  energy_grade:integer \
+  principal:boolean    \
+  belongs_to:user      \
+  belongs_to:city
+```
+The following filter values can be used:
+```ruby
+"street != 'candlewood lane'"
+"street !== 'Candlewood Lane'"
+"number <= 50"
+"price != 50000"
+"build_at <= '#{DateTime.now - 2.years}'"
+"user.email == 'jason@gmail.com'"
+"city.name != 'Berlin'"
+"street != 'candlewood lane' && (city.name != 'Berlin' || user.email == 'jason@gmail.com')"
+```
 
-All of these relations will be propagated to the graphql types.
+### Order query result argument
+`order_by` is a non mandatory string argument used to order the returned data.
+With the model above the following order_by values can be used:
+```ruby
+"street DESC"
+"number ASC"
+"user.email ASC"
+```
+## About mutations
+The graphql-rails-api application service can handle 5 types of mutation on generated models.
+
+create mutation:
+```gql
+mutation($name: String!) {
+  create_city(
+    city: {
+      name: $name
+    }
+  ) {
+    id
+  }
+}
+```
 
+update mutation:
+```gql
+mutation($id: String!, $name: String) {
+  update_city(
+    id: $id
+    city: {
+      name: $name
+    }
+  ) {
+    id
+    name
+  }
+}
+```
 
+destroy mutation:
+```gql
+mutation($id: String!) {
+  destroy_city(id: $id) {
+    id
+  }
+}
+```
 
-## Graphql API example
+bulk_create mutation:
+```gql
+mutation($cities: [CityInputType]!) {
+  bulk_create_city(cities: $cities) {
+    id
+  }
+}
+```
 
-Example of a backend API: https://github.com/Poilon/graphql-rails-api-example
+bulk_update:
+```gql
+mutation($cities: [CityInputType]!) {
+  bulk_update_city(cities: $cities) {
+    id
+  }
+}
+```
 
+You can override the default application service for all mutation by defining your own method into the corresponding graphql service:
 
+Example:
+
+app/graphql/cities/service.rb
+```ruby
+module Cities
+  class Service < ApplicationService
+    def create
+      return graphql_error('Forbidden') if params[:name] == 'Forbidden city'
+
+      super
+    end
+  end
+end
+```
+## Custom mutation resource services
+
+To defined your own custom mutation create a file to defined the mutation type and define the correponding methods.
+
+Example:
+
+app/graphql/cities/mutations/custom.rb
+```ruby
+Cities::Mutations::Custom = GraphQL::Field.define do
+  description 'Im a custom mutation'
+  type Cities::Type
+
+  argument :id, !types.String
+  argument :name, !types.String
+
+  resolve ApplicationService.call(:city, :custom)
+end
+```
+
+app/graphql/cities/service.rb
+```ruby
+module Cities
+  class Service < ApplicationService
+    def custom
+      ...
+    end
+  end
+end
+```
+
+## TODO
+
+- Clean error management : catching on raise ... no unexpected json blabla
+- Query type
+	- remove me type
+	- add paginated_resource types
+- Logging on generators actions
+- Don't create model user if it exists
+- Config variable for max number of result (Per page on query)
+- Handle id type != uuid (generated migration)
+- Filter:
+	- handle association multiple level
+- Order
+	- handle order on multiple field
+	- handle order on association multiple level
+	- case sensitivity on order by string / text
+- Write Spec to test visible_for
+- Write Spec to test filter order and visible for together
+- Write spec to test installation and generators
+- Documentation about user authentication and scope
+  - describe authenticated_user
+  - describe visible_for
+  - describe writable_by
+
+## Graphql API example
+
+Example of a backend API: https://github.com/Poilon/graphql-rails-api-example
 
 ## Contributing
 
 You can post any issue, and contribute by making pull requests.
 Don't hesitate, even the shortest pull request is great help. <3
 
+Need any help or wanna talk with me on discord : Poilon#5412
+
 ## License
 The gem is available as open source under the terms of the [MIT License](http://opensource.org/licenses/MIT).

data/lib/generators/graphql_add_fields/graphql_add_fields_generator.rb

@@ -245,10 +245,10 @@ t.#{@id_db_type} :#{resource.underscore.singularize}_id
     file_name = "app/models/#{model.underscore.singularize}.rb"
     return if !File.exist?(file_name) || File.read(file_name).include?(line)
 
-    line_count = `wc -l "#{file_name}"`.strip.split(' ')[0].to_i
-
+    file = open(file_name)
+    line_count = file.readlines.size
     line_nb = 0
-    File.open(file_name).each do |l|
+    file.each do |l|
       line_nb += 1
       break if l.include?('ApplicationRecord')
     end

data/lib/generators/graphql_all_connections/graphql_all_connections_generator.rb

@@ -8,6 +8,7 @@ class GraphqlAllConnectionsGenerator < Rails::Generators::NamedBase
   end
 
   def generate_connection(dir, resource)
+    FileUtils.mkdir_p(dir) unless File.directory?(dir)
     File.write(
       "#{dir}/connection.rb",
       <<~STRING

data/lib/generators/graphql_mutations/graphql_mutations_generator.rb

@@ -3,7 +3,7 @@ class GraphqlMutationsGenerator < Rails::Generators::NamedBase
   def generate
     resource = file_name.underscore.singularize
     dir = "app/graphql/#{resource.pluralize}/mutations"
-    system("mkdir -p #{dir}")
+    FileUtils.mkdir_p(dir) unless File.directory?(dir)
     generate_create_mutation(dir, resource)
     generate_update_mutation(dir, resource)
     generate_destroy_mutation(dir, resource)

data/lib/generators/graphql_rails_api/install_generator.rb

@@ -8,7 +8,9 @@ module GraphqlRailsApi
 
     def generate_files
       @app_name = File.basename(Rails.root.to_s).underscore
-      system('mkdir -p app/graphql/')
+
+      folder = 'app/graphql/'
+      FileUtils.mkdir_p(folder) unless File.directory?(folder)
 
       write_uuid_extensions_migration
 
@@ -20,9 +22,7 @@ module GraphqlRailsApi
 
       write_controller
 
-      write_websocket_models
-      write_websocket_connection
-      write_subscriptions_channel
+      system 'rails g graphql_resource user first_name:string last_name:string email:string'
 
       write_application_record_methods
       write_initializer
@@ -32,12 +32,6 @@ module GraphqlRailsApi
 
     private
 
-    def write_websocket_models
-      system 'rails g graphql_resource user first_name:string last_name:string email:string'
-      system 'rails g graphql_resource websocket_connection belongs_to:user connection_identifier:string'
-      system 'rails g graphql_resource subscribed_query belongs_to:websocket_connection result_hash:string query:string'
-    end
-
     def write_route
       route_file = File.read('config/routes.rb')
       return if route_file.include?('graphql')
@@ -47,8 +41,7 @@ module GraphqlRailsApi
         route_file.gsub(
           "Rails.application.routes.draw do\n",
           "Rails.application.routes.draw do\n" \
-          "  post '/graphql', to: 'graphql#execute'\n" \
-          "  mount ActionCable.server => '/cable'\n"
+          "  post '/graphql', to: 'graphql#execute'\n"
         )
       )
     end
@@ -89,25 +82,13 @@ module GraphqlRailsApi
     all
   end
 
-  def self.broadcast_queries
-    WebsocketConnection.all.each do |wsc|
-      wsc.subscribed_queries.each do |sq|
-        result = #{@app_name.camelize}Schema.execute(sq.query, context: { current_user: wsc.user })
-        hex = Digest::SHA1.hexdigest(result.to_s)
-        next if sq.result_hash == hex
-
-        sq.update_attributes(result_hash: hex)
-        SubscriptionsChannel.broadcast_to(wsc, query: sq.query, result: result.to_s)
-      end
-    end
-  end
 
-        STRING
-      )
-    end
+  STRING
+  )
+end
 
     def write_require_application_rb
-      write_at('config/application.rb', 5, "require 'graphql/hydrate_query'\nrequire 'rkelly'\n")
+      write_at('config/application.rb', 5, "require 'graphql/hydrate_query'\nrequire 'rkelly'\nrequire 'graphql'\n")
     end
 
     def write_uuid_extensions_migration
@@ -119,8 +100,7 @@ module GraphqlRailsApi
           class UuidPgExtensions < ActiveRecord::Migration[5.2]
 
             def change
-              execute 'CREATE EXTENSION "pgcrypto" SCHEMA pg_catalog;'
-              execute 'CREATE EXTENSION "uuid-ossp" SCHEMA pg_catalog;'
+              enable_extension 'pgcrypto'
             end
 
           end
@@ -139,70 +119,6 @@ module GraphqlRailsApi
       )
     end
 
-    def write_websocket_connection
-      File.write(
-        'app/channels/application_cable/connection.rb',
-        <<~'STRING'
-          module ApplicationCable
-            class Connection < ActionCable::Connection::Base
-
-              identified_by :websocket_connection
-
-              def connect
-                # Check authentication, and define current user
-                self.websocket_connection = WebsocketConnection.create(
-                  # user_id: current_user.id
-                )
-              end
-
-            end
-          end
-        STRING
-      )
-    end
-
-    def write_subscriptions_channel
-      File.write(
-        'app/channels/subscriptions_channel.rb',
-        <<~STRING
-          class SubscriptionsChannel < ApplicationCable::Channel
-
-            def subscribed
-              stream_for(websocket_connection)
-              websocket_connection.update_attributes(connection_identifier: connection.connection_identifier)
-              ci = ActionCable.server.connections.map(&:connection_identifier)
-              WebsocketConnection.all.each do |wsc|
-                wsc.destroy unless ci.include?(wsc.connection_identifier)
-              end
-            end
-
-            def subscribe_to_query(data)
-              websocket_connection.subscribed_queries.find_or_create_by(query: data['query'])
-              SubscriptionsChannel.broadcast_to(
-                websocket_connection,
-                query: data['query'],
-                result: #{@app_name.camelize}Schema.execute(data['query'], context: { current_user: websocket_connection.user })
-              )
-            end
-
-            def unsubscribe_to_query(data)
-              websocket_connection.subscribed_queries.find_by(query: data['query'])&.destroy
-            end
-
-            def unsubscribed
-              websocket_connection.destroy
-              ci = ActionCable.server.connections.map(&:connection_identifier)
-              WebsocketConnection.all.each do |wsc|
-                wsc.destroy unless ci.include?(wsc.connection_identifier)
-              end
-            end
-
-          end
-
-        STRING
-      )
-    end
-
     def write_controller
       File.write(
         'app/controllers/graphql_controller.rb',
@@ -217,7 +133,6 @@ module GraphqlRailsApi
                 context: { current_user: authenticated_user },
                 operation_name: params[:operationName]
               )
-              ApplicationRecord.broadcast_queries
               render json: result
             end
 

data/lib/generators/graphql_resource/graphql_resource_generator.rb

@@ -103,7 +103,7 @@ class GraphqlResourceGenerator < Rails::Generators::NamedBase
   end
 
   def generate_basic_mutations(resource)
-    system("mkdir -p #{@mutations_directory}")
+    FileUtils.mkdir_p(@mutations_directory) unless File.directory?(@mutations_directory)
     system("rails generate graphql_mutations #{resource}")
 
     # Graphql Input Type
@@ -126,7 +126,8 @@ class GraphqlResourceGenerator < Rails::Generators::NamedBase
   end
 
   def generate_graphql_input_type(resource)
-    system("mkdir -p #{@mutations_directory}")
+    FileUtils.mkdir_p(@mutations_directory) unless File.directory?(@mutations_directory)
+
     File.write(
       "#{@mutations_directory}/input_type.rb",
       <<~STRING
@@ -241,6 +242,9 @@ class GraphqlResourceGenerator < Rails::Generators::NamedBase
   end
 
   def generate_service(resource)
+     
+    FileUtils.mkdir_p("app/graphql/#{resource.pluralize}/") unless File.directory?("app/graphql/#{resource.pluralize}/")
+     
     File.write(
       "app/graphql/#{resource.pluralize}/service.rb",
       <<~STRING
@@ -308,13 +312,13 @@ t.#{@id_db_type} :#{resource.underscore.singularize}_id
   end
 
   def add_to_model(model, line)
-    file_name = "app/models/#{model.underscore.singularize}.rb"
+    file_name = File.join("app","models","#{model.underscore.singularize}.rb")
     return if !File.exist?(file_name) || File.read(file_name).include?(line)
 
-    line_count = `wc -l "#{file_name}"`.strip.split(' ')[0].to_i
-
+    file = open(file_name)
+    line_count = file.readlines.size
     line_nb = 0
-    File.open(file_name).each do |l|
+    file.each do |l|
       line_nb += 1
       break if l.include?('ApplicationRecord')
     end

data/lib/graphql/hydrate_query.rb

@@ -1,40 +1,311 @@
-require 'deep_pluck'
-require 'rkelly'
+require "deep_pluck"
+require "rkelly"
 
 module Graphql
   class HydrateQuery
-
-    def initialize(model, context, order_by: nil, filter: nil, id: nil, user: nil, page: nil, per_page: 10)
+    def initialize(model, context, order_by: nil, filter: nil, check_visibility: true, id: nil, user: nil, page: nil, per_page: nil)
       @context = context
       @filter = filter
       @order_by = order_by
       @model = model
-      @models = [model_name.singularize.camelize]
+      @check_visibility = check_visibility
+
+      if id.present? && !valid_id?(id)
+        raise GraphQL::ExecutionError, "Invalid id: #{id}"
+      end
+
       @id = id
       @user = user
-      @page = page
-      @per_page = per_page
+      @page = page&.to_i || 1
+      @per_page = per_page&.to_i || 1000
+      @per_page = 1000 if @per_page > 1000
     end
 
     def run
-      @model = @model.where(transform_filter(@filter)) if @filter
-      @model = @model.order(@order_by) if @order_by
+      if @id
+        @model = @model.where(id: @id)
+        deep_pluck_to_structs(@context&.irep_node).first
+      else
+        @model = @model.limit(@per_page)
+        @model = @model.offset(@per_page * (@page - 1))
+
+        transform_filter if @filter.present?
+        transform_order if @order_by.present?
+
+        deep_pluck_to_structs(@context&.irep_node)
+      end
+    end
+
+    def paginated_run
+      transform_filter if @filter.present?
+      transform_order if @order_by.present?
+
+      @total = @model.length
+      @model = @model.limit(@per_page)
+      @model = @model.offset(@per_page * (@page - 1))
+
+      OpenStruct.new(
+        data: deep_pluck_to_structs(@context&.irep_node&.typed_children&.values&.first.try(:[], "data")),
+        total_count: @total,
+        per_page: @per_page,
+        page: @page,
+      )
+    end
+
+    private
+
+    def transform_order
+      return if @order_by.blank?
+
+      sign = @order_by.split(" ").last.downcase == "desc" ? "desc" : "asc"
+      column = @order_by.split(" ").first.strip
+
+      if column.include?(".")
+        associated_model = column.split(".").first
+        accessor = column.split(".").last
+        assoc = get_assoc!(@model, associated_model)
+        field_type = get_field_type!(assoc.klass, accessor)
+        @model = @model.left_joins(associated_model.to_sym)
+        ordered_field = "#{associated_model.pluralize}.#{accessor}"
+      else
+        field_type = get_field_type!(@model, column)
+        ordered_field = "#{model_name.pluralize}.#{column}"
+      end
+
+      if %i[string text].include?(field_type)
+        @model = @model.order(Arel.sql("upper(#{ordered_field}) #{sign}"))
+      else
+        @model = @model.order(Arel.sql("#{ordered_field} #{sign}"))
+      end
+    end
+
+    def transform_filter
+      return if @filter.blank?
+
+      ast = RKelly::Parser.new.parse(@filter)
+      exprs = ast.value
+      if exprs.count != 1
+        raise GraphQL::ExecutionError, "Invalid filter: #{@filter}, only one expression allowed"
+      end
+
+      @model = handle_node(exprs.first.value, @model)
+
+      if @need_distinct_results
+        @model = @model.distinct
+      end
+
+    rescue RKelly::SyntaxError => e
+      raise GraphQL::ExecutionError, "Invalid filter: #{e.message}"
+    end
+
+    def handle_node(node, model)
+      if node.class == RKelly::Nodes::ParentheticalNode
+        handle_ParentheticalNode(node, model)
+      elsif node.class == RKelly::Nodes::LogicalAndNode
+        handle_LogicalAndNode(node, model)
+      elsif node.class == RKelly::Nodes::LogicalOrNode
+        handle_LogicalOrNode(node, model)
+      elsif node.class == RKelly::Nodes::NotEqualNode
+        handle_NotEqualNode(node, model)
+      elsif node.class == RKelly::Nodes::EqualNode
+        handle_EqualNode(node, model)
+      elsif node.class == RKelly::Nodes::StrictEqualNode
+        handle_StrictEqualNode(node, model)
+      elsif node.class == RKelly::Nodes::NotStrictEqualNode
+        handle_NotStrictEqualNode(node, model)
+      elsif node.class == RKelly::Nodes::GreaterOrEqualNode
+        handle_GreaterOrEqualNode(node, model)
+      elsif node.class == RKelly::Nodes::LessOrEqualNode
+        handle_LessOrEqualNode(node, model)
+      elsif node.class == RKelly::Nodes::LessNode
+        handle_LessNode(node, model)
+      elsif node.class == RKelly::Nodes::GreaterNode
+        handle_GreaterNode(node, model)
+      else
+        raise GraphQL::ExecutionError, "Invalid filter: #{node.class} unknown operator"
+      end
+    end
+
+    def handle_ParentheticalNode(node, model)
+      handle_node(node.value, model)
+    end
+
+    def handle_LogicalAndNode(node, model)
+      handle_node(node.left, model).and(handle_node(node.value, model))
+    end
+
+    def handle_LogicalOrNode(node, model)
+      handle_node(node.left, model).or(handle_node(node.value, model))
+    end
+
+    def handle_dot_accessor_node(node, model)
+      associated_model = node.left.value.value
+      accessor = node.left.accessor
+      assoc = get_assoc!(model, associated_model)
+      field_type = get_field_type!(assoc.klass, accessor)
+
+      if assoc.association_class == ActiveRecord::Associations::HasManyAssociation
+        @need_distinct_results = true
+      end
+
+      model = model.left_joins(associated_model.to_sym)
+      # field = "#{associated_model.pluralize}.#{accessor}"
+      value = value_from_node(node.value, field_type, accessor.to_sym, model)
+      [assoc.klass.arel_table[accessor], model, field_type, value]
+    end
+
+    def handle_resolve_node(node, model)
+      field = node.left.value
+      field_type = get_field_type!(model, field)
+      value = value_from_node(node.value, field_type, field.to_sym, model)
+      [model.klass.arel_table[field], model, field_type, value]
+    end
 
-      @model = @model.limit(@per_page) if @per_page
-      @model = @model.offset(@per_page * (@page - 1)) if @page
+    def handle_operator_node(node, model)
+      if node.left.class == RKelly::Nodes::DotAccessorNode
+        handle_dot_accessor_node(node, model)
+      elsif node.left.class == RKelly::Nodes::ResolveNode
+        handle_resolve_node(node, model)
+      else
+        raise GraphQL::ExecutionError, "Invalid left value: #{node.left.class}"
+      end
+    end
+
+    def value_from_node(node, sym_type, sym, model)
+      if node.class == RKelly::Nodes::StringNode
+        val = node.value.gsub(/^'|'$|^"|"$/, "")
+        if sym_type == :datetime
+          DateTime.parse(val)
+        elsif sym_type == :date
+          Date.parse(val)
+        elsif sym_type == :integer
+          # Enums are handled here : We are about to compare a string with an integer column
+          # If the symbol and the value correspond to an existing enum into the model
+          if model.klass.defined_enums[sym.to_s]&.keys&.include?(val)
+            # return the corresponding enum value
+            model.klass.defined_enums[sym.to_s][val]
+          else
+            raise GraphQL::ExecutionError, "Invalid value: #{val}, compare a string with an integer column #{sym}"
+          end
+        else
+          val
+        end
+      elsif node.class == RKelly::Nodes::NumberNode
+        node.value
+      elsif node.class == RKelly::Nodes::TrueNode
+        true
+      elsif node.class == RKelly::Nodes::FalseNode
+        false
+      elsif node.class == RKelly::Nodes::NullNode
+        nil
+      else
+        raise GraphQL::ExecutionError, "Invalid filter: #{node} unknown rvalue node"
+      end
+    end
+
+    def sanitize_sql_like(value)
+      ActiveRecord::Base::sanitize_sql_like(value)
+    end
+
+    def handle_NotEqualNode(node, model)
+      arel_field, model, type, value = handle_operator_node(node, model)
+
+      if value.nil?
+        model.where.not(arel_field.eq(nil))
+      elsif type == :text || type == :string
+        model.where.not(arel_field.lower.matches(sanitize_sql_like(value.downcase)))
+      else
+        model.where.not(arel_field.eq(value))
+      end
+    end
+
+    def handle_NotStrictEqualNode(node, model)
+      arel_field, model, type, value = handle_operator_node(node, model)
+
+      if value.nil?
+        model.where.not(arel_field.eq(nil))
+      elsif type == :text || type == :string
+        model.where.not(arel_field.matches(sanitize_sql_like(value), false, true))
+      else
+        model.where.not(arel_field.eq(value))
+      end
+    end
+
+    def handle_EqualNode(node, model)
+      arel_field, model, type, value = handle_operator_node(node, model)
+
+      if value.nil?
+        model.where(arel_field.eq(nil))
+      elsif type == :text || type == :string
+        model.where(arel_field.lower.matches(sanitize_sql_like(value.downcase)))
+      else
+        model.where(arel_field.eq(value))
+      end
+    end
 
-      @model = @model.where(id: @id) if @id
-      plucked = DeepPluck::Model.new(@model.visible_for(user: @user), user: @user).add(
-        hash_to_array_of_hashes(parse_fields(@context&.irep_node), @model)
-      ).load_all
-      result = plucked_attr_to_structs(plucked, model_name.singularize.camelize.constantize)&.compact
-      @id ? result.first : result
+    def handle_StrictEqualNode(node, model)
+      arel_field, model, type, value = handle_operator_node(node, model)
+
+      if value.nil?
+        model.where(arel_field.eq(nil))
+      elsif type == :text || type == :string
+        model.where(arel_field.matches(sanitize_sql_like(value), false, true))
+      else
+        model.where(arel_field.eq(value))
+      end
+    end
+
+    def handle_GreaterOrEqualNode(node, model)
+      arel_field, model, type, value = handle_operator_node(node, model)
+      model.where(arel_field.gteq(value))
+    end
+
+    def handle_LessOrEqualNode(node, model)
+      arel_field, model, type, value = handle_operator_node(node, model)
+      model.where(arel_field.lteq(value))
+    end
+
+    def handle_LessNode(node, model)
+      arel_field, model, type, value = handle_operator_node(node, model)
+      model.where(arel_field.lt(value))
+    end
+
+    def handle_GreaterNode(node, model)
+      arel_field, model, type, value = handle_operator_node(node, model)
+      model.where(arel_field.gt(value))
+    end
+
+    def valid_id?(id)
+      valid_uuid?(id) || id.is_a?(Integer)
+    end
+
+    def valid_uuid?(id)
+      id.to_s.match(/\A[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}\z/)
     end
 
-    def transform_filter(filter)
-      parsed_filter = RKelly::Parser.new.parse(filter.gsub('like', ' | ')).to_ecma
-      parsed_filter.gsub(' | ', ' like ').
-        gsub('||', 'OR').gsub('&&', 'AND').gsub('===', '=').gsub('==', '=').delete(';')
+    def get_assoc!(model, assoc_name)
+      assoc = model.reflect_on_association(assoc_name)
+      unless assoc.present?
+        raise GraphQL::ExecutionError, "Invalid filter: #{assoc_name} is not an association of #{model}"
+      end
+      assoc
+    end
+
+    def get_field_type!(model, field_name)
+      field = model.column_for_attribute(field_name.to_sym)
+      unless field.present?
+        raise GraphQL::ExecutionError, "Invalid filter: #{field_name} is not a field of #{model}"
+      end
+      field.type
+    end
+
+    def deep_pluck_to_structs(irep_node)
+      plucked_attr_to_structs(
+        DeepPluck::Model.new(@model.visible_for(user: @user), user: @user).add(
+          ((hash_to_array_of_hashes(parse_fields(irep_node), @model) || []) + [@to_select_to_add]).compact
+        ).load_all,
+        model_name.singularize.camelize.constantize
+      )&.compact
     end
 
     def plucked_attr_to_structs(arr, parent_model)
@@ -56,7 +327,7 @@ module Graphql
     def hash_to_array_of_hashes(hash, parent_class)
       return if parent_class.nil? || hash.nil?
 
-      hash['id'] = nil if hash['id'].blank?
+      hash["id"] = nil if hash["id"].blank?
       fetch_ids_from_relation(hash)
 
       hash.each_with_object([]) do |(k, v), arr|
@@ -64,18 +335,17 @@ module Graphql
         next arr << v if parent_class.new.attributes.key?(v)
 
         klass = evaluate_model(parent_class, k)
-        @models << klass.to_s unless @models.include?(klass.to_s)
         arr << { k.to_sym => hash_to_array_of_hashes(v, klass) } if klass.present? && v.present?
       end
     end
 
     def fetch_ids_from_relation(hash)
-      hash.select { |k, _| k.ends_with?('_ids') }.each do |(k, _)|
-        collection_name = k.gsub('_ids', '').pluralize
+      hash.select { |k, _| k.ends_with?("_ids") }.each do |(k, _)|
+        collection_name = k.gsub("_ids", "").pluralize
         if hash[collection_name].blank?
-          hash[collection_name] = { 'id' => nil }
+          hash[collection_name] = { "id" => nil }
         else
-          hash[collection_name]['id'] = nil
+          hash[collection_name]["id"] = nil
         end
       end
     end
@@ -104,9 +374,7 @@ module Graphql
 
     def parse_fields(irep_node)
       fields = irep_node&.scoped_children&.values&.first
-      if fields.key?('edges')
-        fields = fields['edges'].scoped_children.values.first['node']&.scoped_children&.values&.first
-      end
+      fields = fields["edges"].scoped_children.values.first["node"]&.scoped_children&.values&.first if fields&.key?("edges")
       return if fields.blank?
 
       fields.each_with_object({}) do |(k, v), h|
@@ -115,8 +383,7 @@ module Graphql
     end
 
     def model_name
-      @model.class.to_s.split('::').first.underscore.pluralize
+      @model.class.to_s.split("::").first.underscore.pluralize
     end
-
   end
 end

data/lib/graphql/rails/api/config.rb

@@ -6,23 +6,22 @@ module Graphql
         include Singleton
 
         def self.query_resources
-          Dir.glob("#{File.expand_path('.')}/app/graphql/*/type.rb").map do |dir|
+          Dir.glob("#{::Rails.root}/app/graphql/*/type.rb").map do |dir|
             dir.split('/').last(2).first
           end
         end
 
         def self.mutation_resources
-          mutations = Dir.glob("#{File.expand_path('.')}/app/graphql/*/mutations/*.rb").reject do |e|
+          mutations = Dir.glob("#{::Rails.root}/app/graphql/*/mutations/*.rb").reject do |e|
             e.end_with?('type.rb', 'types.rb')
           end
           mutations = mutations.map { |e| e.split('/').last.gsub('.rb', '') }.uniq
           mutations.each_with_object({}) do |meth, h|
-            h[meth] = Dir.glob("#{File.expand_path('.')}/app/graphql/*/mutations/#{meth}.rb").map do |dir|
+            h[meth] = Dir.glob("#{::Rails.root}/app/graphql/*/mutations/#{meth}.rb").map do |dir|
               dir.split('/').last(3).first
             end
           end
         end
-
       end
     end
   end

data/lib/graphql/rails/api/version.rb

@@ -1,7 +1,7 @@
 module Graphql
   module Rails
     module Api
-      VERSION = '0.8.0'.freeze
+      VERSION = '0.9.2'.freeze
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphql-rails-api
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.9.2
 platform: ruby
 authors:
 - poilon
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-12-12 00:00:00.000000000 Z
+date: 2022-08-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql
@@ -16,14 +16,20 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '1.12'
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: 1.12.13
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '1.12'
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: 1.12.13
 - !ruby/object:Gem::Dependency
   name: deep_pluck_with_authorization
   requirement: !ruby/object:Gem::Requirement
@@ -44,20 +50,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 6.0.0
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 6.0.0
+        version: 5.1.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 6.0.0
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 6.0.0
+        version: 5.1.4
 - !ruby/object:Gem::Dependency
   name: rkelly-remix
   requirement: !ruby/object:Gem::Requirement
@@ -114,7 +114,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
+rubygems_version: 3.3.7
 signing_key: 
 specification_version: 4
 summary: Graphql rails api framework to create easily graphql api with rails