Bug in ParseResource::Base#save

[joshuabowers]

The explicit returns in save are causing the method to exit before returning control back to run_callbacks. This is, in turn, canceling any around_save and after_save callbacks which might be defined on the class. This can be fixed by simply removing the return statements.

In case this doesn't get updated, and others stumble across this problem, you can monkey patch a workaround for this something like the following:

class ParseResource::Base
  self.send(:alias_method, :_original_save, :save)

  def save
    if valid?
      run_callbacks :save do
        if new?
          create
        else
          update
        end
      end
    else
      false
    end
  rescue
    false
  end
end

If you are using this in Rails, you can throw this in an initializer to override the default behavior.

[sbonami]

Hey @joshuabowers, feel free to push up a pull request and we can get this merged.