Remove unused variables from layout builder module

Quite simple code change. Tests are working.

+++ b/core/modules/layout_builder/src/Entity/LayoutBuilderEntityViewDisplay.php
@@ -359,9 +359,9 @@ public function label() {
 
-    foreach ($this->getSections() as $delta => $section) {
+    foreach ($this->getSections() as $section) {
       $this->calculatePluginDependencies($section->getLayout());
-      foreach ($section->getComponents() as $uuid => $component) {
+      foreach ($section->getComponents() as $component) {
         $this->calculatePluginDependencies($component->getPlugin());
       }

People often specify the key in foreaches for readability, so let's remove these hunks from the patch and open a coding standards issue if we want to standardise one way or the other.

The use of the key is not in general defined in the coding standards. The example in the PEAR coding standards shows it with key but all over drupal it is used in both ways. Seems like it is used where the key is needed and if not it is left out. Also in this case the key is not something special needed anywhere. I would say it's ok to keep the hunks.

Rerolled patch to keep the key in foreach.

@pratik_kamble Hello,

Your patch is works well. Thanks for your contribution.

Committed 6ef456a and pushed to 9.0.x. Thanks!