Sites breaks if another modules adds an icon without target_id
If another module adds an icon to the core navigation module, UI Icons 1.0.1 breaks the site, because ui_icons_menu.module line 115 expects a target_id.
`TypeError: ui_icons_menu_generate_markup(): Argument #2 ($icon_full_id) must be of type string, null given, called in /var/www/html/web/modules/contrib/ui_icons/modules/ui_icons_menu/ui_icons_menu.module on line 115 in ui_icons_menu_generate_markup() (line 165 of modules/contrib/ui_icons/modules/ui_icons_menu/ui_icons_menu.module). `
I got this error after upgrading Commerce which adds an icon in its commerce.links.menu.yml. I assume that Commerce has been doing this correctly:
https://www.drupal.org/project/commerce/issues/3565263
```
commerce.admin_commerce:
options:
icon:
pack_id: commerce
icon_id: drupal-cart
```
issue