Endless loop with SelectPreviousLayer

[fs999]

Hello,

There seems to be a endless loop error with the SelectPreviousLayer if a selection has been saved in the Alpha Chanel of the image, the function always returns True.
I use this part of code to go to the bottom layer and iterate every layer up.
I have tried with SelectNextLayer there is no such a problem.
And if no selection is saved in the Alpha Chanel, the function works well.

Code: Select all

      
    FoundLayer = True 
    while FoundLayer == True: 
        FoundLayer = App.Do( Environment, 'SelectPreviousLayer', {
            'GeneralSettings': {
                'ExecutionMode': App.Constants.ExecutionMode.Silent, 
                'AutoActionMode': App.Constants.AutoActionMode.Match, 
                'Version': ((19,0,0),1)
                }
            })

[LeviFiction]

Interesting, shall have to look into that. i wonder if it happens in earlier versions of PSP or if it's new to 2020.

You might find this helpful, this comes from the Scripting Guide provided by Corel. It's a fast way to quickly select the bottom-most layer in a single command

Code: Select all

    # start by selecting the bottommost layer in the image.
    App.Do( Environment, 'SelectLayer', { 'Path': (9999,-9999, [], False ) } )

It leaves out the GeneralSettings as they don't really do anything with commands like this so it's an easy space saver to leave it out.

[fs999]

It is also in PSP 2019.

I will try your solution.

Thank you !

[fs999]

This SelectLayer doesn't work. I've tried it last year, that's why I made that loop.

I have added a message box asking if I used the alpha chanel and not iterating when true...

[LeviFiction]

Interesting.

First I have verified the bug you reference, i added a break condition so it wouldn't loop forever, but I can verify it and I will report it to Corel.

EDIT: I'm updating my solution to be even easier.

One option you can try is after selecting the layer, grab its absolute layer path. If the layer path is (0,0, [], False) then you're on the bottom layer, and can just break the loop.

Code: Select all

    LayerExists = True
    while LayerExists:
        LayerExists = App.Do( Environment, 'SelectPreviousLayer', {})
        p = App.Do( Environment, 'ReturnLayerProperties', {})['Path']
        if p == (0,0,[],False): #Bottom layer path
            break 

Second, I'm honestly suprised the SelectLayer option I gave you didn't work. It works in all of my tests, I can't seem to find a way to break it.

[fs999]

Yes ! That works !

Thank you very much !