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.

Sign up for GitHub

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

Jump to bottom

Namespace for AcquiaCloudApi Connector and Client needs updating #6

Open
wu-edward opened this issue Jun 28, 2020 · 1 comment
Open

Namespace for AcquiaCloudApi Connector and Client needs updating #6

wu-edward opened this issue Jun 28, 2020 · 1 comment

Comments

@wu-edward
Copy link

Got this error on the 2.0 release:
PHP Fatal error: Uncaught Error: Class 'AcquiaCloudApi\CloudApi\Connector' not found in /vendor/lcatlett/blt-drush-alias/src/Blt/Plugin/Commands/DrushAliasCommand.php:193

Looks like the namespaces in typhonius/acquia-php-sdk-v2 were changed in this commit: typhonius/acquia-php-sdk-v2@2d6b412

So at the very least, probably AcquiaCloudApi\CloudApi\Connector needs to be AcquiaCloudApi\Connector\Connector and AcquiaCloudApi\CloudApi\Client needs to be AcquiaCloudApi\Connector\Client. I haven't yet investigated further whether the class method calls need updating as well.
@siliconmeadow
Copy link

As well as changes to the namespaces, there were changes to numbers of arguments required in arguments, etc. I wonder if it might be better to specify a specific version of the typhonius/acquia-php-sdk-v2 in the composer.json file, instead?

@cainaru cainaru mentioned this issue Mar 8, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@siliconmeadow @wu-edward