Is there any idea why this gave undefined error?

[Filip Vandueren]

In the for loop, use < not <=

[Palak Bhatt]

Thanks ☺ Brothers i have no words for you.

I have the other question
I am trying to move all selected layers at every Nth number.

What should i do this code to works?

var d = 0;
var myLayers = app.project.activeItem.selectedLayers;
var myMove = app.project.activeItem;

if (myLayers.length > 0){
for (var i=0; i &lt; myLayers.length; i++){
d = myLayers[i].index;
}
myMove.layer(d).moveAfter(myMove.layer(d+10));

}else{
alert ("No layers selected.");
}


[Palak Bhatt]

The problem is on above code is only moved last number of selected layer, How can I move bulk layers at once? at fixed number of layers

[Palak Bhatt]

I Got it ☺

Check it below
Thank you, Brother

I want to add you on Facebook if you don’t mind
give me your id if possible, You are super pro :)))
many thanks

ver NumberOfMove = 10;
var myLayers = app.project.activeItem.selectedLayers;
var myMove = app.project.activeItem;

if (myLayers.length > 0){
for (var i=0; i &lt; myLayers.length; i++){
var d = myLayers[i].index;
myMove.layer(d).moveAfter(myMove.layer(d+NumberOfMove));
}

}else{
alert ("No layers selected.");
}

[Palak Bhatt]

Corrected and tested
Now its final

var d = 0;
var NumberOfMove = 2;
var myLayers = app.project.activeItem.selectedLayers;
var myMove = app.project.activeItem;

if (myLayers.length > 0){
for (var i=0; i &lt; myLayers.length; i++){
d = myLayers[i].index;

myMove.layer(d).moveAfter(myMove.layer(d+NumberOfMove+myLayers.length));
}

}else{
alert ("No layers selected.");
}

[Filip Vandueren]

I don’t really understand how you want the layers to move with this code, It seems very unpredictable.

If you want the layers to move down by “numberOfMove”, it’s best to cycle through them backwards, because moving a layer will change the indexes of all the layers after it.
So we first order the selectedLayers by index descending (they are in the list in the order that they were clicked on, not in the order of index)

I’ve changed the code a bit so it also works for negative numbers.


var numberOfMove = 2;
var myComp = app.project.activeItem;
var myLayers = myComp.selectedLayers;

myLayers.sort( function(a,b) { return b.index - a.index }); // reverse sort layers by index
if (numberOfMove < 0) myLayers.reverse(); // except if we're moving backwards, then we do want sorted by index ascending

if (myLayers.length > 0){
for (var i=0; i< myLayers.length; i++){
d = Math.min ( d , myComp.numLayers-i); // never move than the number of layers
if (myLayers[i].index!==d) { // can't move a layer after itself
myLayers[i].moveAfter(myComp.layer(d));
}
} else {
d= Math.max (d, 1+i); // never less than 1st layer
if (myLayers[i].index!==d) { // can't move a layer before itself
myLayers[i].moveBefore(myComp.layer(d));
}
}
}

}else{
alert ("No layers selected.");
}