fix : Check for undefined in vertical flyout layout function#7523

[patelb268]

The basics

• [X ] I validated my changes

The details

In layout_ in flyout_vertical.ts we access the block with const block = item.block. The result is nullable

Resolves

so all later uses of block have to have a [non-null assertion]

Fixes
Check for undefined in vertical flyout layout function#7523

Proposed Changes

Check whether block is undefined after setting const block = item.block.
continue if block doesn't exist.
Remove ! from after the remaining uses of block within the for loop.

Reason for Changes

Test Coverage

Documentation

Additional Information

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[github-actions]

Welcome! It looks like this is your first pull request in Blockly, so here are a couple of tips:

• You can find tips about contributing to Blockly and how to
  validate your changes on our
  developer site.

• All contributors must sign the Google Contributor License
  Agreement (CLA). If the google-cla bot leaves a comment on this
  PR, make sure you follow the instructions.

• We use conventional commits
  to make versioning the package easier. Make sure your commit
  message is in the proper format
  or learn how to fix it.

• If any of the other checks on this PR fail, you can click on
  them to learn why. It might be that your change caused a test
  failure, or that you need to double-check the
  style guide.

Thank you for opening this PR! A member of the Blockly team will review it soon.

[maribethb]

Hi @patelb268,

Thank you for opening this pull request! It needs just a few things before it's ready to merge:

• Follow the google-cla bot's instructions to sign our CLA and refresh the check
• Address the comment left in the PR review below
• Run npm run format to automatically format the code to pass the rest of the CI checks

After that, things should be ready to merge! Thank you for your contribution!

[maribethb]

You can simplify this line to just:

if (block) {

[maribethb]

Hi Prayag, another contributor also fixed this issue and we've merged that one. Sorry for the mixup, and thanks for the PR. I hope you enjoyed OSD!