Issue-1611 Fix in updating relations with ids

[amitsuryavanshi]

Fixes #1611

This pull introduces/changes:

• The way we get what relations to delete on update of relationships In case there was a string ID property, the to_i on passed id parameter would cause it be inconsistent with IDs of existing relations and we would end up with deleting all existing relations.

[codecov-commenter]

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ca2fade). Click here to learn what that means. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #1612   +/-   ##
=========================================
  Coverage          ?   94.02%           
=========================================
  Files             ?      126           
  Lines             ?     5721           
  Branches          ?        0           
=========================================
  Hits              ?     5379           
  Misses            ?      342           
  Partials          ?        0           

Impacted Files	Coverage Δ
...node/query/query_proxy_methods_of_mass_updating.rb	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ca2fade...30171e9. Read the comment docs.

[codeclimate[bot]]

Code Climate has analyzed commit 30171e9e and detected 0 issues on this pull request.

View more on Code Climate.