aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #20413 from kddeisz/pluck_postgres_fixYves Senn2015-06-021-2/+2
|\ | | | | Fix postgresql DISTINCT requirement in pluck test
| * Fix postgresql DISTINCT requirement in pluck testKevin Deisz2015-06-021-2/+2
|/
* Merge pull request #20410 from schneems/schneems/boo-global-varsRafael Mendonça França2015-06-013-5/+5
|\ | | | | Use block variable instead of global
| * Use block variable instead of globalschneems2015-06-013-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ```ruby require 'benchmark/ips' Benchmark.ips do |x| x.report("$&") { "foo".sub(/f/) { $&.upcase } } x.report("block var") { "foo".sub(/f/) {|match| match.upcase } } end ``` ``` Calculating ------------------------------------- $& 48.658k i/100ms block var 49.666k i/100ms ------------------------------------------------- $& 873.156k (± 9.3%) i/s - 4.331M block var 969.744k (± 9.2%) i/s - 4.818M ``` It's faster, and gets rid of a few "magic" global variables
* | Merge pull request #20409 from davydovanton/doc-queue-adapterAbdelkader Boudih2015-06-021-0/+2
|\ \ | | | | | | Add documentation for QueueAdapter::queue_adapter
| * | [skip ci] Add documentation for QueueAdapter::queue_adapterAnton Davydov2015-06-021-0/+2
|/ /
* | Merge pull request #20357 from ronakjangir47/remove_mocha1Rafael Mendonça França2015-06-011-27/+52
|\ \ | | | | | | Removed use of mocha from railties actions_test
| * | Removed use of mocha from railties actions_testRonak Jangir2015-06-011-27/+52
| | |
* | | Merge pull request #20406 from yoongkang/add_edge_sprocketsSantiago Pastorino2015-06-011-0/+2
|\ \ \ | | | | | | | | Use sprockets-rails from github repo
| * | | Use sprockets-rails from github repoYoong Kang Lim2015-06-021-0/+2
| |/ / | | | | | | | | | See #20397
* | | Merge pull request #20405 from manish-shrivastava/masterRafael Mendonça França2015-06-015-5/+7
|\ \ \ | | | | | | | | Fixed typos in rails guide
| * | | Fixed typos in guidemanish-shrivastava2015-06-015-5/+7
| | | |
* | | | Merge pull request #19094 from phoet/have_bearer_be_valid_as_wellRafael Mendonça França2015-06-013-2/+19
|\ \ \ \ | | | | | | | | | | Have Bearer be valid as well
| * | | | add changelog entryphoet2015-06-011-0/+10
| | | | |
| * | | | allow `Bearer` as well as `Token`phoet2015-06-012-1/+8
| | | | |
| * | | | actually test what the name saysphoet2015-06-011-1/+1
| | |/ / | |/| |
* | | | Merge pull request #20362 from kddeisz/enumerable_pluckRafael Mendonça França2015-06-014-2/+43
|\ \ \ \ | |_|/ / |/| | | Allow Enumerable#pluck to take a splat.
| * | | Allow Enumerable#pluck to take a splat.Kevin Deisz2015-05-294-2/+43
| | | | | | | | | | | | | | | | | | | | | | | | This allows easier integration with ActiveRecord, such that AR#pluck will now use Enumerable#pluck if the relation is loaded, without needing to hit the database.
* | | | Merge pull request #20138 from tgxworld/deprecated_assert_templateRafael Mendonça França2015-06-0126-880/+166
|\ \ \ \ | | | | | | | | | | Deprecate `assert_template` and `assigns()`.
| * | | | Remove `assigns` and `assert_template`.Guo Xiang Tan2015-05-3026-880/+166
| | | | |
* | | | | Merge pull request #20284 from kaspth/fix-caching-testRafael Mendonça França2015-06-013-3/+9
|\ \ \ \ \ | | | | | | | | | | | | Move expectation to instance level.
| * | | | | Replace expectation with state check.Kasper Timm Hansen2015-05-303-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | The tests would still pass if the cache call in the rendered templates were removed.
* | | | | | Merge pull request #20398 from nkondratyev/fix-time-changeRafael Mendonça França2015-06-013-1/+9
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix a range of values for parameters of the Time#change
| * | | | | | Fix a range of values for parameters of the Time#changeNikolay Kondratyev2015-06-013-1/+9
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | Passing 999999000 < `:nsec` < 999999999 and 999999 < `:usec` < 1000000 to change a time with utc_offset doesn't throw an `ArgumentError`.
* | | | | | Merge pull request #20386 from yoongkang/circular_importRafael Mendonça França2015-06-011-1/+0
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix circular import warning on build
| * | | | | | Fix circular import warning on buildYoong Kang Lim2015-05-311-1/+0
| | | | | | |
* | | | | | | Merge pull request #20401 from yahonda/master_bigint_as_limit_19_in_oracleRafael Mendonça França2015-06-012-2/+6
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Map :bigint as NUMBER(19) sql_type by using `:limit => 19` for Oracle
| * | | | | | | Map :bigint as NUMBER(19) sql_type by using `:limit => 19` for OracleYasuo Honda2015-06-012-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | since NUMBER(8) is not enough to store the maximum number of bigint. Oracle NUMBER(p,0) as handled as integer because there is no dedicated integer sql data type exist in Oracle database. Also NUMBER(p,s) precision can take up to 38. p means the number of digits, not the byte length. bigint type needs 19 digits as follows. $ irb 2.2.2 :001 > limit = 8 => 8 2.2.2 :002 > maxvalue_of_bigint = 1 << ( limit * 8 - 1) => 9223372036854775808 2.2.2 :003 > puts maxvalue_of_bigint.to_s.length 19 => nil 2.2.2 :004 >
* | | | | | | | Merge pull request #20403 from takiy33/patch-1Rafael Mendonça França2015-06-011-2/+2
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Use --no-document option instead of --no-rdoc and --no-ri option
| * | | | | | | Use --no-document option instead of --no-rdoc and --no-ri optiontakiy332015-06-011-2/+2
|/ / / / / / /
* | | | | | | Merge pull request #20371 from davydovanton/doc-adapter-lookupYves Senn2015-06-011-0/+4
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Update documentation for QueueAdapters::lookup [ci skip]
| * | | | | | | [skip ci] Update documentation for QueueAdapters::lookupAnton Davydov2015-06-011-0/+4
| | | | | | | |
* | | | | | | | Merge pull request #20394 from mechanicles/use-symMatthew Draper2015-06-011-2/+5
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | [ci skip] Used 'instance_variables' in example for consistency.
| * | | | | | | [ci skip] Used 'instance_variables' in example for consistency.Santosh Wadghule2015-06-011-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Found that checking instance_variables elements using string object gives us 'false' value. `instance_variables.include? "@articles" # => false` - Used only 'instance_variables' instead of using '.include?' on it. So that it will return recent instance variables list.
* | | | | | | | Merge pull request #20396 from nkondratyev/fix-guidesKasper Timm Hansen2015-06-011-2/+0
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Remove description of `dependency_loading` option
| * | | | | | | Remove description of `dependency_loading` optionNikolay Kondratyev2015-06-011-2/+0
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This option has been removed in e6747d87f3a061d153215715d56acbb0be20191f [ci skip]
* | | | | | | Merge pull request #20393 from yui-knk/fix/guide2Arun Agrawal2015-06-011-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | [ci skip] Fix `above` -> `below`
| * | | | | | | [ci skip] Fix `above` -> `below`yui-knk2015-06-011-1/+1
|/ / / / / / /
* | | | | | | Merge pull request #20392 from equiamos/equiamos-fix-typos-docsZachary Scott2015-05-311-5/+5
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fixed typos [ci skip]
| * | | | | | | Fixed typos. [ci skip]Morgan Owens2015-05-311-5/+5
| | | | | | | |
* | | | | | | | Use `any?` rather than `present?` to check argseileencodes2015-05-312-2/+2
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's better to use Ruby methods when possible over methods defined by Active Support because then it does not need to rely on any dependencies.
* | | | | | | no rails command short-cuts in the getting-started guide. [ci skip]Yves Senn2015-05-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The full command name is more expressive.
* | | | | | | minor formatting changes in changelogs. [ci skip]Yves Senn2015-05-312-7/+7
| |/ / / / / |/| | | | |
* | | | | | Attempt to fix travis failuresSean Griffin2015-05-301-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The version of SQLite used by Travis is outdated. We need to install a newer version.
* | | | | | Ensure symbols passed to `select` are always quotedSean Griffin2015-05-304-10/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Our general contract in Active Record is that strings are assumed to be SQL literals, and symbols are assumed to reference a column. If a from clause is given, we shouldn't include the table name, but we should still quote the value as if it were a column. Upon fixing this, the tests were still failing on SQLite. This was because the column name being returned by the query was `"\"join\""` instead of `"join"`. This is actually a bug in SQLite that was fixed a long time ago, but I was using the version of SQLite included by OS X which has this bug. Since I'm guessing this will be a common case for contributors, I also added an explicit check with a more helpful error message. Fixes #20360
* | | | | | Add docs and changelog entry for 73aab03 [ci skip]Sean Griffin2015-05-302-1/+15
| | | | | |
* | | | | | Merge pull request #20380 from kamipo/add_changelog_for_17654Eileen M. Uchitelle2015-05-301-0/+13
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add CHANGELOG entry for #17654
| * | | | | | Add CHANGELOG entry for #17654 [ci skip]Ryuta Kamizono2015-05-301-0/+13
|/ / / / / /
* | | | | | Merge pull request #20378 from vngrs/fix_doc_of_match_methodKasper Timm Hansen2015-05-301-3/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | match method doc fix [ci skip]
| * | | | | | match method doc fix [ci skip]Mehmet Emin İNAÇ2015-05-301-3/+6
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | match method without setting `:via` option has been deprecated fix minor typo
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-09 22:41:52 +0000