Skip to content
This repository was archived by the owner on Jan 29, 2025. It is now read-only.

Fix src() method in RequireJSHelper.php #72

Closed
wants to merge 2 commits into from
Closed

Fix src() method in RequireJSHelper.php #72

wants to merge 2 commits into from

Conversation

liviucmg
Copy link

First check if the engine supports the $this->requireJsSrc file, and only then check if it exists. Closes #71.

First check if the engine supports the $this->requireJsSrc file, and only then check if it exists.
Ref issue #72.

`$this->engine->supports` throws `InvalidArgumentException` when using PHP templates instead of Twig templates.

```
An exception has been thrown during the rendering of a template ("Template name "/cdnjs.cloudflare.com/ajax/libs/require.js/2.1.8/require.min.js" is not valid (format is "bundle:section:template.format.engine").") in HearsayRequireJSBundle::initialize.html.twig at line 46.
```
@liviucmg liviucmg changed the title Fix RequireJSHelper.php engine support check Fix src() method in RequireJSHelper.php Apr 20, 2014
@liviucmg liviucmg closed this Apr 20, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix src() method in RequireJSHelper.php
1 participant