aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Revert "Merge pull request #11053 from yangchenyun/mine"José Valim2013-06-221-2/+0
| | | | | | | | | | | Those two files are required by active_support/rails.rb loaded in active_model.rb It is common to require common dependencies on the root files to avoid boilerplate in each file. This reverts commit 763635e30f17800ebc0a8ae98d780edc8ebfb4e5, reversing changes made to 4f46ef36aaef217834f3f96d3689d32f6a6761ea.
* test-case to prevent regressions on `Association#build` with an Array.Yves Senn2013-06-221-0/+10
| | | | Closes #11026
* Merge pull request #11053 from yangchenyun/mineYves Senn2013-06-221-0/+2
|\ | | | | Add explicit dependencies of ActiveSupport to enable independent usage of ActiveModel::Name
| * add explicit AS dependencies for ActiveModel::NamingSteven Yang2013-06-221-0/+2
| | | | | | | | | | | | | | | | There are two missing ActiveSupport dependencies to use ActiveModel::Name class or ActiveModel::Naming module independently. Missing dependencies for Module#delegate defined in `active_support/core_ext/module/delegation`, used at [L148](https://github.com/rails/rails/blob/master/activemodel/lib/active_model/naming.rb#L148) Missing dependencies for Object#blank? defined in `active_support/core_ext/object/blank`, used at [L131](https://github.com/rails/rails/blob/master/activemodel/lib/active_model/naming.rb#L131)
* | Merge pull request #10796 from neerajdotname/10669Jon Leighton2013-06-223-2/+19
|\ \ | | | | | | flatten merged join_values before building the joins
| * | flatten merged join_values before building the joinsNeeraj Singh2013-06-223-2/+19
|/ / | | | | | | | | | | | | | | fixes #10669 While joining_values special treatment is given to string values. By flattening the array it ensures that string values are detected as strings and not arrays.
* | Merge pull request #11050 from robin850/remove-rails-test-mentionCarlos Antonio da Silva2013-06-211-2/+0
|\ \ | | | | | | Remove code related to the rails test command
| * | Remove code related to the rails test commandRobin Dupret2013-06-211-2/+0
| | | | | | | | | | | | | | | Remove the mention about this command in the rails command's help message and remove the "t" alias related to it.
* | | Merge pull request #11051 from neerajdotname/10509fJon Leighton2013-06-213-1/+25
|\ \ \ | |/ / |/| | do not load all child records for inverse case
| * | do not load all child records for inverse caseNeeraj Singh2013-06-213-1/+25
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | currently `post.comments.find(Comment.first.id)` would load all comments for the given post to set the inverse association. This has a huge performance penalty. Because if post has 100k records and all these 100k records would be loaded in memory even though the comment id was supplied. Fix is to use in-memory records only if loaded? is true. Otherwise load the records using full sql. Fixes #10509
* | Merge pull request #11049 from neerajdotname/no-need-of-large-numberJon Leighton2013-06-211-1/+5
|\ \ | | | | | | fix bad test by making number that fits for integer
| * | fix bad test by making number that fits for integerNeeraj Singh2013-06-211-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PR https://github.com/rails/rails/pull/10566 had to be reverted because after applying the fix test "test_raise_record_not_found_error_when_invalid_ids_are_passed" started failing. In this test invalid_id is being assigned a really large number which was causing following failure when PR #10566 was applied. ``` RangeError: bignum too big to convert into `long long' SELECT `interests`.* FROM `interests` WHERE `interests`.`man_id` = ? AND `interests`.`id` = ? LIMIT 1 [["man_id", 970345987], ["id", 2394823094892348920348523452345]] ``` This test is not failing in master because when test code `man.interests.find(invalid_id)` is executed then interests are fully loaded in memory and no database query is executed. After PR #10566 was merged then test code `man.interests.find(invalid_id)` started executing sql query and hence the error. In case someone is wondering why the second part of query is not failing, then that's because the actual query does not require any variable substituation where the number is large. In that case the sql generate is following. ``` SELECT `interests`.* FROM `interests` WHERE `interests`.`man_id` = ? AND `interests`.`id` IN (8432342, 2390102913, 2453245234523452) [["man_id", 970345987]] ```
* | | Merge pull request #10537 from ↵Guillermo Iguaran2013-06-211-0/+23
|\ \ \ | | | | | | | | | | | | | | | | jetthoughts/10428_bug_with_config_assets_precompile #10428: Added tests for config.assets.precompile
| * | | #10428: Added tests for config.assets.precompilePaul Nikitochkin2013-06-211-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | Tests for bug in sprocket-rails: do not use value of configuration options which changed after environment loaded
* | | | Merge pull request #11040 from satococoa/fix-create_join_table-documentCarlos Antonio da Silva2013-06-212-4/+4
|\ \ \ \ | | | | | | | | | | Fix documents for create_join_table [ci skip]
| * | | | Fix documents for create_join_tableSatoshi Ebisawa2013-06-212-4/+4
| | | | |
* | | | | Merge pull request #11043 from arunagw/gemcutter_to_rubygemsCarlos Antonio da Silva2013-06-218-8/+8
|\ \ \ \ \ | | | | | | | | | | | | Saying gemcutter => rubygems [ci skip]
| * | | | | Saying gemcutter => rubygems [ci skip]Arun Agrawal2013-06-218-8/+8
| | |_|/ / | |/| | |
* | | | | Merge pull request #11045 from rajars2576/fixing_action_viewXavier Noria2013-06-211-3/+3
|\ \ \ \ \ | |/ / / / |/| | | | fix the path of action view from actionpack/lib/action_view => actionvie...
| * | | | fix the path of action view from actionpack/lib/action_view => ↵Rajarshi Das2013-06-211-3/+3
|/ / / / | | | | | | | | | | | | actionview/lib/action_view
* | | | Merge pull request #11039 from robin850/patch-2Xavier Noria2013-06-211-5/+5
|\ \ \ \ | |/ / / |/| | | Backport #11033 to master
| * | | Update the feedback instructions [ci skip]Robin Dupret2013-06-211-5/+5
|/ / / | | | | | | | | | | | | | | | Since docrails as moved from lifo/docrails to rails/docrails we have to ask for commit rights or submit patches directly to the rails repository
* | | Don't remove trailing slash from PATH_INFO for mounted appsPiotr Sarnacki2013-06-213-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously when app was mounted as following: class Foo def call(env) [200, {}, [env['PATH_INFO']]] end end RackMountRailsBug::Application.routes.draw do mount RackTest.new => "/foo" end trailing slash was removed from PATH_INFO. For example requesting GET /foo/bar/ on routes defined above would result in a response containing "/foo/bar" instead of "/foo/bar/". This commit fixes the issue. (closes #3215)
* | | Merge pull request #11014 from ↵Piotr Sarnacki2013-06-203-0/+33
|\ \ \ | | | | | | | | | | | | | | | | senny/10936_inspect_does_not_crash_without_connection `inspect` for AR model classes does not initiate a new connection.
| * | | `inspect` for AR model classes does not initiate a new connection.Yves Senn2013-06-203-0/+33
| | | |
* | | | Merge pull request #11030 from ↵Piotr Sarnacki2013-06-203-5/+29
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | mjtko/fix/backtrace-silencer-noise-with-multiple-silencers Fix BacktraceSilencer#noise when multiple silencers are configured
| * | | | Fix BacktraceCleaner#noise for multiple silencers.Mark J. Titorenko2013-06-203-5/+29
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous implementation of BacktraceSilencer#noise did not work correctly if more than one silencer was configured -- specifically, it would only return noise which was matched by all silencers. The new implementation is such that anything that has been matched by silencers is removed from the backtrace using Array#- (array difference), ie. we now return all elements within a backtrace that have been matched by any silencer (and are thus removed by #silence). Fixes #11030.
* | | | Merge pull request #11032 from strzalek/extract-actionviewPiotr Sarnacki2013-06-20337-66/+1040
|\ \ \ \ | |/ / / |/| | | Extract ActionView to separate directory
| * | | Check if malformed fixture exists firstŁukasz Strzałkowski2013-06-201-0/+1
| | | | | | | | | | | | | | | | Now if somebody by mistake will remove malformed files test will raise error.
| * | | Restore mistakenly removed malformed file fixturesŁukasz Strzałkowski2013-06-205-0/+5
| | | |
| * | | Rename AP to AV in info about running testsŁukasz Strzałkowski2013-06-201-2/+2
| | | |
| * | | Bump up version, fix module nameŁukasz Strzałkowski2013-06-201-4/+4
| | | |
| * | | Adjust changelog for AV & APŁukasz Strzałkowski2013-06-202-36/+44
| | | |
| * | | Remove heading from AV's CHANGELOGŁukasz Strzałkowski2013-06-201-2/+0
| | | |
| * | | Change from 'actionpack' to 'actionview' for locales pathŁukasz Strzałkowski2013-06-201-1/+1
| | | |
| * | | Fix isolated tests on AVŁukasz Strzałkowski2013-06-201-3/+5
| | | |
| * | | Add ActionView to CIŁukasz Strzałkowski2013-06-202-2/+3
| | | |
| * | | Template test were moved to AVŁukasz Strzałkowski2013-06-201-1/+1
| | | |
| * | | Remove require to AP stuff that leftŁukasz Strzałkowski2013-06-202-2/+0
| | | |
| * | | Fix digestor testsŁukasz Strzałkowski2013-06-201-1/+2
| | | |
| * | | Remove digestor fixtures from APŁukasz Strzałkowski2013-06-2012-25/+0
| | | | | | | | | | | | | | | | They were moved to actionview/ and are not used in actionpack
| * | | Copy company test fixture to AV (fixes failing test)Łukasz Strzałkowski2013-06-201-0/+9
| | | |
| * | | Remove unneeded test fixtures in AVPiotr Sarnacki2013-06-20237-558/+0
| | | |
| * | | Add actionpack as actionview's development dependencyPiotr Sarnacki2013-06-201-0/+2
| | | | | | | | | | | | | | | | ActionView still relies on ActionPack in some of the tests.
| * | | Remove unneeded filesPiotr Sarnacki2013-06-207-88/+0
| | | |
| * | | Move template tests from actionpack to actionviewPiotr Sarnacki2013-06-20351-15/+1514
| | | |
| * | | ActionView version should be 4.1.0Piotr Sarnacki2013-06-201-1/+1
| | | |
| * | | Move actionpack/lib/action_view* into actionview/libPiotr Sarnacki2013-06-2097-95/+71
| | | |
| * | | Add bare actionview gem to the root directoryPiotr Sarnacki2013-06-209-2/+147
|/ / / | | | | | | | | | | | | This commit creates structure for Action View gem and is first of a series of commits extracting Action View from Action Pack.
* | | Merge pull request #11020 from fredwu/master-activemodel-inclusion-chain-fix-iiiRafael Mendonça França2013-06-202-6/+16
|\ \ \ | | | | | | | | ActiveModel::Model inclusion chain backward compatibility