:out-of-service is a boolean field. The previous logic would always have it return true when doing a single-read.
I believe the intention was to return true if the coercion resulted in nil, so I went with nil-checking. If that's not necessary, and nil is acceptable, then I'd back out this change and just get rid of the (or ... true) part.
:out-of-serviceis a boolean field. The previous logic would always have it returntruewhen doing a single-read.I believe the intention was to return
trueif the coercion resulted innil, so I went with nil-checking. If that's not necessary, andnilis acceptable, then I'd back out this change and just get rid of the (or ... true) part.