prevent unnecessary cache clearing

[timse201]

• flush cache when comment is in database and not before

• flush when comment is or was approved

• move comment hooks to comment 'flush hooks'

• rename functions for better code comprehension

• removes user_register and edit_user_profile_update from flush total cache

• because of edit_comment $data we need wp 4.6 or greater

#223

[stklcode]

Thanks for your work.

The changes in general are well suited for point release like 2.4, however they do introduce some breaking changes and I'm not sure whether we should raise the major version for this, if there is an viable way to add the new behavior non-breaking.

[florianbrinkmann]

I would not remove those, like @stklcode wrote in #223 (comment), user info can be displayed in many places.

[stklcode]

Re-added "edit_user_profile_update" for now in decdd88

We can discuss about refining this in future changes.

[stklcode]

because of edit_comment $data we need wp 4.6 or greater

We require 4.7 since #268, so that's no longer an issue here.

[stklcode]

Should be OK now.

A second opinion would not hurt here though.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
2.9% Duplication on New Code

See analysis details on SonarCloud