Wouldn't it be cleaner to have two methods like `enableCache()` and `disableCache()` instead of `setCacheable()`? I guess that's a little easier to read/understand in the first place and you don't need a boolean type cast inside the method.