Merge pull request #1270 from cerebris/cleanup

Remove extra assignments

Author: lgebhardt

lib/jsonapi/active_relation/join_manager.rb

@@ -154,13 +154,13 @@ def perform_joins(records, options)
               next
             end
 
-           records, join_node = self.class.get_join_arel_node(records, options) {|records, options|
-              records  = related_resource_klass.join_relationship(
-                  records: records,
-                  resource_type: related_resource_klass._type,
-                  join_type: join_type,
-                  relationship: relationship,
-                  options: options)
+            records, join_node = self.class.get_join_arel_node(records, options) {|records, options|
+              related_resource_klass.join_relationship(
+                records: records,
+                resource_type: related_resource_klass._type,
+                join_type: join_type,
+                relationship: relationship,
+                options: options)
             }
 
             details = {alias: self.class.alias_from_arel_node(join_node), join_type: join_type}

lib/jsonapi/active_relation_resource.rb

@@ -155,7 +155,7 @@ def find_fragments(filters, options = {})
 
         fragments = {}
         rows = records.pluck(*pluck_fields)
-        rows.collect do |row|
+        rows.each do |row|
           rid = JSONAPI::ResourceIdentity.new(resource_klass, pluck_fields.length == 1 ? row : row[0])
 
           fragments[rid] ||= JSONAPI::ResourceFragment.new(rid)