aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | Improve changelog entries in ARCarlos Antonio da Silva2013-02-081-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Describe better the changes related to raising errors with empty array/hash values in where clauses. [ci skip]
* | | | | | | Merge pull request #9216 from robertomiranda/where-with-empty-arrayGuillermo Iguaran2013-02-086-13/+14
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Activer Record: Change behaviour with empty array in where clause
| * | | | | | | Change behaviour with empty array in where clauserobertomiranda2013-02-086-13/+14
|/ / / / / / /
* | | | | | | Merge pull request #9220 from robertomiranda/where-with-empty-hashMichael Koziarski2013-02-073-3/+11
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Active Record: Change behaviour with empty hash in where clause
| * | | | | | | Change behaviour with empty hash in where clauserobertomiranda2013-02-083-3/+11
|/ / / / / / /
* | | | | | | Merge pull request #9185 from tricknotes/fix-article-for-generator-nameCarlos Antonio da Silva2013-02-071-2/+2
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Fix article for generator name
| * | | | | | Fix article for generator nameRyunosuke SATO2013-02-071-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Use 'some' instead of 'a/an'.
* | | | | | | Merge pull request #9207 from dylanahsmith/mysql-quote-numericGuillermo Iguaran2013-02-078-14/+60
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | active_record: Quote numeric values compared to string columns.
| * | | | | | | active_record: Quote numeric values compared to string columns.Dylan Smith2013-02-078-14/+60
| | | | | | | |
* | | | | | | | Skip schema dumper extensions test if connection does not support itCarlos Antonio da Silva2013-02-071-0/+1
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Travis failed because the postgresql version installed there does not support extensions, so we just need to skip this for now.
* | | | | | | Run schema dumper extensions without creating real extensionsCarlos Antonio da Silva2013-02-061-9/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no need to create the extension in the database just to test if it's dumped, we can stub that instead.
* | | | | | | Do not print anything related to extensions when they don't none existCarlos Antonio da Silva2013-02-062-4/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When extensions are supported but there's no one enabled in the database, we should not print anything related to them in schema.rb.
* | | | | | | Add blank line after extensions to separate from tables in schemaCarlos Antonio da Silva2013-02-061-0/+1
| | | | | | |
* | | | | | | Fix indentation of extensions in schemaCarlos Antonio da Silva2013-02-061-4/+2
| | | | | | |
* | | | | | | Call super to use the abstract adapter implementation insteadCarlos Antonio da Silva2013-02-061-1/+1
| | | | | | |
* | | | | | | Add changelog entry for #9203 about schema dumper with db extensionsCarlos Antonio da Silva2013-02-062-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | Merge pull request #9203 from jaggederest/dumping_pgsql_extensionsAaron Patterson2013-02-065-0/+42
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Adding database extension support to schema.rb
| * | | | | | | add ActiveRecord::AbstractAdapter#extensions and ↵Justin George2013-02-063-0/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ActiveRecord::ConnectionAdapters::PostgreSQLAdapter#extensions to allow dumping of enabled extensions to schema.rb, add ActiveRecord::SchemaDumper#extensions to dump extensions to schema.rb
| * | | | | | | improve tests to check for existence of extensions method, and skip testing ↵Justin George2013-02-062-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dumped extensions if they are unsupported by the database
| * | | | | | | Add some tests to enumerate how extensions should be stored in the schema outputJustin George2013-02-062-0/+15
|/ / / / / / /
* | | | | | | Update changelog from #9128 with author nameCarlos Antonio da Silva2013-02-061-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | And improve AS changelog a bit [ci skip]
* | | | | | | Merge pull request #9128 from jamesotron/iso8601-sub-second-accuracyJeremy Kemper2013-02-063-2/+7
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Modify TimeWithZone#as_json to return 3DP of sub-second accuracy.
| * | | | | | | Modify TimeWithZone#as_json to return 3DP of sub-second accuracy by default, ↵James Harton2013-01-313-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | since it's allowed by the spec and is very useful.
* | | | | | | | Update actionpack/CHANGELOG.mdRafael Mendonça França2013-02-061-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | | Merge pull request #9039 from ↵Rafael Mendonça França2013-02-064-9/+72
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | senny/warn_on_controller_option_with_ruby_constant_syntax ruby constant syntax is not supported as routing `:controller` option.
| * | | | | | | | ruby constant syntax is not supported as routing `:controller` option.Yves Senn2013-02-064-9/+72
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current implementation only works correctly if you supply the `:controller` with directory notation (eg. `:controller => 'admin/posts'`). The ruby constant notation (eg. `:controller => 'Admin::Posts`) leads to unexpected problems with `url_for`. This patch prints a warning for every non supported `:controller` option. I also added documentation how to work with namespaced controllers. The warning links to that documentation in the rails guide.
* | | | | | | | Update .gitignoreJosé Valim2013-02-051-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | Get rid of old .rvmrc and .rbenv-version files.
* | | | | | | | Enable hstore extensions on tests if it is not enabled and database supports itRafael Mendonça França2013-02-051-1/+8
| | | | | | | |
* | | | | | | | Merge pull request #9192 from adammck/ignore-dot-ruby-versionRafael Mendonça França2013-02-051-0/+1
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | Ignore .ruby-version
| * | | | | | | Ignore .ruby-versionAdam Mckaig2013-02-051-0/+1
|/ / / / / / /
* | | | | | | Merge pull request #9178 from robertomiranda/best-standard-support-guideCarlos Antonio da Silva2013-02-041-0/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Update upgrading guide with ActionDispatch::BestStandardsSupport removal [ci skip]
| * | | | | | | Update upgrading guide with ActionDispatch::BestStandardsSupport removalrobertomiranda2013-02-041-0/+2
| | | | | | | |
* | | | | | | | Merge pull request #9177 from zenspider/masterAaron Patterson2013-02-041-0/+12
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Added test:all and test:all:db tasks to speed up full test runs
| * | | | | | | Added test:all and test:all:db tasks to speed up full test runs.Ryan Davis2013-02-041-0/+12
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Speed up depends on the number of test phases (N) you have for your app, but it is roughly a 1/N improvement. In my app, it goes from 15.51s to 5.56s. Inspired by http://ngauthier.com/2012/02/quick-tests-with-bash.html Reviewed by @tenderlove
* | | | | | | Merge pull request #9175 from robertomiranda/add--rc-descGuillermo Iguaran2013-02-041-0/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Add --rc description on the command line
| * | | | | | | Add --rc description on the command linerobertomiranda2013-02-041-0/+3
| | |_|/ / / / | |/| | | | |
* | | | | | | Merge pull request #9174 from robertomiranda/railsrc-cli-optionRafael Mendonça França2013-02-041-2/+6
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Use --rc=PATH instead of --rc PATH
| * | | | | | Use --rc= instead of --rcrobertomiranda2013-02-041-2/+6
|/ / / / / /
* | | | | | Merge pull request #8520 from senny/8516_assert_template_with_localsCarlos Antonio da Silva2013-02-046-3/+46
|\ \ \ \ \ \ | | | | | | | | | | | | | | Improved `assert_template` when matching :locals with partials
| * | | | | | partials inside directory work with `assert_template`Yves Senn2013-02-046-3/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | previously when a partial was placed inside a directory (eg. '/dir/_partial'), `assert_template` did not replace the '_' prefix when looking through rendered tempaltes, which resulted in an error. I modified it to replace both, the leading '_' and the last '_' after a '/'.
| * | | | | | descriptive `assert_template` error when partial wasn't renderedYves Senn2013-02-043-0/+16
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When `assert_template` is used with the :locals option, and the partial was not rendered, a method_missing error was raised. This changes first checks, if the partial actually was rendered and raises a descriptive error.
* | | | | | Take care of whitespaces and changing the hash syntaxRafael Mendonça França2013-02-041-57/+56
| | | | | |
* | | | | | Merge pull request #9152 from dahakawang/masterAaron Patterson2013-02-031-0/+8
|\ \ \ \ \ \ | | | | | | | | | | | | | | test for issue 8931
| * | | | | | test for issue 8931David2013-02-021-0/+8
| | | | | | |
* | | | | | | Merge pull request #9164 from yuki24/put_rdoc_at_the_end_of_RUNNING_UNIT_TESTSXavier Noria2013-02-032-0/+0
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Put .rdoc at the end of the files named RUNNING_UNIT_TESTS
| * | | | | | | Put .rdoc at the end of the files named RUNNING_UNIT_TESTSYuki Nishijima2013-02-032-0/+0
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | So that they are printed nicely on GitHub.
* | | | | | | Merge pull request #9154 from robertomiranda/add-rc-descGuillermo Iguaran2013-02-021-0/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Add --no-rc description on the command line
| * | | | | | | Add --no-rc description on the command linerobertomiranda2013-02-021-0/+3
|/ / / / / / /
* | | | | | | Merge pull request #9151 from gogogarrett/patch-1Rafael Mendonça França2013-02-021-1/+1
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | Fix link to point to new RefineryCMS repo
| * | | | | | Fix link to point to new RefineryCMS repoGarrett Heinlen2013-02-011-1/+1
| |/ / / / /