Back fetching modifier from controller

[melnikaite]

No description provided.

[coveralls]

Coverage decreased (-0.1%) to 99.151% when pulling 8556240 on melnikaite:set_modifier_id into 6948018 on aq1018:master.

[coveralls]

Coverage decreased (-0.1%) to 99.151% when pulling 003d259 on melnikaite:set_modifier_id into 6948018 on aq1018:master.

[coveralls]

Coverage decreased (-0.2%) to 99.06% when pulling d295a99 on melnikaite:set_modifier_id into 6948018 on aq1018:master.

[coveralls]

Coverage decreased (-0.3%) to 98.994% when pulling 628bc67 on melnikaite:set_modifier_id into 6948018 on aq1018:master.

[coveralls]

Coverage decreased (-0.2%) to 99.06% when pulling beb94ad on melnikaite:set_modifier_id into 6948018 on aq1018:master.

[coveralls]

Coverage decreased (-0.1%) to 99.152% when pulling d112219 on melnikaite:set_modifier_id into 6948018 on aq1018:master.

[dblock]

I think this is ok to merge, but I've never heard of Tenacity and maybe we should call that out separately, with documentation and tests?

[dblock]

Other thoughts: maybe Hooks should be a more explicit class name, like Hooks::UpdatedBy or Hooks::Modifier?

[coveralls]

Coverage decreased (-0.3%) to 98.994% when pulling ea2019e on melnikaite:set_modifier_id into 6948018 on aq1018:master.

[coveralls]

Coverage decreased (-0.3%) to 98.994% when pulling 73bd9eb on melnikaite:set_modifier_id into 6948018 on aq1018:master.