From 18ec97992dd5629fbb875fa63a9e701dbc7cbf2d Mon Sep 17 00:00:00 2001
From: Maikel Linke <maikel@email.org.au>
Date: Wed, 15 Jan 2025 12:55:13 +1100
Subject: [PATCH] Remove unneeded HAML upgrade script

We updated a long time ago and I now copied those scripts to a Gist to
share with others:

* https://gist.github.com/mkllnk/bab2d474d686a2e3b88914129aee63e1
---
 lib/haml_up.rb           | 118 ---------------------------------------
 script/haml-up.rb        |  22 --------
 spec/lib/haml_up_spec.rb |  50 -----------------
 3 files changed, 190 deletions(-)
 delete mode 100644 lib/haml_up.rb
 delete mode 100755 script/haml-up.rb
 delete mode 100644 spec/lib/haml_up_spec.rb

diff --git a/lib/haml_up.rb b/lib/haml_up.rb
deleted file mode 100644
index 2ecc6199605..00000000000
--- a/lib/haml_up.rb
+++ /dev/null
@@ -1,118 +0,0 @@
-# frozen_string_literal: true
-
-# Upgrade HAML attribute syntax to prepare for HAML 6.
-#
-# HAML 6 stopped supporting nested hash attributes other than `data` and `aria`.
-# We used to be able to write:
-#
-#     %div{ ng: { class: "upper", bind: "model" } }
-#
-# This needs to be written in a flat structure now:
-#
-#     %div{ "ng-class" => "upper", "ng-bind" => "model" }
-#
-require "fileutils"
-require "haml"
-
-class HamlUp
-  def upgrade_file(filename)
-    template = File.read(filename)
-    rewrite_template(template)
-    File.write(filename, template)
-  end
-
-  def rewrite_template(template)
-    haml_attributes(template).compact.each do |attributes|
-      rewrite_attributes(template, attributes)
-    end
-  end
-
-  def rewrite_attributes(template, original)
-    attributes = parse_attributes(original)
-
-    if attributes.nil? # parser failed
-      puts "Warning: failed to parse:\n" # rubocop:disable Rails/Output
-      puts original # rubocop:disable Rails/Output
-      return
-    end
-
-    parse_deprecated_hashes(attributes)
-
-    to_transform = attributes.select { |_k, v| v.is_a? Hash }
-
-    return if to_transform.empty?
-
-    to_transform.each do |key, hash|
-      add_full_keys(attributes, key, hash)
-      attributes.delete(key)
-    end
-
-    replace_attributes(template, original, attributes)
-  end
-
-  def haml_attributes(template)
-    options = Haml::Options.new
-    parsed_tree = Haml::Parser.new(options).call(template)
-    elements = flatten_tree(parsed_tree)
-    elements.map { |e| e.value[:dynamic_attributes]&.old }
-  end
-
-  def flatten_tree(parent)
-    parent.children.flat_map do |child|
-      [child, *flatten_tree(child)]
-    end
-  end
-
-  def parse_attributes(string)
-    Haml::AttributeParser.parse(string)
-  end
-
-  def parse_deprecated_hashes(hash)
-    hash.each do |key, value|
-      next if ["aria", "data"].include?(key)
-
-      parsed = parse_attributes(value)
-      next unless parsed.is_a? Hash
-
-      parse_deprecated_hashes(parsed)
-      hash[key] = parsed
-    end
-  end
-
-  def add_full_keys(attributes, key, hash)
-    hash.each do |subkey, value|
-      full_key = "#{key}-#{subkey}"
-      if value.is_a? Hash
-        add_full_keys(attributes, full_key, value)
-      else
-        attributes[full_key] = value
-      end
-    end
-  end
-
-  def replace_attributes(template, original, attributes)
-    parsed_lines = original.split("\n")
-    lines_as_regex = parsed_lines.map(&Regexp.method(:escape))
-    pattern = lines_as_regex.join("\n\s*")
-
-    template.gsub!(/#{pattern}/, stringify(attributes))
-  end
-
-  def stringify(hash)
-    entries = hash.map do |key, value|
-      value = stringify(value) if value.is_a? Hash
-
-      # We prefer the Ruby 1.9 hash syntax with symbols followed by a colon
-      # like this:
-      #
-      #   %button{ disabled: true, "ng-class": "primary-button" }
-      #
-      # Symbols start with `:` which we slice off. It gets appended below.
-      key = key.to_sym.inspect.slice(1..-1)
-
-      "#{key}: #{value}"
-    end
-
-    "{ #{entries.join(', ')} }"
-  end
-end
diff --git a/script/haml-up.rb b/script/haml-up.rb
deleted file mode 100755
index 033e09f348e..00000000000
--- a/script/haml-up.rb
+++ /dev/null
@@ -1,22 +0,0 @@
-#!/usr/bin/env ruby
-# frozen_string_literal: true
-
-# Upgrade HAML attribute syntax to prepare for HAML 6.
-#
-# HAML 6 stopped supporting nested hash attributes other than `data` and `aria`.
-# We used to be able to write:
-#
-#     %div{ ng: { class: "upper", bind: "model" } }
-#
-# This needs to be written in a flat structure now:
-#
-#     %div{ "ng-class" => "upper", "ng-bind" => "model" }
-#
-# This script rewrites HAML files automatically. It may be used like:
-#
-#     git ls-files '*.haml' | while read f; do ./haml-up.rb "$f"; done
-#
-require "haml_up"
-
-puts ARGV[0]
-HamlUp.new.upgrade_file(ARGV[0])
diff --git a/spec/lib/haml_up_spec.rb b/spec/lib/haml_up_spec.rb
deleted file mode 100644
index f78b95017ad..00000000000
--- a/spec/lib/haml_up_spec.rb
+++ /dev/null
@@ -1,50 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-require 'haml_up'
-
-RSpec.describe HamlUp, skip: !Gem::Dependency.new("", "~> 5.2").match?("", Haml::VERSION) do
-  describe "#rewrite_template" do
-    it "preserves a simple template" do
-      original = "%p This is a paragraph"
-      template = call(original)
-      expect(template).to eq original
-    end
-
-    it "rewrites non-standard attribute hashes" do
-      original = "%p{ng: {click: 'action', show: 'condition'}} label"
-      template = call(original)
-      expect(template).to eq "%p{ \"ng-click\": 'action', \"ng-show\": 'condition' } label"
-    end
-
-    it "preserves standard attribute hashes" do
-      original = "%p{data: {click: 'action', show: 'condition'}} label"
-      template = call(original)
-      expect(template).to eq original
-    end
-
-    it "preserves standard attribute hashes while rewriting others" do
-      original = "%p{data: {click: 'standard'}, ng: {click: 'not'}} label"
-      template = call(original)
-      expect(template).to eq "%p{ data: {click: 'standard'}, \"ng-click\": 'not' } label"
-    end
-
-    it "rewrites multi-line attributes" do
-      original = <<~HAML
-        %li{ ng: { class: "{active: selector.active}" } }
-          %a{ tooltip: "{{selector.object.value}}", "tooltip-placement": "bottom",
-            ng: { transclude: true, class: "{active: selector.active, 'has-tip': selector.object.value}" } }
-      HAML
-      expected = <<~HAML
-        %li{ "ng-class": "{active: selector.active}" }
-          %a{ tooltip: "{{selector.object.value}}", "tooltip-placement": "bottom", "ng-transclude": true, "ng-class": "{active: selector.active, 'has-tip': selector.object.value}" }
-      HAML
-      template = call(original)
-      expect(template).to eq expected
-    end
-
-    def call(original)
-      original.dup.tap { |t| subject.rewrite_template(t) }
-    end
-  end
-end