4.0: ext/get-the-image.php needs PHP 7.3 compatibility upgrade for compact()

[lkraav]

compact() behavior was changed in PHP 7.3 and get-the-image.php usage of it should probably be rearchitected https://github.com/justintadlock/hybrid-core/blob/4.0.0/ext/get-the-image.php#L298

If left unattended, it's likely that people's error logs will be flooded with Notice | compact(): Undefined variable: post_id | 51 | wp-content/themes/hybrid-core-4.0/ext/get-the-image.php:298.

For example on a category page for me this notice trigger 1800 times per page load (many featured images).

https://www.php.net/manual/en/function.compact.php

EDIT https://core.trac.wordpress.org/ticket/44416

[justintadlock]

I'd recommend switching to Hybrid Carbon, even if on 4.0 of the framework: https://github.com/justintadlock/hybrid-carbon

[lkraav]

Thanks. That's a good idea.

While we get ourselves migrated, can you drop a hint on what could be a good strategy to optimize compact() out of $key = md5( serialize( compact( array_keys( $this->args ) ) ) ); caching?

I'm not entirely sure why it's there in the first place, why not just serialize $this->args directly?