aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #2872 from sikachu/content_tag_html_safeSantiago Pastorino2011-09-052-1/+13
|\ | | | | Make sure that result from content_tag_for with collection is html_safe
| * Make sure that result from content_tag_for with collection is html_safePrem Sichanugrist2011-09-052-1/+13
| | | | | | Thank you @spastorino for catching the empty test in b84cee0, as I totally forgot that I still have to make it html_safe.
* | Merge pull request #2815 from avakhov/current-page-non-get-requestsSantiago Pastorino2011-09-052-2/+16
|\ \ | | | | | | current_page? returns false for non-GET requests
| * | current_page? returns false for non-GET requestsAlexey Vakhov2011-09-022-2/+16
| | |
* | | Merge pull request #2799 from tomstuart/3-1-stableSantiago Pastorino2011-09-052-3/+39
| | | | | | | | | | | | Never return stored content from content_for when a block is given
* | | Merge pull request #2789 from dmathieu/limit_first_lastJon Leighton2011-09-052-2/+32
|\ \ \ | | | | | | | | Use LIMIT SQL word in first - Closes #2783
| * | | Use LIMIT sql word in last when it's possibleDamien Mathieu2011-09-052-2/+23
| | | |
| * | | use LIMIT SQL word in first - Closes #2783Damien Mathieu2011-09-052-1/+10
|/ / /
* | / add blanks between methodsVijay Dev2011-09-051-0/+3
| |/ |/|
* | Merge pull request #2868 from arunagw/batches_test_pg_masterJon Leighton2011-09-051-1/+1
|\ \ | | | | | | Sort here to make sure the order is ok.
| * | Sort here to make sure the order is ok.Arun Agrawal2011-09-051-1/+1
|/ /
* | Merge pull request #2713 from FLOChip/500-fixedJosé Valim2011-09-051-1/+0
|\ \ | | | | | | remove misleading line on 500 error page. #2648
| * | remove misleading line on 500 error page.Teng Siong Ong2011-08-271-1/+0
| | |
| * | fixed 500 error message which is misleading by suggesting all rails apps ↵Jared Tame2011-08-271-1/+1
| | | | | | | | | | | | have exception notifications by default
* | | Remove empty testSantiago Pastorino2011-09-051-3/+0
| | |
* | | Merge pull request #2863 from ↵Santiago Pastorino2011-09-041-0/+5
|\ \ \ | | | | | | | | | | | | | | | | raybaxter/silence_unitialized_instance_variable_warnings Silence unitialized instance variable warnings
| * | | Tests shouldn't produce warningsRay Baxter2011-09-041-0/+5
| | | |
* | | | Merge pull request #2861 from guilleiguaran/assets-fix-changelogSantiago Pastorino2011-09-041-0/+3
|\ \ \ \ | | | | | | | | | | Add entry about Asset behaviour fix to ActionPack ChangeLog
| * | | | Add entry about Asset behaviour fix to ActionPack ChangeLogGuillermo Iguaran2011-09-041-0/+3
|/ / / /
* | | | Merge pull request #2860 from guilleiguaran/fix-assets-behaviorSantiago Pastorino2011-09-042-2/+28
|\ \ \ \ | | | | | | | | | | Fix AssetNotPrecompiledError behavior
| * | | | AssetNotPrecompiledError should be raise when config.assets.digest = false, ↵Guillermo Iguaran2011-09-042-2/+28
| |/ / / | | | | | | | | | | | | config.assets.compile = false and manifest file isn't present
* | | | * is not allowed in windows file names. Closes #2574 #2847Santiago Pastorino2011-09-042-1/+1
| | | |
* | | | Merge pull request #2855 from arunagw/fix_find_in_batches_masterJon Leighton2011-09-043-1/+31
|\ \ \ \ | |/ / / |/| | | We need to recorder here. Need to drop the order from default scope.
| * | | We need to recorder here. Need to drop the order from default scope.Arun Agrawal2011-09-053-1/+31
|/ / / | | | | | | Fixes #2832
* | | Merge pull request #2852 from milandobrota/remove_unneeded_explicit_nil_checksJon Leighton2011-09-047-11/+11
|\ \ \ | | | | | | | | if ... nil? is more expensive than unless
| * | | if ... nil? is more expensive than unlessMilan Dobrota2011-09-047-11/+11
|/ / /
* | | Merge docrails, resolving conflicts in the assets guideVijay Dev2011-09-049-24/+26
|\ \ \
| * | | minor fixes in assets guideVijay Dev2011-09-041-11/+9
| | | |
| * | | Merge pull request #64 from rinaldifonseca/masterGuillermo Iguaran2011-09-031-1/+1
| |\ \ \ | | | | | | | | | | Added |t| to create_table block
| | * | | Added |t| to create_table blockRinaldi Fonseca2011-09-041-1/+1
| |/ / /
| * | | add some missing dots in the docsVijay Dev2011-09-041-4/+4
| | | |
| * | | Fix asset_path example in CSS and ERB sectionGuillermo Iguaran2011-09-021-1/+1
| | | |
| * | | document send_file guesses content type from the file extensionVijay Dev2011-09-031-1/+3
| | | | | | | | | | | | | | | | and remove info about x_send_file option to send_file
| * | | fix YUI descriptionRichard Hulse2011-09-031-1/+1
| | | | | | | | | | | | | | | | | | | | YUI CSS compressor does not extend syntax! Also added link to more info
| * | | Fixed a typo in the section about migration.David Peckham2011-09-021-1/+1
| | | |
| * | | some of the changes for validation earlier reverted from d20281aVijay Dev2011-09-033-4/+6
| | | |
| * | | Change photos_path to photos_url in Using redirect_toFranco Catena2011-09-021-1/+1
| | | | | | | | | | | | | | | | Is more "correct" a complete url than only the path
* | | | Merge pull request #2848 from arunagw/jruby_activesupport_fixJon Leighton2011-09-041-4/+6
|\ \ \ \ | | | | | | | | | | Need to have a check here for JAVA PLATFORM for JRuby
| * | | | We can't simply check the first line of the backtrace, Arun Agrawal2011-09-041-4/+6
| | | | | | | | | | | | | | | because JRuby reports the call to __send__ in the backtrace.
* | | | | Revert "Fixes bug in ActiveRecord::QueryMethods, #1697"Jon Leighton2011-09-042-7/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 0df27c98d982ec87d2fb48cfda82694eb267993e. Reverted due to failing test, see #2845.
* | | | | Merge pull request #2841 from wojtekmach/app-generators-groupJosé Valim2011-09-044-1/+61
|\ \ \ \ \ | | | | | | | | | | | | Add gem group support to generators
| * | | | | Add gem_group support to generatorsWojciech Mach2011-09-044-1/+61
| | | | | |
* | | | | | Merge pull request #2816 from sikachu/content_tag_forJosé Valim2011-09-044-7/+166
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Make `content_tag_for` and `div_for` accepts the array of records
| * | | | | Make `content_tag_for` and `div_for` accepts the array of recordsPrem Sichanugrist2011-09-044-7/+166
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So instead of having to do this: @items.each do |item| content_tag_for(:li, item) do Title: <%= item.title %> end end You can now do this: content_tag_for(:li, @items) do |item| Title: <%= item.title %> end
* | | | | | Remove useless parensSantiago Pastorino2011-09-041-1/+1
| | | | | |
* | | | | | Merge pull request #2844 from arunagw/warnings_removed_masterSantiago Pastorino2011-09-0312-37/+37
|\ \ \ \ \ \ | | | | | | | | | | | | | | Warnings removed from master
| * | | | | | Modified content in guides and commentsArun Agrawal2011-09-044-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | for "assert /" warnings. Removed because if somebody will use this code they will get warnings!
| * | | | | | Warnings removed for "assert_match /" Please doArun Agrawal2011-09-046-25/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | not add more.!
| * | | | | | Warnings removed for using shadow variable.Arun Agrawal2011-09-042-4/+4
|/ / / / / /
* | | | | | restores the API docs of AR::Fixtures, made a quick pass over them, revises ↵Xavier Noria2011-09-033-370/+363
| | | | | | | | | | | | | | | | | | | | | | | | link in fixture template [closes #2840]