Skip to content

Conversation

@nickelser
Copy link

Resolves the wrong number of arguments (given 2, expected 0..1) exception with timeouts.

@j3pic
Copy link

j3pic commented Jun 5, 2018

The TravisCI test failed, but it's a problem with the test environment and not this PR:

The following packages have unmet dependencies:
 mysql-client : Depends: mysql-client-5.5 but it is not going to be installed

@ziggythehamster
Copy link

@nickelser You might want to fix ActiveRecord::RecordNotUnique.new(message, exception) in this PR; it only takes one argument as well.

divoxx pushed a commit to doximity/odbc_adapter that referenced this pull request Aug 11, 2023
* MOJ-366 Two updates for added functionality
- Varient & object fields now support store_accessor attribute on models
- Insert attribute stripper now validates the record before stripping the attributes, but then disables validation when doing the initial save so that it doesn't fail validation when required attributes are stripped off.

* MOJ-366 Switched InsertAttributeStripper back to use calling the base function, so that it will always correctly return false or raise the error depending on if save or save! is called.

* MOJ-366 Need to use options, not temp_options here. temp_options doesn't even exist yet.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants