lib/ecoportal/api/internal/account.rb in ecoportal-api-0.7.3 vs lib/ecoportal/api/internal/account.rb in ecoportal-api-0.7.4

@@ -22,11 +22,11 @@
 
           value.uniq!
           ini_ids = (original_doc && original_doc["policy_group_ids"]) || []
           # preserve original order to avoid false updates
           doc["policy_group_ids"] = (ini_ids & value) + (value - ini_ids)
-          doc["policy_group_ids"].compact!
+          doc["policy_group_ids"].compact
         end
 
         # @return [Array<String>] the policy group ids of this user.
         def policy_group_ids
           doc["policy_group_ids"] ||= []
@@ -35,11 +35,11 @@
         # Sets the `login_provider_ids`
         def login_provider_ids=(value)
           unless value.is_a?(Array)
             raise "login_provider_ids= needs to be passed an Array, got #{value.class}"
           end
-          doc["login_provider_ids"] = value.compact!
+          doc["login_provider_ids"] = value.compact
         end
 
         # @return [Array<String>] the login provider ids of this user.
         def login_provider_ids
           doc["login_provider_ids"] ||= []
@@ -48,10 +48,10 @@
         # Sets the `starred_ids`
         def starred_ids=(value)
           unless value.is_a?(Array)
             raise "starred_ids= needs to be passed an Array, got #{value.class}"
           end
-          doc["starred_ids"] = value.compact!
+          doc["starred_ids"] = value.compact
         end
 
         # @return [Array<String>] the starred page ids of this user.
         def starred_ids
           doc["starred_ids"] ||= []