aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | Use join without default separatorPiotr Niełacny2012-07-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Use lstrip method
* | | | | | | Don't assume resource param is :id when using shallow routesAndrew White2012-07-202-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since #5581 added support for resources with custom params we should not assume that it is :id when using shallow resource routing.
* | | | | | | Support constraints on resource custom params when nestingAndrew White2012-07-202-9/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Mapper looks for a :id constraint in the scope to see whether it should apply a constraint for nested resources. Since #5581 added support for resource params other than :id, we need to check for a constraint on the parent resource's param name and not assume it's :id.
* | | | | | | adding Rails::Queueing::ContainerAaron Patterson2012-07-194-6/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows us to do: In your configuration: Rails.queue[:image_queue] = SomeQueue.new Rails.queue[:mail_queue] = SomeQueue.new In your app code: Rails.queue[:mail_queue].push MailJob.new Both jobs pushed to the same default queue Rails.queue.push DefaultJob.new Rails.queue[:default].push DefaultJob.new
* | | | | | | Merge pull request #7078 from kennyj/logging_query_planRafael Mendonça França2012-07-192-2/+14
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Log query plan when we use count_by_sql method.
| * | | | | | | Log query plan when we use count_by_sql method.kennyj2012-07-182-2/+14
| | | | | | | |
* | | | | | | | Create default join table name using a SymbolCarlos Antonio da Silva2012-07-191-1/+1
| |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | Lets stick with a symbol in join table name, it was removed in 993e1643650b7b0939437f86b7b4cda0d92b0b8f and broke the build.
* | | | | | | Use attributes instead of table names in migration exampleJosé Valim2012-07-191-1/+1
| | | | | | |
* | | | | | | Merge pull request #7097 from lexmag/join_table_gen_attrJosé Valim2012-07-185-8/+24
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | Add fkey attributes to `join_table` migration generator
| * | | | | | Add fkey attributes to `join_table` migration generatorAleksey Magusev2012-07-195-8/+24
| | | | | | |
* | | | | | | Revert "Merge pull request #7084 from LTe/logger_default_separator"Aaron Patterson2012-07-182-14/+3
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit c08f30ff5fcda7e07cd9275a073acb2091e4b3f7, reversing changes made to e243a8a32eb4c8777f07ca4b974bd7e38d9477d3.
* | | | | | Merge pull request #7095 from davekaro/masterRafael Mendonça França2012-07-183-7/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | revert Default timestamps to non-null
| * | | | | | revert Default timestamps to non-nullDave Kroondyk2012-07-183-7/+7
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 3dbedd2 added NOT NULL constraints to timestamps. Commit fcef728 started to revert this, but was incomplete. With this commit, 3dbedd2 should be fully reverted and timestamps will no longer default to NOT NULL.
* | | | | | Merge pull request #6614 from carlosgaldino/data-confirmRafael Mendonça França2012-07-1811-74/+58
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove `:confirm` in favor of `:data => { :confirm => 'Text' }` option
| * | | | | | Remove `:confirm` in favor of `:data => { :confirm => 'Text' }` optionCarlos Galdino2012-07-1811-74/+58
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This applies to the following helpers: `button_to` `button_tag` `image_submit_tag` `link_to` `submit_tag`
* | | | | | Merge pull request #7084 from LTe/logger_default_separatorJosé Valim2012-07-182-3/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | Don't use default separator in tagged logger
| * | | | | | Don't use default separatorPiotr Niełacny2012-07-182-3/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | When the default separator is set logger will create incorrect output
* | | | | | | Update changelog for migration generator changeJosé Valim2012-07-181-0/+7
| | | | | | |
* | | | | | | Merge pull request #7082 from jalkoby/lazy_lookup_in_abstract_controllerJosé Valim2012-07-182-1/+20
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | add lazy look up in abstract controller's translate method
| * | | | | | add lazy look up in abstract controller's translate methodSergey Pchelincev2012-07-182-1/+20
|/ / / / / /
* | | | | | Merge pull request #7028 from lexmag/join_table_indexesJosé Valim2012-07-187-16/+66
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add indexes to create_join_table method
| * | | | | | Add join table migration generatorAleksey Magusev2012-07-187-16/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For instance, running rails g migration CreateMediaJoinTable artists musics:uniq will create a migration with create_join_table :artists, :musics do |t| # t.index [:artist_id, :music_id] t.index [:music_id, :artist_id], unique: true end
* | | | | | | Merge pull request #6626 from route/render_formatsJosé Valim2012-07-1712-4/+33
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Add additional formats to lookup context
| * | | | | | | Common behavior with adding formats to lookup_context for TemplateRenderer ↵Dmitry Vorotilin2012-07-1812-4/+33
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | and PartialRenderer
* | | | | | | Merge pull request #7081 from jfirebaugh/to_param_regression_masterJon Leighton2012-07-172-1/+7
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | AR::Integration must be included after AM::Conversion
| * | | | | | | AR::Integration must be included after AM::ConversionJohn Firebaugh2012-07-172-1/+7
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Integration's definition of #to_param must override Conversion's. Otherwise, there is a regression from 3.1 in the behavior of a non-persisted AR::Base instance which nevertheless has an id.
* | / / / / / Add support for optional root segments containing slashesAndrew White2012-07-172-1/+28
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Optional segments with a root scope need to have the leading slash outside of the parentheses, otherwise the generated url will be empty. However if the route has non-optional elements then the leading slash needs to remain inside the parentheses otherwise the generated url will have two leading slashes, e.g: Blog::Application.routes.draw do get '/(:category)', :to => 'posts#index', :as => :root get '/(:category)/author/:name', :to => 'posts#author', :as => :author end $ rake routes root GET /(:category)(.:format) posts#index author GET (/:category)/author/:name(.:format) posts#author This change adds support for optional segments that contain a slash, allowing support for urls like /page/2 for the root path, e.g: Blog::Application.routes.draw do get '/(page/:page)', :to => 'posts#index', :as => :root end $ rake routes root GET /(page/:page)(.:format) posts#index Fixes #7073
* | | | | | Merge pull request #7076 from kennyj/fix_class_evalAaron Patterson2012-07-175-6/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix class_eval without __FILE__ and __LINE__.
| * | | | | | Fix class_eval without __FILE__ and __LINE__.kennyj2012-07-185-6/+6
| | | | | | |
* | | | | | | Merge pull request #7072 from beerlington/fix_validations_with_scopeJon Leighton2012-07-172-1/+17
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Use database value for uniqueness validation scope
| * | | | | | Fixes "Cannot visit ..." with validates_uniqueness_ofbeerlington2012-07-172-1/+17
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes issue with overrding ActiveRecord reader methods with a composed object and using that attribute as the scope of a validates_uniqueness_of validation.
* | | | | | Merge pull request #7006 from jaredbeck/issue_7005_explain_asset_debugPiotr Sarnacki2012-07-161-1/+1
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | A more informative comment for config.asset.debug
| * | | | | A more informative comment for config.asset.debugJared Beck2012-07-161-1/+1
| | | | | |
* | | | | | Merge pull request #7066 from wkang/masterRafael Mendonça França2012-07-161-1/+1
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Fix activerecord model to_ary method comment 'see also' link
| * | | | | Fix activerecord model to_ary method comment 'see also' linkKang Wen2012-07-161-1/+1
|/ / / / /
* | | | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-07-1511-38/+218
|\ \ \ \ \
| * | | | | session_id column is string in defaultkennyj2012-07-151-1/+1
| | | | | |
| * | | | | add note about requiring gem dependencies before initialization [ci skip]Francesco Rodriguez2012-07-131-6/+28
| | | | | |
| * | | | | Added note about feature requests and rails-core list.Steve Klabnik2012-07-131-0/+2
| | | | | |
| * | | | | Updated with_routing example to 3.x syntax [ci skip]Wojciech Wnętrzak2012-07-131-7/+4
| | | | | |
| * | | | | Change Assertion language to Expected/Actual from obj1/obj2Scott Stewart2012-07-121-5/+5
| | | | | |
| * | | | | Fixes typo in CommentsController#create example Andy Lindeman2012-07-111-1/+1
| | | | | | | | | | | | | | | | | | * comments.build will not actually persist the comment
| * | | | | Fix typoChun-wei Kuo2012-07-111-1/+1
| | | | | |
| * | | | | [getting started] update a code listingSimon Hengel2012-07-101-0/+1
| | | | | |
| * | | | | use 'use_zone' method in example, instead of reimplementing itJakub Kuźma2012-07-101-5/+5
| | | | | |
| * | | | | Add info note about sass/coffeescript use in engine assets to guideMarten Veldthuis2012-07-091-0/+2
| | | | | |
| * | | | | Fixed second assert_match - previous one didn't make any sense (if the first ↵Jakub Kuźma2012-07-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | passed, the second passed as well)
| * | | | | It's better to pass strings to assert_match - it converts them to regular ↵Jakub Kuźma2012-07-091-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | expressions, escaping special chars like '.'. It seems email.encoded contains some unexpected line breaks (due to encoding), it's better to assert_match email.body.to_s.
| * | | | | Revert "Select missing formbuilder typo."Dan Patrick2012-07-081-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit bf97795e7327e15774666d2197f1c0b8ada57a84.
| * | | | | Select missing formbuilder typo.Dan Patrick2012-07-081-2/+2
| | | | | |
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-09 07:42:52 +0000