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

Undeclared Altis core dependency breaks composer updating #563

Open
tomjn opened this issue Jan 29, 2023 · 0 comments
Open

Undeclared Altis core dependency breaks composer updating #563

tomjn opened this issue Jan 29, 2023 · 0 comments
Labels
bug Existing functionality isn't behaving as expected

Comments

@tomjn
Copy link
Contributor

tomjn commented Jan 29, 2023

Previously there was an issue where local server became dependent on Altis Core, but the package was undeclared: #501 This is not the same problem, but a consequence of the resolution implemented.

The resolution for this was not to declare the dependency as a dependency though, and an alternative method was used.

Now, when upgrading the Altis local server package, because v13 still doesn't declare Altis core as a dependency it isn't installed and the command fails with the warning that was added to resolve that issue.

However, this also puts the project in a limbo state where the required package cannot be installed. All attempts to install altis/core via composer commands fail as the previous resolution interrupted package installation/removal processes.

The immediate result is that it tries to scan the local server package folder for files to autoload but fails because it was removed, then suffers a hard crash because it can't load wp-config.php from the package folder.

So now I'm stuck, to install altis/core I need to install altis/core. Fixing this forced me to nuke my vendor folder.

Steps to reproduce:

  1. Install Local Server v11 or a version that did not require Altis core
  2. Attempt to upgrade local server and encounter the problem in Call to undefined function Altis\get_config() #501
  3. Add altis/core to composer.json
  4. Attempt to composer update to install the new packages and upgrade the old ones
@tomjn tomjn added the bug Existing functionality isn't behaving as expected label Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Existing functionality isn't behaving as expected
Projects
None yet
Development

No branches or pull requests

1 participant