js-routes 2.2.0 → 2.2.3

data/lib/routes.js

@@ -2,7 +2,7 @@
  * File generated by js-routes RubyVariables.GEM_VERSION
  * Based on Rails RubyVariables.RAILS_VERSION routes of RubyVariables.APP_CLASS
  */
-RubyVariables.WRAPPER((that) => {
+RubyVariables.WRAPPER(() => {
     const hasProp = (value, key) => Object.prototype.hasOwnProperty.call(value, key);
     let NodeTypes;
     (function (NodeTypes) {
@@ -15,8 +15,7 @@ RubyVariables.WRAPPER((that) => {
         NodeTypes[NodeTypes["SLASH"] = 7] = "SLASH";
         NodeTypes[NodeTypes["DOT"] = 8] = "DOT";
     })(NodeTypes || (NodeTypes = {}));
-    const Root = that;
-    const isBroswer = typeof window !== "undefined";
+    const isBrowser = typeof window !== "undefined";
     const ModuleReferences = {
         CJS: {
             define(routes) {
@@ -70,11 +69,11 @@ RubyVariables.WRAPPER((that) => {
             },
         },
         NIL: {
-            define(routes) {
-                Utils.namespace(Root, RubyVariables.NAMESPACE, routes);
+            define() {
+                // Defined using RubyVariables.WRAPPER
             },
             isSupported() {
-                return !RubyVariables.NAMESPACE || !!Root;
+                return true;
             },
         },
         DTS: {
@@ -422,15 +421,15 @@ RubyVariables.WRAPPER((that) => {
         }
         current_host() {
             var _a;
-            return (isBroswer && ((_a = window === null || window === void 0 ? void 0 : window.location) === null || _a === void 0 ? void 0 : _a.hostname)) || "";
+            return (isBrowser && ((_a = window === null || window === void 0 ? void 0 : window.location) === null || _a === void 0 ? void 0 : _a.hostname)) || "";
         }
         current_protocol() {
             var _a, _b;
-            return ((isBroswer && ((_b = (_a = window === null || window === void 0 ? void 0 : window.location) === null || _a === void 0 ? void 0 : _a.protocol) === null || _b === void 0 ? void 0 : _b.replace(/:$/, ""))) || "http");
+            return ((isBrowser && ((_b = (_a = window === null || window === void 0 ? void 0 : window.location) === null || _a === void 0 ? void 0 : _a.protocol) === null || _b === void 0 ? void 0 : _b.replace(/:$/, ""))) || "http");
         }
         current_port() {
             var _a;
-            return (isBroswer && ((_a = window === null || window === void 0 ? void 0 : window.location) === null || _a === void 0 ? void 0 : _a.port)) || "";
+            return (isBrowser && ((_a = window === null || window === void 0 ? void 0 : window.location) === null || _a === void 0 ? void 0 : _a.port)) || "";
         }
         is_object(value) {
             return (typeof value === "object" &&
@@ -445,21 +444,6 @@ RubyVariables.WRAPPER((that) => {
         is_reserved_option(key) {
             return ReservedOptions.includes(key);
         }
-        namespace(object, namespace, routes) {
-            const parts = (namespace === null || namespace === void 0 ? void 0 : namespace.split(".")) || [];
-            if (parts.length === 0) {
-                return;
-            }
-            for (let index = 0; index < parts.length; index++) {
-                const part = parts[index];
-                if (index < parts.length - 1) {
-                    object = object[part] || (object[part] = {});
-                }
-                else {
-                    object[part] = routes;
-                }
-            }
-        }
         configure(new_config) {
             this.configuration = { ...this.configuration, ...new_config };
             return this.configuration;
@@ -502,4 +486,4 @@ RubyVariables.WRAPPER((that) => {
     };
     Utils.define_module(RubyVariables.MODULE_TYPE, result);
     return result;
-})(this);
+})();

data/lib/routes.ts

@@ -84,7 +84,6 @@ declare const RubyVariables: {
   SPECIAL_OPTIONS_KEY: string;
   DEFAULT_URL_OPTIONS: RouteParameters;
   SERIALIZER: Serializer;
-  NAMESPACE: string;
   ROUTES_OBJECT: RouteHelpers;
   MODULE_TYPE: ModuleType;
   WRAPPER: <T>(callback: T) => T;
@@ -97,7 +96,7 @@ declare const define:
 declare const module: { exports: any } | undefined;
 
 RubyVariables.WRAPPER(
-  (that: unknown): RouterExposedMethods => {
+  (): RouterExposedMethods => {
     const hasProp = (value: unknown, key: string) =>
       Object.prototype.hasOwnProperty.call(value, key);
     enum NodeTypes {
@@ -128,8 +127,7 @@ RubyVariables.WRAPPER(
       [T in keyof RouteNodes]: RouteNode<T>;
     }[keyof RouteNodes];
 
-    const Root = that;
-    const isBroswer = typeof window !== "undefined";
+    const isBrowser = typeof window !== "undefined";
     type ModuleDefinition = {
       define: (routes: RouterExposedMethods) => void;
       isSupported: () => boolean;
@@ -186,11 +184,11 @@ RubyVariables.WRAPPER(
         },
       },
       NIL: {
-        define(routes) {
-          Utils.namespace(Root, RubyVariables.NAMESPACE, routes);
+        define() {
+          // Defined using RubyVariables.WRAPPER
         },
         isSupported() {
-          return !RubyVariables.NAMESPACE || !!Root;
+          return true;
         },
       },
       DTS: {
@@ -628,17 +626,17 @@ RubyVariables.WRAPPER(
       }
 
       current_host(): string {
-        return (isBroswer && window?.location?.hostname) || "";
+        return (isBrowser && window?.location?.hostname) || "";
       }
 
       current_protocol(): string {
         return (
-          (isBroswer && window?.location?.protocol?.replace(/:$/, "")) || "http"
+          (isBrowser && window?.location?.protocol?.replace(/:$/, "")) || "http"
         );
       }
 
       current_port(): string {
-        return (isBroswer && window?.location?.port) || "";
+        return (isBrowser && window?.location?.port) || "";
       }
 
       is_object(value: unknown): value is Record<string, unknown> {
@@ -660,25 +658,6 @@ RubyVariables.WRAPPER(
         return ReservedOptions.includes(key as any);
       }
 
-      namespace(
-        object: any,
-        namespace: string | null | undefined,
-        routes: unknown
-      ): void {
-        const parts = namespace?.split(".") || [];
-        if (parts.length === 0) {
-          return;
-        }
-        for (let index = 0; index < parts.length; index++) {
-          const part = parts[index];
-          if (index < parts.length - 1) {
-            object = object[part] || (object[part] = {});
-          } else {
-            object[part] = routes;
-          }
-        }
-      }
-
       configure(new_config: Partial<Configuration>): Configuration {
         this.configuration = { ...this.configuration, ...new_config };
         return this.configuration;
@@ -734,4 +713,4 @@ RubyVariables.WRAPPER(
     Utils.define_module(RubyVariables.MODULE_TYPE, result);
     return result;
   }
-)(this);
+)();

data/spec/js_routes/module_types/dts/routes.spec.d.ts

@@ -66,7 +66,6 @@ declare const RubyVariables: {
     SPECIAL_OPTIONS_KEY: string;
     DEFAULT_URL_OPTIONS: RouteParameters;
     SERIALIZER: Serializer;
-    NAMESPACE: string;
     ROUTES_OBJECT: RouteHelpers;
     MODULE_TYPE: ModuleType;
     WRAPPER: <T>(callback: T) => T;

data/spec/js_routes/module_types/nil_spec.rb

@@ -45,6 +45,7 @@ describe JsRoutes, "compatibility with NIL (legacy browser)" do
     let(:_presetup) { "" }
     before do
       evaljs("var window = this;")
+      evaljs("window.PHM = {}")
       evaljs(_presetup)
       evaljs(generated_js)
     end

data/spec/js_routes/options_spec.rb

@@ -399,13 +399,16 @@ describe JsRoutes, "options" do
       end
 
       let(:_presetup) do
-        window = {location: {
+        location =  {
           protocol: current_protocol,
           hostname: current_hostname,
           port: current_port,
           host: current_host,
-        }}
-        "const window = #{ActiveSupport::JSON.encode(window)}"
+        }
+        [
+          "const window = this;",
+          "window.location = #{ActiveSupport::JSON.encode(location)};",
+        ].join("\n")
       end
 
       context "without specifying a default host" do

data/spec/js_routes/rails_routes_compatibility_spec.rb

@@ -2,8 +2,11 @@ require "spec_helper"
 
 describe JsRoutes, "compatibility with Rails"  do
 
+  let(:generated_js) do
+    JsRoutes.generate({module_type: nil, namespace: 'Routes'})
+  end
   before(:each) do
-    evaljs(JsRoutes.generate({module_type: nil, namespace: 'Routes'}))
+    evaljs(generated_js)
   end
 
   it "should generate collection routing" do

data/spec/spec_helper.rb

@@ -42,10 +42,14 @@ end
 def evaljs(string, force: false, filename: 'context.js')
   jscontext(force).eval(string, filename: filename)
 rescue MiniRacer::ParseError => e
-  message = e.message
-  _, _, line, _ = message.split(':')
-  code = line && string.split("\n")[line.to_i-1]
-  raise "#{message}. Code: #{code.strip}";
+  trace = e.message
+  _, _, line, _ = trace.split(':')
+  if line
+    code = string.split("\n")[line.to_i-1]
+    raise "#{trace}. Code: #{code.strip}";
+  else
+    raise e
+  end
 rescue MiniRacer::RuntimeError => e
   raise e
 end
@@ -62,6 +66,10 @@ def planner_routes
   Planner::Engine.routes.url_helpers
 end
 
+def log(string)
+  evaljs("console.log(#{string})")
+end
+
 ActiveSupport::Inflector.inflections do |inflect|
   inflect.irregular "budgie", "budgies"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: js-routes
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 2.2.3
 platform: ruby
 authors:
 - Bogdan Gusiev
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-12-21 00:00:00.000000000 Z
+date: 2022-03-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
@@ -170,10 +170,13 @@ files:
 - js-routes.gemspec
 - lib/js-routes.rb
 - lib/js_routes.rb
+- lib/js_routes/configuration.rb
 - lib/js_routes/engine.rb
 - lib/js_routes/generators/middleware.rb
 - lib/js_routes/generators/webpacker.rb
+- lib/js_routes/instance.rb
 - lib/js_routes/middleware.rb
+- lib/js_routes/route.rb
 - lib/js_routes/version.rb
 - lib/routes.d.ts
 - lib/routes.js