Removes link attr_reader from Cells (roo-rb#457)

* Removes link attr_reader from Cells * Adds deprecation method for link * Deprecated methods should still return a value
chobiwa · Jun 18, 2019 · 49af97c · 49af97c
1 parent 29992c9
commit 49af97c
Show file tree

Hide file tree

Showing 8 changed files with 16 additions and 9 deletions.
diff --git a/lib/roo/excelx/cell/base.rb b/lib/roo/excelx/cell/base.rb
@@ -71,9 +71,16 @@ def to_s
           formatted_value
         end
 
-        # DEPRECATED: Please use link instead.
+        # DEPRECATED: Please use link? instead.
         def hyperlink
-          warn '[DEPRECATION] `hyperlink` is deprecated.  Please use `link` instead.'
+          warn '[DEPRECATION] `hyperlink` is deprecated.  Please use `link?` instead.'
+          link?
+        end
+
+        # DEPRECATED: Please use link? instead.
+        def link
+          warn '[DEPRECATION] `link` is deprecated.  Please use `link?` instead.'
+          link?
         end
 
         # DEPRECATED: Please use cell_value instead.

diff --git a/lib/roo/excelx/cell/boolean.rb b/lib/roo/excelx/cell/boolean.rb
@@ -4,7 +4,7 @@ module Roo
   class Excelx
     class Cell
       class Boolean < Cell::Base
-        attr_reader :value, :formula, :format, :cell_value, :link, :coordinate
+        attr_reader :value, :formula, :format, :cell_value, :coordinate
 
         attr_reader_with_default default_type: :boolean, cell_type: :boolean
 

diff --git a/lib/roo/excelx/cell/date.rb b/lib/roo/excelx/cell/date.rb
@@ -4,7 +4,7 @@ module Roo
   class Excelx
     class Cell
       class Date < Roo::Excelx::Cell::DateTime
-        attr_reader :value, :formula, :format, :cell_type, :cell_value, :link, :coordinate
+        attr_reader :value, :formula, :format, :cell_type, :cell_value, :coordinate
 
         attr_reader_with_default default_type: :date
 

diff --git a/lib/roo/excelx/cell/datetime.rb b/lib/roo/excelx/cell/datetime.rb
@@ -8,7 +8,7 @@ class Cell
       class DateTime < Cell::Base
         SECONDS_IN_DAY = 60 * 60 * 24
 
-        attr_reader :value, :formula, :format, :cell_value, :link, :coordinate
+        attr_reader :value, :formula, :format, :cell_value, :coordinate
 
         attr_reader_with_default default_type: :datetime
 

diff --git a/lib/roo/excelx/cell/empty.rb b/lib/roo/excelx/cell/empty.rb
@@ -3,7 +3,7 @@ module Roo
   class Excelx
     class Cell
       class Empty < Cell::Base
-        attr_reader :value, :formula, :format, :cell_type, :cell_value, :hyperlink, :coordinate
+        attr_reader :value, :formula, :format, :cell_type, :cell_value, :coordinate
 
         attr_reader_with_default default_type: nil, style: nil
 

diff --git a/lib/roo/excelx/cell/number.rb b/lib/roo/excelx/cell/number.rb
@@ -4,7 +4,7 @@ module Roo
   class Excelx
     class Cell
       class Number < Cell::Base
-        attr_reader :value, :formula, :format, :cell_value, :link, :coordinate
+        attr_reader :value, :formula, :format, :cell_value, :coordinate
 
         # FIXME: change default_type to number. This will break brittle tests.
         attr_reader_with_default default_type: :float

diff --git a/lib/roo/excelx/cell/string.rb b/lib/roo/excelx/cell/string.rb
@@ -2,7 +2,7 @@ module Roo
   class Excelx
     class Cell
       class String < Cell::Base
-        attr_reader :value, :formula, :format, :cell_value, :link, :coordinate
+        attr_reader :value, :formula, :format, :cell_value, :coordinate
 
         attr_reader_with_default default_type: :string, cell_type: :string
 

diff --git a/lib/roo/excelx/cell/time.rb b/lib/roo/excelx/cell/time.rb
@@ -4,7 +4,7 @@ module Roo
   class Excelx
     class Cell
       class Time < Roo::Excelx::Cell::DateTime
-        attr_reader :value, :formula, :format, :cell_value, :link, :coordinate
+        attr_reader :value, :formula, :format, :cell_value, :coordinate
 
         attr_reader_with_default default_type: :time