aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/sprockets/helpers
Commit message (Collapse)AuthorAgeFilesLines
* Revert "Revert "Respect `config.digest = false` for `asset_path`""Rafael Mendonça França2012-09-201-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ça2012-09-201-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 Wagenet2012-09-171-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 suitePiotr Sarnacki2012-06-081-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 dotPiotr Sarnacki2012-06-071-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 existsSergey Nartimov2012-05-211-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 inJorge Bejar2012-05-031-2/+2
| | | | sprockets/helpers/rails_helper.rb discard duplicated assets
* Fix GH #5411. When precompiling, params method is undefined.kennyj2012-03-221-1/+1
|
* Pass extensions to javascript_path and stylesheet_path helpers. Closes #3417Santiago Pastorino2012-01-031-2/+2
|
* Add font_path helper methodSantiago Pastorino2012-01-031-0/+5
|
* Remove broken #compute_source_pathJeremy Kemper2011-12-231-5/+0
|
* Use ProcessedAsset#pathname in Sprockets helpers when debugging is on. ↵Guillermo Iguaran2011-12-101-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 Pike2011-12-051-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 '.'lest2011-11-221-1/+1
|
* stylesheet_link_tag('/stylesheets/application') and similar helpers doesn't ↵Santiago Pastorino2011-10-051-0/+2
| | | | throw Sprockets::FileOutsidePaths exception anymore
* Fix regression when calling helpers inside sprockets.José Valim2011-10-051-1/+1
|
* Fix the lame config.action_controller.present? check scattered throughout ↵José Valim2011-10-052-3/+13
| | | | | | | | assets_path. Conflicts: actionpack/test/template/sprockets_helper_test.rb
* javascript_path and stylesheet_path should honor the asset pipeliningSantiago Pastorino2011-09-281-0/+10
|
* Refactor image_path to use asset_path directlyChristos Zisopoulos2011-09-261-1/+1
|
* Correctly override image_path in sprockets rails_helperChristos Zisopoulos2011-09-261-1/+2
|
* Merge pull request #3135 from christos/respect_assets_digest_valueSantiago Pastorino2011-09-261-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 digestsChristos Zisopoulos2011-09-261-1/+1
| |
* | image_tag should use /assets if asset pipelining is turned on. Closes #3126Santiago Pastorino2011-09-261-0/+4
|/
* Remove Sprockets compute_public_path, AV compute_public_path can be used ↵Guillermo Iguaran2011-09-231-5/+0
| | | | directly
* Fixes #1489 again, with updated code and numerous tests to confirmdocunext2011-09-221-1/+1
|
* Allow asset tag helper methods to accept :digest => false option in order to ↵Santiago Pastorino2011-09-141-13/+15
| | | | completely avoid the digest generation.
* Fix asset_path relative_url_root methodSantiago Pastorino2011-09-131-4/+0
|
* Fix for relative url root in assetsGuillermo Iguaran2011-09-131-4/+2
|
* always rewrite_relative_url_rootSantiago Pastorino2011-09-131-5/+1
|
* Set relative url root in assets when controller isn't available for ↵Guillermo Iguaran2011-09-131-3/+13
| | | | | | Sprockets. Fix #2435 See https://github.com/rails/sass-rails/issues/42 for details
* Merge pull request #2851 from dasch/patch-6Jon Leighton2011-09-061-5/+3
|\ | | | | No need to use a begin block there
| * No need to use a begin block thereDaniel Schierbeck2011-09-041-5/+3
| |
* | AssetNotPrecompiledError should be raise when config.assets.digest = false, ↵Guillermo Iguaran2011-09-041-2/+2
|/ | | | config.assets.compile = false and manifest file isn't present
* Move references to Rails.application.config out of Sprockets AssetPathsSam Pohlenz2011-09-011-5/+14
|
* Backport f443f9cb0c64 to masterGuillermo Iguaran2011-08-311-20/+9
|
* Read digests of assets from manifest.yml if config.assets.manifest is onGuillermo Iguaran2011-08-311-1/+16
|
* Revert "Merge pull request #2647 from dmathieu/no_rescue"Santiago Pastorino2011-08-301-2/+6
| | | | This reverts commit 125b1b0b9180ec8f0135da185e77250d2b8a4bea.
* Avoid extra recursive call in Sprockets helpersGuillermo Iguaran2011-08-281-2/+2
|
* Merge pull request #2696 from guilleiguaran/string-keys-sprockets-helpersSantiago Pastorino2011-08-251-4/+4
|\ | | | | Removing redundant stringify_keys in Sprockets helpers
| * Removing redundant stringify_keys in Sprockets helpersGuillermo Iguaran2011-08-251-4/+4
| |
* | Don't require a specific value for the ?debug flagDaniel Schierbeck2011-08-251-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 readableDaniel Schierbeck2011-08-251-4/+4
|
* Preserve stylesheet_link_tag and javascript_include_tag options on debugGuillermo Iguaran2011-08-241-4/+3
|
* Use super in javascript_include_tag and stylesheet_link_tag instead of ↵Guillermo Iguaran2011-08-241-14/+2
| | | | generate tags manually
* Remove redundant join and call to html_safeGuillermo Iguaran2011-08-241-2/+2
|
* Merge pull request #2644 from guilleiguaran/allow-assets-debuggingSantiago Pastorino2011-08-241-3/+4
|\ | | | | Add config.allow_debugging option
| * Add config.allow_debugging option to determine if the debug_assets query ↵Guillermo Iguaran2011-08-241-3/+4
| | | | | | | | param can be passed by user
* | Merge pull request #2668 from guilleiguaran/debug-assets-media-typeSantiago Pastorino2011-08-241-2/+3
|\ \ | |/ |/| Debug assets shouldn't ignore media type for stylesheets. Closes #2625
| * Debug assets shouldn't ignore media type for stylesheets. Closes #2625Guillermo Iguaran2011-08-231-2/+3
| |
* | remove useless rescueDamien Mathieu2011-08-241-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.