Move constructor code into check

laminas / laminas-diagnostics

A set of components for performing diagnostic tests in PHP applications

https://docs.laminas.dev/laminas-diagnostics/

BSD 3-Clause "New" or "Revised" License

55 stars 32 forks source link

Move constructor code into check #29

Closed lostfocus closed 2 years ago

lostfocus commented 3 years ago

Q	A
Bugfix	yes

This moves the instantiation of the AdvisoryAnalyzer into the check function to make sure we only download the advisories when actually checking them. Closes #28

Ocramius commented 3 years ago

@lostfocus since this is a bugfix, I'm thinking this could go against 1.10.x, right?

Ocramius commented 2 years ago

Thanks @lostfocus - sorry for the long delay