8000 Fix many 2 many relation meta bug for version resources. by raven-chen · Pull Request #256 · qor/qor · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix many 2 many relation meta bug for version resources. #256

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
Merged
merged 40 commits into from
Jun 30, 2021
Merged

Fix many 2 many relation meta bug for version resources. #256

merged 40 commits into from
Jun 30, 2021

Conversation

raven-chen
Copy link
Member

No description provided.

@raven-chen
Setup test for reproduce the bug.
f11d1dc
@raven-chen
Add test to many2many relationship setter.
dcfad7b
@raven-chen
Expand many2many relation test to support versions. (test fails becau…
c765cde 
…se the feature is not implemented)
@raven-chen
Make many2many relation meta supports id & version_name composite pri…
10fceea 
…mary key.
@raven-chen
Remove condition expression that never occurs.
6a32ac0
@raven-chen
eliminate potential version_name condition on the main object, we don…
710c48f 
…'t need it when querying associated records. it usually added by qor/publish2.
@raven-chen
Whenever fetching the associated records do not apply version_name co…
c27b13b 
…ndition for main object.
@raven-chen
Use real-env frontend parameter to process metaValue.Value.
3dca068
@raven-chen
Optimize belongs to relation meta, add test to it.
410800e
@raven-chen
Skip potential incorrect main object version_name condition when quer…
6534a48 
…ying associated many2many records.
@raven-chen
Use CompositePrimaryKeySeparator to separate id and version_name inst…
3811ba0 
…ead of separate them into two pieces. This can avoid frontend change.
@raven-chen
Skip blank string when it is not the only element
4916e04
@raven-chen
Ability to support automatically apply compositePrimaryKey in valuer.
129e51c
@raven-chen
Ability to handle versioning belongs_to.
41a0ac6
@raven-chen
Skip main object version_name condition while query associated record…
5843cab 
… value.
@raven-chen
Remove debug code.
9fd9b01
@raven-chen
Check foreignVersionField presence before construct composite old pri…
1d2f65f 
…maryKey. So it can work properly for other types of multiple foreign keys.
@raven-chen
Update qor/admin ref.
23a77c4
@raven-chen
Add comment to explain the logic.
dbce795
@raven-chen
Fix belongs to with versioning resource cannot set blank value bug.
397d9e4
@raven-chen
Handle pointer associations.
34ee132
@raven-chen
Set default db driver as postgres
22552c0
@raven-chen
Support add associations when creating a new version of record.
49b1465
@raven-chen
Skip main object's "version_name = xxx" condition correctly. the prev…
90bfabb 
…ious one was incorrect.
@raven-chen
Merge branch 'master' into fix-many2many-not-work-for-versioned-resou…
39cfd70 
…rce-bug
@raven-chen
Extract setCompositePrimaryKey logic into function and add test to it.
5a200c6
@raven-chen
Extract fieldIsStructAndHasVersion function and add test to it.
4f48614
@raven-chen
Add double assurance to belongs_to association setter. see comment in…
8fb29cc 
… the code for detail.
@raven-chen
extract normal belongs_to logic into a function with test.
ad72994
@raven-chen
Extract versioning belongs_to to a function, add test to it.
d527cb6
@raven-chen raven-chen merged commit bfbb16e into master Jun 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@raven-chen
0