aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Support for specifying transaction isolation levelJon Leighton2012-09-219-15/+263
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If your database supports setting the isolation level for a transaction, you can set it like so: Post.transaction(isolation: :serializable) do # ... end Valid isolation levels are: * `:read_uncommitted` * `:read_committed` * `:repeatable_read` * `:serializable` You should consult the documentation for your database to understand the semantics of these different levels: * http://www.postgresql.org/docs/9.1/static/transaction-iso.html * https://dev.mysql.com/doc/refman/5.0/en/set-transaction.html An `ActiveRecord::TransactionIsolationError` will be raised if: * The adapter does not support setting the isolation level * You are joining an existing open transaction * You are creating a nested (savepoint) transaction The mysql, mysql2 and postgresql adapters support setting the transaction isolation level. However, support is disabled for mysql versions below 5, because they are affected by a bug (http://bugs.mysql.com/bug.php?id=39170) which means the isolation level gets persisted outside the transaction.
* Merge pull request #5248 from ↵Jon Leighton2012-09-212-0/+17
|\ | | | | | | | | jcoleman/should-unset-association-when-an-existing-record-is-destroyed Unset association when existing record is destroyed.
| * Unset association when existing record is destroyed.James Coleman2012-03-022-0/+17
| | | | | | | | To avoid foreign key errors (and invalid data) in the database, when a belongs_to association is destroyed, it should also be nil'd out on the parent object.
* | Get rid of global variable in AR transactions testCarlos Antonio da Silva2012-09-201-7/+7
| |
* | Update delegate docs with new hash syntax [ci skip]Carlos Antonio da Silva2012-09-201-10/+10
| |
* | Add changelog entry and docs about class delegation using the symbol :classCarlos Antonio da Silva2012-09-202-5/+36
| | | | | | | | See #7613. [ci skip]
* | Merge pull request #7721 from frodsan/ar_changelog_improvementSantiago Pastorino2012-09-201-7/+2
|\ \ | | | | | | remove unnecessary entry and make minor edits to AR/CHANGELOG [ci skip]
| * | remove unnecessary entry and make minor edits to AR/CHANGELOG [ci skip]Francesco Rodriguez2012-09-201-7/+2
|/ /
* | Merge pull request #7613 from marcandre/delegate_to_classJeremy Kemper2012-09-202-0/+14
|\ \ | | | | | | Nice and easy delegation to the class
| * | Nice and easy delegation to the classMarc-Andre Lafortune2012-09-112-0/+14
| | |
* | | Merge pull request #7720 from frodsan/rename_ar_tagJon Leighton2012-09-206-14/+19
|\ \ \ | | | | | | | | rename AR::Model::Tag to AR::Tag
| * | | rename AR::Model::Tag to AR::Tag - fixes #7714Francesco Rodriguez2012-09-206-14/+19
| | | |
* | | | Merge pull request #7719 from frodsan/strong_params_docs_part_twoRafael Mendonça França2012-09-203-15/+30
|\ \ \ \ | |/ / / |/| | | Strong Parameters documentation
| * | | add :nodoc: to AMo::DeprecatedMassAssignmentSecurity::ClassMethods [ci skip]Francesco Rodriguez2012-09-201-3/+3
| | | |
| * | | change AMo::ForbiddenAttributesProtection#sanitize_for_mass_assignment to ↵Francesco Rodriguez2012-09-201-7/+8
| | | | | | | | | | | | | | | | protected
| * | | update AMo::ForbiddenAttributesError documentation [ci skip]Francesco Rodriguez2012-09-201-0/+12
| | | |
| * | | fix typos and improve AC::StrongParameters documentation [ci skip]Francesco Rodriguez2012-09-201-6/+8
| | | |
| * | | add :nodoc: directives to AMo::DeprecatedMassAssignmentSecurity [ci skip]Francesco Rodriguez2012-09-201-3/+3
|/ / /
* | | Remove the environment variables in the GemfileRafael Mendonça França2012-09-201-27/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you want to set a local path for the gems you can use the local git repositories feature from the Bundler 1.2. For example to set the local arel repository: bundle config local.arel ~/Work/git/arel To unset it: bundle config --delete local.arel Bundler will check if the branch of you local repository is the same that specified in the Gemfile. If you want to disable these branch checks you can override it by setting this option: bundle config disable_local_branch_check true See more about this feature at http://gembundler.com/v1.2/git.html#local
* | | Freeze columns only once per ResultSantiago Pastorino2012-09-201-6/+9
| | |
* | | Merge pull request #7703 from kennyj/fix_6962Rafael Mendonça França2012-09-202-1/+12
|\ \ \ | | | | | | | | Fix #6962. AS::TimeWithZone#strftime responds incorrectly to %:z and %::z format strings.
| * | | Fix #6962. AS::TimeWithZone#strftime responds incorrectly to %:z and %::z ↵kennyj2012-09-202-1/+12
| | | | | | | | | | | | | | | | format strings.
* | | | Freeze columns just before using them as hash keysSantiago Pastorino2012-09-201-2/+5
| | | |
* | | | Merge pull request #7631 from jeremyevans/patch-1Aaron Patterson2012-09-191-1/+1
|\ \ \ \ | | | | | | | | | | Freeze columns before using them as hash keys
| * | | | Freeze columns before using them as hash keysJeremy Evans2012-09-131-1/+1
| | | | | | | | | | | | | | | This reduces the number of allocated strings from columns * (rows + 1) to just columns.
* | | | | Merge pull request #7702 from guilleiguaran/remove-manifest-optionSantiago Pastorino2012-09-195-13/+11
|\ \ \ \ \ | | | | | | | | | | | | Remove highly uncommon `config.assets.manifest` option
| * | | | | Allow to use a local copy of sprockets-rails during developmentGuillermo Iguaran2012-09-191-1/+6
| | | | | |
| * | | | | Remove highly uncommon `config.assets.manifest` option for moving the ↵Guillermo Iguaran2012-09-194-12/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | manifest path. This option is now unsupported in sprockets-rails.
* | | | | | Merge pull request #7692 from frodsan/strong_params_docsSantiago Pastorino2012-09-193-14/+225
|\ \ \ \ \ \ | | | | | | | | | | | | | | update docs for strong parameters merge
| * | | | | | update AC::StrongParameters documentationFrancesco Rodriguez2012-09-191-8/+53
| | | | | | |
| * | | | | | define permitted? method instead of use an alias to fix rdocFrancesco Rodriguez2012-09-191-2/+12
| | | | | | |
| * | | | | | update AC::Parameters documentation [ci skip]Francesco Rodriguez2012-09-191-1/+149
| | | | | | |
| * | | | | | update AC::ParameterMissing documentation [ci skip]Francesco Rodriguez2012-09-191-2/+9
| | | | | | |
| * | | | | | update AR::AttributeAssignment documentation [ci skip]Francesco Rodriguez2012-09-191-5/+5
| | | | | | |
| * | | | | | update AC::ParamsWrapper documentation [ci skip]Francesco Rodriguez2012-09-191-3/+4
|/ / / / / /
* | | | | | Merge pull request #7710 from guilleiguaran/refactor-ar-attributes-methodRafael Mendonça França2012-09-191-10/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | AR::Base#attributes= is just an alias for AR::Base#assign_attributes
| * | | | | | AR::Base#attributes= is just an alias for AR::Base#assign_attributesGuillermo Iguaran2012-09-191-10/+4
| | | | | | |
* | | | | | | Merge pull request #7709 from alexreisner/masterRafael Mendonça França2012-09-191-1/+1
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | Fix Broken Link in ActiveRecord Transaction Documentation [ci skip]
| * | | | | | Fix out-of-date URL.Alex Reisner2012-09-191-1/+1
|/ / / / / /
* | | | | | Merge pull request #7707 from guilleiguaran/changelogs-updateRafael Mendonça França2012-09-194-0/+50
|\ \ \ \ \ \ | | | | | | | | | | | | | | Update changelogs to add entries about strong_parameters integration
| * | | | | | Update changelogs to add entries about strong_parameters integrationGuillermo Iguaran2012-09-194-0/+50
|/ / / / / /
* | | | | | extracts manual development dependencies installation to a new non-promoted ↵Xavier Noria2012-09-202-158/+176
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | guide The purpose of this refactor is twofold: * Emphasize the Rails development box as the recommended way to setup a dev environment by taking out the rest of the instructions. * Lighten the contributing guide, this non-essential information takes a disproportionate amount of space now that we have the rails-dev-box.
* | | | | | Load the environment after creating the databaseJeremy Kemper2012-09-191-2/+2
| | | | | |
* | | | | | Revert "Make sure :environment task is executed before db:schema:load or ↵Jeremy Kemper2012-09-191-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | db:structure:load " Breaks db:setup because it tries to load the environment before creating the database. This reverts commit 5ca11fefce6d83f5db399aa4412f1f1a0d42b2e6.
* | | | | | Merge pull request #7701 from guilleiguaran/set-permit-all-parameters-explicitlyRafael Mendonça França2012-09-191-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Set AC::Parameters.permit_all_parameters explicitly to false when config.action_controller.permit_all_parameters is not present
| * | | | | | Set AC::Parameters.permit_all_parameters explicitly to false when ↵Guillermo Iguaran2012-09-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | config.action_controller.permit_all_parameters is not present
* | | | | | | Improve the CHANGELOG entry for #6971Rafael Mendonça França2012-09-191-1/+14
|/ / / / / /
* | | | | | Merge pull request #6971 from dmathieu/empty_hashRafael Mendonça França2012-09-193-3/+18
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | fix querying with an empty hash
| * | | | | fix querying with an empty hashDamien Mathieu2012-09-193-3/+18
| | | | | | | | | | | | | | | | | | | | | | | | Closes #6960
* | | | | | Merge pull request #7482 from route/syncing_tests_for_sprocketsRafael Mendonça França2012-09-192-33/+37
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Use last version of sprockets-rails and sync tests for assets.