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

Undefined strings for tmp directory #850

Closed
michael-milette opened this issue Dec 9, 2024 · 6 comments · Fixed by #852
Closed

Undefined strings for tmp directory #850

michael-milette opened this issue Dec 9, 2024 · 6 comments · Fixed by #852

Comments

@michael-milette
Copy link

michael-milette commented Dec 9, 2024

What happened?

During an upgrade of the plugin (most recent version last updated in November 2024 on GitHub) in Moodle 4.1 (English), I noticed that strings are undefined / undefined:

image

Here is a list of the missing strings:

  • tempdir
  • tempdirdefault
  • tempdir_help

Let me know if you have any questions.

Best regards,

Michael Milette

@danmarsden
Copy link
Member

Those strings appear to.be there to me.. do you have local changes to the lang string file that have caused a merge conflict which wasn't fixed properly?

@danmarsden
Copy link
Member

@Limekiller @keevan the tempdirdefault value there doesn't look like it should have landed in the lang strings to me. We should pull that out too.

@keevan
Copy link
Contributor

keevan commented Dec 10, 2024

Thanks for catching that one @danmarsden, I've pushed a PR which should fix this in 404 and 39 branches.

@danmarsden
Copy link
Member

Cool thanks!

@michael-milette
Copy link
Author

Those strings appear to.be there to me.. do you have local changes to the lang string file that have caused a merge conflict which wasn't fixed properly?

I have no local changes to the strings. It was working fine before I upgraded the plugin from a version from October 2023.

@danmarsden
Copy link
Member

yeah - sorry @michael-milette you were correct they were missing in one of the branches - @keevan pushed a fix through last night though so it should be sorted now.

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants