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

fix for if the same resource name is in a root and sub dir. #77

Merged

Conversation

jcompagner
Copy link
Contributor

then with local or all it should not hit the root one.

everything is the same as lessc compiler is outputting except the
"off.css" which url15 works like the others that we do resolve it even
if it shouldn't according to lessc
Because url14 'test.jpg' is already mapping on the root image so
so url15 '../test.jpg' shouldn't be there (its above root).
But we fallback to always map it on the root anyway,

all it should not hit the root one.

everything is the same as lessc compiler is outputitng except the
"off.css" which url15 works like the others that we do resolve it even
if it shouldn't according to lessc
Because url14 'test.jpg' is already mapping on the root image so
so url15 '../test.jpg' shouldn't be there (its above root). But we fall
back to always map it on the root anyway,
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.002%) to 92.598% when pulling 96f8e2b on jcompagner:datauri_same_resources_name_fix into 2752b71 on i-net-software:master.

@Horcrux7
Copy link
Member

Horcrux7 commented Dec 8, 2020

It should be called the ReaderFactory.openStream with the 3 parameters. In the other case it would break for the users which use its own implementation of ReaderFactory.

@jcompagner
Copy link
Contributor Author

done now openStream(url) is not called anymore from the data-uri code only the 3 args
But in the end that ends up to be the same thing (The actual url that is being loaded)

@Horcrux7 Horcrux7 merged commit b04fc52 into i-net-software:master Dec 8, 2020
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.

None yet

3 participants