Error thrown for count where api responds with no records matching about fmrest-ruby

[pilaf]

@woller @leehericks Thanks for reporting back! I just released v0.4.1 with this fix.

[leehericks]

I am currently extending every model from this Record parent to resolve the issue within the models but simply using the scopes finished with .count isn't covered...

class FileMaker::Record < FmRest::Spyke::Base

  def self.find(id)
    query(fm_id: id).find_one
  rescue FmRest::APIError::ParameterError => error
    raise unless error.code == 401
    nil
  end

  def find_one(query)
    query.find_one
  rescue FmRest::APIError::ParameterError => error
    raise unless error.code == 401
    nil
  end

  def find_some(query)
    query.find_some
  rescue FmRest::APIError::ParameterError => error
    raise unless error.code == 401
    []
  end

  def count(query)
    query.count
  rescue FmRest::APIError::ParameterError => error
    raise unless error.code == 401
    0
  end
end

[pilaf]

@leehericks @woller

I just pushed a fix for this to the fix-no-matching-records-error branch (changes here: dc0a629).

Would you mind sourcing the gem from that branch and checking if it solves the issue?

[leehericks]

I will do this as soon as I get back to work on Monday.

Foolish question but this would be changing my Gemfile to point to the github branch, right?

Thank you for your prompt work on this!

[pilaf]

@leehericks

this would be changing my Gemfile to point to the github branch, right?

Yeah, I think this should do the trick:

gem "fmrest", github: "beezwax/fmrest-ruby", branch: "fix-no-matching-records-error"

[woller]

@pilaf
It works! Thank you for fixing this.

[leehericks]

@pilaf
Works on all three situations, find_one => nil, find_some => [], count => 0

Thank you for fixing this! Looking forward to a gem version push.