flexirest 1.11.1 → 1.11.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '036069384b97e0fbcf4e73c9e75154569cffbb9cbfcb8dfcda4ccfc4ef75cca2'
-  data.tar.gz: 3099f07f5284601a68ebba405a60e485975635b5306b72f2ead2090509bdb901
+  metadata.gz: '069a842d7b7b6db2e2aa7cff28191a02d70e0ebe3bc6f49aadf569b8006b7cc4'
+  data.tar.gz: 9db8212da8ed8698bc4a1d48137fe2e0e9ce8d75c94366d72fe1f78d986855e0
 SHA512:
-  metadata.gz: 18d64a00ceea9f1a9ee61dcdaedf64dbcc8eae7b9f4f3d20943e7b9209dd7d98acd08138a9c149d0f7cb7aa03740131030580c45f4d153e2f47df999e6415a61
-  data.tar.gz: 026a430bdc41b68adc6e46c42dc69983fca9ef431fb3cc068c96fb3714c687209393d15e66879bb46c3f26ba3f9e38dd08b94620a7d8b2f20d392f08690031fd
+  metadata.gz: fb73957659a29bb35ab8f08b9127e32f36e733e9a37830c09e42f221774531c18edbbb4767bccd1fcc4713a938fb9c195daa4863af39c128b312d582b6245fc2
+  data.tar.gz: b9e91c14da366548a6a618eae2d634ad26cf60e1ed20d957b577bb4e1e777c32eea594ebc2a37990f61eeb59c41e34912ed591477f0a80005bfc7435dac7aa3d

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 1.11.2
+
+Bugfix:
+
+- When a model had multiple lazy loaders specified, they would all have their object class set to the first object class that was found when iterating over them. (thanks to Joshua Samberg for the PR)
+
 ## 1.11.1
 
 Enhancement:

data/docs/associations.md

@@ -86,37 +86,56 @@ When the `:array` option includes an attribute, it is assumed the values were re
 
 ## Type 2 - Lazy Loading From Other URLs
 
-When mapping the method, passing a list of attributes will cause any requests for those attributes to mapped to the URLs given in their responses. The response for the attribute may be one of the following:
+If the call for an attribute should
 
-```ruby
-"attribute" : "URL"
-"attribute" : ["URL", "URL"]
-"attribute" : { "url" : "URL"}
-"attribute" : { "href" : "URL"}
-"attribute" : { "something" : "URL"}
-```
-
-The difference between the last 3 examples is that a key of `url` or `href` signifies it's a single object that is lazy loaded from the value specified. Any other keys assume that it's a nested set of URLs (like in the array situation, but accessible via the keys - e.g. object.attribute.something in the above example).
++ Use the value held within the specified attribute as a URL(s) from which to load the associated resource(s)
++ **THEN**, create an instance of your API object from the result
++ **THEN**, call subsequent chained methods on that instance
 
-It is required that the URL is a complete URL including a protocol starting with "http". To configure this use code like:
+you can specify this when mapping the method by passing attribtues to the `:lazy` option.
 
 ```ruby
+class Book < Flexirest::Base
+  get :find, "/books/:name"
+end
+
 class Person < Flexirest::Base
-  get :find, "/people/:id", :lazy => [:orders, :refunds]
+  get :find, "/people/:id", :lazy => { books: Book }
 end
 ```
 
-And use it like this:
+Use it like this:
 
 ```ruby
 # Makes a call to /people/1
 @person = Person.find(1)
 
-# Makes a call to the first URL found in the "books":[...] array in the article response
-# only makes the HTTP request when first used though
+# Lazily makes a call to the first URL found in the "books":[...] array in the Person.find response
+# - Only makes the HTTP request when first used
+# - Instantiates a Book object from the response and then accesses its "name" property
 @person.books.first.name
 ```
 
+### URLs in API responses
+To provide a URL(s) for lazy loading an attribute, the API response may contain one of the following for the attribute (**`book`** or **`books`** is the attribute in all examples below):
+
+```ruby
+# all of the following will lazy load a single object from the specified URL
+"book" : "https://example.com/books/1"
+# or
+"book" : { "url" : "https://example.com/books/1"}
+# or
+"book" : { "href" : "https://example.com/books/1"}
+
+# books will be an array whose elements will be lazy loaded one-by-one, from the URL in the corresponding array position, whenever each element is first accessed
+"books" : ["https://example.com/books/1", "https://example.com/books/2"]
+
+# book will be an object where the values will be lazy loaded one-by-one, from the URL in the corresponding key, whenever each key is first accessed (e.g. the first time object.book.author is accessed)
+"book" : { "author" : "https://example.com/author/1"}
+```
+
+It is required that each URL is a complete URL including a protocol starting with `http`.
+
 ## Type 3 - HAL Auto-loaded Resources
 
 You don't need to define lazy attributes if they are defined using [HAL](http://stateless.co/hal_specification.html) (with an optional embedded representation). If your resource has an `_links` item (and optionally an `_embedded` item) then it will automatically treat the linked resources (with the `_embedded` cache) as if they were defined using `:lazy` as per type 2 above.

data/lib/flexirest/request.rb

@@ -713,12 +713,13 @@ module Flexirest
       @method[:options][:has_many] ||= {}
       name = name.to_sym rescue nil
       if @method[:options][:has_many][name]
-        overridden_name = name
+        parent_name = name
         object = @method[:options][:has_many][name].new
       elsif @method[:options][:has_one][name]
-        overridden_name = name
+        parent_name = name
         object = @method[:options][:has_one][name].new
       else
+        parent_name = nil
         object = create_object_instance
       end
 
@@ -735,8 +736,7 @@ module Flexirest
         else
           k = k.to_sym
         end
-        overridden_name = select_name(k, overridden_name)
-        set_corresponding_value(v, k, object, overridden_name)
+        set_corresponding_value(v, k, object, select_name(k, parent_name))
       end
       object.clean! unless object_is_class?
 

data/lib/flexirest/version.rb

@@ -1,3 +1,3 @@
 module Flexirest
-  VERSION = "1.11.1"
+  VERSION = "1.11.2"
 end

data/spec/lib/lazy_association_loader_spec.rb

@@ -12,6 +12,24 @@ class YearExample < Flexirest::Base
   get :find, "/year/:id", lazy: { months: MonthExample }, fake: "{\"months\": [\"http://www.example.com/months/1\"] }"
 end
 
+class CenturyExample < Flexirest::Base
+  base_url "http://www.example.com"
+
+  get :find, "century/:id"
+end
+
+class DecadeExample < Flexirest::Base
+  base_url "http://www.example.com"
+
+  get :find, "/decade/:id", lazy: { years: YearExample, century: CenturyExample }, fake: %Q{
+    {
+      "years": ["http://www.example.com/years/1"],
+      "century": "http://www.example.com/century/1"
+    }
+  }
+end
+
+
 describe Flexirest::LazyAssociationLoader do
   let(:url1) { "http://www.example.com/some/url" }
   let(:url2) { "http://www.example.com/some/other" }
@@ -132,4 +150,13 @@ describe Flexirest::LazyAssociationLoader do
     association = YearExample.find(1)
     expect(association.months.instance_variable_get('@request').instance_variable_get('@object').class).to eq(MonthExample)
   end
+
+  context "has multiple associations" do
+    it "should correctly map each association name to the class specified for that association in the 'lazy' declaration " do
+      association = DecadeExample.find(1)
+      expect(association.years.instance_variable_get('@request').instance_variable_get('@object').class).to eq(YearExample)
+      expect(association.century.instance_variable_get('@request').instance_variable_get('@object').class).to eq(CenturyExample)
+    end
+  end
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flexirest
 version: !ruby/object:Gem::Version
-  version: 1.11.1
+  version: 1.11.2
 platform: ruby
 authors:
 - Andy Jeffries
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-06-15 00:00:00.000000000 Z
+date: 2023-07-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler