aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | Merge pull request #9001 from schneems/schneems/routes-path-jsRafael Mendonça França2013-01-215-12/+134
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | In Browser Path Matching with Javascript
| * | | | | | | In Browser Path Matching with Javascriptschneems2013-01-205-12/+134
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When debugging routes ,it can sometimes be difficult to understand exactly how the paths are matched. This PR adds a JS based path matching widget to the `/rails/info/routes` output. You can enter in a path, and it will tell you which of the routes that path matches, while preserving order (top match wins). The matching widget in action: ![](http://f.cl.ly/items/3A2F0v2m3m1Z1p3P3O3k/path-match.gif) Prior to this PR the only way to check matching paths is via mental math, or typing in a path in the url bar and seeing where it goes. This feature will be an invaluable debugging tool by dramatically decreasing the time needed to check a path match. ATP actionpack
* | | | | | | | Merge pull request #9010 from vipulnsward/remove_extra_sort_from_testRafael Mendonça França2013-01-211-3/+3
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Remove extra sort from test
| * | | | | | | | Remove extra sort from testVipul A M2013-01-211-3/+3
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | Cleanup change_schema tests to remove extra sorts on columns.
* / | | | | | | Fix setting expected value in translation testsCarlos Antonio da Silva2013-01-211-2/+2
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | It was being set to nil instead due to the wrong assignment.
* | | | | | | Merge pull request #9007 from dpree/masterRafael Mendonça França2013-01-201-35/+46
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Enhanced tests for AbstractController::Translation module
| * | | | | | Removed ActionController::Base dependency from abstract controller ↵Jens Bissinger2013-01-201-41/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | translation tests.
| * | | | | | Test abstract controller's localize method.Jens Bissinger2013-01-201-0/+6
| | | | | | |
* | | | | | | Merge pull request #8913 from ↵Carlos Antonio da Silva2013-01-202-0/+10
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | seejee/regression_test_for_chained_preloaded_scopes Added test case to prevent regression of chained, preloaded scopes.
| * | | | | | | Added test case to prevent regression of chained, preloaded scopes. (#7490)Chris Geihsler2013-01-192-0/+10
| | | | | | | |
* | | | | | | | Refactor grep call to remove .eachCarlos Antonio da Silva2013-01-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Grep already yields the matching keys to the given block.
* | | | | | | | Add missing assert callsCarlos Antonio da Silva2013-01-201-2/+2
| | | | | | | |
* | | | | | | | Use 1.9 hash style in docs/comments [ci skip]Carlos Antonio da Silva2013-01-201-3/+3
| | | | | | | |
* | | | | | | | Fix markdown formatting to highlight block in changelog [ci skip]Carlos Antonio da Silva2013-01-201-2/+2
| | | | | | | |
* | | | | | | | Merge pull request #9008 from ranjaykrishna/dbcreateGuillermo Iguaran2013-01-201-0/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Added a new rake task in activerecord
| * | | | | | | | added db:drop task that executes mysql:drop_databses and ↵Ranjay Krishna2013-01-201-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | postgresql:drop_databases
* | | | | | | | | Merge pull request #9006 from wangjohn/activerecord_transaction_stateAaron Patterson2013-01-204-11/+49
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Created state for a transaction and added tests.
| * | | | | | | | | Added a changelog entry.wangjohn2013-01-201-0/+5
| | | | | | | | | |
| * | | | | | | | | Removed reflects_transaction_state.wangjohn2013-01-201-1/+0
| | | | | | | | | |
| * | | | | | | | | Created state for a transaction and added tests.wangjohn2013-01-203-11/+45
| | | | | | | | | |
* | | | | | | | | | Merge pull request #9005 from ranjaykrishna/dbcreateAaron Patterson2013-01-201-0/+4
|\ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / | |/| | | | | | | | added a db create rake task.
| * | | | | | | | | removed the puts statement and simplified the execute statementRanjay Krishna2013-01-201-6/+1
| | | | | | | | | |
| * | | | | | | | | added rake db:create that executes mysql:build_databases and ↵Ranjay Krishna2013-01-201-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | postgresql:build_databases
* | | | | | | | | | Merge pull request #9004 from arunagw/warning_removed_shadowing_outerGuillermo Iguaran2013-01-201-3/+3
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Removing warning: shadowing outer local variable
| * | | | | | | | | | Removing warning: shadowing outer local variable Arun Agrawal2013-01-201-3/+3
|/ / / / / / / / / /
* | | | / / / / / / strong parameters filters permitted scalarsXavier Noria2013-01-204-39/+273
| |_|_|/ / / / / / |/| | | | | | | |
* | | | | | | | | Don't use anonymous classes in connection testsGuillermo Iguaran2013-01-191-3/+5
| |/ / / / / / / |/| | | | | | |
* | | | | | | | Anonymous class is not allowed. For reference see #8934 and bc43763247e25Guillermo Iguaran2013-01-191-4/+6
| | | | | | | |
* | | | | | | | Merge pull request #9002 from dahakawang/masterAaron Patterson2013-01-192-0/+15
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | issue #8934 sovled
| * | | | | | | fix anonymous class issueDavid2013-01-202-0/+15
|/ / / / / / /
* | | | | | | Remove warning: shadowing outer local variableCarlos Antonio da Silva2013-01-191-1/+1
| | | | | | |
* | | | | | | Fix failing test about date.order translation valuesCarlos Antonio da Silva2013-01-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduced in b718998f3e8e6f3d01ad138e40f08fbcc3736285.
* | | | | | | Merge pull request #8978 from chrismcg/remove_i18n_symbol_dependencyAaron Patterson2013-01-193-6/+12
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Remove i18n symbol dependency
| * | | | | | | Don't change the original i18n dataChris McGrath2013-01-171-1/+1
| | | | | | | |
| * | | | | | | Remove i18n symbol dependencyChris McGrath2013-01-173-6/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | date.order is the only key in rails i18n that is required to be a symbol. This patch allows for symbols or strings which means: * No requirement for symbol type in .yml files. A future YAML.safe_load wouldn't need to load symbols * Rails could actually use json rather than yml as the backend
* | | | | | | | Merge pull request #8081 from JoeyButler/refactor_cache_storeRafael Mendonça França2013-01-191-25/+41
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Refactor cache store
| * | | | | | | | Refactor ActiveSupport::Cache::Store#fetch.Joey Butler2013-01-191-25/+41
| | | | | | | | |
* | | | | | | | | Merge pull request #9000 from sherwu/masterRafael Mendonça França2013-01-191-1/+1
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | Updated Gemfile to use jQuery-rails v 2.2.0
| * | | | | | | | Updated version of jQuery rails in gemfileSherwin Wu2013-01-191-1/+1
|/ / / / / / / /
* | | | | | | | Restore and adapt the implementation reverted atRafael Mendonça França2013-01-195-37/+114
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/rails/rails/commit/cc1c3c5be061e7572018f734e5239750ab449e3f Now instead of raise, we log by default in development and test
* | | | | | | | Merge pull request #8999 from thomasfedb/raise_or_log_unpermitted_paramsRafael Mendonça França2013-01-194-6/+117
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Raise or log unpermitted params.
| * | | | | | | | Added ability to raise or log on unpermitted params.Thomas Drake-Brockman2013-01-204-6/+117
|/ / / / / / / /
* | | | | | | | Improve relation docs about to_sql and where_values_hashCarlos Antonio da Silva2013-01-191-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * User class instead of Users. * #where_values_hash does not change the value to downcase as the example was showing. [ci skip]
* | | | | | | | Merge pull request #8977 from Soylent/masterAaron Patterson2013-01-182-1/+5
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Resovled issue rails#7774
| * | | | | | | | Fix ActionDispatch::Request#formats when HTTP_ACCEPT header is empty stringKonstantin Papkovskiy2013-01-172-1/+5
| | |_|_|_|_|/ / | |/| | | | | |
* | | | | | | | Fix syntax error :grin:Guillermo Iguaran2013-01-181-1/+1
| | | | | | | |
* | | | | | | | Merge pull request #8995 from schneems/schneems/formatting-the-formattersGuillermo Iguaran2013-01-181-6/+13
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Remove "Application" section title from routes
| * | | | | | | | Remove "Application" section title from routesschneems2013-01-181-6/+13
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR standardizes the output of the HTML and console based routes to not include the title for "Application Routes" those that are defined by the application. Instead only routes defined in engines get any special treatment. Based on this conversation: https://github.com/rails/rails/commit/af5c0fd85fce1adb311083dd1ecf96432ee8caa3#commitcomment-2458823 ATP actionpack /cc @carlosantoniodasilva
* | | | | | | | Merge pull request #8994 from Springest/fix_default_scope_update_all_delete_allJon Leighton2013-01-183-2/+33
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix .update_all and .delete_all when using a condition on a joined table in a default_scope
| * | | | | | | | Fix .update_all and .delete_all when using a condition on a joined tableDerek Kraan2013-01-113-2/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | in a default_scope. `Model.joins(...).where(condition_on_joined_table).update_all` / `delete_all` worked, but the same operation implemented with a default_scope generated a SQL error because ActiveRecord ignored the join but implemented the where condition anyways.
generated by cgit v1.2.3 (git 2.25.1) at 2025-03-22 19:57:49 +0000