Admin: remove modules filtered by jetpack_get_available_modules

[eliorivero]

Before this PR, filtering modules out could result in

#### Changes proposed in this Pull Request: - checks that hardcoded modules are among those that are not filtered out by using the filter `jetpack_get_available_modules` - removes card from At a Glance that belongs to a module removed through the filter #### Testing instructions: - paste this where it can be executed

add_filter( 'jetpack_get_available_modules', function ( $modules ) {
    unset( $modules['protect'] );
    unset( $modules['carousel'] );
    unset( $modules['post-by-email'] );
    unset( $modules['sharedaddy'] );
    unset( $modules['json-api'] );
    unset( $modules['photon'] );
    return $modules;
} );

• go to Jetpack > Settings and make sure those cards don't look empty like before, but like

- in Jetpack > Dashboard > At a Glance, make sure that cards that belong to a filtered out module are not displayed.

[zinigor]

Tested, works well for now! Added one commit that refactors one conditional for better readability. LGTM!

[RCowles]

A user in the forums reported that he's unable to remove VaultPress from the new UI using the jetpack_get_available_modules filter:

That functionality is no longer working for for a couple items under the Security Tab (security scanning and site backups specifically) in the new React interface. Previously I could block “vaultpress” in the modules list using that code, but it’s not working anymore. Is there a different module name in the latest version?

Original report:
https://wordpress.org/support/topic/hide-modules-in-react-interface/

[jeherve]

@RCowles Might be worth creating a brand new issue for that, since this PR is already merged.