condition.rb in byebug-4.0.0

- old
+ new

@@ -1,5 +1,7 @@
+require 'byebug/command'
+
 module Byebug
   #
   # Implements conditions on breakpoints.
   #
   # Adds the ability to stop on breakpoints only under certain conditions.
@@ -10,39 +12,41 @@
     def regexp
       /^\s* cond(?:ition)? (?:\s+(\d+)(?:\s+(.*))?)? \s*$/x
     end
 
     def execute
-      return puts(ConditionCommand.help) unless @match[1]
+      return puts(self.class.help) unless @match[1]
 
-      breakpoints = Byebug.breakpoints.sort_by { |b| b.id }
-      return errmsg('No breakpoints have been set') unless breakpoints.any?
+      breakpoints = Byebug.breakpoints.sort_by(&:id)
+      return errmsg(pr('condition.errors.no_breakpoints')) if breakpoints.empty?
 
       pos, err = get_int(@match[1], 'Condition', 1)
       return errmsg(err) if err
 
       breakpoint = breakpoints.find { |b| b.id == pos }
-      return errmsg('Invalid breakpoint id. Use "info breakpoint" to find ' \
-                    'out the correct id') unless breakpoint
+      return errmsg(pr('break.errors.no_breakpoint')) unless breakpoint
 
-      return errmsg("Incorrect expression \"#{@match[2]}\", " \
-                    'breakpoint not changed') unless syntax_valid?(@match[2])
+      unless syntax_valid?(@match[2])
+        return errmsg(pr('break.errors.not_changed', expr: @match[2]))
+      end
 
       breakpoint.expr = @match[2]
     end
 
     class << self
       def names
         %w(condition)
       end
 
       def description
-        %(cond[ition] <n>[ expr]
+        prettify <<-EOD
+          cond[ition] <n>[ expr]
 
           Specify breakpoint number <n> to break only if <expr> is true. <n> is
           an integer and <expr> is an expression to be evaluated whenever
           breakpoint <n> is reached. If no expression is specified, the
-          condition is removed.)
+          condition is removed.
+        EOD
       end
     end
   end
 end