canvas_link_migrator 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7d147093c7e6b397c3eb184aba4af1595f782e14735a50e9d4986be628af75f3
-  data.tar.gz: cb48c35f35dd333b02ff7dcba5e3bd79d1a1a7b736b7bb14c56de11b6c91565e
+  metadata.gz: 6c077949f9cef706c71bff4537a9df6a3e53cb5306dadab87fb0281c79754a16
+  data.tar.gz: d1f6d129570271a82b293f6422b00cb1b6533233713adaf1479fd5d7efda47cb
 SHA512:
-  metadata.gz: 4074cbb1bfa20b00f4843327d62ee0e1cf739f7450b46698ee1a6a240054f27acd34046605e2ae4f54bbdf02b529272ba165769352e10abff1f5950c08af6ad8
-  data.tar.gz: e1deecf7b1d6586c432234092e338c9d3059f26b744072eab1cb6e644c25dbcec9ffacfb53239b6c9ffc0610c210bf80399f097f2d7efa2265686caf314a8fee
+  metadata.gz: b07d22a498fbb5188874aa67a6702be9ee349ed2be2efb5c4f42050d45b7914afb0edeae7d2f2127dae44531631f371cb1d8df62923133f8a88b96bcbe8f997f
+  data.tar.gz: f1c6fa682fc28072571d3841d4418e14bc1240b453618ec19be98df2d09ab8a217c3638310bc0383e1a63f32ade812008f69a3198ad436167dd12f5ffc672673

data/lib/canvas_link_migrator/link_parser.rb

@@ -166,6 +166,10 @@ module CanvasLinkMigrator
         else
           result[:old_value] = node[attr]
           result[:placeholder] = placeholder(result[:old_value])
+          # replace the inner html of an anchor tag if it matches the href
+          if node.name == "a" && attr == "href" && node["href"] == node.inner_html.delete("\n").strip
+            node.inner_html = result[:placeholder]
+          end
           node[attr] = result[:placeholder]
         end
         add_unresolved_link(result, item_type, mig_id, field)
@@ -177,7 +181,7 @@ module CanvasLinkMigrator
     end
 
     def resolved(new_url = nil)
-      { resolved: true, new_url: new_url }
+      { resolved: true, new_url: new_url}
     end
 
     # returns a hash with resolution status and data to hold onto if unresolved
@@ -186,7 +190,7 @@ module CanvasLinkMigrator
       query_values = parsed_url.query_values
       media_attachment = query_values.try(:delete, "media_attachment") == "true"
       if media_attachment
-        parsed_url.query_values = query_values.present? ? query_values : nil
+        parsed_url.query_values = query_values.presence || nil
         url = Addressable::URI.unencode(parsed_url)
       end
 

data/spec/canvas_link_migrator/link_parser_spec.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+#
+# Copyright (C) 2023 - present Instructure, Inc.
+#
+# This file is part of Canvas.
+#
+# Canvas is free software: you can redistribute it and/or modify it under
+# the terms of the GNU Affero General Public License as published by the Free
+# Software Foundation, version 3 of the License.
+#
+# Canvas is distributed in the hope that it will be useful, but WITHOUT ANY
+# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
+# A PARTICULAR PURPOSE. See the GNU Affero General Public License for more
+# details.
+#
+# You should have received a copy of the GNU Affero General Public License along
+# with this program. If not, see <http://www.gnu.org/licenses/>.
+#
+
+require "spec_helper"
+
+describe CanvasLinkMigrator::LinkParser do
+  def parser
+    migration_query_service_mock = double()
+    allow(migration_query_service_mock).to receive(:supports_embedded_images).and_return(true)
+    allow(migration_query_service_mock).to receive(:fix_relative_urls?).and_return(true)
+    CanvasLinkMigrator::LinkParser.new(migration_query_service_mock)
+  end
+
+  describe "convert_link" do
+    it "converts inner html of anchor tags when appropriate" do
+      doc = Nokogiri::HTML5.fragment("<a href=\"$WIKI_REFERENCE$/pages/1\">$WIKI_REFERENCE$/pages/1</a>")
+      parser.convert_link(doc.at_css('a'), "href","wiki_page", "migrationid", "")
+      expect(doc.at_css('a')['href']).to include("LINK.PLACEHOLDER")
+      expect(doc.at_css('a').inner_html).to include("LINK.PLACEHOLDER")
+    end
+
+    it "doesn't convert inner html of anchor tags if unnecessary" do
+      doc = Nokogiri::HTML5.fragment("<a href=\"$WIKI_REFERENCE$/pages/1\">$WIKI_REFERENCE$/pages/5</a>")
+      parser.convert_link(doc.at_css('a'), "href","wiki_page", "migrationid", "")
+      expect(doc.at_css('a')['href']).to include("LINK.PLACEHOLDER")
+      expect(doc.at_css('a').inner_html).not_to include("LINK.PLACEHOLDER")
+    end
+  end
+end

metadata

@@ -1,16 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: canvas_link_migrator
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - Mysti Lilla
 - James Logan
 - Sarah Gerard
+- Math Costa
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-11-04 00:00:00.000000000 Z
+date: 2023-11-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -129,6 +130,7 @@ email:
 - mysti@instructure.com
 - james.logan@instructure.com
 - sarah.gerard@instructure.com
+- luis.oliveira@instructure.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -141,6 +143,7 @@ files:
 - lib/canvas_link_migrator/resource_map_service.rb
 - lib/canvas_link_migrator/version.rb
 - spec/canvas_link_migrator/imported_html_converter_spec.rb
+- spec/canvas_link_migrator/link_parser_spec.rb
 - spec/canvas_link_migrator/link_resolver_spec.rb
 - spec/canvas_link_migrator_spec.rb
 - spec/fixtures/canvas_resource_map.json