Skip to content

Commit a85ae3c

Browse files
authored
Merge pull request #6227 from mamhoff/no-javascripts-in-dummy-apps
Dummy app generator: Only configure app/assets/javascripts if present
2 parents 61f675e + ca1af55 commit a85ae3c

2 files changed

Lines changed: 3 additions & 0 deletions

File tree

core/lib/generators/spree/dummy/dummy_generator.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,7 @@ def generate_test_dummy
5454
def test_dummy_config
5555
@lib_name = options[:lib_name]
5656
@database = options[:database]
57+
@has_javascripts = Dir.exist?("#{dummy_path}/app/assets/javascripts")
5758

5859
template "rails/database.yml", "#{dummy_path}/config/database.yml", force: true
5960
template "rails/storage.yml", "#{dummy_path}/config/storage/test.yml", force: true

core/lib/generators/spree/dummy/templates/rails/manifest.js renamed to core/lib/generators/spree/dummy/templates/rails/manifest.js.tt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
11
//= link_tree ../images
2+
<% if @has_javascripts %>
23
//= link_directory ../javascripts .js
4+
<% end %>
35
//= link_directory ../stylesheets .css

0 commit comments

Comments
 (0)