Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | | | | | | Merge pull request #20226 from EpicH0liday/reversible-remove-foreign-key | Yves Senn | 2015-06-12 | 4 | -2/+50 | |
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make remove_foreign_key reversible Conflicts: activerecord/CHANGELOG.md | |||||
| * | | | | | | | | | | Add an invert method for remove_foreign_key | Aster Ryan | 2015-06-11 | 4 | -2/+51 | |
| | | | | | | | | | | | ||||||
* | | | | | | | | | | | Merge pull request #19813 from igas/enum-prefix | Sean Griffin | 2015-06-12 | 6 | -6/+104 | |
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | Add prefix option to enum definition | |||||
| * | | | | | | | | | | | Add enum prefix/suffix option to enum definition | Igor Kapkov | 2015-06-12 | 6 | -6/+104 | |
| | |_|_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #17511 and #17415 | |||||
* | | | | | | | | | | | Merge pull request #20526 from y-yagi/fix_doc_about_foreign_key_name | Abdelkader Boudih | 2015-06-12 | 1 | -1/+2 | |
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | fix doc about foreign key name [ci skip] | |||||
| * | | | | | | | | | | | fix doc about foreign key name [ci skip] | yuuji.yaginuma | 2015-06-12 | 1 | -1/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this behavior was changed in b8e1f202676b4788c56241b124c401beff9f4014 | |||||
* | | | | | | | | | | | | Merge pull request #20529 from pwnall/kwargs_docs | Rafael Mendonça França | 2015-06-12 | 1 | -1/+1 | |
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / |/| | | | | | | | | | | | Update RDoc for ActionController::TestCase for kwargs. | |||||
| * | | | | | | | | | | | Update RDoc for ActionController::TestCase for kwargs. | Victor Costan | 2015-06-12 | 1 | -1/+1 | |
|/ / / / / / / / / / / | ||||||
* | | | | | | | | | | | Merge pull request #20058 from farukaydin/cache-digests-dependency-rake | Rafael Mendonça França | 2015-06-11 | 2 | -1/+22 | |
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | add error log that notify 'file not found' when using cache_digest dependency rake | |||||
| * | | | | | | | | | | | add error log that notify 'file not found' when using ↵ | Faruk AYDIN | 2015-05-07 | 2 | -1/+22 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cache_digests:dependency and cache_digests:nested_dependency tasks | |||||
* | | | | | | | | | | | | Fix failing tests for Rails-API | Sean Griffin | 2015-06-11 | 1 | -1/+1 | |
| |_|/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This looks to be just out of sync tests | |||||
* | | | | | | | | | | | Merge pull request #20522 from colby-swandale/doc-update | Sean Griffin | 2015-06-11 | 1 | -1/+1 | |
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | fixed sring to be string in ActiveRecord::Base params documentation [ci skip] | |||||
| * | | | | | | | | | | | fixed sring to be string in ActiveRecord::Base params documentation | Colby Swandale | 2015-06-12 | 1 | -1/+1 | |
| | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | assigns was removed from Rails | Rafael Mendonça França | 2015-06-11 | 1 | -1/+0 | |
| | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | Add a missing test case for the persistence behavior of `serialize` | Sean Griffin | 2015-06-11 | 1 | -0/+13 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `serialize` makes the contract that if it is given a class name, it will never return something other than an instance of that class. This means that it must cast `nil` to the empty form of that object. As such, we should then persist empty forms of that object as `nil`. While this is techincally under the contract of ``` model.attribute = value assert_equal model.attribute, model.tap(&:save).reload.attribute ``` which we can't actually test universally without property based testing, it has come up more than once and is worth calling out specifically since we aren't looking to change it. | |||||
* | | | | | | | | | | | | Credit the author of #20515 in the previous commit | Sean Griffin | 2015-06-11 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [Sean Griffin & jmondo] | |||||
* | | | | | | | | | | | | Correctly handle array columns with defaults in the schema dumper | Sean Griffin | 2015-06-11 | 4 | -0/+17 | |
| |_|/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the subtype provides custom schema dumping behavior, we need to defer to it. We purposely choose not to handle any values other than an array (which technically should only ever be `nil`, but I'd rather code defensively here). Fixes #20515. | |||||
* | | | | | | | | | | | Merge pull request #19832 from spastorino/rails-api | Santiago Pastorino | 2015-06-11 | 45 | -32/+1497 | |
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | Rails api | |||||
| * | | | | | | | | | | | head :no_content is implicitly called | Santiago Pastorino | 2015-06-11 | 1 | -2/+0 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Mention that doing nothing in Rails API controllers returns 204 | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Return 204 if render is not called in API controllers | Jorge Bejar | 2015-06-11 | 5 | -6/+31 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Add test coverage for implicit render in empty actions | Jorge Bejar | 2015-06-11 | 1 | -0/+14 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Checking if controller responds to wrap_parameter is not longer required | Jorge Bejar | 2015-06-11 | 1 | -1/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Enable wrap_parameter by default in rails api applications | Jorge Bejar | 2015-06-11 | 2 | -5/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Include ParamsWrapper in AC::API | Jorge Bejar | 2015-06-11 | 2 | -1/+31 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ParamsWrapper was initially removed from API controllers according to the following discusision: https://github.com/rails-api/rails-api/issues/33 However, we're including it again so Rails API devs can decide whether to enable or disable it. | |||||
| * | | | | | | | | | | | Add rake-cors gem (commented) in Gemfile for rails api apps | Jorge Bejar | 2015-06-11 | 3 | -0/+20 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Make Rails API apps return the full resource on update | Santiago Pastorino | 2015-06-11 | 2 | -2/+2 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | It's rails new my_api --api | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Change guide heading from - to = | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Add CHANGELOG entries for API apps functionality | Santiago Pastorino | 2015-06-11 | 2 | -0/+13 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Add API only apps guide | Santiago Pastorino | 2015-06-11 | 1 | -0/+435 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Document Generators.api_only! method | Santiago Pastorino | 2015-06-11 | 1 | -0/+4 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | http only => API only | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Add AMS 0.10.0.rc1 by default for api apps | Santiago Pastorino | 2015-06-11 | 2 | -0/+6 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Fix scaffold generator test for resource routes | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Fix class_option description for api generators | Santiago Pastorino | 2015-06-11 | 2 | -2/+2 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Remove unneeded option from ResourceRouteGenerator | Santiago Pastorino | 2015-06-11 | 1 | -3/+0 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Fix MimeResponds example in AC::API documentation | Santiago Pastorino | 2015-06-11 | 1 | -3/+5 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Revert changes related with api apps in RouteWrapper | Jorge Bejar | 2015-06-11 | 1 | -11/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See the following commit to have context about this change: https://github.com/rails/rails/commit/757a2bc3e3e52a5d9418656928db993db42b741b | |||||
| * | | | | | | | | | | | Rename test methods in api conditional get controller tests | Jorge Bejar | 2015-06-11 | 1 | -2/+2 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Do not say that Api Controllers are faster than regular ones in docs | Jorge Bejar | 2015-06-11 | 1 | -4/+4 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Api only apps should include tmp and vendor folders | Jorge Bejar | 2015-06-11 | 2 | -2/+6 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Remove Compatibility module since we don't remember why it was added :smile: | Jorge Bejar | 2015-06-11 | 2 | -20/+0 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Exclude cache_digests:dependencies rake task in api app | Jorge Bejar | 2015-06-11 | 1 | -2/+4 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Routes resources avoid :new and :edit endpoints if api_only is enabled | Jorge Bejar | 2015-06-11 | 5 | -21/+111 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Remove extra whitespaces | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Use new hash syntax | Santiago Pastorino | 2015-06-11 | 3 | -8/+8 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Refactor internal? to query internal_controller? and internal_asset? methods | Santiago Pastorino | 2015-06-11 | 1 | -1/+11 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Remove Unneeded ApiPublicExceptions middleware, PublicExceptions already ↵ | Santiago Pastorino | 2015-06-11 | 3 | -84/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | does the work | |||||
| * | | | | | | | | | | | Remove api_rendering is not needed | Santiago Pastorino | 2015-06-11 | 3 | -19/+1 | |
| | | | | | | | | | | | |