htmlbeautifier 1.2.1 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 657518afa5b160297e6b490855c02526f48ad144
-  data.tar.gz: bfdfbca49d50e276f2314a893ecc0b70b6392115
+SHA256:
+  metadata.gz: 4ad32fe5c137d828c711e50642ff3b00af14e3b8cdf18105506c1149de2c6cde
+  data.tar.gz: d42bcce139d8a3e9a96f73735fbb068044f1f43c8d9951c89cd642bf4e996bef
 SHA512:
-  metadata.gz: 9a96bc60da54b69d972b80d5faf47a547dc7ab403e19c4ab6685dce4896a71a76bf9b7f49ce3b2c8039d8f0b661fe5c9b21c740116ac49e7e49313a11bd0a903
-  data.tar.gz: cfe7bb9dceaad5997e65ba6fdc397a0ce5a30ee3012e295c1a1fdc2dedcf3a34661d662b92c83ec28ea4d675f99c82a05c7a2924f8f3f13d32f12db094d82fb8
+  metadata.gz: 89dc1309a74d24da1f91ec626e3a0291918c9e0ef64370d10893ee30c961bcb788efa2bc99fbdb89af96d643cf418333b6b5b6eb3ec3e54dfb164398d0cdf0b5
+  data.tar.gz: accce8738133c66d40ddcbdc21aee322f7b68e2d9bcca81102fbecc680a5a3b021d8911587e31e0a1963f3cae166e2f436a76800feebde59da411d96cb5fe628

data/bin/htmlbeautifier

@@ -1,11 +1,14 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
+
 require "htmlbeautifier"
 require "optparse"
 require "fileutils"
+require "stringio"
 
 def beautify(name, input, output, options)
   output.puts HtmlBeautifier.beautify(input, options)
-rescue => e
+rescue StandardError => e
   raise "Error parsing #{name}: #{e}"
 end
 
@@ -14,18 +17,18 @@ executable = File.basename(__FILE__)
 options = { indent: "  " }
 parser = OptionParser.new do |opts|
   opts.banner = "Usage: #{executable} [options] [file ...]"
-  opts.separator <<END
+  opts.separator <<~STRING
 
-#{executable} has two modes of operation:
+    #{executable} has two modes of operation:
 
-1. If no files are listed, it will read from standard input and write to
-   standard output.
-2. If files are listed, it will modify each file in place, overwriting it
-   with the beautified output.
+    1. If no files are listed, it will read from standard input and write to
+       standard output.
+    2. If files are listed, it will modify each file in place, overwriting it
+       with the beautified output.
 
-The following options are available:
+    The following options are available:
 
-END
+  STRING
   opts.on(
     "-t", "--tab-stops NUMBER", Integer,
     "Set number of spaces per indent (default #{options[:tab_stops]})"
@@ -50,6 +53,26 @@ END
   ) do |num|
     options[:stop_on_errors] = num
   end
+  opts.on(
+    "-b", "--keep-blank-lines NUMBER", Integer,
+    "Set number of consecutive blank lines"
+  ) do |num|
+    options[:keep_blank_lines] = num
+  end
+  opts.on(
+    "-l", "--lint-only",
+    "Lint only, error on files which would be modified",
+    "This is not available when reading from standard input"
+  ) do |num|
+    options[:lint_only] = num
+  end
+  opts.on(
+    "-v", "--version",
+    "Display version and exit"
+  ) do
+    puts HtmlBeautifier::VERSION::STRING
+    exit
+  end
   opts.on(
     "-h", "--help",
     "Display this help message and exit"
@@ -58,16 +81,34 @@ END
     exit
   end
 end
+
 parser.parse!
 
 if ARGV.any?
+  failures = []
   ARGV.each do |path|
     input = File.read(path)
-    temppath = path + ".tmp"
-    File.open(temppath, "w") do |output|
+    if options[:lint_only]
+      output = StringIO.new
       beautify path, input, output, options
+      failures << path unless input == output.string
+    else
+      temppath = "#{path}.tmp"
+      File.open(temppath, "w") do |file|
+        beautify path, input, file, options
+      end
+      FileUtils.mv temppath, path
     end
-    FileUtils.mv temppath, path
+  end
+  unless failures.empty?
+    relative_paths = failures.map { |path|
+      Pathname.new(path).relative_path_from Dir.pwd
+    }
+    $stderr.puts [
+      "Lint failed - files would be modified:",
+      *relative_paths
+    ].join("\n")
+    exit 1
   end
 else
   beautify "standard input", $stdin.read, $stdout, options

data/lib/htmlbeautifier/builder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "htmlbeautifier/parser"
 require "htmlbeautifier/ruby_indenter"
 
@@ -6,14 +8,16 @@ module HtmlBeautifier
     DEFAULT_OPTIONS = {
       indent: "  ",
       initial_level: 0,
-      stop_on_errors: false
-    }
+      stop_on_errors: false,
+      keep_blank_lines: 0
+    }.freeze
 
     def initialize(output, options = {})
       options = DEFAULT_OPTIONS.merge(options)
       @tab = options[:indent]
       @stop_on_errors = options[:stop_on_errors]
       @level = options[:initial_level]
+      @keep_blank_lines = options[:keep_blank_lines]
       @new_line = false
       @empty = true
       @ie_cc_levels = []
@@ -25,6 +29,7 @@ module HtmlBeautifier
 
     def error(text)
       return unless @stop_on_errors
+
       raise text
     end
 
@@ -38,14 +43,15 @@ module HtmlBeautifier
     end
 
     def emit(*strings)
+      strings_join = strings.join("")
       @output << "\n" if @new_line && !@empty
-      @output << (@tab * @level) if @new_line
-      @output << strings.join("")
+      @output << (@tab * @level) if @new_line && !strings_join.strip.empty?
+      @output << strings_join
       @new_line = false
       @empty = false
     end
 
-    def new_line(*)
+    def new_line
       @new_line = true
     end
 
@@ -64,7 +70,7 @@ module HtmlBeautifier
 
     def emit_reindented_block_content(code)
       lines = code.strip.split(%r{\n})
-      indentation = lines.first[%r{^\s+}]
+      indentation = foreign_block_indentation(code)
 
       indent
       new_line
@@ -75,55 +81,63 @@ module HtmlBeautifier
       outdent
     end
 
+    def foreign_block_indentation(code)
+      code.split(%r{\n}).find { |ln| !ln.strip.empty? }[%r{^\s+}]
+    end
+
     def preformatted_block(opening, content, closing)
       new_line
       emit opening, content, closing
       new_line
     end
 
-    def standalone_element(e)
-      emit e
-      new_line if e =~ %r{^<br[^\w]}
+    def standalone_element(elem)
+      emit elem
+      new_line if elem =~ %r{^<br[^\w]}
     end
 
-    def close_element(e)
+    def close_element(elem)
       outdent
-      emit e
+      emit elem
     end
 
-    def close_block_element(e)
-      close_element e
+    def close_block_element(elem)
+      close_element elem
       new_line
     end
 
-    def open_element(e)
-      emit e
+    def open_element(elem)
+      emit elem
       indent
     end
 
-    def open_block_element(e)
+    def open_block_element(elem)
       new_line
-      open_element e
+      open_element elem
     end
 
-    def close_ie_cc(e)
+    def close_ie_cc(elem)
       if @ie_cc_levels.empty?
         error "Unclosed conditional comment"
       else
         @level = @ie_cc_levels.pop
       end
-      emit e
+      emit elem
     end
 
-    def open_ie_cc(e)
-      emit e
+    def open_ie_cc(elem)
+      emit elem
       @ie_cc_levels.push @level
       indent
     end
 
-    def text(t)
-      emit t.chomp
-      new_line if t.end_with?("\n")
+    def new_lines(*content)
+      blank_lines = content.first.scan(%r{\n}).count - 1
+      blank_lines = [blank_lines, @keep_blank_lines].min
+      @output << ("\n" * blank_lines)
+      new_line
     end
+
+    alias_method :text, :emit
   end
 end

data/lib/htmlbeautifier/html_parser.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "htmlbeautifier/parser"
 
 module HtmlBeautifier
@@ -35,20 +37,21 @@ module HtmlBeautifier
        :preformatted_block],
       [%r{<#{HTML_VOID_ELEMENTS}(?: #{ELEMENT_CONTENT})?/?>}om,
        :standalone_element],
-      [%r{<\w+(?: #{ELEMENT_CONTENT})?/>}om,
-       :standalone_element],
       [%r{</#{HTML_BLOCK_ELEMENTS}>}om,
        :close_block_element],
       [%r{<#{HTML_BLOCK_ELEMENTS}(?: #{ELEMENT_CONTENT})?>}om,
        :open_block_element],
       [%r{</#{ELEMENT_CONTENT}>}om,
        :close_element],
-      [%r{<#{ELEMENT_CONTENT}>}om,
+      [%r{<#{ELEMENT_CONTENT}[^/]>}om,
        :open_element],
-      [%r{\s*\r?\n\s*}om,
-       :new_line],
-      [%r{[^<]+},
-       :text]]
+      [%r{<[\w\-]+(?: #{ELEMENT_CONTENT})?/>}om,
+       :standalone_element],
+      [%r{(\s*\r?\n\s*)+}om,
+       :new_lines],
+      [%r{[^<\n]+},
+       :text]
+    ].freeze
 
     def initialize
       super do |p|

data/lib/htmlbeautifier/parser.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "strscan"
 
 module HtmlBeautifier
@@ -29,13 +31,15 @@ module HtmlBeautifier
     def dispatch(receiver)
       _, method = @maps.find { |pattern, _| @scanner.scan(pattern) }
       raise "Unmatched sequence" unless method
+
       receiver.__send__(method, *extract_params(@scanner))
-    rescue => ex
-      raise "#{ex.message} on line #{source_line_number}"
+    rescue StandardError => e
+      raise "#{e.message} on line #{source_line_number}"
     end
 
     def extract_params(scanner)
       return [scanner[0]] unless scanner[1]
+
       params = []
       i = 1
       while scanner[i]

data/lib/htmlbeautifier/ruby_indenter.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 module HtmlBeautifier
   class RubyIndenter
-    INDENT_KEYWORDS = %w[ if elsif else unless while until begin for ]
-    OUTDENT_KEYWORDS = %w[ elsif else end ]
-    RUBY_INDENT  = %r{
+    INDENT_KEYWORDS = %w[if elsif else unless while until begin for].freeze
+    OUTDENT_KEYWORDS = %w[elsif else end].freeze
+    RUBY_INDENT = %r{
       ^ ( #{INDENT_KEYWORDS.join("|")} )\b
-      | \b ( do | \{ ) ( \s* \| [^\|]+ \| )? $
+      | \b ( do | \{ ) ( \s* \| [^|]+ \| )? $
     }xo
     RUBY_OUTDENT = %r{ ^ ( #{OUTDENT_KEYWORDS.join("|")} | \} ) \b }xo
 

data/lib/htmlbeautifier/version.rb

@@ -1,7 +1,9 @@
-module HtmlBeautifier #:nodoc:
-  module VERSION #:nodoc:
+# frozen_string_literal: true
+
+module HtmlBeautifier # :nodoc:
+  module VERSION # :nodoc:
     MAJOR = 1
-    MINOR = 2
+    MINOR = 4
     TINY  = 1
 
     STRING = [MAJOR, MINOR, TINY].join(".")

data/lib/htmlbeautifier.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "htmlbeautifier/builder"
 require "htmlbeautifier/html_parser"
 require "htmlbeautifier/version"
@@ -15,12 +17,14 @@ module HtmlBeautifier
   # is false, i.e. continue to process the rest of the document.
   # initial_level - The entire output will be indented by this number of steps.
   # Default is 0.
+  # keep_blank_lines - an integer for the number of consecutive empty lines
+  # to keep in output.
   #
   def self.beautify(html, options = {})
     if options[:tab_stops]
       options[:indent] = " " * options[:tab_stops]
     end
-    "".tap { |output|
+    String.new.tap { |output|
       HtmlParser.new.scan html.to_s, Builder.new(output, options)
     }
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: htmlbeautifier
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.4.1
 platform: ruby
 authors:
 - Paul Battley
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-11-22 00:00:00.000000000 Z
+date: 2021-12-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '13'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '13'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -44,14 +44,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.30.0
+        version: '1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.30.0
+        version: '1'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
 description: A normaliser/beautifier for HTML that also understands embedded Ruby.
 email: pbattley@gmail.com
 executables:
@@ -80,15 +108,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 1.9.2
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 3.2.22
 signing_key: 
 specification_version: 4
 summary: HTML/ERB beautifier