wp-media / imagify-plugin

Speed up your website with lighter images without losing quality.
https://imagify.io
73 stars 27 forks source link

[Feature Request] Remove unclear PHP line comment #929

Open tyrann0us opened 3 days ago

tyrann0us commented 3 days ago

Is your feature request related to a problem? Please describe. https://github.com/wp-media/imagify-plugin/blob/49c5593c59c56182a3ceebeb38eafff7707faea0/classes/Media/WP.php#L222 This unusual comment consisting of an emoji-like sequence (that possibly expresses a shocked or surprised face, according to ChatGPT) is triggering false positives in security scans of a hosting provider (https://maxcluster.de/en/knowledge-base/shopsecurity/what-is-shopsecurity). See the following screenshot: Image

Describe the solution you'd like It would be good if the comment could be removed to avoid further false positives and because its existence is unclear.

Describe alternatives you've considered Alternatively, the comment should be converted into an explanatory sentence.

Additional context It was introduced in https://github.com/wp-media/imagify-plugin/pull/435/files#diff-3b8e1c62f65c1f8867b62aed7de21bbdc91c753291e1d3dac8735d252a0ff3f7R220 by @Screenfeed; maybe he can say more about it.