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.

Sign up for GitHub

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

Jump to bottom

Add support for "./" in findField path #130

Open

DegrangeM wants to merge 1 commit into h5p:master from DegrangeM:patch-1

DegrangeM commented Apr 13, 2021

Currently, using ./ will not work. We must use foo instead of ./foo.

This PR add this support. This might seems stupid to allow that as we just have to not write the ./ but:

./ is a valid path and is often used in html
it usefull to allow a field to either be a value, or to point to an other field which will hold the value : we just have to detect if the value start with a . (this is something I use in a widget library I am developping and that I will use in two content type I am developping, I currently have to manually remove the ./ in the path to make it work).


          Add support for "./" in findField path

6e60865

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment