Refactor pull_hotlinked_images (PR #10342)

GitHub

      return if post.blank?
      raise Discourse::InvalidParameters.new(:post_id) if @post_id.blank?

This is trivial but usually it’s easier for our brains to parse regular ifs rather than unless/not.

      post_changed_elsewhere = (start_raw != post.raw)
      raw_changed_here = (raw != post.raw)

Very trivial but I find this easier to read because it makes the one assignment very clear.