[16.0][FIX] fs_attachment,fs_attachment_s3,fs_storage,fs_storage_backup: fix server_environment split migration and add uninstall hooks

[yankinmax]

The PR was merged:

• [16.0] [IMP] fs_storage: Remove dependancy on server_environment #627

This PR fixes the broken migration scripts, which set ir_module_module::state directly via SQL instead of calling button_install, so the new *_environment modules never actually got installed on upgrade.
Scripts are moved to the upgrades/ folder (same version, no version bump) and fixed to call button_install.

Also adds uninstall_hook to each *_environment module, using restore_env_managed_columns from server_environment to restore the DB columns dropped by server.env.mixin when the module is uninstalled.

Depends on:

• [IMP] server_environment: module uninstallation server-env#276

[oca-cla-bot]

All commit authors signed the Contributor License Agreement.

[OCA-git-bot]

Hi @lmignon,
some modules you are maintaining are being modified, check this out!

[lmignon]

I haven't had any upgrade issues with the current implementation.... which doesn't mean the fix doesn't improve the robustness of the approach.

[yankinmax]

image_tag split is handled separately, because it doesn't introduce any server_env_fields override, so only need the upgrade script:

• [16.0][ADD] image_tag_environment: make server_environment an optional dependency #575