Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ULT_SetMediaItemNote not working in Lua #702

Closed
matthewjumpsoffbuildings opened this issue Feb 19, 2015 · 2 comments
Closed

ULT_SetMediaItemNote not working in Lua #702

matthewjumpsoffbuildings opened this issue Feb 19, 2015 · 2 comments

Comments

@matthewjumpsoffbuildings

im using Lua, SWS 2.6.1 with Reaper 5 pre 13. I have also tested this with 5 pre 12, running SWS 2.6.1 and 2.6.2

If i create a new item with reaper.AddMediaItemToTrack() then attempt to run ULT_SetMediaItemNote() on it, nothing seems to happen, no notes are added.

@matthewjumpsoffbuildings
Copy link
Author

@matthewjumpsoffbuildings matthewjumpsoffbuildings commented Feb 19, 2015

just did a simple test

item = reaper.GetSelectedMediaItem(0, 0)

reaper.ULT_SetMediaItemNote(item, "test")

ran it on a single selected empty item, did nothing. is this function broken in Lua?

GetMediaItemNote works. and also if you do a Set on an item that has notes, instead of just failing and leaving the original notes, it deletes all notes. but it doesnt trigger a UI refresh, so you have to zoom/scroll to see it

Loading

@matthewjumpsoffbuildings matthewjumpsoffbuildings changed the title ULT_SetMediaItemNote not working on newly ReaScript created item ULT_SetMediaItemNote not working in Lua Feb 21, 2015
@Jeff0S
Copy link
Member

@Jeff0S Jeff0S commented Mar 18, 2015

Fixed for the next pre-release, thanks for the report.
Note about refresh: your script has to take care of it with UpdateArrange()

Loading

@Jeff0S Jeff0S closed this Mar 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants