aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Revert "Only include Rake::DSL if it's defined."Akira Matsuda2012-06-011-1/+1
| | | | | | | | | | | | | | | | | | This reverts commit 82c3aca17e78d25f217702e530586673f2a219d7. Reason: Ruby 1.9.3 is shipped with Rake > 0.9
* | | Merge pull request #6584 from amatsuda/ar_explain_inspect_readabilityXavier Noria2012-06-011-1/+7
|\ \ \ | | | | | | | | improve readability of AR explain result
| * | | improve readability of AR explain resultAkira Matsuda2012-06-011-1/+7
|/ / /
* | | Merge pull request #6579 from amatsuda/null_relation_modulizeJosé Valim2012-05-313-11/+15
|\ \ \ | |/ / |/| | modulize AR::NullRelation
| * | unused method argumentsAkira Matsuda2012-06-011-7/+6
| | |
| * | modulize AR::NullRelationAkira Matsuda2012-06-013-4/+9
|/ / | | | | | | now we can invoke previously added scope extension methods
* | Rake::DSL should always be availableJeremy Kemper2012-05-311-1/+1
| |
* | Merge branch 'master-sec'Aaron Patterson2012-05-315-4/+65
|\ \ | | | | | | | | | | | | | | | * master-sec: Strip [nil] from parameters hash. Thanks to Ben Murphy for reporting this! predicate builder should not recurse for determining where columns. Thanks to Ben Murphy for reporting this
| * | Strip [nil] from parameters hash.Aaron Patterson2012-05-302-1/+28
| | | | | | | | | | | | | | | | | | Thanks to Ben Murphy for reporting this! CVE-2012-2660
| * | predicate builder should not recurse for determining where columns.Aaron Patterson2012-05-303-3/+37
| | | | | | | | | | | | | | | | | | Thanks to Ben Murphy for reporting this CVE-2012-2661
* | | Explicitly require rake so its DSL is availableJeremy Kemper2012-05-311-0/+1
| | |
* | | Rake is in 1.9 stdlib, no need for gem depJeremy Kemper2012-05-311-4/+2
| | |
* | | Psych is always available on 1.9Jeremy Kemper2012-05-312-29/+23
| | |
* | | Merge pull request #6571 from ↵Rafael Mendonça França2012-05-313-7/+50
|\ \ \ | | | | | | | | | | | | | | | | jeremyf/extract-force-ssl-filter-logic-to-separate-method Extracted redirect logic from ActionController::Force::ClassMethods.forc...
| * | | Extracted redirect logic from ActionController::Force::ClassMethods.force_sslJeremy Friesen2012-05-313-7/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prior to this patch the existing .force_ssl method handles both defining the filter and handling the logic for performing the redirect. With this patch the logic for redirecting to the HTTPS protocol is separated from the filter logic that determines if a redirect should occur. By separating the two levels of behavior, an instance method for ActionController (i.e. #force_ssl_redirect) is exposed and available for more granular SSL enforcement. Cleaned up indentation.
* | | | Reduce warning messages when running Active Support testsAndrew White2012-05-312-2/+8
| | | | | | | | | | | | | | | | Eliminate the warnings generated by redefining methods and constants.
* | | | Merge pull request #6566 from arunagw/add-javascripts-to-rake-stats-taskRafael Mendonça França2012-05-312-2/+10
|\ \ \ \ | |/ / / |/| | | Add code statistics for Javascript and CoffeeScript files
| * | | Add code statistics for Javascript andArun Agrawal2012-05-312-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | CoffeeScript files to `rake stats` task Orignal PR was #2270 Thanks to @nfm
* | | | Fix changelog example for count method with block [ci skip]Carlos Antonio da Silva2012-05-311-1/+1
| | | | | | | | | | | | | | | | Missed that too, thanks again @splattael.
* | | | Fix wrong example for count method with block [ci skip]Carlos Antonio da Silva2012-05-311-1/+1
| | | | | | | | | | | | | | | | Thanks @splattael.
* | | | Simplify link_to using content_tagCarlos Antonio da Silva2012-05-312-12/+26
| | | | | | | | | | | | | | | | Add some tests for link_to with blocks and escaping content.
* | | | Make link_to arguments explicitCarlos Antonio da Silva2012-05-311-8/+4
| | | |
* | | | Make button_to arguments explicit and refactor a bitCarlos Antonio da Silva2012-05-311-10/+6
| | | | | | | | | | | | | | | | Prefer Hash#[]= over Hash#merge when setting a value.
* | | | Simplify logic to initialize valid conditions in RouteSetCarlos Antonio da Silva2012-05-311-7/+3
|/ / / | | | | | | | | | | | | | | | Remove :to_sym call from public_instance_methods iteration, as such methods in Ruby 1.9 already return symbols. Initialize valid conditions with controller/action instead of setting them afterwards.
* | | Work around undiagnosed bug that's draining a relation's bind_valuesJeremy Kemper2012-05-311-0/+1
| | |
* | | Fix backward compatibility with stored Hash values. Wrap coders to convert ↵Jeremy Kemper2012-05-303-15/+51
| | | | | | | | | | | | serialized values to indifferent access.
* | | Set RACK_ENV to nil in the dbconsole testRafael Mendonça França2012-05-301-0/+1
| | | | | | | | | | | | This will fix the travis-ci build
* | | Fix app_generator_test with the changes introduced atRafael Mendonça França2012-05-301-1/+1
| | | | | | | | | | | | f4d7af67ffc90f2542afa50c7579fc83ea4f45f2
* | | Merge pull request #6560 from martinoconnor/generator_fixRafael Mendonça França2012-05-301-1/+1
|\ \ \ | | | | | | | | Changed symbol platform to platforms for the commented out call to gem 'therubyracer'.
| * | | Changed symbol platform to platforms for the commented out call to gem ↵Martin O'Connor2012-05-301-1/+1
|/ / / | | | | | | | | | | | | | | | 'therubyracer'. Dependency.rb expects the symbol to be named :platforms as opposed to platform. RubyMine's inspections indicate that the symbol should be named :platforms.
* | | Merge pull request #6276 from samoli/fix_console_argumentsRafael Mendonça França2012-05-307-132/+182
|\ \ \ | | | | | | | | Fix various bugs with console arguments and improve test coverage
| * | | Remove support for rails server RAILS_ENV=env-nameSam Oliver2012-05-302-7/+2
| | | |
| * | | Fix various bugs with console arguments.Sam Oliver2012-05-305-125/+180
|/ / / | | | | | | | | | Allow hyphens in environment names again.
* | | Merge pull request #6477 from steveklabnik/close_discovered_pg_connectionRafael Mendonça França2012-05-301-1/+2
|\ \ \ | | | | | | | | Properly discover a connection is closed in postgresql_adapter
| * | | Properly discover a connection is closed in postgresql_adapterSteve Klabnik2012-05-241-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PQstatus doesn't properly test if future operations will succeed. A PQping function is added to libpq in PostgreSQL 9.1, but if we rely on it, everyone on earlier versions of Postgres is out of luck, and the pg gem wouldn't have the 'fix' until the next release. Thanks to @cbrecabarren and @ged for handling all the dirty details. Closes #3392.
* | | | Merge pull request #6556 from lest/patch-1Rafael Mendonça França2012-05-304-5/+53
|\ \ \ \ | | | | | | | | | | accept a block in button_to helper
| * | | | accept a block in button_to helperSergey Nartimov2012-05-303-3/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make possible to use a block in button_to helper if button text is hard to fit into the name parameter, e.g.: <%= button_to [:make_happy, @user] do %> Make happy <strong><%= @user.name %></strong> <% end %> # => "<form method="post" action="/users/1/make_happy" class="button_to"> # <div> # <button type="submit"> # Make happy <strong>Name</strong> # </button> # </div> # </form>"
| * | | | no need to pass an empty block to button_to helperSergey Nartimov2012-05-301-2/+2
| | | | |
* | | | | Merge pull request #6554 from erichmenge/remove-json-variablePiotr Sarnacki2012-05-304-14/+7
|\ \ \ \ \ | |/ / / / |/| | | | Remove deprecated ActiveSupport::JSON::Variable.
| * | | | Remove deprecated ActiveSupport::JSON::Variable.Erich Menge2012-05-304-14/+7
| | | | |
* | | | | Merge pull request #6541 from ↵Rafael Mendonça França2012-05-301-3/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tjouan/tj/activesupport-fix-tests-requiring-unsafe-umask Make tests pass with umask
| * | | | | Make File#atomic_write tests pass with umaskThibault Jouan2012-05-291-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Assertions on file permissions only pass with some combinations of file create mode and the process file mode creation mask. This mask should be applied on the file create mode before related assertions.
* | | | | | Merge pull request #6555 from schneems/schneems/remove-reload-codeRafael Mendonça França2012-05-301-2/+0
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove Unused Route Reloading Code
| * | | | | | remove unused route reloading codeschneems2012-05-301-2/+0
|/ / / / / / | | | | | | | | | | | | Since the environment is initialized each time rake is run, routes don't need to be re-loaded. https://github.com/rails/rails/pull/6461#r869953
* | | | | | Merge pull request #5810 from kennyj/fix_5797Aaron Patterson2012-05-304-5/+15
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix #5797. Error calling dup method on AR model with serialized field
| * | | | | | Fix #5797. Error calling dup method on AR model with serialized fieldkennyj2012-05-304-5/+15
| | | | | | |
* | | | | | | Merge pull request #6386 from kennyj/fix_logs_name_consistencyAaron Patterson2012-05-307-33/+123
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix logs name consistency.
| * | | | | | | Fix logs name consistency.kennyj2012-05-197-33/+123
| | | | | | | |
* | | | | | | | cut down spaces [ci skip]Vijay Dev2012-05-301-3/+3
| | | | | | | |
* | | | | | | | Revert "Revert "[AR querying guide] Add examples for take(limit), ↵Vijay Dev2012-05-301-0/+48
| |_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | first(limit) and last(limit)"" This reverts commit 5559a2ae98dcda6854f38890025b52edfb2836f5. Reason: These are for selecting multiple objects and there isn't a need to club them with the selecting single objects section, as discussed with the author.