Skip to content

mediaPath does not need trailing slash as provided by scan.php#419

Open
hayden-t wants to merge 1 commit into
givanz:masterfrom
hayden-t:patch-5
Open

mediaPath does not need trailing slash as provided by scan.php#419
hayden-t wants to merge 1 commit into
givanz:masterfrom
hayden-t:patch-5

Conversation

@hayden-t

Copy link
Copy Markdown

No description provided.

@hayden-t

hayden-t commented Oct 14, 2025

Copy link
Copy Markdown
Author

actually though its not needed, from a path point of view, it seems it's possibly being tested for here

let src = (value.indexOf("//") > -1 || value.indexOf("media/") > -1 || value.indexOf("image-cache/") > -1 ? '' : Vvveb.themeBaseUrl) + value
as to if user selected image or not and if the theme url should be appended to the path, which the whole thing is a bit messy, and im going to propose a change to that test

@hayden-t

hayden-t commented Oct 14, 2025

Copy link
Copy Markdown
Author

actually the // i think is being used to test if its an absolute path such as http:// but this i will expand on to support root-relative path eg '/path'

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant