operations 0.7.0 → 0.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 37feed93ae3f90b8e08f2ab960faf4195c8891e6bc4d60a78d03666c5c035de3
-  data.tar.gz: 29b2bad869d32e4bf22f188eb4c57a199300a315431f373d5cf988f15f63ab8b
+  metadata.gz: 41842fbe905d259c1c39bd2922caa0dc50583cdfceac0c0b78ce7e4e8e1d26f4
+  data.tar.gz: 37e703dca9a9db80167e2750e77af20e59494a3fb1c0f208720e08ae2bb83a8e
 SHA512:
-  metadata.gz: 186e9327e813bbf7ddd5d5aea52b5bf00cc7a62a9c8b235b7fd25f36585a2468192922df3ff9435d7320b164807e31acf73523eaf9b106c67de6f2f27da19400
-  data.tar.gz: 43d61db8dbe71a4f9269755a6fc73c46214ba3baddeed80e650a7ae2ced43c08fb361dd19cef005954504970c06bfa573a1a55f760e7ac711c68c19f4e34601b
+  metadata.gz: 33ebaee09aff4d2cd32742566e1c3959c5f43558b6e3c57760936a31f8431af2806a8b6b8807da10588fbf2916de5e34857cc9ce5378b93d70099dd09b049d78
+  data.tar.gz: 1f733f0567064105528b480d47f766aa6acd9b2dd2766d011e11edeb542f3eea6233f2e30cb810d10f4e9412ab9548a186643072dd80414cf07b9aecc7189570

data/CHANGELOG.md

@@ -1,10 +1,16 @@
 # Changelog
 
-## [0.7.0](https://github.com/BookingSync/operations/tree/main)
+## [0.7.1](https://github.com/BookingSync/operations/tree/v0.7.1)
 
 ### Added
 
-- Implement new forms system detaching it from operations [\#47](https://github.com/BookingSync/operations/pull/47) ([pyromaniac](https://github.com/pyromaniac))
+- Added `persisted:` option to the new forms definition. [\#48](https://github.com/BookingSync/operations/pull/48) ([pyromaniac](https://github.com/pyromaniac))
+
+## [0.7.0](https://github.com/BookingSync/operations/tree/v0.7.0)
+
+### Added
+
+- Implement new forms system detaching it from operations. Please check [UPGRADING_FORMS.md](UPGRADING_FORMS.md) for more details. [\#47](https://github.com/BookingSync/operations/pull/47) ([pyromaniac](https://github.com/pyromaniac))
 
 ### Improvements
 
@@ -15,7 +21,7 @@
 
 - In some cases, `Operation::Command#form_class` was evaluated before `form_base` was evaluated [\#41](https://github.com/BookingSync/operations/pull/41) ([pyromaniac](https://github.com/pyromaniac))
 
-## [0.6.2]
+## [0.6.3](https://github.com/BookingSync/operations/tree/v0.6.3)
 
 ### Added
 

data/README.md

@@ -853,6 +853,8 @@ In this case, `Order::MarkAsCompleted.system.call(...)` will be used in, say, co
 
 ### Form objects
 
+Form objects were refactored to be separate from Command. Please check [UPGRADING_FORMS.md](UPGRADING_FORMS.md) for more details.
+
 While we normally recommend using frontend-backend separation, it is still possible to use this framework with Rails view helpers:
 
 ```ruby
@@ -885,7 +887,7 @@ class Post::Update
 end
 ```
 
-Then, the form can be used as any other form object:
+Then, the form can be used as any other form object. Unfortunately, there is no way to figure out the correct route for the operation form object, so it have to be provided manually:
 
 ```erb
 # views/posts/edit.html.erb
@@ -939,7 +941,18 @@ class Post::Update::ModelMap
 end
 ```
 
-In forms, params input is already transformed to extract the nested data with the form name. `form_for @post_update_form` will generate the form that send params nested under the `params[:post_update_form]` key. By default operation forms extract this form data and send it to the operation at the top level, so `{ id: 42, post_update_form: { title: "Post Title" } }` params will be sent to the operation as `{ id: 42, title: "Post Title" }`. Strong params are also accepted by the form, though they are being converted with `to_unsafe_hash`.
+In forms, params input is already transformed to extract the nested data with the form name. `form_for @post_update_form` will generate the form that send params nested under the `params[:post_update_form]` key. By default operation forms extract this form data and send it to the operation at the top level, so `{ id: 42, post_update_form: { title: "Post Title" } }` params will be sent to the operation as `{ id: 42, title: "Post Title" }`. Strong params are also accepted by the form, though they are being converted with `to_unsafe_hash`. Though the form name can be customized if necessary:
+
+```ruby
+class Post::Update
+  def self.default_form
+    @default_form ||= Operations::Form.new(
+      default,
+      model_name: "custom_post_update_form", # form name can be customized
+    )
+  end
+end
+```
 
 It is possible to add more params transfomations to the form in cases when operation contract is different from the params structure:
 
@@ -948,7 +961,6 @@ class Post::Update
   def self.default_form
     @default_form ||= Operations::Form.new(
       default,
-      model_name: "post_update_form", # form name can be customized
       params_transformations: [
         ParamsMap.new(id: :post_id),
         NestedAttributes.new(:sections)
@@ -994,6 +1006,21 @@ class NestedAttributes
 end
 ```
 
+By default, the top-level form objects instantiated from the form will have `persisted?` flag set to `true`. This will result the form to use the `PATCH` werb like for any persisted AR object. If it is required to generate a form with the `POST` verb in case of operation, say, creating some objects, this default behavior can be customised:
+
+```ruby
+class Post::Create
+  def self.default_form
+    @default_form ||= Operations::Form.new(
+      default,
+      persisted: false
+    )
+  end
+end
+```
+
+Note that operation itself is agnostic to the persistence layer, so there is no way for it to figure out this semanticsa automatically.
+
 ## Development
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.

data/UPGRADING_FORMS.md

@@ -0,0 +1,101 @@
+# New form objects
+
+In version 0.7.0, a new form objects system were introduced. The old way of using form objects is deprecated and will be removed in 1.0.0. In order to upgrade your code to start using the new way please follow the guide:
+
+## Replace `Operations::Form` with `Operations::Form::Base`
+
+If you have any classes that were inherited from `Operations::Form`, please change them to inherit from `Operations::Form::Base`
+
+## Define forms as separate objects on top of the existing operations:
+
+```ruby
+# Before
+class Post::Update
+  def self.default
+    @default ||= Operations::Command.new(
+      ...,
+      form_hydrator: Post::Update::Hydrator.new,
+      form_model_map: {
+        [%r{.+}] => "Post"
+      }
+    )
+  end
+end
+
+# After
+class Post::Update
+  def self.default
+    @default ||= Operations::Command.new(...)
+  end
+
+  def self.default_form
+    @default_form ||= Operations::Form.new(
+      default,
+      hydrator: Post::Update::Hydrator.new,
+      model_map: Post::Update::ModelMap.new,
+      params_transformations: [
+        ParamsMap.new(id: :post_id)
+      ]
+    )
+  end
+end
+```
+
+Where `Post::Update::ModelMap` can be a copy of [Operations::Form::DeprecatedLegacyModelMapImplementation](https://github.com/BookingSync/operations/blob/main/lib/operations/form/deprecated_legacy_model_map_implementation.rb) or your own implementation.
+
+And `ParamsMap` can be as simple as:
+
+```ruby
+class ParamsMap
+  extend Dry::Initializer
+
+  param :params_map
+
+  def call(_form_class, params, **_context)
+    params.transform_keys { |key| params_map[key] || key }
+  end
+end
+```
+
+## Change the way you use forms in you controllers and views:
+
+```ruby
+# Before
+class PostsController < ApplicationController
+  def edit
+    @post_update = Post::Update.default.callable(
+      { post_id: params[:id] },
+      current_user: current_user
+    )
+
+    respond_with @post_update
+  end
+
+  def update
+    # With operations we don't need strong parameters as the operation contract takes care of this.
+    @post_update = Post::Update.default.call(
+      { **params[:post_update_default_form], post_id: params[:id] },
+      current_user: current_user
+    )
+
+    respond_with @post_update, location: edit_post_url(@post_update.context[:post])
+  end
+end
+
+# After
+class PostsController < ApplicationController
+  def edit
+    @post_update_form = Post::Update.default_form.build(params, current_user: current_user)
+
+    respond_with @post_update_form
+  end
+
+  def update
+    @post_update_form = Post::Update.default_form.persist(params, current_user: current_user)
+
+    respond_with @post_update_form, location: edit_post_url(@post_update_form.operation_result.context[:post])
+  end
+end
+```
+
+Notice that `callable` and `call` methond are replaced with `build` and `persist` respectively.

data/lib/operations/command.rb

@@ -387,7 +387,8 @@ class Operations::Command
         key_map: contract.class.schema.key_map,
         model_map: Operations::Form::DeprecatedLegacyModelMapImplementation.new(form_model_map),
         namespace: operation.class,
-        class_name: form_class_name
+        class_name: form_class_name,
+        persisted: true
       )
   end
 

data/lib/operations/form/base.rb

@@ -47,6 +47,7 @@ class Operations::Form::Base
 
       base.class_attribute :attributes, instance_accessor: false, default: {}
       base.class_attribute :primary_key, instance_accessor: false, default: :id
+      base.class_attribute :persisted, instance_accessor: false, default: nil
 
       base.define_method :initialize do |*args, **kwargs|
         args.empty? && kwargs.present? ? super(kwargs, **{}) : super(*args, **kwargs)
@@ -132,7 +133,7 @@ class Operations::Form::Base
     end
 
     def persisted?
-      !has_attribute?(self.class.primary_key) || read_attribute(self.class.primary_key).present?
+      self.class.persisted.nil? ? read_attribute(self.class.primary_key).present? : self.class.persisted
     end
 
     def new_record?

data/lib/operations/form/builder.rb

@@ -11,18 +11,19 @@ class Operations::Form::Builder
 
   option :base_class, Operations::Types::Instance(Class)
 
-  def build(key_map:, model_map:, namespace: nil, class_name: nil, model_name: nil)
+  def build(key_map:, model_map:, namespace: nil, class_name: nil, model_name: nil, persisted: nil)
     return namespace.const_get(class_name) if namespace && class_name && namespace.const_defined?(class_name)
 
-    traverse(key_map, model_map, namespace, class_name, model_name, [])
+    traverse(key_map, model_map, namespace, class_name, model_name, [], persisted: persisted)
   end
 
   private
 
-  def traverse(key_map, model_map, namespace, class_name, model_name, path)
+  def traverse(key_map, model_map, namespace, class_name, model_name, path, persisted: nil)
     form = Class.new(base_class)
     namespace.const_set(class_name, form) if namespace&.name && class_name
     define_model_name(form, model_name) if model_name && !form.name
+    form.persisted = persisted
 
     key_map.each { |key| define_attribute(form, model_map, key, path) }
     form

data/lib/operations/form.rb

@@ -21,8 +21,8 @@
 #
 class Operations::Form
   include Dry::Core::Constants
-  include Dry::Equalizer(:command, :model_map, :params_transformations, :hydrator, :form_class)
-  include Operations::Inspect.new(:model_name, :model_map, :params_transformations, :hydrator, :form_class)
+  include Dry::Equalizer(:command, :model_map, :persisted, :params_transformations, :hydrator, :form_class)
+  include Operations::Inspect.new(:model_name, :model_map, :persisted, :params_transformations, :hydrator, :form_class)
 
   # We need to make deprecated inheritance from Operations::Form act exactly the
   # same way as from Operations::Form::Base. In order to do this, we are encapsulating all the
@@ -48,6 +48,7 @@ class Operations::Form
     param :command, type: Operations::Types.Interface(:operation, :contract, :call)
     option :model_name, type: Operations::Types::String.optional, default: proc {}, reader: false
     option :model_map, type: Operations::Types.Interface(:call).optional, default: proc {}
+    option :persisted, type: Operations::Types::Bool, default: proc { true }
     option :params_transformations, type: Operations::Types::Coercible::Array.of(Operations::Types.Interface(:call)),
       default: proc { [] }
     option :hydrator, type: Operations::Types.Interface(:call).optional, default: proc {}
@@ -64,7 +65,7 @@ class Operations::Form
 
   def form_class
     @form_class ||= Operations::Form::Builder.new(base_class: base_class)
-      .build(key_map: key_map, model_map: model_map, model_name: model_name)
+      .build(key_map: key_map, model_map: model_map, model_name: model_name, persisted: persisted)
   end
 
   private

data/lib/operations/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Operations
-  VERSION = "0.7.0"
+  VERSION = "0.7.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: operations
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.7.1
 platform: ruby
 authors:
 - Arkadiy Zabazhanov
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-07-28 00:00:00.000000000 Z
+date: 2024-07-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: appraisal
@@ -154,6 +154,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- UPGRADING_FORMS.md
 - bin/console
 - bin/setup
 - gemfiles/rails.5.2.gemfile