aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Missing requiresAkira Matsuda2013-01-062-0/+2
|
* These are already required through AS/railsAkira Matsuda2013-01-065-6/+0
| | | | | | | * core_ext/object/blank * concern * core_ext/class/attribute * deprecation
* Rescue from LoadError when trying to load the debugger gem for consoleCarlos Antonio da Silva2013-01-061-1/+1
| | | | | | | | Avoid swallowing all exceptions that might happen when requiring debugger, just catch a LoadError if it cannot be loaded. Check for more background: https://github.com/rails/rails/commit/28fd867c9bc790636d37a28f288791cd0089a6fd#commitcomment-2386952
* Rename route_wrapper partial layout to tableCarlos Antonio da Silva2013-01-062-1/+1
| | | | | | It is used by the table formatter only, and it's already inside a routes directory that namespaces it properly, so calling it just "table" seems simpler.
* Move table routes formatter class to the inspector and rename itCarlos Antonio da Silva2013-01-064-20/+20
| | | | | | | It feels more consistent to have this class called "HtmlTableFormatter", and to have it here with the routes inspector and console formatter, since it's used for both routing error exceptions and the rails info page.
* Merge pull request #8777 from goshakkk/delegate-classAndrew White2013-01-065-5/+5
|\ | | | | Delegate to :class rather than 'self.class'
| * delegate to :class rather than 'self.class'Gosha Arinich2013-01-065-5/+5
| |
* | Refactor the logic that checks whether or not to emit the hidden id fieldCarlos Antonio da Silva2013-01-061-7/+6
| | | | | | | | | | By checking for object.persisted? first, we avoid the hash lookups for new objects.
* | Move the hidden :id field logic to where it belongs toCarlos Antonio da Silva2013-01-061-6/+9
| | | | | | | | | | | | | | | | When dealing with nested forms, Rails automatically generates a hidden field with the id value of the current object being generated by fields_for. This logic was inside the method that's available from the template object, but we just need it when really dealing with nested attributes, so moving the code to here makes more sense.
* | Eliminate the usage of parent_builder option from form_forCarlos Antonio da Silva2013-01-061-19/+15
| | | | | | | | Just use it internally from fields_for until we come up with a better solution.
* | Do not call fields_for from form_for, to avoid instantiating two buildersCarlos Antonio da Silva2013-01-062-6/+18
| |
* | Refactor to not call path.ast twiceCarlos Antonio da Silva2013-01-061-2/+3
| |
* | Merge pull request #8779 from goshakkk/refactor-route-astCarlos Antonio da Silva2013-01-061-5/+4
|\ \ | | | | | | Refactor Route#ast to use or-equals and block, instead of return with a check
| * | refactor Route#ast to use or-equals and block, instead of return with a condGosha Arinich2013-01-061-5/+4
| | |
* | | Improve javascript in welcome page [ci skip]Carlos Antonio da Silva2013-01-061-4/+5
| | |
* | | This is not that common of a change to warrant top-line billing and you ↵David Heinemeier Hansson2013-01-061-6/+1
| | | | | | | | | | | | should read more about the consequences than just toggling a boolean to make best use of it
* | | Include the standard reminder that you need to restart for changes in ↵David Heinemeier Hansson2013-01-062-0/+4
|/ / | | | | | | config/initializers to take effect
* | Update generated app's README [ci skip]Akira Matsuda2013-01-061-7/+5
| | | | | | | | | | | | | | * a controller that destroying Weblogs might be WeblogsController * a controller that indexing Posts might be PostsController * Post.all returns a Relation * modernize Model.inspect results
* | Remove unnecessary begin..rescue..end, use only rescueAkira Matsuda2013-01-0615-123/+94
| |
* | The controller generator invokes assets generator in additionAkira Matsuda2013-01-061-1/+1
| |
* | Fix generated migration file name in model generator USAGEAkira Matsuda2013-01-061-7/+7
| |
* | Update mailer generator USAGEAkira Matsuda2013-01-061-3/+2
|/ | | | | * it does not generate fixtures * it generates .text.erb templates
* Merge pull request #8774 from goshakkk/multiline-block-do-endCarlos Antonio da Silva2013-01-053-6/+6
|\ | | | | Prefer do-end for multiline block [ci skip]
| * prefer do-end for multiline blockGosha Arinich2013-01-063-6/+6
|/
* Merge pull request #8772 from goshakkk/guides-warning-fixRafael Mendonça França2013-01-051-1/+1
|\ | | | | Fix guides' warning bottom padding
| * fix guides' warning bottom paddingGosha Arinich2013-01-061-1/+1
|/
* Remove the content_tag_for change from CHANGELOG since it was revertedRafael Mendonça França2013-01-051-5/+3
|
* Merge pull request #8771 from lucasmazza/unobtrusive-javascript-examplesSteve Klabnik2013-01-051-31/+33
|\ | | | | New code examples on "Working with JavaScript in Rails" guide.
| * Update some code examples on the "Working with JavaScript in Rails" guide.Lucas Mazza2013-01-051-31/+33
| |
* | Fix documentation to content_tag_for [ci skip]Rafael Mendonça França2013-01-051-2/+2
|/
* Move config.filter_parameters to you own initializer fileRafael Mendonça França2013-01-053-5/+4
| | | | This is very rarely changed and does not deserve to be in application.rb
* Merge pull request #8766 from jcoglan/session_token_docsAndrew White2013-01-051-7/+4
|\ | | | | | | | | Remove suggestion that Procs can be used as session secrets. [ci skip]
| * Remove suggestion that Procs can be used as session secrets.James Coglan2013-01-051-7/+4
| |
* | Needless fancy structure that did not anythingDavid Heinemeier Hansson2013-01-051-1/+3
| |
* | Assert config.filter_parameters should be able to be set in a initializerRafael Mendonça França2013-01-051-0/+10
| |
* | Fix indention issue in generated templateDavid Heinemeier Hansson2013-01-051-2/+2
| |
* | Fix indention issue in generated templateDavid Heinemeier Hansson2013-01-051-2/+2
| |
* | Merge pull request #8769 from senny/deprecate_assert_blankDavid Heinemeier Hansson2013-01-0516-43/+60
|\ \ | | | | | | deprecate `assert_blank` and `assert_present`.
| * | deprecate `assert_blank` and `assert_present`.Yves Senn2013-01-0516-43/+60
| | | | | | | | | | | | | | | They don't add any benefits over `assert object.blank?` and `assert object.present?`
* | | No need to give an example where there is generated code right there to ↵David Heinemeier Hansson2013-01-051-5/+1
| | | | | | | | | | | | serve as the example
* | | Make sure assets is disable when --skip-sprockets is usedRafael Mendonça França2013-01-051-1/+2
| | |
* | | config.assets.enabled is now true by defaultDavid Heinemeier Hansson2013-01-055-9/+14
|/ /
* | Merge pull request #8768 from goshakkk/about-env-togglingGuillermo Iguaran2013-01-051-7/+11
|\ \ | | | | | | Allow toggling "about app's env", not just showing
| * | allow toggling about env, not just showingGosha Arinich2013-01-051-7/+11
|/ /
* / Remove warning, remove not used variable, and make methods privateCarlos Antonio da Silva2013-01-051-3/+5
|/ | | | Warning: "shadowing outer local variable - routes".
* Close container div tag in routing error pageCarlos Antonio da Silva2013-01-051-9/+11
|
* Move style to head to make routes page valid html5Carlos Antonio da Silva2013-01-053-4/+8
|
* Redirect using action hash instead of hardcoding full routeCarlos Antonio da Silva2013-01-051-1/+1
|
* Change Rails Info controller to use the new table routes formatterCarlos Antonio da Silva2013-01-052-5/+2
|
* Action Pack changelog improvements [ci skip]Carlos Antonio da Silva2013-01-051-32/+35
|
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-09 17:36:54 +0000