webpacker-routes 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e97672c8e8fa59f10e40210007b6620c2a59bf0a257e6172731874f446cfb659
-  data.tar.gz: ffb9e82fcf9f6ed0bae01e66a7177092737aae410e738bd749b5344efd4f11e5
+  metadata.gz: e0ace4cd3e92b5e779788f42c032b8227f8352cc124f72d5725ac08280125097
+  data.tar.gz: ca63b493a4a7f3f3115ca19a87883628cfdf864e2862b00d35c9e807af795e63
 SHA512:
-  metadata.gz: 3405c23a0877f10fc5b42404fb8b6f0fb2305fc5d158b649dbe83dcbce05151ac89fe2afc1b5032826f9a393a2a0ba7f19fa97e04addc1a76f9f0fc0a3d36abc
-  data.tar.gz: 1408d938a137462f15ee47f5f06e66d9e47e44478c0cefd57a4a16fb446b2e36c44660fc24d96a8940cc5ef703bbcaef895e98e191829bc354289edf623ffc64
+  metadata.gz: b2dec026375850eda7b48f59c0ffa89e17ccb36fc7f4ea87311047416aabd7fdfbaac2f53ec47a552c0e8c39b551a0ad8859df8a7dba5f4106a2bd53f23781de
+  data.tar.gz: ccf44fcf0f9e8c02b1de95b392a9992e3ed6646bd08bf11301d36bda1317347b256bcab93a83a197fd094436c8b7f27da742f499304b879b1d5b6662fce4b67d

data/lib/webpacker/routes.rb

@@ -24,28 +24,38 @@ module Webpacker
 
         default_url_options_var = var_name.call('default_url_options')
 
-        File.atomic_write(Webpacker.config.routes_path.join('index.js')) do |file|
-          file.write(<<-JAVASCRIPT.strip_heredoc)
-            import { urlFor, pathFor } from 'webpacker-routes'
-            const #{default_url_options_var} = #{js(default_url_options)}
-          JAVASCRIPT
-
-          app.routes.named_routes.sort_by(&:first).each do |name, route|
-            raise `Invalid route name for javascript: ${name}` unless JAVASCRIPT_VARIABLE_NAME_REGEX =~ name
-
-            spec = route.path.spec.to_s
-            segment_keys = route.segment_keys.uniq
-            options = route.defaults.except(*IGNORED_OPTIONS)
-
-            spec_var = var_name.call("#{name}_spec")
-            url_var = var_name.call("#{name}_url")
-            path_var = var_name.call("#{name}_path")
-
-            file.write(<<-JAVASCRIPT.strip_heredoc)
-              const #{spec_var} = [#{js(spec)}, #{js(segment_keys)}, { ...#{default_url_options_var}, ...#{js(options)} }]
-              export const #{url_var} = (...args) => urlFor(#{spec_var}, ...args)
-              export const #{path_var} = (...args) => pathFor(#{spec_var}, ...args)
+        js_file = Webpacker.config.routes_path.join('index.js')
+
+        catch(:identical) do
+          File.atomic_write(js_file) do |temp_file|
+            temp_file.write(<<-JAVASCRIPT.strip_heredoc)
+              import { urlFor, pathFor } from 'webpacker-routes'
+              const #{default_url_options_var} = #{js(default_url_options)}
             JAVASCRIPT
+
+            app.routes.named_routes.sort_by(&:first).each do |name, route|
+              raise `Invalid route name for javascript: ${name}` unless JAVASCRIPT_VARIABLE_NAME_REGEX =~ name
+
+              spec = route.path.spec.to_s
+              segment_keys = route.segment_keys.uniq
+              options = route.defaults.except(*IGNORED_OPTIONS)
+
+              spec_var = var_name.call("#{name}_spec")
+              url_var = var_name.call("#{name}_url")
+              path_var = var_name.call("#{name}_path")
+
+              temp_file.write(<<-JAVASCRIPT.strip_heredoc)
+                const #{spec_var} = [#{js(spec)}, #{js(segment_keys)}, { ...#{default_url_options_var}, ...#{js(options)} }]
+                export const #{url_var} = (...args) => urlFor(#{spec_var}, ...args)
+                export const #{path_var} = (...args) => pathFor(#{spec_var}, ...args)
+              JAVASCRIPT
+            end
+
+            temp_file.close
+            if identical?(js_file.to_s, temp_file.path)
+              temp_file.unlink
+              throw :identical
+            end
           end
         end
       end
@@ -55,6 +65,12 @@ module Webpacker
       def js(obj)
         ERB::Util.json_escape(obj.to_json)
       end
+
+      def identical?(path1, path2)
+        FileUtils.compare_file(path1, path2)
+      rescue Errno::ENOENT
+        false
+      end
     end
   end
 end

data/lib/webpacker/routes/railtie.rb

@@ -8,13 +8,12 @@ module Webpacker
       config.webpacker.routes.camel_case = false
 
       config.after_initialize do |app|
-        generate = -> { Webpacker::Routes.generate(app.tap(&:reload_routes!)) }
         if Rails::VERSION::MAJOR >= 5
-          app.reloader.to_run(&generate)
+          app.reloader.to_run(:after) { Webpacker::Routes.generate(app) }
         else
-          ActionDispatch::Reloader.to_prepare(&generate)
+          ActionDispatch::Reloader.to_prepare(:after) { Webpacker::Routes.generate(app) }
         end
-        generate.call unless ENV['WEBPACKER_ROUTES_INSTALL'] == 'true'
+        Webpacker::Routes.generate(app.tap(&:reload_routes!)) unless ENV['WEBPACKER_ROUTES_INSTALL']
       end
     end
   end

data/lib/webpacker/routes/version.rb

@@ -1,5 +1,5 @@
 module Webpacker
   module Routes
-    VERSION = '0.1.1'
+    VERSION = '0.1.2'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: webpacker-routes
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - David Harsha
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-03-18 00:00:00.000000000 Z
+date: 2019-12-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -100,8 +100,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Convert Rails routes to JavaScript modules