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

Could not translate external modules from different folder #41

Open
yurabakhtin opened this issue Feb 17, 2021 · 2 comments
Open

Could not translate external modules from different folder #41

yurabakhtin opened this issue Feb 17, 2021 · 2 comments
Assignees

Comments

@yurabakhtin
Copy link
Contributor

When I clone some external module from GitHub it is cloned by default in to the folder with prefix humhub-modules-:

module_dirs

and the modules works well, but this module cannot find the message files from such folders because it expects if module has an ID = twofa then the message files are located in the folder twofa too and not in humhub-modules-twofa:

error

Do we really need the 3rd restriction?

@buddh4
Copy link
Contributor

buddh4 commented Feb 17, 2021

Since the module was designed to only be used in our translation environment I wanted to make it as strict as possible in order to prevent any kind of vulnerability, feel free to loosen the restriction as long as you don't open any vulnerabilities. Maybe you could disable some restrictions by configurations?

@yurabakhtin
Copy link
Contributor Author

@buddh4 If we want to keep the 3rd condition || !$this->validateParent($moduleId, $expectedModuleId) then I can suggest this solution #42.

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