rails-footnotes 5.0.0 → 7.0.1

data/{README.rdoc → README.md}

@@ -1,4 +1,4 @@
-= Rails 6 Footnotes
+# Rails 7 Footnotes
 
 Rails footnotes displays footnotes in your application for easy debugging, such as sessions,
 request parameters, cookies, filter chain, routes, queries, etc.
@@ -6,34 +6,37 @@ request parameters, cookies, filter chain, routes, queries, etc.
 Even more, it contains links to open files directly in your editor including
 your backtrace lines.
 
-== Installation
+### Installation
 
-Add to your `Gemfile`:
+1. Add to your `Gemfile` with `bundle add rails-footnotes`
+2. Run `bundle install`
+3. Generate the initializer with `bin/rails generate rails_footnotes:install`
 
-    gem "rails-footnotes", "~> 5.0"
-
-Run `bundle install`, then the generator:
+This will create an initializer with default config and some examples.
 
-  rails generate rails_footnotes:install
+### Hooks
 
-This will create an initializer with default config and some examples.
+You can run blocks before and after footnotes are evaluated.
 
-=== Hooks
+```
+Footnotes.setup do |config|
+  config.before do |controller, filter|
+    filter.notes = (controller.class.name =~ /Message/ && controller.action_name == 'index' ? [:assigns] : [])
+  end
 
-    Footnotes.setup do |config|
-      config.before {|controller, filter| filter.notes = controller.class.name =~ /Message/ && \
-        controller.action_name == 'index' ? [:assigns] : []}
-      config.before {|controller, filter| filter.notes |= [:params] if controller.class.name =~ /Profile/ && \
-        controller.action_name == 'edit' }
-    end
+  config.before do |controller, filter|
+    filter.notes |= [:params] if controller.class.name =~ /Profile/ && controller.action_name == 'edit'
+  end
+end
+```
 
-=== Editor links
+### Editor links
 
-Textmate, MacVim and Sublime Text 3 are compatible.
+By default, files are linked to open in TextMate, but you can use any editor with a URL scheme. Here are some examples for other editors:
 
-*MacVim*
+**MacVim**
 
-In the rails-footnotes initializer do :
+In `config/initializers/rails-footnotes.rb` do:
 
     f.prefix = 'mvim://open?url=file://%s&line=%d&column=%d'
 
@@ -44,26 +47,26 @@ the second %d is replaced by the column number.
 Take note that the order in which the file name (%s), line number (%d) and column number (%d) appears is important.
 We assume that they appear in that order. "foo://line=%d&file=%s" (%d precedes %s) would throw out an error.
 
-*Sublime* *Text* *3*
+**Sublime Text 3**
 
-Install {subl}[https://github.com/dhoulb/subl], then use:
+Install [subl](https://github.com/dhoulb/subl), then use:
 
     f.prefix = 'subl://open?url=file://%s&line=%d&column=%d'
 
-*Use* *with* *Vagrant* (*and* *other* *virtual* *machines*)
+**Use with Docker, Vagrant, or other virtual machines**
 
-If you're running your app in Vagrant, you'll find that the edit links won't work because the paths point to the Vagrant directory not your native directory. To solve this, you can use a lambda for the prefix and modify the pathname accordingly.
+If you're running your app in a container or VM, you'll find that the edit links won't work because the paths point to the VM directory and not your native directory. To solve this, you can use a lambda for the prefix and modify the pathname accordingly.
 
 For example,
 
     f.prefix = ->(*args) do
-      filename = args[0].sub '/vagrant', '/Users/jamie/projects/myproject'
+      filename = args[0].sub '/docker', '/Users/name/projects/myproject'
       "subl://open?url=file://#{filename}&line=#{args[1]}&column=#{args[2]}"
     end
 
-replaces the vm directory /vagrant with OS X directory where the code is being edited.
+replaces the VM directory /docker with the macOS directory containing the source code.
 
-*Footnotes* *Display* *Options*
+**Footnotes Display Options**
 
 By default, footnotes are appended at the end of the page with default stylesheet. If you want
 to change their position, you can define a div with id "footnotes_holder" or define your own stylesheet
@@ -90,7 +93,7 @@ Finally, you can control which notes you want to show. The default are:
 
 Setting <tt>f.notes = []</tt> will show none of the available notes, although the supporting CSS and JavaScript will still be included. To completely disable all rails-footnotes content on a page, include <tt>params[:footnotes] = 'false'</tt> in the request.
 
-== Creating your own notes
+### Creating your own notes
 
 Creating your notes to integrate with Footnotes is easy.
 
@@ -100,49 +103,52 @@ Creating your notes to integrate with Footnotes is easy.
 
 For example, to create a note that shows info about the user logged in your application you just have to do:
 
-  module Footnotes
-    module Notes
-      class CurrentUserNote < AbstractNote
-        # This method always receives a controller
-        #
-        def initialize(controller)
-          @current_user = controller.instance_variable_get("@current_user")
-        end
-
-        # Returns the title that represents this note.
-        #
-        def title
-          "Current user: #{@current_user.name}"
-        end
-
-        # This Note is only valid if we actually found an user
-        # If it's not valid, it won't be displayed
-        #
-        def valid?
-          @current_user
-        end
-
-        # The fieldset content
-        #
-        def content
-          escape(@current_user.inspect)
+    module Footnotes
+      module Notes
+        class CurrentUserNote < AbstractNote
+          # This method always receives a controller
+          #
+          def initialize(controller)
+            @current_user = controller.instance_variable_get("@current_user")
+          end
+
+          # Returns the title that represents this note.
+          #
+          def title
+            "Current user: #{@current_user.name}"
+          end
+
+          # This Note is only valid if we actually found an user
+          # If it's not valid, it won't be displayed
+          #
+          def valid?
+            @current_user
+          end
+
+          # The fieldset content
+          #
+          def content
+            escape(@current_user.inspect)
+          end
         end
       end
     end
-  end
 
 Then put in your environment, add in your initializer:
 
-  f.notes += [:current_user]
+    f.notes += [:current_user]
 
-== Footnote position
+### Footnote position
 
-By default the notes will be showed at the bottom of your page (appended just before </body>).
+By default the notes will be showed at the bottom of your page (appended just before `</body>`).
 If you'd like the footnote, to be at a different place (perhaps for aesthetical reasons) you can edit one of your views and add:
-  <div id="footnotes_holder"></div>
+
+    <div id="footnotes_holder"></div>
+
 at an appropriate place, your notes will now appear inside div#footnotes_holder
 
-== Bugs and Feedback
+### Bugs and Feedback
 
 If you discover any bugs, please open an issue.
 If you just want to give some positive feedback or drop a line, that's fine too!
+

data/lib/generators/templates/rails_footnotes.rb

@@ -21,6 +21,9 @@ Footnotes.setup do |f|
   # Change font size :
   # f.font_size = '11px'
 
+  # Change default limit :
+  # f.default_limit = 25
+
   # Allow to open multiple notes :
   # f.multiple_notes = true
 end if defined?(Footnotes) && Footnotes.respond_to?(:setup)

data/lib/rails-footnotes/filter.rb

@@ -1,26 +1,37 @@
 module Footnotes
   class Filter
-    @@no_style = false
-    @@multiple_notes = false
     @@klasses = []
-    @@lock_top_right = false
-    @@font_size = '11px'
-
-    # Default link prefix is textmate
-    @@prefix = 'txmt://open?url=file://%s&line=%d&column=%d'
-
-    # Edit notes
-    @@notes = [ :controller, :view, :layout, :partials, :stylesheets, :javascripts ]
-    # Show notes
-    @@notes += [ :assigns, :session, :cookies, :params, :filters, :routes, :env, :queries, :log]
 
+    # :default_limit  => Default limit for ActiveRecord:Relation in assigns note
+    # :font_size      => CSS font-size property
+    # :lock_top_right => Lock a btn to toggle notes to the top right of the browser
+    # :multiple_notes => Set to true if you want to open several notes at the same time
     # :no_style       => If you don't want the style to be appended to your pages
-    # :notes          => Class variable that holds the notes to be processed
     # :prefix         => Prefix appended to FootnotesLinks
-    # :multiple_notes => Set to true if you want to open several notes at the same time
-    # :lock_top_right => Lock a btn to toggle notes to the top right of the browser
-    # :font_size      => CSS font-size property
-    cattr_accessor :no_style, :notes, :prefix, :multiple_notes, :lock_top_right, :font_size
+    # :notes          => Class variable that holds the notes to be processed
+    thread_cattr_accessor :default_limit, default: 25
+    thread_cattr_accessor :font_size, default: '11px'
+    thread_cattr_accessor :lock_top_right, default: false
+    thread_cattr_accessor :multiple_notes, default: false
+    thread_cattr_accessor :no_style, default: false
+    thread_cattr_accessor :prefix, default: 'txmt://open?url=file://%s&line=%d&column=%d'
+    thread_cattr_accessor :notes, default: [
+      :assigns,
+      :controller,
+      :cookies,
+      :env,
+      :filters,
+      :javascripts,
+      :layout,
+      :log,
+      :params,
+      :partials,
+      :queries,
+      :routes,
+      :session,
+      :stylesheets,
+      :view
+    ]
 
     class << self
       include Footnotes::EachWithRescue
@@ -33,7 +44,7 @@ module Footnotes
         self.each_with_rescue(Footnotes.before_hooks) {|hook| hook.call(controller, self)}
 
         @@klasses = []
-        self.each_with_rescue(@@notes.flatten) do |note|
+        self.each_with_rescue(notes.flatten) do |note|
           klass = "Footnotes::Notes::#{note.to_s.camelize}Note".constantize
           klass.start!(controller) if klass.respond_to?(:start!)
           @@klasses << klass
@@ -42,17 +53,17 @@ module Footnotes
 
       # If none argument is sent, simply return the prefix.
       # Otherwise, replace the args in the prefix.
-      #
+      alias_method :read_prefix, :prefix
       def prefix(*args)
         if args.empty?
-          @@prefix
+          read_prefix
         else
           args.map! { |arg| arg.to_s.split("/").map{|s| ERB::Util.url_encode(s) }.join("/") }
 
-          if @@prefix.respond_to? :call
-            @@prefix.call *args
+          if read_prefix.respond_to? :call
+            read_prefix.call(*args)
           else
-            format(@@prefix, *args)
+            format(read_prefix, *args)
           end
         end
       end
@@ -65,7 +76,9 @@ module Footnotes
       @notes = []
 
       revert_pos(controller.response_body) do
-        @body = controller.response.body
+        if controller.response.stream.respond_to?(:body)
+          @body = controller.response.body
+        end
       end
     end
 
@@ -93,7 +106,7 @@ module Footnotes
 
       def add_footnotes_without_validation!
         initialize_notes!
-        insert_styles unless @@no_style
+        insert_styles unless no_style
         insert_footnotes
       end
 
@@ -147,7 +160,7 @@ module Footnotes
 
       def insert_styles
         #TODO More customizable(reset.css, from file etc.)
-        if @@lock_top_right
+        if lock_top_right
           extra_styles = <<-STYLES
             #footnotes_debug {position: fixed; top: 0px; right: 0px; width: 100%; z-index: 10000; margin-top: 0;}
             #footnotes_debug #toggle_footnotes {position: absolute; right: 0; top: 0; background: #fff; border: 1px solid #ccc; color: #9b1b1b; font-size: 20px; text-align: center; padding: 8px; opacity: 0.9;}
@@ -163,7 +176,7 @@ module Footnotes
         insert_text :before, /<\/head>/i, <<-HTML
         <!-- Footnotes Style -->
         <style type="text/css">
-          #footnotes_debug {font-size: #{@@font_size}; font-family: Consolas, monaco, monospace; font-weight: normal; margin: 2em 0 1em 0; text-align: center; color: #444; line-height: 16px; background: #fff;}
+          #footnotes_debug {font-size: #{font_size}; font-family: Consolas, monaco, monospace; font-weight: normal; margin: 2em 0 1em 0; text-align: center; color: #444; line-height: 16px; background: #fff;}
           #footnotes_debug th, #footnotes_debug td {color: #444; line-height: 18px;}
           #footnotes_debug a {color: #9b1b1b; font-weight: inherit; text-decoration: none; line-height: 18px;}
           #footnotes_debug table {text-align: left; width: 100%;}
@@ -188,7 +201,7 @@ module Footnotes
         # Fieldsets method should be called first
         content = fieldsets
         element_style = ''
-        if @@lock_top_right
+        if lock_top_right
           element_style = 'style="display: none;"'
         end
         footnotes_html = <<-HTML
@@ -204,7 +217,7 @@ module Footnotes
             var Footnotes = function() {
 
               function hideAll(){
-                #{close unless @@multiple_notes}
+                #{close unless multiple_notes}
               }
 
               function hideAllAndToggle(id) {

data/lib/rails-footnotes/notes/assigns_note.rb

@@ -1,24 +1,24 @@
+require "active_record"
+
 module Footnotes
   module Notes
     class AssignsNote < AbstractNote
-      @@ignored_assigns = [
-                            :@real_format,
-                            :@before_filter_chain_aborted,
-                            :@performed_redirect,
-                            :@performed_render,
-                            :@_params,
-                            :@_response,
-                            :@url,
-                            :@template,
-                            :@_request,
-                            :@db_rt_before_render,
-                            :@db_rt_after_render,
-                            :@view_runtime,
-                            :@marked_for_same_origin_verification
-                          ]
-      cattr_accessor :ignored_assigns, :instance_writer => false
-      @@ignored_assigns_pattern = /^@_/
-      cattr_accessor :ignored_assigns_pattern, :instance_writer => false
+      thread_cattr_accessor :ignored_assigns, instance_writer: false, default: [
+        :@real_format,
+        :@before_filter_chain_aborted,
+        :@performed_redirect,
+        :@performed_render,
+        :@_params,
+        :@_response,
+        :@url,
+        :@template,
+        :@_request,
+        :@db_rt_before_render,
+        :@db_rt_after_render,
+        :@view_runtime,
+        :@marked_for_same_origin_verification
+      ]
+      thread_cattr_accessor :ignored_assigns_pattern, instance_writer: false, default: /^@_/
 
       def initialize(controller)
         @controller = controller
@@ -40,8 +40,11 @@ module Footnotes
         def to_table
           table = assigns.inject([]) do |rr, var|
             class_name = assigned_value(var).class.name
-            var_name = var.to_s
-            rr << ["<strong>#{var.to_s}</strong>" + "<br /><em>#{class_name}</em>", escape(assigned_value(var).inspect)]
+            var_value = assigned_value(var)
+            if var_value.is_a?(ActiveRecord::Relation) && var_value.limit_value.nil?
+              var_value = var_value.limit(Footnotes::Filter.default_limit)
+            end
+            rr << ["<strong>#{var.to_s}</strong>" + "<br /><em>#{class_name}</em>", escape(var_value.inspect)]
           end
 
           table.unshift(['Name', 'Value'])

data/lib/rails-footnotes/notes/controller_note.rb

@@ -14,7 +14,7 @@ module Footnotes
       end
 
       def valid?
-        prefix? && controller_filename && File.exists?(controller_filename)
+        prefix? && controller_filename && File.exist?(controller_filename)
       end
 
       protected
@@ -23,7 +23,7 @@ module Footnotes
         end
 
         def controller_filename
-          @controller_filename ||= Gem.find_files(controller_path).first # tnx https://github.com/MasterLambaster
+          @controller_filename ||= Gem.find_files(controller_path).first
         end
 
         def controller_text

data/lib/rails-footnotes/notes/log_note/note_logger.rb

@@ -15,44 +15,6 @@ module Footnotes
           formatter = @formatter || Logger::Formatter.new
           @logs << formatter.call(format_severity(severity), Time.now, message, progname)
         end
-
-        ## Backport from rails 4 for handling logging broadcast, should be removed when rails 3 is deprecated :
-
-        # Broadcasts logs to multiple loggers.
-        def self.broadcast(logger) # :nodoc:
-          Module.new do
-            define_method(:add) do |*args, &block|
-              logger.add(*args, &block)
-              super(*args, &block)
-            end
-
-            define_method(:<<) do |x|
-              logger << x
-              super(x)
-            end
-
-            define_method(:close) do
-              logger.close
-              super()
-            end
-
-            define_method(:progname=) do |name|
-              logger.progname = name
-              super(name)
-            end
-
-            define_method(:formatter=) do |formatter|
-              logger.formatter = formatter
-              super(formatter)
-            end
-
-            define_method(:level=) do |level|
-              logger.level = level
-              super(level)
-            end
-          end
-        end
-
       end
     end
   end

data/lib/rails-footnotes/notes/log_note.rb

@@ -4,8 +4,8 @@ module Footnotes
 
       autoload :NoteLogger, 'rails-footnotes/notes/log_note/note_logger'
 
-      cattr_accessor :logs
-      cattr_accessor :original_logger
+      thread_cattr_accessor :logs
+      thread_cattr_accessor :original_logger
 
       def self.start!(controller)
         self.logs = []
@@ -18,9 +18,12 @@ module Footnotes
           else
             defined?(ActiveSupport::Logger) ? ActiveSupport::Logger::SimpleFormatter.new : Logger::SimpleFormatter.new
           end
-        # Rails 3 don't have ActiveSupport::Logger#broadcast so we backported it
-        extend_module = defined?(ActiveSupport::Logger) ? ActiveSupport::Logger.broadcast(note_logger) : NoteLogger.broadcast(note_logger)
-        Rails.logger = self.original_logger.clone.extend(extend_module)
+
+        if ::Rails::VERSION::STRING < "7.1"
+          ::Rails.logger.extend(::ActiveSupport::Logger.broadcast(note_logger))
+        else
+          ::Rails.logger = ::ActiveSupport::BroadcastLogger.new(::Rails.logger, note_logger)
+        end
       end
 
       def title

data/lib/rails-footnotes/notes/partials_note.rb

@@ -2,12 +2,12 @@ module Footnotes
   module Notes
     class PartialsNote < AbstractNote
 
-      cattr_accessor :partials
+      thread_cattr_accessor :partials
 
       def self.start!(controller)
         self.partials = []
         @subscriber ||= ActiveSupport::Notifications.subscribe('render_partial.action_view') do |*args|
-          event = ActiveSupport::Notifications::Event.new *args
+          event = ActiveSupport::Notifications::Event.new(*args)
           self.partials << {:file => event.payload[:identifier], :duration => event.duration}
         end
       end

data/lib/rails-footnotes/notes/queries_note.rb

@@ -1,12 +1,10 @@
 module Footnotes
   module Notes
     class QueriesNote < AbstractNote
-      cattr_accessor :alert_db_time, :alert_sql_number, :orm, :ignored_regexps, :instance_writer => false
-      @@alert_db_time    = 16.0
-      @@alert_sql_number = 8
-      @@query_subscriber = nil
-      @@orm              = [:active_record, :data_mapper]
-      @@ignored_regexps  = [%r{(pg_table|pg_attribute|pg_namespace|show\stables|pragma|sqlite_master)}i]
+      thread_cattr_accessor :alert_db_time, default: 16.0, instance_writer: false
+      thread_cattr_accessor :alert_sql_number, default: 8, instance_writer: false
+      thread_cattr_accessor :orm, default: [:active_record, :data_mapper], instance_writer: false
+      thread_cattr_accessor :ignored_regexps, default: [%r{(pg_table|pg_attribute|pg_namespace|show\stables|pragma|sqlite_master)}i], instance_writer: false
 
       def self.start!(controller)
         self.query_subscriber.reset!

data/lib/rails-footnotes/notes/view_note.rb

@@ -1,11 +1,11 @@
 module Footnotes
   module Notes
     class ViewNote < AbstractNote
-      cattr_accessor :template
+      thread_cattr_accessor :template
 
       def self.start!(controller)
         @subscriber ||= ActiveSupport::Notifications.subscribe('render_template.action_view') do |*args|
-          event = ActiveSupport::Notifications::Event.new *args
+          event = ActiveSupport::Notifications::Event.new(*args)
           self.template = {:file => event.payload[:identifier], :duration => event.duration}
         end
       end
@@ -19,7 +19,7 @@ module Footnotes
       end
 
       def title
-        "View (#{"%.3f" % self.template[:duration]}ms)"
+        "View (#{"%.3f" % template[:duration]}ms)"
       end
 
       def link
@@ -27,14 +27,14 @@ module Footnotes
       end
 
       def valid?
-        prefix? && filename && File.exists?(filename)
+        prefix? && filename && File.exist?(filename)
       end
 
       protected
 
         def filename
           return @filename if defined?(@filename)
-          @filename = self.class.template.try(:[], :file)
+          @filename = template.try(:[], :file)
         end
 
     end

data/lib/rails-footnotes/version.rb

@@ -1,3 +1,3 @@
 module Footnotes
-  VERSION = "5.0.0"
+  VERSION = "7.0.1"
 end

data/lib/rails-footnotes.rb

@@ -1,5 +1,6 @@
 require 'rails'
 require 'action_controller'
+require 'active_support/core_ext/module/attribute_accessors_per_thread'
 require 'rails-footnotes/abstract_note'
 require 'rails-footnotes/each_with_rescue'
 require 'rails-footnotes/filter'
@@ -7,14 +8,9 @@ require 'rails-footnotes/notes/all'
 require 'rails-footnotes/extension'
 
 module Footnotes
-  mattr_accessor :before_hooks
-  @@before_hooks = []
-
-  mattr_accessor :after_hooks
-  @@after_hooks = []
-
-  mattr_accessor :enabled
-  @@enabled = false
+  thread_mattr_accessor :before_hooks
+  thread_mattr_accessor :after_hooks
+  thread_mattr_accessor :enabled, default: false
 
   class << self
     delegate :notes, :to => Filter
@@ -34,25 +30,28 @@ module Footnotes
 
     delegate :font_size, :to => Filter
     delegate :font_size=, :to => Filter
+
+    delegate :default_limit, :to => Filter
+    delegate :default_limit=, :to => Filter
   end
 
   def self.before(&block)
-    @@before_hooks << block
+    before_hooks << block
   end
 
   def self.after(&block)
-    @@after_hooks << block
+    after_hooks << block
   end
 
   def self.enabled?(controller)
-    if @@enabled.is_a? Proc
-      if @@enabled.arity == 1
-        @@enabled.call(controller)
+    if enabled.is_a? Proc
+      if enabled.arity == 1
+        enabled.call(controller)
       else
-        @@enabled.call
+        enabled.call
       end
     else
-      !!@@enabled
+      !!enabled
     end
   end
 
@@ -61,8 +60,11 @@ module Footnotes
   end
 end
 
+Footnotes.before_hooks = []
+Footnotes.after_hooks = []
+
 ActiveSupport.on_load(:action_controller) do
   ActionController::Base.send(:include, Footnotes::RailsFootnotesExtension)
 end
 
-load Rails.root.join('.rails_footnotes') if Rails.root && Rails.root.join('.rails_footnotes').exist?
+load Rails.root.join('.rails_footnotes') if Rails.root&.join('.rails_footnotes')&.exist?

data/rails-footnotes.gemspec

@@ -1,6 +1,4 @@
-# -*- encoding: utf-8 -*-
-$:.push File.expand_path("../lib", __FILE__)
-require "rails-footnotes/version"
+require_relative "./lib/rails-footnotes/version"
 
 Gem::Specification.new do |s|
   s.name        = "rails-footnotes"
@@ -12,8 +10,8 @@ Gem::Specification.new do |s|
   s.summary     = %q{Every Rails page has footnotes that gives information about your application and links back to your editor.}
   s.description = %q{Every Rails page has footnotes that gives information about your application and links back to your editor.}
 
-  s.add_dependency "rails", "~> 6.0"
-  s.required_ruby_version = "~> 3.0"
+  s.add_dependency "rails", "~> 7.0"
+  s.required_ruby_version = ">= 3.0"
 
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")