diff options
| author | Geremia Taglialatela <[email protected]> | 2023-06-10 11:48:13 +0200 |
|---|---|---|
| committer | Geremia Taglialatela <[email protected]> | 2023-06-10 11:48:13 +0200 |
| commit | ee14c1bf5c6aaa96867eb4b90b510f91c3a614f8 (patch) | |
| tree | 63a893a73f1cc51003edf218fed178bb93c604d7 /lib/axlsx/drawing/pic.rb | |
| parent | cfb516062de3d0725b3245cc328f96d702d662d3 (diff) | |
| download | caxlsx-ee14c1bf5c6aaa96867eb4b90b510f91c3a614f8.tar.gz caxlsx-ee14c1bf5c6aaa96867eb4b90b510f91c3a614f8.zip | |
Fix safe Style/ColonMethodCall offenses
Caxlsx is using both `.` and `::`, 220 occurrences vs 280 to invoke
methods on `Axlsx` module.
This commit standardizes the approach towards `.`, which will also allow
shorter lines.
Performance is not affected
```
Comparison:
Axlsx.validate: 8515252.3 i/s
Axlsx::validate: 8512863.7 i/s - same-ish: difference falls within error
```
Diffstat (limited to 'lib/axlsx/drawing/pic.rb')
| -rw-r--r-- | lib/axlsx/drawing/pic.rb | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/axlsx/drawing/pic.rb b/lib/axlsx/drawing/pic.rb index 08beb2d9..630d8d96 100644 --- a/lib/axlsx/drawing/pic.rb +++ b/lib/axlsx/drawing/pic.rb @@ -78,7 +78,7 @@ module Axlsx end def image_src=(v) - Axlsx::validate_string(v) + Axlsx.validate_string(v) if remote? RegexValidator.validate('Pic.image_src', /\A#{URI::DEFAULT_PARSER.make_regexp}\z/, v) RestrictionValidator.validate 'Pic.image_src', ALLOWED_MIME_TYPES, MimeTypeUtils.get_mime_type_from_uri(v) @@ -91,13 +91,13 @@ module Axlsx end # @see name - def name=(v) Axlsx::validate_string(v); @name = v; end + def name=(v) Axlsx.validate_string(v); @name = v; end # @see descr - def descr=(v) Axlsx::validate_string(v); @descr = v; end + def descr=(v) Axlsx.validate_string(v); @descr = v; end # @see remote - def remote=(v) Axlsx::validate_boolean(v); @remote = v; end + def remote=(v) Axlsx.validate_boolean(v); @remote = v; end def remote? remote == 1 || remote.to_s == 'true' |
