You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In a plugin I am building I want to get all video items from a presentation. To strip out anything not-video I do a check with isVideo. However, this returns false for a single looping video. After some digging I suspect that the problem is the asterisk notation behind the extension of a looping video (found in the FilePlaylist property).
This problem consists of 2 parts:
Part 1: getValue returns the file path including the asterisk notation for looping (*0*2 for example). (When I printed the value that getValue returned I also got *0*[times already looped] back).
It seems that setValue was fixed for #295/#296 but getValue was not?
Part 2: isVideo looks at the prop:srcitem (the same thing that getValue returns) and when this includes the asterisk info the regex to check if it is a video fails. Because the regex does not allow anything after the extension.
The fix would therefore probably be to have getValue not return the asterisk-part, and let isVideo() use getValue(). A workaround for me now is to add [\*,\d]* before the dollar sign of the regex.
The text was updated successfully, but these errors were encountered:
In a plugin I am building I want to get all video items from a presentation. To strip out anything not-video I do a check with
isVideo
. However, this returns false for a single looping video. After some digging I suspect that the problem is the asterisk notation behind the extension of a looping video (found in theFilePlaylist
property).This problem consists of 2 parts:
Part 1:
getValue
returns the file path including the asterisk notation for looping (*0*2
for example). (When I printed the value thatgetValue
returned I also got*0*[times already looped]
back).It seems that setValue was fixed for #295/#296 but getValue was not?
Part 2:
isVideo
looks at theprop:srcitem
(the same thing thatgetValue
returns) and when this includes the asterisk info the regex to check if it is a video fails. Because the regex does not allow anything after the extension.The fix would therefore probably be to have
getValue
not return the asterisk-part, and letisVideo()
usegetValue()
. A workaround for me now is to add[\*,\d]*
before the dollar sign of the regex.The text was updated successfully, but these errors were encountered: