fix: prevent mkdef from creating partial objects on validation error

[viniciusferrao]

Fixes xcat2#3706.

Summary

• Treat only_if validation failures as real object errors instead of only printing an error.
• Prevent errored objects from being marked updated or written through %allupdates.
• Validate replacement objects before mkdef -f removes an existing definition.
• Preserve valid paths where the condition is satisfied explicitly, from existing DB attributes, or by group inheritance.
• Add focused unit coverage for only_if matching and mkdef regression cases for partial writes and -f preservation.

Validation

• prove -v xCAT-test/unit/dbobjutils_only_if.t
• perl -c perl-xCAT/xCAT/DBobjUtils.pm
• XCATROOT=/opt/xcat PERL5LIB=perl-xCAT:xCAT-server/lib/perl:xCAT-server/lib perl -c xCAT-server/lib/xcat/plugins/DBobjectdefs.pm
• git diff --check
• Direct temporary SQLite xCAT DB checks for invalid, valid, and mixed valid/invalid object writes.

perlcritic --severity 5 still reports existing findings already present on master; this patch did not introduce a new severity-5 class of issue in the touched files.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3c16d99bb8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".