extract_plist: fix passing an alternative url

[bevanjkay]

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same change?
• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes? Here's an example.
• Have you successfully run brew style with your changes locally?
• Have you successfully run brew typecheck with your changes locally?
• Have you successfully run brew tests with your changes locally?

---

A previous change to the way the arguments are passed in to the extract_plist strategy meant that the cask object is passed in its entirety rather than passing the livecheck url - this reinstates the previous functionality.

First noticed when preparing - Homebrew/homebrew-cask#173163

CC: @samford

[samford]

Makes sense to me. Thanks for catching this, @bevanjkay!

I tested this on some ExtractPlist casks that don't use a url override (i.e., url :url) and ones that do (e.g., the cloudflare-warp PR) and it works as expected. With the existing setup, the url wasn't being passed into ExtractPlist#find_versions, so the override functionality wasn't being used.

[MikeMcQuaid]

Thanks again @bevanjkay!