Link to issue number:

Fix regression introduced in pr #14256

Summary of the issue:

Sometimes a UIA focus event handler can't create an NVDAObject for the element being focused. Normally, in this situation, the event would be silently dropped.
However, with the introduction of extra debugging for UIA events in pr #14256, a return was accidently removed, which meant that the shouldAllowUIAFocusEvent attribute tried to be looed up on the None (failed NVDAObject). This caused a noisy traceback in the log at level error.

Description of user facing changes

A noisy error traceback is no longer produced when NVDA cannot handle certain broken UIA focus events.

Description of development approach

Make sure to return if instantiating the NVDAObject produces None.

Testing strategy:

Run NVDA for an hour or so performing normal daily tasks. Previously the error would have been seen multiple times.

Known issues with pull request:

None known.

Change log entries:

None needed - regression only in master.
New features
Changes
Bug fixes
For Developers

Code Review Checklist:

• Pull Request description:
  • description is up to date
  • change log entries
• Testing:
  • Unit tests
  • System (end to end) tests
  • Manual testing
• API is compatible with existing add-ons.
• Documentation:
  • User Documentation
  • Developer / Technical Documentation
  • Context sensitive help for GUI changes
• UX of all users considered:
  • Speech
  • Braille
  • Low Vision
  • Different web browsers
  • Localization in other languages / culture than English
• Security precautions taken.