From 7c9eba891963451a1feb2e5bbef90fdcac1496ff Mon Sep 17 00:00:00 2001 From: Stan Hu Date: Sat, 18 Jun 2016 10:55:45 -0700 Subject: Fix RangeError exceptions when referring to issues or merge requests outside of max database values When using #XYZ in Markdown text, if XYZ exceeds the maximum value of a signed 32-bit integer, we get an exception when the Markdown render attempts to run `where(iids: XYZ)`. Introduce a method that will throw out out-of-bounds values. Closes #18777 --- app/models/concerns/referable.rb | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'app/models/concerns/referable.rb') diff --git a/app/models/concerns/referable.rb b/app/models/concerns/referable.rb index ce064f675ae..dee940a3f88 100644 --- a/app/models/concerns/referable.rb +++ b/app/models/concerns/referable.rb @@ -49,6 +49,10 @@ module Referable raise NotImplementedError, "#{self} does not implement #{__method__}" end + def reference_valid?(reference) + true + end + def link_reference_pattern(route, pattern) %r{ (? -- cgit v1.2.1