Skip to content

Commit 23a06b3

Browse files
committed
append doesn’t need a idGenerator
1 parent 65800ca commit 23a06b3

1 file changed

Lines changed: 1 addition & 7 deletions

File tree

src/core/stepper/Stepper.js

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -172,11 +172,10 @@ export class Stepper extends StepState {
172172
/**
173173
* Appends one or more objects as new states to the current node
174174
* @param {Object|Array<Object>} items - Single object or array of objects to append as new states
175-
* @param {Function} [idGenerator] - Optional function to generate custom IDs for each item
176175
* @returns {Stepper} Returns this instance for method chaining
177176
* @throws {Error} If items is not an object or array, or if adding items would exceed maxStepperRows
178177
*/
179-
append(items, idGenerator = null) {
178+
append(items) {
180179
this._log.debug('\tappend', items)
181180
// Convert single item to array if needed
182181
const itemsToAdd = Array.isArray(items) ? items : [items]
@@ -193,11 +192,6 @@ export class Stepper extends StepState {
193192

194193
// Try to add each item individually
195194
itemsToAdd.forEach((item) => {
196-
// If an idGenerator function is provided, use it to generate the ID
197-
if (typeof idGenerator === 'function') {
198-
item.id = idGenerator(item)
199-
}
200-
201195
// Check if this specific item would create a duplicate path
202196
if (this._hasDuplicatePaths(item)) {
203197
this._log.warn(`Warning: Skipping item that would create duplicate path`)

0 commit comments

Comments
 (0)