aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | | | | | Use ! instead of notSantiago Pastorino2010-11-111-1/+1
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Fixed bug in active_record/nested_attributes where an empty string id caused ↵Santiago Pastorino2010-11-112-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | an ActiveRecord::RecordNotFound error. Found by [Ben Tillman] [#5638 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | | | | | | | | | | | | | Bump Arel up to 2.0.2Santiago Pastorino2010-11-111-1/+1
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | skip testing framework if app created with --skip-test-unit [#5796 ↵Rob Zolkos2010-11-112-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | | | | | | | | | | | | | Add missing autoload.José Valim2010-11-111-0/+1
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Remove unused Configurable in Rails::Engine and Rails::Application. Remove ↵zhengjia2010-11-113-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | unnecessary railties load path in cli [#5871 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | | | | | | | | | | | | | provide better error message if path is uri [#5914 state:resolved]Neeraj Singh2010-11-112-3/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | | | | | | | | | | | | | The model generator shouldn't throw warnings when using mass nouns [#5363 ↵Jeff Kreeftmeijer2010-11-112-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | | | | | | | | | | | | | fields_for should treat ActiveRecord::Relation as an arrayNeeraj Singh2010-11-113-2/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [#5795 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | | | | | | | | | | | | | Remove inline comment.José Valim2010-11-111-17/+17
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Make after_filter halt when before_filter renders or redirects [#5648 ↵Marcelo Giorgi2010-11-112-2/+81
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | | | | | | | | | | | | | Fix render partial with layout and no blockCarlos Antonio da Silva2010-11-112-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using a render :partial with :layout call, without giving a block, if the given :partial had another render :partial call, the layout was not being rendered. This commit fixes this context by storing variables before rendering the partial, so they are not overrided in any successive call to render partials down the path. All ActionPack tests are ok.
* | | | | | | | | | | | | | Fix issue with rendering partials and then yielding inside a render layout callCarlos Antonio da Silva2010-11-115-0/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The given block was never being executed when another render :partial call existed before the yield call, due to the block being overriden. This commit also adds some more tests to render with :layout option.
* | | | | | | | | | | | | | patching to ensure separator is printed with order [#5816 state:resolved]Aditya Sanghi2010-11-112-0/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | | | | | | | | | | | | | Allow AR Session Store to be renewedJosé Valim2010-11-112-0/+21
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Duck typing hereSantiago Pastorino2010-11-111-1/+1
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Fix ActiveSupport::TimeWithZone#localtime method with DateTimeAlvaro Bautista2010-11-112-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [#5344 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | | | | | | | | | | | | | restores the example for fixed-width font in API guidelinesXavier Noria2010-11-101-0/+10
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Remove copy_instance_variables_from from guidesSantiago Pastorino2010-11-102-45/+0
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Remove unused methodSantiago Pastorino2010-11-102-62/+0
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Remove useless cgiDenis Odorcic2010-11-104-36/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [#5943 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | | | | | | | | | | | | | syncs CHANGELOGs for 3.0.2 in 3-0-stable with the ones in masterXavier Noria2010-11-096-2/+49
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Double negation of an already boolean value produces the same resultSantiago Pastorino2010-11-091-1/+1
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Don't check if persisted is defined just initialize it properlySantiago Pastorino2010-11-093-4/+1
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Execute less operationsSantiago Pastorino2010-11-091-2/+1
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | use persisted? instead of new_record? wherever possibleDavid Chelimsky2010-11-0935-196/+203
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - persisted? is the API defined in ActiveModel - makes it easier for extension libraries to conform to ActiveModel APIs without concern for whether the extended object is specifically ActiveRecord [#5927 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | | | | | | | | | | | | | registers number_to_currency's :negative_format in the CHANGELOGXavier Noria2010-11-091-0/+2
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Strip regexp anchors from rake routes output [#5934 state:resolved]Andrew White2010-11-092-1/+11
| |_|_|_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | | |
* | | | | | | | | | | | | Test cache.key_file_path with separators ht. Jim Wilson [#5611]Santiago Pastorino2010-11-071-0/+5
| | | | | | | | | | | | |
* | | | | | | | | | | | | Fix FileStore cache incorrectly regenerating its key from a pathname when a ↵Denis Odorcic2010-11-072-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | regexp is used in expire_fragment [#5850 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | | | | | | | | | | | | Update the validates_uniqueness_of documentation to tell you to use a unique ↵Michael Koziarski2010-11-081-25/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | index. The other options mentioned were not good ideas and disclaimed as such in the documentation.
* | | | | | | | | | | | | Make safe_append= live on AV::OutputBuffer not AS::SafeBufferMichael Koziarski2010-11-082-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: actionpack/lib/action_view/template/handlers/erb.rb
* | | | | | | | | | | | | Added support for Erubis <%== tagJan Maurits Faber2010-11-083-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | <%== x %> is syntactic sugar for <%= raw(x) %> Signed-off-by: Michael Koziarski <michael@koziarski.com> [#5918 status:committed]
* | | | | | | | | | | | | OrderedHash#select now preserves order [#5843 state:resolved]James A. Rosen2010-11-072-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | | | | | | | | | | | | Make cookies hash in ActionDispatch::TestProcess indifferent access [#5761 ↵David Trasbo2010-11-072-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | | | | | | | | | | | | Ensure that HashWithIndifferentAccess duplication preserves class (for ↵laserlemon2010-11-072-3/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | sublclasses) and default value [#5724 state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | | | | | | | | | | | | use column names as order by complex function parameters, fix for postgresql ↵Raimonds Simanovskis2010-11-072-13/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | adapter to correctly remove ASC/DESC Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | | | | | | | | | | | | reduce duplication in assiciations #construct()Aaron Patterson2010-11-061-9/+3
| | | | | | | | | | | | |
* | | | | | | | | | | | | Move @assigns from the controller to the test itselfSantiago Pastorino2010-11-062-1/+1
| | | | | | | | | | | | |
* | | | | | | | | | | | | Fix problems trying to functional test AC::Metal controllersSantiago Pastorino2010-11-063-11/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [#5393 state:committed]
* | | | | | | | | | | | | make sure we keep parent records in orderAaron Patterson2010-11-061-6/+6
| | | | | | | | | | | | |
* | | | | | | | | | | | | only call to_s on the association variable onceAaron Patterson2010-11-051-2/+5
| | | | | | | | | | | | |
* | | | | | | | | | | | | simplify instantiate in the join parts objectAaron Patterson2010-11-051-8/+7
| | | | | | | | | | | | |
* | | | | | | | | | | | | index is not used, so removing itAaron Patterson2010-11-051-1/+1
| | | | | | | | | | | | |
* | | | | | | | | | | | | reduce method calls to the join base objectAaron Patterson2010-11-051-4/+6
| | | | | | | | | | | | |
* | | | | | | | | | | | | Correctly handle the case of an API response that returns a hash by treating ↵Chris Eppstein2010-11-062-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a single hash argument as the resource instead of as options.
* | | | | | | | | | | | | stop recursively calling exists?Aaron Patterson2010-11-051-3/+5
| | | | | | | | | | | | |
* | | | | | | | | | | | | Bring back ruby-debug19 for Ruby versions < 1.9.3Santiago Pastorino2010-11-051-0/+5
| | | | | | | | | | | | |
* | | | | | | | | | | | | Avoid warning: shadowing outer local variableSantiago Pastorino2010-11-051-2/+2
| | | | | | | | | | | | |
* | | | | | | | | | | | | Fixes queries using limits and punctuation in order, removes order("col1, ↵Santiago Pastorino2010-11-055-8/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | col2") usage in favor of order(["col1", "col2"}) [#4597 state:committed]
generated by cgit v1.2.3 (git 2.25.1) at 2025-07-15 16:54:54 +0000