Replace the unique subform id via random bytes by just an increasing …

…integer in the fields type rendering process.

layouts/joomla/form/field/subform/repeatable-table.php

@@ -66,9 +66,6 @@
 		'.subform-table-sublayout-section .controls { margin-left: 0px }'
 	);
 }
-
-// @TODO
-$unique_subform_id = bin2hex(openssl_random_pseudo_bytes(4));
 ?>
 
 <div class="row-fluid">

layouts/joomla/form/field/subform/repeatable-table/section-byfieldsets.php

@@ -18,7 +18,6 @@
  * @var array   $buttons    Array of the buttons that will be rendered
  */
 extract($displayData);
-
 ?>
 
 <tr

layouts/joomla/form/field/subform/repeatable.php

@@ -34,9 +34,6 @@
 }
 
 $sublayout = empty($groupByFieldset) ? 'section' : 'section-byfieldsets';
-
-// @TODO
-$unique_subform_id = bin2hex(openssl_random_pseudo_bytes(4));
 ?>
 
 <div class="row-fluid">
@@ -47,6 +44,7 @@
 			data-bt-move="a.group-move-<?php echo $unique_subform_id; ?>"
 			data-repeatable-element="div.subform-repeatable-group-<?php echo $unique_subform_id; ?>"
 			data-minimum="<?php echo $min; ?>" data-maximum="<?php echo $max; ?>">
+
 			<?php if (!empty($buttons['add'])) : ?>
 			<div class="btn-toolbar">
 				<div class="btn-group">

libraries/joomla/form/fields/subform.php

@@ -274,6 +274,12 @@ protected function getInput()
 		$data['fieldname'] = $this->fieldname;
 		$data['groupByFieldset'] = $this->groupByFieldset;
 
+		// For each rendering process of a subform element, we want to have a
+		// separate unique subform id present to could distinguish the eventhandlers
+		// regarding adding/moving/removing rows from nested subforms from their parents.
+		static $unique_subform_id = 0;
+		$data['unique_subform_id'] = ('sr-' . $unique_subform_id++);
+
 		// Prepare renderer
 		$renderer = $this->getRenderer($this->layout);