From a7ef80074d869a6c8c60b25b88f3839afef6f0c9 Mon Sep 17 00:00:00 2001 From: Earlopain <14981592+Earlopain@users.noreply.github.com> Date: Sun, 28 Apr 2024 11:59:37 +0200 Subject: [PATCH] [Cleanup] Resolve some easy ruby warnings Mostly duplicate method warnings --- app/logical/danbooru/has_bit_flags.rb | 12 ++++++------ app/logical/storage_manager.rb | 1 - app/models/forum_post.rb | 8 -------- app/views/layouts/default.html.erb | 2 +- test/unit/parse_value_test.rb | 6 ------ 5 files changed, 7 insertions(+), 22 deletions(-) diff --git a/app/logical/danbooru/has_bit_flags.rb b/app/logical/danbooru/has_bit_flags.rb index dc3536dc3..10074948f 100644 --- a/app/logical/danbooru/has_bit_flags.rb +++ b/app/logical/danbooru/has_bit_flags.rb @@ -10,15 +10,15 @@ module Danbooru def has_bit_flags(attributes, options = {}) field = options[:field] || :bit_flags + define_singleton_method("flag_value_for") do |key| + index = attributes.index(key) + raise IndexError if index.nil? + 1 << index + end + attributes.each.with_index do |attribute, i| bit_flag = 1 << i - define_singleton_method("flag_value_for") do |key| - index = attributes.index(key) - raise IndexError if index.nil? - 1 << index - end - define_method(attribute) do send(field) & bit_flag > 0 end diff --git a/app/logical/storage_manager.rb b/app/logical/storage_manager.rb index a03f7601b..00a5e4a6d 100644 --- a/app/logical/storage_manager.rb +++ b/app/logical/storage_manager.rb @@ -97,7 +97,6 @@ class StorageManager def protected_params(url, post, secret: Danbooru.config.protected_file_secret) user_id = CurrentUser.id - ip = CurrentUser.ip_addr time = (Time.now + 15.minute).to_i secret = secret hmac = Digest::MD5.base64digest("#{time} #{url} #{user_id} #{secret}").tr("+/","-_").gsub("==",'') diff --git a/app/models/forum_post.rb b/app/models/forum_post.rb index 1372e9a29..06082f9f7 100644 --- a/app/models/forum_post.rb +++ b/app/models/forum_post.rb @@ -196,14 +196,6 @@ class ForumPost < ApplicationRecord self.is_hidden = false if is_hidden.nil? end - def creator_name - User.id_to_name(creator_id) - end - - def updater_name - User.id_to_name(updater_id) - end - def forum_topic_page (ForumPost.where("topic_id = ? and created_at <= ?", topic_id, created_at).count / Danbooru.config.records_per_page.to_f).ceil end diff --git a/app/views/layouts/default.html.erb b/app/views/layouts/default.html.erb index 9a22b0212..6db37b891 100644 --- a/app/views/layouts/default.html.erb +++ b/app/views/layouts/default.html.erb @@ -3,7 +3,7 @@ <%= render "layouts/head" %> -<%= tag.body **body_attributes(CurrentUser.user) do %> +<%= tag.body(**body_attributes(CurrentUser.user)) do %> <%= render "layouts/theme_include" %>
diff --git a/test/unit/parse_value_test.rb b/test/unit/parse_value_test.rb index 007367a52..3ff5b2689 100644 --- a/test/unit/parse_value_test.rb +++ b/test/unit/parse_value_test.rb @@ -44,12 +44,6 @@ class ParseValueTest < ActiveSupport::TestCase assert_equal(0.0, eq_value("10:0", :ratio)) end - should "parse floats" do - assert_equal(10.0, eq_value("10", :float)) - assert_equal(0.1, eq_value(".1", :float)) - assert_equal(1.234, eq_value("1.234", :float)) - end - should "parse filesizes" do assert_equal(10, eq_value("10", :filesize)) assert_equal(102, eq_value(".1kb", :filesize))