Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Revert "Revert "Respect `config.digest = false` for `asset_path`"" | Rafael Mendonça França | 2012-09-20 | 1 | -1/+3 |
| | | | | | | This reverts commit 54f55746a70a7091341e84236498203118a7fbb4. Reason: the last commit fixed the failing case | ||||
* | Revert "Respect `config.digest = false` for `asset_path`" | Rafael Mendonça França | 2012-09-20 | 1 | -3/+1 |
| | | | | | | | | | | | | This reverts commit 1ac19c11792a745cd654f02dfaed8e93f95b3c70. Conflicts: actionpack/CHANGELOG.md Reason: This is causing failures in the railties build. See http://travis-ci.org/#!/rails/rails/jobs/2491787 Related with #7672 | ||||
* | Respect `config.digest = false` for `asset_path` | Peter Wagenet | 2012-09-17 | 1 | -1/+3 |
| | | | | | | | Previously, the `asset_path` internals only respected the `:digest` option, but ignored the global config setting. This meant that `config.digest = false` could not be used in conjunction with `config.compile = false` this corrects the behavior. | ||||
* | Fix railties test suite | Piotr Sarnacki | 2012-06-08 | 1 | -2/+1 |
| | | | | | | Apparently asset_environment should not be invoked if it's not needed. This fixes broken build by getting back to the code more similar to the version changed here: 5b0a891 | ||||
* | Fix asset tags for files with more than one dot | Piotr Sarnacki | 2012-06-07 | 1 | -1/+3 |
| | | | | | | | | | | | | | | | After the fix done in 39f9f02a, there are cases that will not work correctly. If you have file with "2 extensions", like foo.min.js and you reference the file without extension, like: javascript_include_tag "foo.min" it will fail because sprockets finds foo.min.js with foo.min argument. This commit fixes this case and will get the right file even when referrencing it without extension. (closes #6598) | ||||
* | Assets: don't add extension if other given and file exists | Sergey Nartimov | 2012-05-21 | 1 | -2/+7 |
| | | | | | | | | | | | | | | We should lookup if asset without appended extension exists. When sprockets are disabled the asset tag helpers incorporate this logic. When sprockets are enabled we should have the same logic. For example, we have style.ext file in app/assets/stylesheets and we use stylesheet_link_tag in the layout. In this case we should have /assets/style.ext instead of /assets/style.ext.css in the output. Closes #6310 | ||||
* | Make sure that javascript_include_tag/stylesheet_link_tag methods in | Jorge Bejar | 2012-05-03 | 1 | -2/+2 |
| | | | | sprockets/helpers/rails_helper.rb discard duplicated assets | ||||
* | Fix GH #5411. When precompiling, params method is undefined. | kennyj | 2012-03-22 | 1 | -1/+1 |
| | |||||
* | Pass extensions to javascript_path and stylesheet_path helpers. Closes #3417 | Santiago Pastorino | 2012-01-03 | 1 | -2/+2 |
| | |||||
* | Add font_path helper method | Santiago Pastorino | 2012-01-03 | 1 | -0/+5 |
| | |||||
* | Remove broken #compute_source_path | Jeremy Kemper | 2011-12-23 | 1 | -5/+0 |
| | |||||
* | Use ProcessedAsset#pathname in Sprockets helpers when debugging is on. ↵ | Guillermo Iguaran | 2011-12-10 | 1 | -3/+3 |
| | | | | | | Closes #3333 #3348 #3361. Is wrong use ProcessedAsset#to_s since it returns the content of the file. | ||||
* | Let's do the same trick for asset_path that we do for ↵ | Adrian Pike | 2011-12-05 | 1 | -8/+9 |
| | | | | | | [image,javascript,stylesheet]_path to avoid namespace conflicts on named routes. Closes #3427 | ||||
* | javascript_include_tag should add '.js' to sources that contain '.' | lest | 2011-11-22 | 1 | -1/+1 |
| | |||||
* | stylesheet_link_tag('/stylesheets/application') and similar helpers doesn't ↵ | Santiago Pastorino | 2011-10-05 | 1 | -0/+2 |
| | | | | throw Sprockets::FileOutsidePaths exception anymore | ||||
* | Fix regression when calling helpers inside sprockets. | José Valim | 2011-10-05 | 1 | -1/+1 |
| | |||||
* | Fix the lame config.action_controller.present? check scattered throughout ↵ | José Valim | 2011-10-05 | 2 | -3/+13 |
| | | | | | | | | assets_path. Conflicts: actionpack/test/template/sprockets_helper_test.rb | ||||
* | javascript_path and stylesheet_path should honor the asset pipelining | Santiago Pastorino | 2011-09-28 | 1 | -0/+10 |
| | |||||
* | Refactor image_path to use asset_path directly | Christos Zisopoulos | 2011-09-26 | 1 | -1/+1 |
| | |||||
* | Correctly override image_path in sprockets rails_helper | Christos Zisopoulos | 2011-09-26 | 1 | -1/+2 |
| | |||||
* | Merge pull request #3135 from christos/respect_assets_digest_value | Santiago Pastorino | 2011-09-26 | 1 | -1/+1 |
|\ | | | | | Give precedence to `config.digest = false` over the existence of manifest.yml asset digests | ||||
| * | Give precedence to `config.digest = false` over manifest.yml asset digests | Christos Zisopoulos | 2011-09-26 | 1 | -1/+1 |
| | | |||||
* | | image_tag should use /assets if asset pipelining is turned on. Closes #3126 | Santiago Pastorino | 2011-09-26 | 1 | -0/+4 |
|/ | |||||
* | Remove Sprockets compute_public_path, AV compute_public_path can be used ↵ | Guillermo Iguaran | 2011-09-23 | 1 | -5/+0 |
| | | | | directly | ||||
* | Fixes #1489 again, with updated code and numerous tests to confirm | docunext | 2011-09-22 | 1 | -1/+1 |
| | |||||
* | Allow asset tag helper methods to accept :digest => false option in order to ↵ | Santiago Pastorino | 2011-09-14 | 1 | -13/+15 |
| | | | | completely avoid the digest generation. | ||||
* | Fix asset_path relative_url_root method | Santiago Pastorino | 2011-09-13 | 1 | -4/+0 |
| | |||||
* | Fix for relative url root in assets | Guillermo Iguaran | 2011-09-13 | 1 | -4/+2 |
| | |||||
* | always rewrite_relative_url_root | Santiago Pastorino | 2011-09-13 | 1 | -5/+1 |
| | |||||
* | Set relative url root in assets when controller isn't available for ↵ | Guillermo Iguaran | 2011-09-13 | 1 | -3/+13 |
| | | | | | | Sprockets. Fix #2435 See https://github.com/rails/sass-rails/issues/42 for details | ||||
* | Merge pull request #2851 from dasch/patch-6 | Jon Leighton | 2011-09-06 | 1 | -5/+3 |
|\ | | | | | No need to use a begin block there | ||||
| * | No need to use a begin block there | Daniel Schierbeck | 2011-09-04 | 1 | -5/+3 |
| | | |||||
* | | AssetNotPrecompiledError should be raise when config.assets.digest = false, ↵ | Guillermo Iguaran | 2011-09-04 | 1 | -2/+2 |
|/ | | | | config.assets.compile = false and manifest file isn't present | ||||
* | Move references to Rails.application.config out of Sprockets AssetPaths | Sam Pohlenz | 2011-09-01 | 1 | -5/+14 |
| | |||||
* | Backport f443f9cb0c64 to master | Guillermo Iguaran | 2011-08-31 | 1 | -20/+9 |
| | |||||
* | Read digests of assets from manifest.yml if config.assets.manifest is on | Guillermo Iguaran | 2011-08-31 | 1 | -1/+16 |
| | |||||
* | Revert "Merge pull request #2647 from dmathieu/no_rescue" | Santiago Pastorino | 2011-08-30 | 1 | -2/+6 |
| | | | | This reverts commit 125b1b0b9180ec8f0135da185e77250d2b8a4bea. | ||||
* | Avoid extra recursive call in Sprockets helpers | Guillermo Iguaran | 2011-08-28 | 1 | -2/+2 |
| | |||||
* | Merge pull request #2696 from guilleiguaran/string-keys-sprockets-helpers | Santiago Pastorino | 2011-08-25 | 1 | -4/+4 |
|\ | | | | | Removing redundant stringify_keys in Sprockets helpers | ||||
| * | Removing redundant stringify_keys in Sprockets helpers | Guillermo Iguaran | 2011-08-25 | 1 | -4/+4 |
| | | |||||
* | | Don't require a specific value for the ?debug flag | Daniel Schierbeck | 2011-08-25 | 1 | -3/+1 |
|/ | | | | | Previously, only "1" and "true" were accepted. This change makes any value, or no value at all, have the same effect. | ||||
* | Make Sprockets::Helpers::RailsHelper#debug_assets? more readable | Daniel Schierbeck | 2011-08-25 | 1 | -4/+4 |
| | |||||
* | Preserve stylesheet_link_tag and javascript_include_tag options on debug | Guillermo Iguaran | 2011-08-24 | 1 | -4/+3 |
| | |||||
* | Use super in javascript_include_tag and stylesheet_link_tag instead of ↵ | Guillermo Iguaran | 2011-08-24 | 1 | -14/+2 |
| | | | | generate tags manually | ||||
* | Remove redundant join and call to html_safe | Guillermo Iguaran | 2011-08-24 | 1 | -2/+2 |
| | |||||
* | Merge pull request #2644 from guilleiguaran/allow-assets-debugging | Santiago Pastorino | 2011-08-24 | 1 | -3/+4 |
|\ | | | | | Add config.allow_debugging option | ||||
| * | Add config.allow_debugging option to determine if the debug_assets query ↵ | Guillermo Iguaran | 2011-08-24 | 1 | -3/+4 |
| | | | | | | | | param can be passed by user | ||||
* | | Merge pull request #2668 from guilleiguaran/debug-assets-media-type | Santiago Pastorino | 2011-08-24 | 1 | -2/+3 |
|\ \ | |/ |/| | Debug assets shouldn't ignore media type for stylesheets. Closes #2625 | ||||
| * | Debug assets shouldn't ignore media type for stylesheets. Closes #2625 | Guillermo Iguaran | 2011-08-23 | 1 | -2/+3 |
| | | |||||
* | | remove useless rescue | Damien Mathieu | 2011-08-24 | 1 | -6/+3 |
|/ | | | | | params is a method, defined in every controller, which always returns a hash. If it raises a NoMethodError, it means there's a bug somewhere else, which we want to know about. |