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

Cycle ManyToMany relation is not working #449

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

albertborsos
Copy link

@albertborsos albertborsos commented Apr 8, 2022

Q A
Is bugfix?
New feature?
Breaks BC?
Fixed issues -

I just want to inform you to know that the ManyToMany relation is not working correctly even in this demo application.
(Or I use it wrong, because I can not solve this issue for days.)

I want to store multiple tags for posts. These tags maybe new, maybe already exists in the database. If it is new, then it works great. But if I want to assing an existing Tag to a new Post, it fails.

Related issue in Cycle ORM: cycle/orm#329

I also updated the tests for this demo app, because I did not understand how it could work here. But it is not working.

@rustamwin rustamwin added the type:test Test label Apr 8, 2022
@albertborsos
Copy link
Author

#439 is a related issue too

@samdark
Copy link
Member

samdark commented Apr 12, 2022

@albertborsos how exactly does it fail?
@roxblnfk would you please take a look?

@albertborsos
Copy link
Author

@samdark I modified the tests to make them fail.

I get the exact same error in my application:
https://github.com/yiisoft/demo/runs/5888867520?check_suite_focus=true

@TiaNex-Com
Copy link

TiaNex-Com commented May 26, 2022

yes,
temp solution

cycle/orm#329

Copy link
Member

@roxblnfk roxblnfk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it will be better to set ORM version to ^2.1.1

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

Successfully merging this pull request may close these issues.

5 participants