jsonapi.rb 1.5.7 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 521d608a58983b2c99246e34faf27eae9bb38b32
-  data.tar.gz: 02a2e7a0086ede91a907d8c7d097150f0222b768
+SHA256:
+  metadata.gz: 2192bbffc57d4ecbe419a90e25728951db4464c5f299e8866afefe72fa393cad
+  data.tar.gz: f7f83b43169207e7cf6d06e6e99cc8c8f705de09c4c83ff9dba5238540577a9e
 SHA512:
-  metadata.gz: 23f3ebb21b586e5995b6cb4f2c2ae8f3636f93f5f0e11b9fdbbabdb4087c32850bcda172aeda4aabb8f58f762a23e8c034128f4b48bb7ed38d5ee2c573778ff4
-  data.tar.gz: b0cdb369d4265dd306a1af8206a46f837322ec00142ec2fac9c40c0ebcc6010cd4f2b7982e05f7ff2f489a960def72431ffc140f3460bffcf9ebecf65aee710d
+  metadata.gz: 577c2b0d1801ef2191c25925e407449ff12ba95d7fc5c8433fcbb2b20e532a1b7a51a884f147c5042c38ec27ca9eed6567bfd746aad4ad8895b7a3f90e9043b1
+  data.tar.gz: f6ca1bc49fd89c1ea942f3c047234d76c4fa1d7604bd06aa5d67c43dad93603cb19e6c542ec13837d1db5e3fbe923ec4799fd865c96c08822dcab7cb160d7f7e

data/README.md

@@ -31,7 +31,7 @@ Main goals:
 
 The available features include:
 
- * object serialization (powered by Fast JSON API)
+ * object serialization (powered by JSON:API Serializer, was `fast_jsonapi`)
  * [error handling](https://jsonapi.org/format/#errors) (parameters,
    validation, generic errors)
  * fetching of the data (support for
@@ -39,16 +39,26 @@ The available features include:
    [sparse fields](https://jsonapi.org/format/#fetching-sparse-fieldsets))
  * [filtering](https://jsonapi.org/format/#fetching-filtering) and
    [sorting](https://jsonapi.org/format/#fetching-sorting) of the data
-   (powered by Ransack)
+   (powered by Ransack, soft-dependency)
  * [pagination](https://jsonapi.org/format/#fetching-pagination) support
 
 ## But how?
 
-Mainly by leveraging [Fast JSON API](https://github.com/Netflix/fast_jsonapi)
+Mainly by leveraging [JSON:API Serializer](https://github.com/jsonapi-serializer/jsonapi-serializer)
 and [Ransack](https://github.com/activerecord-hackery/ransack).
 
 Thanks to everyone who worked on these amazing projects!
 
+## Sponsors
+
+I'm grateful for the following companies for supporting this project!
+
+<p align="center">
+<a href="https://www.luneteyewear.com"><img src="https://user-images.githubusercontent.com/112147/136836142-2bfba96e-447f-4eb6-b137-2445aee81b37.png"/></a>
+<a href="https://www.startuplandia.io"><img src="https://user-images.githubusercontent.com/112147/136836147-93f8ab17-2465-4477-a7ab-e38255483c66.png"/></a>
+</p>
+
+
 ## Installation
 
 Add this line to your application's Gemfile:
@@ -100,7 +110,7 @@ The naming scheme follows the `ModuleName::ClassNameSerializer` for an instance
 of the `ModuleName::ClassName`.
 
 Please follow the
-[Fast JSON API guide](https://github.com/Netflix/fast_jsonapi#serializer-definition)
+[JSON:API Serializer guide](https://github.com/jsonapi-serializer/jsonapi-serializer#serializer-definition)
 on how to define a serializer.
 
 To provide a different naming scheme implement the `jsonapi_serializer_class`
@@ -231,6 +241,8 @@ to filter and sort over a collection of records.
 The support is pretty extended and covers also relationships and composite
 matchers.
 
+Please add `ransack` to your `Gemfile` in order to benefit from this functionality!
+
 Here's an example:
 
 ```ruby
@@ -290,6 +302,7 @@ class MyController < ActionController::Base
       render jsonapi: paginated
     end
   end
+
 end
 ```
 
@@ -306,6 +319,17 @@ use the `jsonapi_pagination_meta` method:
   end
 
 ```
+
+If you want to change the default number of items per page or define a custom logic to handle page size, use the
+`jsonapi_page_size` method:
+
+```ruby
+  def jsonapi_page_size(pagination_params)
+    per_page = pagination_params[:size].to_f.to_i
+    per_page = 30 if per_page > 30 || per_page < 1
+    per_page
+  end
+```
 ### Deserialization
 
 `JSONAPI::Deserialization` provides a helper to transform a `JSONAPI` document

data/lib/jsonapi/active_model_error_serializer.rb

@@ -3,9 +3,6 @@ require 'jsonapi/error_serializer'
 module JSONAPI
   # [ActiveModel::Errors] serializer
   class ActiveModelErrorSerializer < ErrorSerializer
-    set_id :object_id
-    set_type :error
-
     attribute :status do
       '422'
     end

data/lib/jsonapi/error_serializer.rb

@@ -1,11 +1,10 @@
-require 'fast_jsonapi'
+require 'jsonapi/serializer'
 
 module JSONAPI
   # A simple error serializer
   class ErrorSerializer
-    include FastJsonapi::ObjectSerializer
+    include JSONAPI::Serializer
 
-    set_id :object_id
     set_type :error
 
     # Object/Hash attribute helpers.
@@ -15,6 +14,12 @@ module JSONAPI
       end
     end
 
+    # Overwrite the ID extraction method, to skip validations
+    #
+    # @return [NilClass]
+    def self.id_from_record(_record, _params)
+    end
+
     # Remap the root key to `errors`
     #
     # @return [Hash]

data/lib/jsonapi/filtering.rb

@@ -1,5 +1,10 @@
-require 'ransack/predicate'
-require_relative 'patches'
+begin
+  require 'active_record'
+  require 'ransack'
+  require_relative 'patches'
+rescue LoadError
+  warn('Install `ransack` gem before using `JSONAPI::Filtering`!')
+end
 
 # Filtering and sorting support
 module JSONAPI

data/lib/jsonapi/pagination.rb

@@ -43,6 +43,8 @@ module JSONAPI
       original_url = request.base_url + request.path + '?'
 
       pagination.each do |page_name, number|
+        next if page_name == :records
+
         original_params[:page][:number] = number
         links[page_name] = original_url + CGI.unescape(
           original_params.to_query
@@ -63,7 +65,7 @@ module JSONAPI
       numbers = { current: page }
 
       if resources.respond_to?(:unscope)
-        total = resources.unscope(:limit, :offset, :order).count()
+        total = resources.unscope(:limit, :offset, :order).size
       else
         # Try to fetch the cached size first
         total = resources.instance_variable_get(:@original_size)
@@ -82,6 +84,10 @@ module JSONAPI
         numbers[:last] = last_page
       end
 
+      if total.present?
+        numbers[:records] = total
+      end
+
       numbers
     end
 
@@ -89,16 +95,30 @@ module JSONAPI
     #
     # @return [Array] with the offset, limit and the current page number
     def jsonapi_pagination_params
-      def_per_page = self.class.const_get(:JSONAPI_PAGE_SIZE).to_i
-
       pagination = params[:page].try(:slice, :number, :size) || {}
-      per_page = pagination[:size].to_f.to_i
-      per_page = def_per_page if per_page > def_per_page || per_page < 1
+      per_page = jsonapi_page_size(pagination)
       num = [1, pagination[:number].to_f.to_i].max
 
       [(num - 1) * per_page, per_page, num]
     end
 
+    # Retrieves the default page size
+    #
+    # @param per_page_param [Hash] opts the paginations params
+    # @option opts [String] :number the page number requested
+    # @option opts [String] :size the page size requested
+    #
+    # @return [Integer]
+    def jsonapi_page_size(pagination_params)
+      per_page = pagination_params[:size].to_f.to_i
+
+      return self.class
+              .const_get(:JSONAPI_PAGE_SIZE)
+              .to_i if per_page < 1
+
+      per_page
+    end
+
     # Fallback to Rack's parsed query string when Rails is not available
     #
     # @return [Hash]

data/lib/jsonapi/rails.rb

@@ -42,11 +42,12 @@ module JSONAPI
         many = JSONAPI::Rails.is_collection?(resource, options[:is_collection])
         resource = [resource] unless many
 
-        return JSONAPI::ErrorSerializer.new(resource, options)
-          .serialized_json unless resource.is_a?(ActiveModel::Errors)
+        return JSONAPI::Rails.serializer_to_json(
+          JSONAPI::ErrorSerializer.new(resource, options)
+        ) unless resource.is_a?(ActiveModel::Errors)
 
         errors = []
-        model = resource.instance_variable_get('@base')
+        model = resource.instance_variable_get(:@base)
 
         if respond_to?(:jsonapi_serializer_class, true)
           model_serializer = jsonapi_serializer_class(model, false)
@@ -54,8 +55,18 @@ module JSONAPI
           model_serializer = JSONAPI::Rails.serializer_class(model, false)
         end
 
-        details = resource.messages
-        details = resource.details if resource.respond_to?(:details)
+        details = {}
+        if ::Rails.gem_version >= Gem::Version.new('6.1')
+          resource.each do |error|
+            attr = error.attribute
+            details[attr] ||= []
+            details[attr] << error.detail.merge(message: error.message)
+          end
+        elsif resource.respond_to?(:details)
+          details = resource.details
+        else
+          details = resource.messages
+        end
 
         details.each do |error_key, error_hashes|
           error_hashes.each do |error_hash|
@@ -66,9 +77,11 @@ module JSONAPI
           end
         end
 
-        JSONAPI::ActiveModelErrorSerializer.new(
-          errors, params: { model: model, model_serializer: model_serializer }
-        ).serialized_json
+        JSONAPI::Rails.serializer_to_json(
+          JSONAPI::ActiveModelErrorSerializer.new(
+            errors, params: { model: model, model_serializer: model_serializer }
+          )
+        )
       end
     end
 
@@ -100,21 +113,21 @@ module JSONAPI
           serializer_class = JSONAPI::Rails.serializer_class(resource, many)
         end
 
-        serializer_class.new(resource, options).serialized_json
+        JSONAPI::Rails.serializer_to_json(
+          serializer_class.new(resource, options)
+        )
       end
     end
 
     # Checks if an object is a collection
     #
-    # Stolen from [FastJsonapi::ObjectSerializer], instance method.
+    # Basically forwards it to a [JSONAPI::Serializer] as there's no public API
     #
     # @param resource [Object] to check
     # @param force_is_collection [NilClass] flag to overwrite
     # @return [TrueClass] upon success
     def self.is_collection?(resource, force_is_collection = nil)
-      return force_is_collection unless force_is_collection.nil?
-
-      resource.respond_to?(:size) && !resource.respond_to?(:each_pair)
+      JSONAPI::ErrorSerializer.is_collection?(resource, force_is_collection)
     end
 
     # Resolves resource serializer class
@@ -126,5 +139,17 @@ module JSONAPI
 
       "#{klass.name}Serializer".constantize
     end
+
+    # Lazily returns the serializer JSON
+    #
+    # @param serializer [Object] to evaluate
+    # @return [String]
+    def self.serializer_to_json(serializer)
+      if serializer.respond_to?(:serialized_json)
+        serializer.serialized_json
+      else
+        serializer.serializable_hash.to_json
+      end
+    end
   end
 end

data/lib/jsonapi/version.rb

@@ -1,3 +1,3 @@
 module JSONAPI
-  VERSION = '1.5.7'
+  VERSION = '2.0.0'
 end

data/spec/deserialization_spec.rb

@@ -0,0 +1,87 @@
+require 'spec_helper'
+
+RSpec.describe JSONAPI::Deserialization do
+  let(:jsonapi_deserialize) { UsersController.new.method(:jsonapi_deserialize) }
+  let(:document) do
+    {
+      data: {
+        id: 1,
+        type: 'note',
+        attributes: {
+          title: 'Title 1',
+          date: '2015-12-20'
+        },
+        relationships: {
+          author: {
+            data: {
+              type: 'user',
+              id: 2
+            }
+          },
+          second_author: {
+            data: nil
+          },
+          notes: {
+            data: [
+              {
+                type: 'note',
+                id: 3
+              },
+              {
+                type: 'note',
+                id: 4
+              }
+            ]
+          }
+        }
+      }
+    }
+  end
+
+  describe '#jsonapi_deserialize' do
+    it do
+      expect(jsonapi_deserialize.call(document)).to eq(
+        'id' => 1,
+        'date' => '2015-12-20',
+        'title' => 'Title 1',
+        'author_id' => 2,
+        'second_author_id' => nil,
+        'note_ids' => [3, 4]
+      )
+    end
+
+    context 'with `only`' do
+      it do
+        expect(jsonapi_deserialize.call(document, only: :notes)).to eq(
+          'note_ids' => [3, 4]
+        )
+      end
+    end
+
+    context 'with `except`' do
+      it do
+        expect(
+          jsonapi_deserialize.call(document, except: [:date, :title])
+        ).to eq(
+          'id' => 1,
+          'author_id' => 2,
+          'second_author_id' => nil,
+          'note_ids' => [3, 4]
+        )
+      end
+    end
+
+    context 'with `polymorphic`' do
+      it do
+        expect(
+          jsonapi_deserialize.call(
+            document, only: :author, polymorphic: :author
+          )
+        ).to eq(
+          'author_id' => 2,
+          'author_type' => User.name
+        )
+      end
+    end
+  end
+end

data/spec/dummy.rb

@@ -0,0 +1,162 @@
+require 'securerandom'
+require 'rails/all'
+require 'ransack'
+require 'jsonapi'
+
+Rails.logger = Logger.new(STDOUT)
+Rails.logger.level = ENV['LOG_LEVEL'] || Logger::WARN
+
+JSONAPI::Rails.install!
+
+ActiveRecord::Base.logger = Rails.logger
+ActiveRecord::Base.establish_connection(
+  ENV['DATABASE_URL'] || 'sqlite3::memory:'
+)
+
+ActiveRecord::Schema.define do
+  create_table :users, force: true do |t|
+    t.string :first_name
+    t.string :last_name
+    t.timestamps
+  end
+
+  create_table :notes, force: true do |t|
+    t.string :title
+    t.integer :user_id
+    t.integer :quantity
+    t.timestamps
+  end
+end
+
+class User < ActiveRecord::Base
+  has_many :notes
+end
+
+class Note < ActiveRecord::Base
+  validates_format_of :title, without: /BAD_TITLE/
+  validates_numericality_of :quantity, less_than: 100, if: :quantity?
+  belongs_to :user, required: true
+end
+
+class CustomNoteSerializer
+  include JSONAPI::Serializer
+
+  set_type :note
+  belongs_to :user
+  attributes(:title, :quantity, :created_at, :updated_at)
+end
+
+class UserSerializer
+  include JSONAPI::Serializer
+
+  has_many :notes, serializer: CustomNoteSerializer
+  attributes(:last_name, :created_at, :updated_at)
+
+  attribute :first_name do |object, params|
+    if params[:first_name_upcase]
+      object.first_name.upcase
+    else
+      object.first_name
+    end
+  end
+end
+
+class Dummy < Rails::Application
+  secrets.secret_key_base = '_'
+  config.hosts << 'www.example.com' if config.respond_to?(:hosts)
+
+  routes.draw do
+    scope defaults: { format: :jsonapi } do
+      resources :users, only: [:index]
+      resources :notes, only: [:update]
+    end
+  end
+end
+
+class UsersController < ActionController::Base
+  include JSONAPI::Fetching
+  include JSONAPI::Filtering
+  include JSONAPI::Pagination
+  include JSONAPI::Deserialization
+
+  def index
+    allowed_fields = [
+      :first_name, :last_name, :created_at,
+      :notes_created_at, :notes_quantity
+    ]
+    options = { sort_with_expressions: true }
+
+    jsonapi_filter(User.all, allowed_fields, options) do |filtered|
+      result = filtered.result
+
+      if params[:sort].to_s.include?('notes_quantity')
+        render jsonapi: result.group('id').to_a
+        return
+      end
+
+      result = result.to_a if params[:as_list]
+
+      jsonapi_paginate(result) do |paginated|
+        render jsonapi: paginated
+      end
+    end
+  end
+
+  private
+  def jsonapi_meta(resources)
+    {
+      many: true,
+      pagination: jsonapi_pagination_meta(resources)
+    }
+  end
+
+  def jsonapi_serializer_params
+    {
+      first_name_upcase: params[:upcase]
+    }
+  end
+end
+
+class NotesController < ActionController::Base
+  include JSONAPI::Errors
+  include JSONAPI::Deserialization
+
+  def update
+    raise_error! if params[:id] == 'tada'
+
+    note = Note.find(params[:id])
+
+    if note.update(note_params)
+      render jsonapi: note
+    else
+      note.errors.add(:title, message: 'has typos') if note.errors.key?(:title)
+
+      render jsonapi_errors: note.errors, status: :unprocessable_entity
+    end
+  end
+
+  private
+  def render_jsonapi_internal_server_error(exception)
+    Rails.logger.error(exception)
+    super(exception)
+  end
+
+  def jsonapi_serializer_class(resource, is_collection)
+    JSONAPI::Rails.serializer_class(resource, is_collection)
+  rescue NameError
+    klass = resource.class
+    klass = resource.first.class if is_collection
+    "Custom#{klass.name}Serializer".constantize
+  end
+
+  def note_params
+    # Will trigger required attribute error handling
+    params.require(:data).require(:attributes).require(:title)
+
+    jsonapi_deserialize(params)
+  end
+
+  def jsonapi_meta(resources)
+    { single: true }
+  end
+end