Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | Merge pull request #7972 from scottwillson/fix-double-presence-of-errors-2 | José Valim | 2012-10-17 | 3 | -2/+16 | |
|\ \ \ | |_|/ |/| | | Fix bug with presence validation of associations. | |||||
| * | | Fix bug with presence validation of associations. | Scott Willson | 2012-10-16 | 3 | -2/+16 | |
|/ / | | | | | | | Would incorrectly add duplicated errors when the association was blank. Bug introduced in 1fab518c6a75dac5773654646eb724a59741bc13. | |||||
* | | Merge pull request #7371 from csmuc/fix_dup_validation_errors | Santiago Pastorino | 2012-10-16 | 3 | -0/+20 | |
|\ \ | | | | | | | Dup'ed ActiveRecord objects may not share the errors object | |||||
| * | | Call super to nullify the reference to the original errors object in the ↵ | Christian Seiler | 2012-10-16 | 3 | -0/+20 | |
| | | | | | | | | | | | | dup'ed object (call ActiveModel::Validations#initialize_dup). Closes #7291 | |||||
* | | | Merge pull request #7963 from arunagw/redcarpet_not_for_jruby | Carlos Antonio da Silva | 2012-10-16 | 1 | -1/+1 | |
|\ \ \ | | | | | | | | | Add :platform to redcarpet gem, pointing to :ruby only. | |||||
| * | | | redcarpet gem will not work with JRuby! | Arun Agrawal | 2012-10-16 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Revert "remove unnecessary object/conversions file" | Xavier Noria | 2012-10-16 | 4 | -2/+11 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This file is used at least by Active Merchant, its existence is maybe not necessary but no big deal either. This reverts commit ae9b3d7cecd77b9ace38671b183e1a360bf632b6. | |||||
* | | | | Merge pull request #7958 from mattdbridges/rails-guides-active-support-addition | Santiago Pastorino | 2012-10-15 | 1 | -0/+4 | |
|\ \ \ \ | | | | | | | | | | | Adding reference a few Time and Date aliases in Rails 4.0 release notes | |||||
| * | | | | Adding reference a few Time and Date aliases | Matt Bridges | 2012-10-15 | 1 | -0/+4 | |
|/ / / / | | | | | | | | | | | | | | | | | I originally contributed these aliases 10 months ago and just never got it added to the Rails 4.0 release notes. Here they are. | |||||
* | | | | Move changelog entry from #7439 to the top [ci skip] | Carlos Antonio da Silva | 2012-10-15 | 1 | -6/+6 | |
| | | | | ||||||
* | | | | Merge pull request #7439 from featurist/master | Aaron Patterson | 2012-10-15 | 3 | -1/+10 | |
|\ \ \ \ | | | | | | | | | | | ActiveRecord: sum expression returns string '0' for no records, fixed | |||||
| * | | | | ActiveRecord: sum expression returns string '0' for no records, fixed | Tim Macfarlane | 2012-10-15 | 3 | -1/+10 | |
|/ / / / | ||||||
* | | / | use `setup` for setting up the test | Aaron Patterson | 2012-10-15 | 1 | -5/+1 | |
| |_|/ |/| | | ||||||
* | | | Merge pull request #7956 from josh/ignore-asset-url-query-or-anchor | Rafael Mendonça França | 2012-10-15 | 2 | -2/+21 | |
|\ \ \ | | | | | | | | | Ignore asset url query string or anchor when generating path | |||||
| * | | | Ignore asset url query string or anchor when appending extensions and ↵ | Joshua Peek | 2012-10-15 | 2 | -2/+21 | |
| | | | | | | | | | | | | | | | | computing public path | |||||
* | | | | buckets hash isn't public, so use symbol keys to avoid string | Aaron Patterson | 2012-10-15 | 1 | -8/+8 | |
| | | | | | | | | | | | | | | | | allocations | |||||
* | | | | Move two hotspots to use Hash[] rather than Hash#dup | Aaron Patterson | 2012-10-15 | 1 | -3/+5 | |
|/ / / | | | | | | | | | | https://bugs.ruby-lang.org/issues/7166 | |||||
* | | | Merge pull request #7927 from josh/asset-path-helper | Rafael Mendonça França | 2012-10-15 | 11 | -505/+289 | |
|\ \ \ | | | | | | | | | Refactor AssetUrlHelper | |||||
| * | | | Use sprockets-rails master | Joshua Peek | 2012-10-15 | 2 | -3/+3 | |
| | | | | ||||||
| * | | | Extract compute_asset_extname and allow extname to be disabled | Joshua Peek | 2012-10-15 | 2 | -13/+30 | |
| | | | | ||||||
| * | | | Sprockets-rails branch | Joshua Peek | 2012-10-15 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Use sprockets-rails branch for now | Joshua Peek | 2012-10-15 | 1 | -2/+2 | |
| | | | | ||||||
| * | | | Merge branch 'master' into asset-path-helper | Joshua Peek | 2012-10-15 | 123 | -626/+681 | |
| |\ \ \ | |/ / / |/| | | | | | | | | | | | Conflicts: railties/test/application/configuration_test.rb | |||||
* | | | | ruby-prof isn't compatible with ruby trunk after 36925 | Jeremy Kemper | 2012-10-15 | 1 | -2/+2 | |
| | | | | ||||||
* | | | | Merge pull request #7953 from kryzhovnik/sanitize_helper_cleanup | Santiago Pastorino | 2012-10-15 | 1 | -4/+0 | |
|\ \ \ \ | | | | | | | | | | | Removed useless "extend SanitizeHelper::ClassMethods" | |||||
| * | | | | Removed useless "extend SanitizeHelper::ClassMethods" | Andrey Samsonov | 2012-10-15 | 1 | -4/+0 | |
| | |/ / | |/| | | | | | | | | | | | | | | Since SanitizeHelper includes ActiveSupport::Concern, extending of it ClassMethods is no needed. | |||||
* | | | | Merge pull request #7952 from ayrton/minor_cleanup | Santiago Pastorino | 2012-10-15 | 1 | -5/+1 | |
|\ \ \ \ | |/ / / |/| | | | Minor cleanup, helper method was only used once | |||||
| * | | | Minor cleanup, helper method was only used once | Ayrton De Craene | 2012-10-15 | 1 | -5/+1 | |
|/ / / | ||||||
* | | | Merge pull request #7947 from acapilleri/uniqueness_validate | Rafael Mendonça França | 2012-10-14 | 1 | -1/+2 | |
|\ \ \ | | | | | | | | | refactoring of uniqueness validate_each | |||||
| * | | | refactoring of uniqueness validate_each | Angelo Capilleri | 2012-10-14 | 1 | -1/+2 | |
|/ / / | | | | | | | | | | get scope_value only one time dependig on reflection | |||||
* | | | Merge pull request #7942 from IPGlider/master | Rafael Mendonça França | 2012-10-14 | 2 | -1/+14 | |
|\ \ \ | | | | | | | | | Fix typo in inet and cidr saving | |||||
| * | | | Fix typo in inet and cidr saving | Miguel Herranz | 2012-10-14 | 2 | -1/+14 | |
| | | | | ||||||
* | | | | Fix typo in #7941 | Jeremy Kemper | 2012-10-14 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | The debugger gem isn't compatible with 1.9.3-p286 yet. Omit it for now. | Jeremy Kemper | 2012-10-14 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Merge pull request #7941 from robin850/master | Jeremy Kemper | 2012-10-14 | 94 | -447/+447 | |
|\ \ \ \ | | | | | | | | | | | Use 1.9 Hash syntax in railties | |||||
| * | | | | Use Ruby 1.9 Hash syntax in railties | Robin Dupret | 2012-10-14 | 94 | -447/+447 | |
| | | | | | ||||||
* | | | | | Merge pull request #7943 from NARKOZ/patch-4 | Jeremy Kemper | 2012-10-14 | 1 | -2/+4 | |
|\ \ \ \ \ | |/ / / / |/| | | | | remove 'then' from conditional statement | |||||
| * | | | | remove 'then' from conditional statement | Nihad Abbasov | 2012-10-14 | 1 | -2/+4 | |
|/ / / / | ||||||
* | | | | Merge pull request #7937 from ↵ | Rafael Mendonça França | 2012-10-13 | 6 | -15/+61 | |
|\| | | | | | | | | | | | | | | | | | | | arturopie/7914-defaults-on-domains-and-multiple-schemas Fixes #7914 - PostgreSQL adapter doesn't fetch column defaults when using multiple schemas and domains | |||||
| * | | | #7914 Remove code for unsupported postgreSQL version. | Arturo Pie | 2012-10-13 | 2 | -6/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove parsing of character type default values for 8.1 formatting since Rails doesn't support postgreSQL 8.1 anymore. Remove misleading comment unrelated to code. | |||||
| * | | | #7914 Using a better way to get the defaults from db. | Arturo Pie | 2012-10-13 | 2 | -6/+7 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to postgreSQL documentation: (http://www.postgresql.org/docs/8.2/static/catalog-pg-attrdef.html) we should not be using 'adsrc' field because this field is unaware of outside changes that could affect the way that default values are represented. Thus, I changed the queries to use "pg_get_expr(adbin, adrelid)" instead of the historical "adsrc" field. | |||||
| * | | | #7914 Add change of previous commit to CHANGELOG.md | Arturo Pie | 2012-10-13 | 1 | -0/+4 | |
| | | | | ||||||
| * | | | #7914 get default value when type uses schema name | Arturo Pie | 2012-10-13 | 4 | -3/+50 | |
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PostgreSQL adapter properly parses default values when using multiple schemas and domains. When using domains across schemas, PostgresSQL prefixes the type of the default value with the name of the schema where that type (or domain) is. For example, this query: ``` SELECT a.attname, d.adsrc FROM pg_attribute a LEFT JOIN pg_attrdef d ON a.attrelid = d.adrelid AND a.attnum = d.adnum WHERE a.attrelid = "defaults"'::regclass AND a.attnum > 0 AND NOT a.attisdropped ORDER BY a.attnum; ``` could return something like "'<default_value>'::pg_catalog.text" or "(''<default_value>'::pg_catalog.text)::text" for the text columns with defaults. I modified the regexp used to parse this value so that it ignores anything between ':: and \b(?:character varying|bpchar|text), and it allows to have optional parens like in the above second example. | |||||
* | | | Merge pull request #7935 from leereilly/minor-gemfile-twerk | Rafael Mendonça França | 2012-10-13 | 1 | -1/+1 | |
|\ \ \ | | | | | | | | | Clean up Gemfile entry (replace double quotes with single quotes) [ci skip] | |||||
| * | | | Replace double quotes with single quotes | Lee Reilly | 2012-10-13 | 1 | -1/+1 | |
|/ / / | ||||||
* | | | Test that a Rails.queue consumer is automatically started in production | Jeremy Kemper | 2012-10-13 | 3 | -3/+14 | |
| | | | ||||||
* | | | Backpedal from class-oriented config.queue. Set an actual queue instance. | Jeremy Kemper | 2012-10-12 | 10 | -26/+14 | |
| | | | ||||||
* | | | Tighten up asset precompile tests | Jeremy Kemper | 2012-10-12 | 2 | -4/+3 | |
| | | | ||||||
* | | | Remove the queue container. Premature consolidation. Set up and maintain ↵ | Jeremy Kemper | 2012-10-12 | 5 | -68/+9 | |
| | | | | | | | | | | | | queues in the classes that use them instead. | |||||
* | | | Merge pull request #7891 from schneems/schneems/rake_command_warning | José Valim | 2012-10-12 | 1 | -23/+32 | |
|\ \ \ | | | | | | | | | Prompt to run rake when accidentally typed rails |