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.0]: Language files are not removed on mods deleting #8094

Open
dragomano opened this issue Feb 9, 2024 · 5 comments
Open

[3.0]: Language files are not removed on mods deleting #8094

dragomano opened this issue Feb 9, 2024 · 5 comments
Assignees
Milestone

Comments

@dragomano
Copy link
Contributor

Basic Information

sshot-7

Steps to reproduce

  1. Install any mod with hooks and language files for SMF 2.1
  2. Try to uninstall it

Expected result

<remove-dir name="$languagedir/modname" />

I think that we need to check both paths on deleting: $boarddir/Languages, and $themedir/languages (if compatibility mode is enabled)

Actual result

No response

Version/Git revision

3.0 Alpha 2

Database Engine

MySQL

Database Version

10.11.6-MariaDB

PHP Version

8.0.30

Logs

No response

Additional Information

No response

@jdarwood007 jdarwood007 self-assigned this Feb 9, 2024
@jdarwood007 jdarwood007 added this to the 3.0 Alpha 2 milestone Feb 9, 2024
@jdarwood007
Copy link
Member

You are correct, I will look into this when I get a chance unless somebody else gets a chance. This may not be the cleanest logic, but would only need to be done when we are running in compatibility mode.

@Sesquipedalian
Copy link
Member

I cannot reproduce this myself.

@dragomano, does the issue still exist for you? Perhaps more recent changes have solved it already, but I don't know.

@dragomano
Copy link
Contributor Author

What mods did you use for testing?

P. S. I have installed the current version for testing and got an error on the Package manage index area - "Fatal error: Uncaught TypeError: in_array(): Argument #2 ($haystack) must be of type array, string given in PackageManager\PackageManager.php:1639", fix - replace "$items = Utils::jsonDecode($data['data'], true);" with "$items = Utils::jsonDecode($data['data'], true) ?? [];"

@Sesquipedalian
Copy link
Member

Several of my own. Can you give me a mod that does produce the error? Then I can figure out what is causing it.

@dragomano
Copy link
Contributor Author

Try to install the Simillar Topics, in the compatibility mode.

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

No branches or pull requests

3 participants