When I asked the discord, the conversation said that as of 2.1.8 it's
- get recipe quality
- apply quality_change
- apply quality_min/quality_max
- if affected_by_quality, apply quality effect
- if affected_by_quality, apply quality_min/quality_max
- resulting quality is product quality
I propose we add a note to quality_min and quality_max to mention it.
I am not tied to those locations specifically, but I personally feel it makes the most sense.
Something like
The wording might be improvable, but I like that it also implicitly mentions that quality change applies before the quality roll.When processing a recipe, the clamping applies after the quality was affected by `quality_change`, then after it was rolled if `affected_by_quality` is true.
