aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #13367 from abhishekjain16/new_branchYves Senn2013-12-181-1/+1
|\ | | | | | | | | change to DateTime.civil_from_format from DateTime.civil_from_fromat in ... [ci skip]
| * change to DateTime.civil_from_format from DateTime.civil_from_fromat in ↵abhishek2013-12-181-1/+1
| | | | | | | | changelog [ci skip]
* | Merge pull request #13340 from akshay-vishnoi/typoGodfrey Chan2013-12-181-5/+4
|\ \ | | | | | | #none documentation updated [ci skip]
| * | #none documentation updated [ci skip]Akshay Vishnoi2013-12-181-5/+4
| | |
* | | Use annotated git tags for release taskJeremy Kemper2013-12-182-2/+2
| | |
* | | we should use annotated tags for releases.Yves Senn2013-12-182-2/+2
| | | | | | | | | | | | Annotated tags will provide us with Tagger and Time information.
* | | Merge pull request #13363 from kuldeepaggarwal/f-video-optionsGuillermo Iguaran2013-12-172-5/+11
|\ \ \ | | | | | | | | allow video_tag to accept `size` as `Number` for square shaped videos
| * | | allow video_tag to accept `size` as `Number` for square shaped videosKuldeep Aggarwal2013-12-182-5/+11
| | | |
* | | | Merge pull request #13364 from corprew/patch-1Arun Agrawal2013-12-171-1/+1
|\ \ \ \ | | | | | | | | | | typo: should be 'DateTime.civil_from_format' and not 'DateTime.civil_from_fromat'
| * | | | typo: should be 'DateTime.civil_from_format' and not ↵Corprew Reed2013-12-171-1/+1
|/ / / / | | | | | | | | | | | | 'DateTime.civil_from_fromat'
* | | | Add a failing test for assigning nil to a polymorphic belongs_to not ↵Jeremy Kemper2013-12-171-0/+13
| | | | | | | | | | | | | | | | nullifying its _type column
* | | | Merge pull request #13358 from JuanitoFatas/patch/release-notesCarlos Antonio da Silva2013-12-171-7/+9
|\ \ \ \ | | | | | | | | | | Prettify the removals from Action Controller in 4.1 release notes. [ci skip]
| * | | | [ci skip] Prettify the removals from Action Controller in 4.1 release notes.Juanito Fatas2013-12-181-7/+9
|/ / / /
* | | | Its beta1 all aroundDavid Heinemeier Hansson2013-12-172-2/+2
| | | |
* | | | Fix dependency orderDavid Heinemeier Hansson2013-12-171-1/+1
| | | |
* | | | Get ready to release 4.1.0.beta1David Heinemeier Hansson2013-12-177-7/+7
| | | |
* | | | Remove the navigator for now. Its ugly and it promises more than it can ↵David Heinemeier Hansson2013-12-171-6/+1
| | | | | | | | | | | | | | | | deliver. These other views (properties, routes) are not designed to be loaded in isolation
* | | | Clearly limit new CSRF protection to GET requestsJeremy Kemper2013-12-172-2/+17
| | | |
* | | | Mention new CSRF protection gotcha in upgrade guideJeremy Kemper2013-12-171-1/+26
| | | |
* | | | Discuss Action Pack's new CSRF protection from remote script tags and what ↵Jeremy Kemper2013-12-171-132/+146
| | | | | | | | | | | | | | | | it means for your app. In short, update your tests if they don't already use the 'xhr :post' style.
* | | | Fix integration test to pass same-origin verificationJeremy Kemper2013-12-171-1/+1
| | | |
* | | | Merge pull request #13345 from jeremy/get-csrfJeremy Kemper2013-12-176-28/+143
|\ \ \ \ | | | | | | | | | | CSRF protection from cross-origin <script> tags
| * | | | CSRF protection from cross-origin <script> tagsJeremy Kemper2013-12-176-28/+143
| | | | | | | | | | | | | | | | | | | | Thanks to @homakov for sounding the alarm about JSONP-style data leaking
* | | | | Merge pull request #13314 from laurocaetano/blacklist_array_methodsJeremy Kemper2013-12-174-30/+59
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | Create a blacklist to disallow mutator methods to be delegated to Array Conflicts: guides/source/upgrading_ruby_on_rails.md
| * | | | Create a blacklist to disallow mutator methods to be delegated to `Array`.Lauro Caetano2013-12-174-30/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change was necessary because the whitelist wouldn't work. It would be painful for users trying to update their applications. This blacklist intent to prevent odd bugs and confusion in code that call mutator methods directely on the `Relation`.
* | | | | Add changelog entries about secrets.yml and secrets.secret_key_baseGuillermo Iguaran2013-12-171-0/+29
| | | | |
* | | | | Merge pull request #13346 from chancancode/release_notes_etcYves Senn2013-12-174-47/+147
|\ \ \ \ \ | | | | | | | | | | | | Release notes fixes [ci skip]
| * | | | | Some assorted fixes for the 4.1 release notes:Godfrey Chan2013-12-174-47/+147
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Added release notes for secrets.yml and mentioned it in the highlights * Added release notes for Mailer previews and mentioned it in the highlights * Added release notes for Module#concerning * Removed mention for AV extraction from the highlights * Rearranged the major features to put highlighted features first * Various improvements and typo fixes [ci skip]
* | | | | Merge pull request #13341 from carlosantoniodasilva/ca-i18nCarlos Antonio da Silva2013-12-1710-14/+94
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | Default I18n.enforce_available_locales to true We will default this option to true from now on to ensure users properly handle their list of available locales whenever necessary. This option was added as a security measure and thus Rails will follow it defaulting to secure option. Also improve the handling of I18n config options in its railtie, taking the new enforce_available_locales option into account, by setting it as the last one in the process. This ensures no other configuration will trigger a deprecation warning due to that setting.
| * | | | Require I18n >= 0.6.9Carlos Antonio da Silva2013-12-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The option enforce_available_locales is only available on latest versions, so require the last available one which has the option + other related fixes and should not have backward compatibility issues.
| * | | | Add changelog and upgrading notice related to I18n enforce_available_locales ↵Carlos Antonio da Silva2013-12-172-5/+30
| | | | | | | | | | | | | | | | | | | | handling
| * | | | Disable available locales checks to avoid warnings running the testsCarlos Antonio da Silva2013-12-175-0/+15
| | | | |
| * | | | Default I18n.enforce_available_locales to trueCarlos Antonio da Silva2013-12-172-8/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We will default this option to true from now on to ensure users properly handle their list of available locales whenever necessary. This option was added as a security measure and thus Rails will follow it defaulting to secure option. Also improve the handling of I18n config options in its railtie, taking the new enforce_available_locales option into account, by setting it as the last one in the process. This ensures no other configuration will trigger a deprecation warning due to that setting.
* | | | | concern doesn't include the module automaticallyRafael Mendonça França2013-12-171-1/+0
| | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | Merge pull request #13343 from akshay-vishnoi/test-casesRafael Mendonça França2013-12-171-0/+4
|\ \ \ \ \ | | | | | | | | | | | | test case for #limit added - picking latest value from limit
| * | | | | test case for #limit added - picking latest value from limitAkshay Vishnoi2013-12-171-0/+4
| | | | | |
* | | | | | release notes link to fixed versions of the API. [ci skip]Yves Senn2013-12-172-2/+2
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | As discussed with @fxn the release notes are a snapshot document. The links going out to the API should represent that exact snapshot. This means we always link to the full final release. For example the 3.2 release notes link to http://api.rubyonrails.org/v3.2.0.
* | | | | Fix indent on AS changelog [ci skip]Carlos Antonio da Silva2013-12-171-18/+18
| | | | |
* | | | | minor doc reword. Upgrade Rails itself not to Rails. [ci skip]Yves Senn2013-12-171-3/+3
| |/ / / |/| | | | | | | | | | | /cc @chancancode
* | | | Merge pull request #13347 from jeremy/concerningJeremy Kemper2013-12-174-0/+219
|\ \ \ \ | | | | | | | | | | Introduce Module#concerning
| * | | | Introduce Module#concerningJeremy Kemper2013-12-174-0/+219
|/ / / / | | | | | | | | | | | | | | | | | | | | A natural, low-ceremony way to separate responsibilities within a class. Imported from https://github.com/37signals/concerning#readme
* | | | MySQL: remove the old-school 'packets out of order' messageJeremy Kemper2013-12-171-6/+0
| | | | | | | | | | | | | | | | Blast from the past, MySQL 4 era, when the password hashing style changed.
* | | | use `bin/spring` in release notes. [ci skip]Yves Senn2013-12-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow up to: https://github.com/rails/rails/commit/828a8f214535e59d709fd4862605902d1cc21632#commitcomment-4879462 This will be available after https://github.com/jonleighton/spring/commit/c6e25804b3338959d87a29f40967a333c650c031 is released.
* | | | Merge pull request #13332 from rails/pixeltrix/mail_viewGuillermo Iguaran2013-12-1621-26/+765
|\ \ \ \ | | | | | | | | | | WIP: Integration of mail_view gem
| * | | | Add mailer previews feature based on mail_view gemAndrew White2013-12-1717-6/+736
| | | | |
| * | | | Add an application controller for internal controllersAndrew White2013-12-173-18/+23
| | | | |
| * | | | Fix method redefinition warningAndrew White2013-12-171-2/+6
|/ / / /
* | | | Merge pull request #12873 from martinemde/fix-where-assoc-nilCarlos Antonio da Silva2013-12-163-3/+104
|\ \ \ \ | |/ / / |/| | | Better support for AR:B.where(belongs_to_assoc: val) which previously generated invalid SQL
| * | | Better support for `where()` conditions that use an association name.Martin Emde2013-12-163-3/+104
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using the name of an association in `where` previously worked only if the value was a single `ActiveRecrd::Base` object. e.g. Post.where(author: Author.first) Any other values, including `nil`, would cause invalid SQL to be generated. This change supports arguments in the `where` query conditions where the key is a `belongs_to` association name and the value is `nil`, an `Array` of `ActiveRecord::Base` objects, or an `ActiveRecord::Relation` object. # Given the Post model class Post < ActiveRecord::Base belongs_to :author end # nil value finds records where the association is not set Post.where(author: nil) # SELECT "posts".* FROM "posts" WHERE "posts"."author_id" IS NULL # Array values find records where the association foreign key # matches the ids of the passed ActiveRecord models, resulting # in the same query as Post.where(author_id: [1,2]) authors_array = [Author.find(1), Author.find(2)] Post.where(author: authors_array) # ActiveRecord::Relation values find records using the same # query as Post.where(author_id: Author.where(last_name: "Emde")) Post.where(author: Author.where(last_name: "Emde")) Polymorphic `belongs_to` associations will continue to be handled appropriately, with the polymorphic `association_type` field added to the query to match the base class of the value. This feature previously only worked when the value was a single `ActveRecord::Base`. class Post < ActiveRecord::Base belongs_to :author, polymorphic: true end Post.where(author: Author.where(last_name: "Emde")) # Generates a query similar to: Post.where(author_id: Author.where(last_name: "Emde"), author_type: "Author")
* | | Remove the link for code.whytheluckystiff.netRafael Mendonça França2013-12-161-1/+1
| | | | | | | | | | | | | | | This is not a valid URL. [ci skip]