FIX: use different method name for topic rake task

https://kevinjalbert.com/defined_methods-in-rake-tasks-you-re-gonna-have-a-bad-time/ cc @gschlager
discourse · Sep 29, 2017 · df09599 · df09599 · gschlager · Sep 29, 2017
1 parent 2c2fe7e
commit df09599
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/tasks/topics.rake b/lib/tasks/topics.rake
@@ -1,4 +1,4 @@
-def print_status(label, current, max)
+def print_status_with_label(label, current, max)
   print "\r%s%9d / %d (%5.1f%%)" % [label, current, max, ((current.to_f / max.to_f) * 100).round(1)]
 end
 
@@ -21,7 +21,7 @@ def close_old_topics(category)
 
   topics.find_each do |topic|
     topic.update_status("closed", true, Discourse.system_user)
-    print_status("    closing old topics: ", topics_closed += 1, total)
+    print_status_with_label("    closing old topics: ", topics_closed += 1, total)
   end
 end
 
@@ -47,7 +47,7 @@ def apply_auto_close(category)
 
   topics.find_each do |topic|
     topic.inherit_auto_close_from_category
-    print_status("    applying auto-close to topics: ", topics_closed += 1, total)
+    print_status_with_label("    applying auto-close to topics: ", topics_closed += 1, total)
   end
 end