Skip to content

Conversation

@yethee
Copy link
Contributor

@yethee yethee commented Sep 20, 2015

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #14764
License MIT
Doc PR n/a

@Tobion
Copy link
Contributor

Tobion commented Sep 20, 2015

Why is that needed? What fails? What is the error?

@yethee
Copy link
Contributor Author

yethee commented Sep 20, 2015

Windows uses backslash as a directory separator, so the reference to a template may be incorrect. For example, @app/foo\bar.html.twig instead of @app/foo/bar.html.twig.

@Tobion
Copy link
Contributor

Tobion commented Sep 20, 2015

When accesing files via php, you can use both forms of slashes. It works either way. Also twig normalizes the path anyway. So again, what error are you facing?

@yethee
Copy link
Contributor Author

yethee commented Sep 21, 2015

You are right. I check again, twig really uses cache file which it has been generated by not normalized path. Won't fix.

@yethee yethee closed this Sep 21, 2015
@yethee yethee deleted the fix-14764 branch September 26, 2015 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants