don't rename non-existing catapult_fwd

Phillip Davis created 1 week ago

Change summary

lib/admin_actions/number_change.rb | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)

Detailed changes

lib/admin_actions/number_change.rb

@@ -111,9 +111,11 @@ class AdminAction
 		# If old_tel exists, it's still possible no catapult_fwd has been set.
 		# However, if one exists, we should rename it.
 		# If old_tel does not exist, it's not possible for catapult_fwd
-		# to exist, and it's not possible for there to be value.
+		# to exist, and it's not possible for there to be value
 		# to set catapult_fwd to. That's out-of-scope for this command.
 		def change_catapult_fwd!
+			return unless old_tel
+
 			REDIS.rename("catapult_fwd-#{old_tel}", "catapult_fwd-#{new_tel}")
 		end