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

Question #92

Open
harikt opened this issue Sep 23, 2023 · 6 comments · Fixed by monofony/Monofony#507
Open

Question #92

harikt opened this issue Sep 23, 2023 · 6 comments · Fixed by monofony/Monofony#507

Comments

@harikt
Copy link

harikt commented Sep 23, 2023

Hi,

I was experimenting monofony. So when I ran

composer create-project monofony/skeleton project_name

I came across a strange error.

Warning: Ambiguous class resolution, "Sylius\Component\Customer\Context\CustomerContextInterface" was found in both "/Users/harikt/projects/monofony-experiments/vendor/monofony/customer/Context/CustomerContextInterface.php" and "/Users/harikt/projects/monofony-experiments/vendor/sylius/customer/Context/CustomerContextInterface.php", the first will be used.
Warning: Ambiguous class resolution, "Sylius\Component\Customer\Model\CustomerGroup" was found in both "/Users/harikt/projects/monofony-experiments/vendor/monofony/customer/Model/CustomerGroup.php" and "/Users/harikt/projects/monofony-experiments/vendor/sylius/customer/Model/CustomerGroup.php", the first will be used.
Warning: Ambiguous class resolution, "Sylius\Component\Customer\Model\Customer" was found in both "/Users/harikt/projects/monofony-experiments/vendor/monofony/customer/Model/Customer.php" and "/Users/harikt/projects/monofony-experiments/vendor/sylius/customer/Model/Customer.php", the first will be used.
Warning: Ambiguous class resolution, "Sylius\Component\Customer\Model\CustomerInterface" was found in both "/Users/harikt/projects/monofony-experiments/vendor/monofony/customer/Model/CustomerInterface.php" and "/Users/harikt/projects/monofony-experiments/vendor/sylius/customer/Model/CustomerInterface.php", the first will be used.
Warning: Ambiguous class resolution, "Sylius\Component\Customer\Model\CustomerGroupInterface" was found in both "/Users/harikt/projects/monofony-experiments/vendor/monofony/customer/Model/CustomerGroupInterface.php" and "/Users/harikt/projects/monofony-experiments/vendor/sylius/customer/Model/CustomerGroupInterface.php", the first will be used.
Warning: Ambiguous class resolution, "Sylius\Component\Customer\Model\CustomerAwareInterface" was found in both "/Users/harikt/projects/monofony-experiments/vendor/monofony/customer/Model/CustomerAwareInterface.php" and "/Users/harikt/projects/monofony-experiments/vendor/sylius/customer/Model/CustomerAwareInterface.php", the first will be used.

Looking into this, I can see the monofony/customer is a fork of sylius/customer itself and namespaces are same. Any reason then why this package exists ?

You can reproduce the above via

composer dumpautoload -o
@loic425
Copy link
Member

loic425 commented Sep 24, 2023

Hi, thx a lot for reporting that warning. Monofony customer fork should not be used anymore. I've to check where I forgot to remove that dependency.

@harikt
Copy link
Author

harikt commented Sep 25, 2023

Hey @loic425 ,

These are used by https://github.com/Monofony/CoreContracts and https://github.com/Monofony/SyliusUserBridge .

composer depends monofony/customer
monofony/core-contracts     v0.10.1 requires monofony/customer (^1.10)
monofony/sylius-user-bridge v0.10.1 requires monofony/customer (^1.10)

@harikt
Copy link
Author

harikt commented Sep 26, 2023

Is it possible to remove the unused packages from packagist. So it will help to find the issues quickly.

@harikt
Copy link
Author

harikt commented Sep 26, 2023

Which is the default branch ? master or main ? May be some hints in the readme will be helpful for the newbies to contribute.

@loic425
Copy link
Member

loic425 commented Sep 29, 2023

Which is the default branch ? master or main ? May be some hints in the readme will be helpful for the newbies to contribute.

Indeed, it's not clear, maybe provide a github template. For now it's seems to be the "master" branch. The 1.10 branch seems to be outdated that's not normal. I've to clean that.

@loic425
Copy link
Member

loic425 commented Sep 29, 2023

Is it possible to remove the unused packages from packagist. So it will help to find the issues quickly.

Yes, we can "archive" it.

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

Successfully merging a pull request may close this issue.

2 participants