Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | We need [] method here. previously it's an array. | Arun Agrawal | 2011-08-13 | 1 | -0/+4 |
| | |||||
* | moving test_generate to an integration test with one assert per test | Aaron Patterson | 2011-08-12 | 2 | -108/+183 |
| | |||||
* | We actually don't need a reverse_merge! here. dup removed was giving error ↵ | Arun Agrawal | 2011-08-12 | 1 | -1/+1 |
| | | | | with nil class. | ||||
* | when calling url_for with a hash, additional (likely unwanted) values (such ↵ | Andrew Kaspick | 2011-08-11 | 2 | -1/+13 |
| | | | | as :host) would be returned in the hash... calling #dup on the hash prevents this | ||||
* | Remove 'parameters_for_url' parameter from 'form_tag' method signature | Gonzalo Rodriguez and Leonardo Capillera | 2011-08-10 | 1 | -4/+4 |
| | | | | | If you use that parameter it will end calling to url_for with two arguments, which fails because url_for expects only one | ||||
* | Remove lame comment. | José Valim | 2011-08-09 | 1 | -1/+0 |
| | |||||
* | rake assets:precompile defaults to production env | Santiago Pastorino | 2011-08-09 | 1 | -0/+1 |
| | |||||
* | adding missing require to fist railties tests | Aaron Patterson | 2011-08-09 | 1 | -0/+2 |
| | |||||
* | deprecating process_view_paths | Aaron Patterson | 2011-08-09 | 5 | -4/+6 |
| | |||||
* | avoid object creation via useless duping and freezing | Aaron Patterson | 2011-08-09 | 1 | -3/+2 |
| | |||||
* | just use map and case / when rather than modifying the iterating array | Aaron Patterson | 2011-08-09 | 1 | -5/+8 |
| | |||||
* | only typecast what we need to typecast | Aaron Patterson | 2011-08-09 | 1 | -11/+9 |
| | |||||
* | Favor composition over inheritance. | Aaron Patterson | 2011-08-09 | 1 | -6/+48 |
| | |||||
* | Array#+ automatically dups, no double duping | Aaron Patterson | 2011-08-09 | 1 | -2/+2 |
| | |||||
* | Just remove the sort_locals method | Aaron Patterson | 2011-08-09 | 1 | -12/+1 |
| | |||||
* | the freeze trick does nothing on arrays used as hash keys. | Aaron Patterson | 2011-08-09 | 1 | -2/+1 |
| | |||||
* | use functional style to build a list of template objects | Aaron Patterson | 2011-08-09 | 1 | -9/+12 |
| | |||||
* | refactor the optimized build_query a bit | Aaron Patterson | 2011-08-08 | 1 | -8/+3 |
| | |||||
* | reduce file stats by improving our dir glob pattern | Aaron Patterson | 2011-08-08 | 1 | -1/+1 |
| | |||||
* | hash on the template directory in order to improve cache hits | Aaron Patterson | 2011-08-08 | 1 | -2/+3 |
| | |||||
* | use meaningful names with our variables | Aaron Patterson | 2011-08-08 | 1 | -6/+6 |
| | |||||
* | use binread to read the files | Aaron Patterson | 2011-08-08 | 1 | -1/+2 |
| | |||||
* | be explicit about arguments passed around | Aaron Patterson | 2011-08-08 | 1 | -2/+2 |
| | |||||
* | Initialize config.assets.version the same way it's done in Sprockets | Santiago Pastorino | 2011-08-08 | 1 | -1/+1 |
| | |||||
* | Merge pull request #2448 from igrigorik/master | Joshua Peek | 2011-08-08 | 3 | -1/+15 |
|\ | | | | | Asset pipeline fixes: clear out tmp cache and use environment in digest generation | ||||
| * | use assets.version in asset checksum to allow user to easily override | Ilya Grigorik | 2011-08-05 | 1 | -1/+1 |
| | | |||||
| * | generate environment dependent asset digests | Ilya Grigorik | 2011-08-04 | 2 | -0/+14 |
| | | | | | | | | | | | | If two different environments are configured to use the pipeline, but one has an extra step (such as compression) then without taking the environment into account you may end up serving wrong assets | ||||
| * | Clear out tmp/cache when assets:clean is invoked. | Ilya Grigorik | 2011-08-04 | 1 | -1/+1 |
| | | | | | | | | | | | | Otherwise, if bad data is cached in tmp/clear then the next invocation of assets:precompile (or a regular incoming request) will pickup files from tmp without regenerating them from source. | ||||
* | | edit changelog to mention about x_sendfile_header default change | Vijay Dev | 2011-08-29 | 1 | -0/+2 |
| | | |||||
* | | x_sendfile_header now defaults to nil and production.rb env file doesn't | Santiago Pastorino | 2011-08-07 | 2 | -2/+2 |
|/ | | | | | | | set a particular value for it. This allows servers to set it through X-Sendfile-Type, read https://github.com/rack/rack/blob/master/lib/rack/sendfile.rb for more info. Anyways you can force this value in your production.rb | ||||
* | The trailing '/' isn't being picked up by Github anyway, and the link works ↵ | Rashmi Yadav | 2011-08-04 | 1 | -1/+1 |
| | | | | as is. | ||||
* | document meta method | Vijay Dev | 2011-08-04 | 1 | -3/+3 |
| | |||||
* | Merge pull request #2421 from pivotal-casebook/master | Santiago Pastorino | 2011-08-04 | 2 | -3/+3 |
|\ | | | | | Simple_format should not edit it in place. (Fixes https://github.com/rails/rails/issues/1980) | ||||
| * | ActionView::Helpers::TextHelper#simple_format should not change the text in ↵ | Casebook Developer | 2011-08-04 | 2 | -3/+3 |
| | | | | | | | | place. Now it duplicates it. | ||||
* | | Merge pull request #2356 from waynn/fix_symbol | Santiago Pastorino | 2011-08-04 | 2 | -4/+28 |
|\ \ | |/ |/| | fix stringify_keys destructive behavior for most FormTagHelper functions | ||||
| * | fix stringify_keys destructive behavior for most FormTagHelper functions | Waynn Lue | 2011-08-03 | 2 | -4/+28 |
| | | | | | | | | add four new tests to verify that the other three methods that called stringify_keys! are fixed. verified that the tests break in master without the code patch. Closes #2355 | ||||
* | | Fix the issue where default_url_options is being cached on test cases. ↵ | thoefer | 2011-08-01 | 3 | -0/+35 |
| | | | | | | | | | | | | Closes #1872. Closes #2031. Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | | Rename new method to_path to to_partial_path to avoid conflicts with ↵ | José Valim | 2011-08-01 | 4 | -10/+10 |
| | | | | | | | | File#to_path and similar. | ||||
* | | Merge pull request #2379 from arunagw/extra_require_removal | Santiago Pastorino | 2011-07-31 | 1 | -1/+0 |
|\ \ | | | | | | | Removing extra requires from the test. Already loaded in abstract_unit. | ||||
| * | | Removing extra requires from the test. Already loaded in abstract_unit. | Arun Agrawal | 2011-07-31 | 1 | -1/+0 |
| |/ | |||||
* / | remove extra require for 'active_support/dependencies' as it is required in ↵ | Vishnu Atrai | 2011-07-31 | 1 | -1/+0 |
|/ | | | | abstract_unit.rb | ||||
* | remove redundant calls to stringify_keys | Waynn Lue | 2011-07-29 | 1 | -2/+2 |
| | |||||
* | Instead of removing the instance variable just set it to nil, resolves the ↵ | Brian Cardarella | 2011-07-29 | 1 | -2/+2 |
| | | | | warnings because of a missing instance variable | ||||
* | Merge pull request #2327 from cesario/patch-1 | José Valim | 2011-07-28 | 1 | -1/+0 |
|\ | | | | | We don't need to require erb here. | ||||
| * | We don't need to require erb here. | Franck Verrot | 2011-07-28 | 1 | -1/+0 |
| | | |||||
* | | Merge pull request #2330 from thedarkone/resources-router-fix | José Valim | 2011-07-28 | 2 | -8/+16 |
|\ \ | |/ |/| | Inline resources router fix | ||||
| * | Make use of the inherited initializer. | thedarkone | 2011-07-28 | 1 | -3/+2 |
| | | |||||
| * | There is no need to be destructive with the passed-in options. | thedarkone | 2011-07-28 | 2 | -6/+15 |
| | | | | | | | | This fixes a bug that is caused by Resource/SingletonResource mangling resource options when using inline "multi"-resource declarations. | ||||
* | | Move the cache to a nested hash which performs better than a hash with array ↵ | José Valim | 2011-07-28 | 1 | -11/+17 |
| | | | | | | | | as keys. | ||||
* | | Rename class method to_path to _to_path and make it explicit that it is an ↵ | José Valim | 2011-07-28 | 1 | -2/+2 |
| | | | | | | | | internal method. |