aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Merge pull request #7972 from scottwillson/fix-double-presence-of-errors-2José Valim2012-10-173-2/+16
|\ \ \ | |_|/ |/| | Fix bug with presence validation of associations.
| * | Fix bug with presence validation of associations.Scott Willson2012-10-163-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_errorsSantiago Pastorino2012-10-163-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 Seiler2012-10-163-0/+20
| | | | | | | | | | | | dup'ed object (call ActiveModel::Validations#initialize_dup). Closes #7291
* | | Merge pull request #7963 from arunagw/redcarpet_not_for_jrubyCarlos Antonio da Silva2012-10-161-1/+1
|\ \ \ | | | | | | | | Add :platform to redcarpet gem, pointing to :ruby only.
| * | | redcarpet gem will not work with JRuby!Arun Agrawal2012-10-161-1/+1
| | | |
* | | | Revert "remove unnecessary object/conversions file"Xavier Noria2012-10-164-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-additionSantiago Pastorino2012-10-151-0/+4
|\ \ \ \ | | | | | | | | | | Adding reference a few Time and Date aliases in Rails 4.0 release notes
| * | | | Adding reference a few Time and Date aliasesMatt Bridges2012-10-151-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 Silva2012-10-151-6/+6
| | | |
* | | | Merge pull request #7439 from featurist/masterAaron Patterson2012-10-153-1/+10
|\ \ \ \ | | | | | | | | | | ActiveRecord: sum expression returns string '0' for no records, fixed
| * | | | ActiveRecord: sum expression returns string '0' for no records, fixedTim Macfarlane2012-10-153-1/+10
|/ / / /
* | | / use `setup` for setting up the testAaron Patterson2012-10-151-5/+1
| |_|/ |/| |
* | | Merge pull request #7956 from josh/ignore-asset-url-query-or-anchorRafael Mendonça França2012-10-152-2/+21
|\ \ \ | | | | | | | | Ignore asset url query string or anchor when generating path
| * | | Ignore asset url query string or anchor when appending extensions and ↵Joshua Peek2012-10-152-2/+21
| | | | | | | | | | | | | | | | computing public path
* | | | buckets hash isn't public, so use symbol keys to avoid stringAaron Patterson2012-10-151-8/+8
| | | | | | | | | | | | | | | | allocations
* | | | Move two hotspots to use Hash[] rather than Hash#dupAaron Patterson2012-10-151-3/+5
|/ / / | | | | | | | | | https://bugs.ruby-lang.org/issues/7166
* | | Merge pull request #7927 from josh/asset-path-helperRafael Mendonça França2012-10-1511-505/+289
|\ \ \ | | | | | | | | Refactor AssetUrlHelper
| * | | Use sprockets-rails masterJoshua Peek2012-10-152-3/+3
| | | |
| * | | Extract compute_asset_extname and allow extname to be disabledJoshua Peek2012-10-152-13/+30
| | | |
| * | | Sprockets-rails branchJoshua Peek2012-10-151-1/+1
| | | |
| * | | Use sprockets-rails branch for nowJoshua Peek2012-10-151-2/+2
| | | |
| * | | Merge branch 'master' into asset-path-helperJoshua Peek2012-10-15123-626/+681
| |\ \ \ | |/ / / |/| | | | | | | | | | | Conflicts: railties/test/application/configuration_test.rb
* | | | ruby-prof isn't compatible with ruby trunk after 36925Jeremy Kemper2012-10-151-2/+2
| | | |
* | | | Merge pull request #7953 from kryzhovnik/sanitize_helper_cleanupSantiago Pastorino2012-10-151-4/+0
|\ \ \ \ | | | | | | | | | | Removed useless "extend SanitizeHelper::ClassMethods"
| * | | | Removed useless "extend SanitizeHelper::ClassMethods"Andrey Samsonov2012-10-151-4/+0
| | |/ / | |/| | | | | | | | | | | | | | Since SanitizeHelper includes ActiveSupport::Concern, extending of it ClassMethods is no needed.
* | | | Merge pull request #7952 from ayrton/minor_cleanupSantiago Pastorino2012-10-151-5/+1
|\ \ \ \ | |/ / / |/| | | Minor cleanup, helper method was only used once
| * | | Minor cleanup, helper method was only used onceAyrton De Craene2012-10-151-5/+1
|/ / /
* | | Merge pull request #7947 from acapilleri/uniqueness_validateRafael Mendonça França2012-10-141-1/+2
|\ \ \ | | | | | | | | refactoring of uniqueness validate_each
| * | | refactoring of uniqueness validate_eachAngelo Capilleri2012-10-141-1/+2
|/ / / | | | | | | | | | get scope_value only one time dependig on reflection
* | | Merge pull request #7942 from IPGlider/masterRafael Mendonça França2012-10-142-1/+14
|\ \ \ | | | | | | | | Fix typo in inet and cidr saving
| * | | Fix typo in inet and cidr savingMiguel Herranz2012-10-142-1/+14
| | | |
* | | | Fix typo in #7941Jeremy Kemper2012-10-141-1/+1
| | | |
* | | | The debugger gem isn't compatible with 1.9.3-p286 yet. Omit it for now.Jeremy Kemper2012-10-141-1/+1
| | | |
* | | | Merge pull request #7941 from robin850/masterJeremy Kemper2012-10-1494-447/+447
|\ \ \ \ | | | | | | | | | | Use 1.9 Hash syntax in railties
| * | | | Use Ruby 1.9 Hash syntax in railtiesRobin Dupret2012-10-1494-447/+447
| | | | |
* | | | | Merge pull request #7943 from NARKOZ/patch-4Jeremy Kemper2012-10-141-2/+4
|\ \ \ \ \ | |/ / / / |/| | | | remove 'then' from conditional statement
| * | | | remove 'then' from conditional statementNihad Abbasov2012-10-141-2/+4
|/ / / /
* | | | Merge pull request #7937 from ↵Rafael Mendonça França2012-10-136-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 Pie2012-10-132-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 Pie2012-10-132-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.mdArturo Pie2012-10-131-0/+4
| | | |
| * | | #7914 get default value when type uses schema nameArturo Pie2012-10-134-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-twerkRafael Mendonça França2012-10-131-1/+1
|\ \ \ | | | | | | | | Clean up Gemfile entry (replace double quotes with single quotes) [ci skip]
| * | | Replace double quotes with single quotesLee Reilly2012-10-131-1/+1
|/ / /
* | | Test that a Rails.queue consumer is automatically started in productionJeremy Kemper2012-10-133-3/+14
| | |
* | | Backpedal from class-oriented config.queue. Set an actual queue instance.Jeremy Kemper2012-10-1210-26/+14
| | |
* | | Tighten up asset precompile testsJeremy Kemper2012-10-122-4/+3
| | |
* | | Remove the queue container. Premature consolidation. Set up and maintain ↵Jeremy Kemper2012-10-125-68/+9
| | | | | | | | | | | | queues in the classes that use them instead.
* | | Merge pull request #7891 from schneems/schneems/rake_command_warningJosé Valim2012-10-121-23/+32
|\ \ \ | | | | | | | | Prompt to run rake when accidentally typed rails