Still cannot get indices of selected FXs in track FX chain view by ReaScript?[SOLVED]

[pigstoe]

In my script, indices of selected FXs in track FX chain view are needed. And I found this thread.
https://forum.cockos.com/showthread.php?t=168718

Is there still no functions for this? And still impossible to get this information?

[cfillion]

Using CF_GetTrackFXChain and CF_EnumSelectedFX from SWS 2.12:

function enumSelectedTrackFX(track)
  local fxChain = reaper.CF_GetTrackFXChain(track)
  local i = -1

  return function()
    i = reaper.CF_EnumSelectedFX(fxChain, i)
    if i < 0 then i = nil end
    return i
  end
end

local track = reaper.GetTrack(nil, 0)
for i in enumSelectedTrackFX(track) do
  reaper.ShowConsoleMsg(string.format('%d\n', i))
end

[pigstoe]

Originally Posted by cfillion

Using CF_GetTrackFXChain and CF_EnumSelectedFX from SWS 2.12:

function enumSelectedTrackFX(track)
  local fxChain = reaper.CF_GetTrackFXChain(track)
  local i = -1

  return function()
    i = reaper.CF_EnumSelectedFX(fxChain, i)
    if i < 0 then i = nil end
    return i
  end
end

local track = reaper.GetTrack(nil, 0)
for i in enumSelectedTrackFX(track) do
  reaper.ShowConsoleMsg(string.format('%d\n', i))
end

I tested your code and found a bug for CF_EnumSelectedFX() function. If CF_EnumSelectedFX() is used at the track which has track name, it always returns 0. This function works fine only at unnamed track.

Thank you, anyway. I'm going to report it.

[cfillion]

Update SWS to 2.12. https://www.sws-extension.org/download/pre-release/

[pigstoe]

Originally Posted by cfillion

Update SWS to 2.12. https://www.sws-extension.org/download/pre-release/

After updated, It always returns 0, even at unnamed track.

[cfillion]

Indices are zero-based so a return value of 0 means the first FX is selected (above the given starting index). Is that value incorrect?

[pigstoe]

Originally Posted by cfillion

Indices are zero-based so a return value of 0 means the first FX is selected (above the given starting index). Is that value incorrect?

Incorrect. When I select nothing, everything, something and even when there is no fx in track, it always returns 0.

[cfillion]

Mhh... I can't duplicate that here. Can you share a minimal script (assuming it's not exactly as in the snippet above) and/or a minimal project that produce that? (Also, just in case, which OS?)

EDIT: CF_GetTrackFXChain seems to always returns nil now in REAPER 6.12/macOS. Not sure exactly in which version it broke or why, I'll look into it tomorrow. In the case of a nil fx chain, CF_EnumSelectedFX returns -1, not 0 though...

[pigstoe]

Originally Posted by cfillion

Mhh... I can't duplicate that here. Can you share a minimal script (assuming it's not exactly as in the snippet above) and/or a minimal project that produce that? (Also, just in case, which OS?)

EDIT: CF_GetTrackFXChain seems to always returns nil now in REAPER 6.12/macOS. Not sure exactly in which version it broke or why, I'll look into it tomorrow. In the case of a nil fx chain, CF_EnumSelectedFX returns -1, not 0 though...

Found. In process of minimizing my code to share, I found that PreventUIRefresh(1) prevents CF_EnumSelectedFX works fine. Is that a bug? I can't understand why UI refreshing affects getting indices of selected FXs. But, I should remove preventing to refresh UI in this script.

Anyway, thanks!

[cfillion]

Originally Posted by pigstoe

Found. In process of minimizing my code to share, I found that PreventUIRefresh(1) prevents CF_EnumSelectedFX works fine. Is that a bug? I can't understand why UI refreshing affects getting indices of selected FXs. But, I should remove preventing to refresh UI in this script.

Ah yes. It's a limitation of the way the function asks REAPER to synchronously update the title of the FX chain window. PreventUIRefresh inhibits TrackList_AdjustWindows.

I discovered a better way to do that while fixing it for v6 on non-Windows so, in the next SWS update, CF_GetTrackFXChain will be usable along with PreventUIRefresh.

[pigstoe]

Good news. I'm looking forward to it. Thank you.