Revert "Merge pull request #22901 from jrafanie/drop_default_value_for"

[jrafanie]

This reverts commit c773b2b, reversing changes made to bb72015.

Reverting #22901.

This was causing errors in tests where these attribute defaults were triggering "changed" on the attribute, such as in https://github.com/ManageIQ/manageiq-providers-ansible_tower/blob/d5be6cc09a1b2804c1c510c990d5f7061cf2240f/app/models/manageiq/providers/ansible_tower/automation_manager.rb#L28

[miq-bot]

Checked commit jrafanie@6a3ef60 with ruby 2.7.8, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
51 files checked, 7 offenses detected

app/models/blacklisted_event.rb

• ❗ - Line 6, Col 3 - Rails/ActiveRecordCallbacksOrder - after_destroy is supposed to appear before after_save.
• ❗ - Line 7, Col 3 - Rails/ActiveRecordCallbacksOrder - after_create is supposed to appear before after_destroy.

app/models/notification.rb

• ❗ - Line 19, Col 33 - Style/EmptyLiteral - Use hash literal {} instead of Hash.new.

app/models/user.rb

• ❗ - Line 58, Col 34 - Style/EmptyLiteral - Use hash literal {} instead of Hash.new.

spec/lib/extensions/ar_yaml_spec.rb

• ❗ - Line 13, Col 144 - Style/ColonMethodCall - Do not use :: for method calls.
• ❗ - Line 21, Col 144 - Style/ColonMethodCall - Do not use :: for method calls.
• ❗ - Line 28, Col 144 - Style/ColonMethodCall - Do not use :: for method calls.

[Fryguy]

Backported to radjabov in commit 9acc4b2.

commit 9acc4b2b5ed80f256b678a66eb6d7f0d6ab4913e
Author: Adam Grare <[email protected]>
Date:   Thu Feb 29 13:20:03 2024 -0500

    Merge pull request #22922 from jrafanie/revert_22901
    
    Revert "Merge pull request #22901 from jrafanie/drop_default_value_for"
    
    (cherry picked from commit 340776c59b8ae6199439e2190487e3f3703f8404)