Fix item visibility check when building GI Vct

[iche033]

🦟 Bug fix

Summary

Add better check when adding an item to the voxelizer when building GI

Previously, the objData.mOwner[k]->getVisible() check may fail and cause a crash if objData is not derived from an Ogre::Item. This PR makes sure that it's an Ogre::Item first before checking visibility.

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• Consider updating Python bindings (if the library has them)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers
• Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Generated-by: Remove this if GenAI was not used.

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

[codebot]

Thanks @iche033 ! This fixes a crash we were seeing. 🙇

[iche033]

@Mergifyio backport gz-rendering10 gz-rendering9 gz-rendering8

[mergify]

backport gz-rendering10 gz-rendering9 gz-rendering8

✅ Backports have been created

Details

• #1222 Fix item visibility check when building GI Vct (backport #1216) has been created for branch gz-rendering10
• #1223 Fix item visibility check when building GI Vct (backport #1216) has been created for branch gz-rendering9
• #1224 Fix item visibility check when building GI Vct (backport #1216) has been created for branch gz-rendering8