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

3.1.30 config_load scope global #274

Closed
blaaat opened this issue Aug 17, 2016 · 2 comments
Closed

3.1.30 config_load scope global #274

blaaat opened this issue Aug 17, 2016 · 2 comments

Comments

@blaaat
Copy link

blaaat commented Aug 17, 2016

I'm getting the following error;

{config_load file='mailtemplate.conf' scope="global"}" illegal value ''global'' for "scope" attribute

Even though scope global is still documented as valid.

@dimadeush
Copy link

+1, the same problem )

uwetews added a commit that referenced this issue Aug 20, 2016
@uwetews
Copy link
Contributor

uwetews commented Aug 20, 2016

Config variables do not have a real global storage area like template variables,
Smarty was now checking for real available scopes which caused the error.

Correct would be

{config_load file='mailtemplate.conf' scope="smarty"}

Use that.

So the documentation needs to be corrected.

Anyway I patched the master branch that scope="global" will fallback to scope="smarty"

@uwetews uwetews closed this as completed Aug 20, 2016
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

No branches or pull requests

3 participants