query_reviewer 0.1.6 → 0.1.7

data/lib/query_reviewer.rb

@@ -12,9 +12,9 @@ module QueryReviewer
     CONFIGURATION.merge!(default_config["all"] || {})
     CONFIGURATION.merge!(default_config[Rails.env || "test"] || {})
 
-    app_config_file = Rails.root + "config/query_reviewer.yml"
+    app_config_file = File.join(Rails.root, 'config/query_reviewer.yml')
 
-    if app_config_file.exist?
+    if File.file?(app_config_file)
       app_config = YAML.load(ERB.new(IO.read(app_config_file)).result)
       CONFIGURATION.merge!(app_config["all"] || {})
       CONFIGURATION.merge!(app_config[Rails.env || "test"] || {})

data/lib/query_reviewer/controller_extensions.rb

@@ -30,8 +30,6 @@ module QueryReviewer
     end
 
     def add_query_output_to_view(total_time)
-      return unless Thread.current["query_reviewer_enabled"]
-
       if request.xhr?
         if cookies["query_review_enabled"]
           if !response.content_type || response.content_type.include?("text/html")
@@ -41,7 +39,7 @@ module QueryReviewer
           end
         end
       else
-        if response.body.is_a?(String) && response.body.match(/<\/body>/i) && Thread.current["queries"]
+        if response.body.is_a?(String) && response.body.match(/<\/body>/i)
           idx = (response.body =~ /<\/body>/i)
           html = query_review_output(false, total_time)
           response.body = response.body.insert(idx, html)
@@ -50,8 +48,8 @@ module QueryReviewer
     end
 
     def perform_action_with_query_review(*args)
-      Thread.current["query_reviewer_enabled"] = (CONFIGURATION["enabled"] == true               and cookies["query_review_enabled"]) ||
-                                                 (CONFIGURATION["enabled"] == "based_on_session" and session["query_review_enabled"])
+      return unless (CONFIGURATION["enabled"] == true               and cookies["query_review_enabled"]) ||
+                    (CONFIGURATION["enabled"] == "based_on_session" and session["query_review_enabled"])
 
       t1 = Time.now
       r = defined?(Rails::Railtie) ? process_action_without_query_review(*args) : perform_action_without_query_review(*args)

data/lib/query_reviewer/views/_box_disabled.html.erb

@@ -1,2 +1,2 @@
 <p>SQL analysis has been disabled for you. A cookie must be set to enable analysis. This generally slows down your browser, so it's only recommended for users analyzing SQL queries.</p>
-<p id="query_review_enable_link"><b><a href="#" onclick="createCookie('query_review_enabled', '1'); query_review_hide('query_review_enable_link'); alert('Cookie successfully set.');">Enabled it</a> on next page load and from now on.</b></p>
+<p id="query_review_enable_link"><b><a href="#" onclick="createCookie('query_review_enabled', '1'); query_review_hide('query_review_enable_link'); alert('Cookie successfully set.');">Enable it</a> on next page load and from now on.</b></p>

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: query_reviewer
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.7
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-15 00:00:00.000000000 Z
+date: 2013-02-06 00:00:00.000000000 Z
 dependencies: []
 description: Runs explain before each select query and displays results in an overlayed
   div
@@ -50,7 +50,8 @@ files:
 - lib/query_reviewer/views/query_review_box_helper.rb
 - lib/query_reviewer.rb
 homepage: https://github.com/nesquena/query_reviewer
-licenses: []
+licenses:
+- MIT
 post_install_message: 
 rdoc_options: []
 require_paths: