There's an alternative that I tried out, this whole block can be reduced to:

```
value = 'arbitrary_string'
value = value[0...[attribute_limit, value.length].compact.min]
```

I thought that this was a bit convoluted, so I left it out _but_ it is shorter and avoids a magic number too. Would this be preferable? If not, if I introduce a constant should I change `value = 'aribtrary_string'` to `value = 'a' * FANCY_NEW_CONSTANT`?