Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | | | | | | | | | | Remove unneeded merge with default false options | Akira Matsuda | 2011-05-16 | 1 | -4/+0 | |
| | | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | | Don't add DEFAULT_TOKENIZER to every instance of LengthValidator. Just use ↵ | Akira Matsuda | 2011-05-16 | 1 | -2/+2 | |
|/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | it only when needed. | |||||
* | | | | | | | | | | | | Merge pull request #573 from smartinez87/attr | José Valim | 2011-05-15 | 1 | -0/+3 | |
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | Add doc to #attribute_names | |||||
| * | | | | | | | | | | | | Add doc to #attribute_names | Sebastian Martinez | 2011-05-15 | 1 | -0/+3 | |
|/ / / / / / / / / / / / | ||||||
* | | | | | | | | | | | | Merge pull request #570 from sikachu/decouple_actionpack | José Valim | 2011-05-15 | 6 | -14/+35 | |
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | Make ParamsWrapper use a well-defined API and not rely on AR methods | |||||
| * | | | | | | | | | | | | Make ParamsWrapper calling newly introduced `Model.attribute_names` instead ↵ | Prem Sichanugrist | 2011-05-15 | 3 | -14/+12 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | of `.column_names` | |||||
| * | | | | | | | | | | | | Add ActiveRecord::attribute_names to retrieve a list of attribute names. ↵ | Prem Sichanugrist | 2011-05-15 | 3 | -0/+23 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This method will also return an empty array on an abstract class or a model that the table doesn't exists. | |||||
* | | | | | | | | | | | | | Merge pull request #568 from carlosantoniodasilva/fields_for_fixes | José Valim | 2011-05-15 | 2 | -36/+40 | |
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | | | Fields for fixes for non nested attributes and without object | |||||
| * | | | | | | | | | | | | Use record_name to make fields for methods more consistent | Carlos Antonio da Silva | 2011-05-14 | 1 | -6/+6 | |
| | | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | | Refactor fields_for to make the api more clear, and fix usage with non ↵ | Carlos Antonio da Silva | 2011-05-14 | 2 | -26/+33 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nested attributes and without object | |||||
| * | | | | | | | | | | | | Cleanup instantiate builder method definition | Carlos Antonio da Silva | 2011-05-14 | 1 | -5/+2 | |
| | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-05-15 | 2 | -8/+8 | |
|\ \ \ \ \ \ \ \ \ \ \ \ \ | ||||||
| * | | | | | | | | | | | | | Use more English language | Pirogov Evgenij | 2011-05-15 | 1 | -4/+4 | |
| | | | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | | | Fixing Changelog link | Guillermo Iguaran | 2011-05-15 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | | | Clean up redundant backslashes | Pirogov Evgenij | 2011-05-14 | 1 | -4/+4 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | reviewed the first ~70 lines of the configuration guide | Xavier Noria | 2011-05-15 | 1 | -20/+21 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | Merge pull request #564 from sikachu/fix_wrapper | Aaron Patterson | 2011-05-14 | 2 | -2/+15 | |
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / |/| | | | | | | | | | | | | | Do not try to call `column_names` on the abstract class. | |||||
| * | | | | | | | | | | | | | Do not try to call `column_names` on the abstract class. | Prem Sichanugrist | 2011-05-15 | 2 | -2/+15 | |
|/ / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Normally the table for abstract class won't be existed, so we should not trying to call `#column_names` on it. | |||||
* | | | | | | | | | | | | | These extra array operations appear to be unnecessary. Reasoning: | Jon Leighton | 2011-05-14 | 1 | -8/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * It is not necessary to subtract 'id' from the list of copied attributes because record and mem_record are equal, so therefore their id attributes are also equal (so there is no harm in copying it, and this reduces the complexity of the code) * It is not necessary to intersect the attribute names, since record and mem_record are equal, so they have the same id and class, so they have the same columns in the database. If record has non-column attributes then it seems reasonable to also copy them onto mem_record (though I am not sure what situation this would ever happen in) | |||||
* | | | | | | | | | | | | | CollectionAssociation#merge_target_lists should write to the underlying ↵ | Jon Leighton | 2011-05-14 | 4 | -5/+17 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | attributes when copying, rather than using the assignment method | |||||
* | | | | | | | | | | | | | An attempt to make CollectionAssociation#merge_target_lists make more sense. | Jon Leighton | 2011-05-14 | 1 | -16/+35 | |
| |/ / / / / / / / / / / |/| | | | | | | | | | | | ||||||
* | | | | | | | | | | | | Added some docs to controller runtime. | José Valim | 2011-05-14 | 1 | -1/+4 | |
| | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | Merge pull request #554 from ↵ | José Valim | 2011-05-14 | 2 | -1/+19 | |
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tomafro/reset-activerecord-runtime-before-each-request ActiveRecord::LogSubscriber.runtime should be reset at the start of each request | |||||
| * | | | | | | | | | | | Reset ActiveRecord::LogSubscriber runtime at the start of each request | Tom Ward | 2011-05-14 | 2 | -0/+18 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the runtime was reset implicitly when #cleanup_view_runtime was called at the end of most requests. However, this doesn't happen when the request redirects, or send_file is called. Consequently, the ActiveRecord runtime recorded in the logs included the time taken for both the current request and the previous redirect. Explicitly resetting at the start of each request ensures that this can't happen, no matter what occurs previously. | |||||
| * | | | | | | | | | | | Escape regex in controller_runtime_test to actually check that the ↵ | Tom Ward | 2011-05-14 | 1 | -1/+1 | |
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ActiveRecord message appears | |||||
* | | | | | | | | | | | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-05-14 | 42 | -220/+319 | |
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/date_helper.rb railties/lib/rails/generators/rails/app/templates/config/initializers/wrap_parameters.rb.tt | |||||
| * | | | | | | | | | | | minor edits after going through what's new in docrails | Xavier Noria | 2011-05-14 | 3 | -10/+13 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | [GUIDES] Changed the `LANGUAGE` variable to `GUIDES_LANGUAGE` to minimize ↵ | Karel Minarik | 2011-05-14 | 1 | -4/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | conflicts with regular environment variables (eg. in Ubuntu) | |||||
| * | | | | | | | | | | | minor corrections | Vijay Dev | 2011-05-14 | 1 | -5/+5 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Link the API documentation to non-edge instead | Prem Sichanugrist | 2011-05-13 | 1 | -1/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Adding guide for wrapping JSON/XML parameters, which also links to the API ↵ | Prem Sichanugrist | 2011-05-13 | 1 | -0/+26 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | documentation. Note that it currently links to http://edgeapi.rubyonrails.org because it's Rails 3.1 feature. | |||||
| * | | | | | | | | | | | Prefer topic branches instead of master branch for users contributions | Guillermo Iguaran | 2011-05-12 | 1 | -5/+8 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | added an example for AC::UrlFor usage to make usage simpler. | Nick Sutterer | 2011-05-12 | 1 | -1/+13 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | added docs for AbC::UrlFor and AC::UrlFor. | Nick Sutterer | 2011-05-12 | 2 | -0/+12 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | fix spellings | Vijay Dev | 2011-05-11 | 2 | -2/+2 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | indentation fixes | Vijay Dev | 2011-05-11 | 1 | -24/+23 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2011-05-11 | 11 | -92/+98 | |
| |\ \ \ \ \ \ \ \ \ \ \ | ||||||
| | * | | | | | | | | | | | fix bad encoding on contributing to RoR page | Dan Pickett | 2011-05-10 | 1 | -1/+1 | |
| | | | | | | | | | | | | | ||||||
| | * | | | | | | | | | | | Follow code conventions on docs | Sebastian Martinez | 2011-05-10 | 5 | -13/+13 | |
| | | | | | | | | | | | | | ||||||
| | * | | | | | | | | | | | Fix styiling issues on DateHelper docs | Sebastian Martinez | 2011-05-10 | 1 | -64/+70 | |
| | | | | | | | | | | | | | ||||||
| | * | | | | | | | | | | | Fix styiling issues on AtomFeedHelper docs | Sebastian Martinez | 2011-05-10 | 1 | -4/+4 | |
| | | | | | | | | | | | | | ||||||
| | * | | | | | | | | | | | Remove extra whitespace. | Sebastian Martinez | 2011-05-10 | 1 | -1/+1 | |
| | | | | | | | | | | | | | ||||||
| | * | | | | | | | | | | | Fix styiling issue on TranslationHelper docs | Sebastian Martinez | 2011-05-10 | 1 | -4/+4 | |
| | | | | | | | | | | | | | ||||||
| | * | | | | | | | | | | | Fix styiling issue on ActionView::Template docs | Sebastian Martinez | 2011-05-10 | 1 | -5/+5 | |
| | | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | | minor fix in assets usage | Vijay Dev | 2011-05-10 | 1 | -2/+2 | |
| |/ / / / / / / / / / / | ||||||
| * | | | | | | | | | | | typo fix | Trent Ogren | 2011-05-10 | 1 | -1/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Explain the implication of using :ignore_date on time_select | Aditya Sanghi | 2011-05-10 | 1 | -1/+2 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | :touch => :symbol also updates the updated_at attribute | Bruno Michel | 2011-05-10 | 1 | -1/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | Missing an end in routing docs | James Miller | 2011-05-09 | 1 | -0/+1 | |
| | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | more fixed-fonts here | Sebastian Martinez | 2011-05-09 | 1 | -23/+23 | |
| | | | | | | | | | | | |