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

[2013-02-13] ManyManyList->removeAll() not working with JOINs #1681

Closed
silverstripe-issues opened this issue Apr 3, 2013 · 5 comments
Closed

Comments

@silverstripe-issues
Copy link

created by: @chillu (ischommer)
created at: 2013-02-13
original ticket: http://open.silverstripe.org/ticket/8258


ManyManyList causes a single DELETE query, as opposed to DataList which does one DELETE query per item. This means the single DELETE query needs to replicate all selection criteria if ot was "converted" from a SELECT query before.

Emulating joins in deletes is possible, but its going to be gricky to satisfy all potential query permutations. My recommendation is to change to single-row DELETE queries which work on the results from executing the SELECT queries.

For context, this problem was discovered with "Group" queries breaking on usage of both subsites and advancedworkflow

@simonwelsh simonwelsh added this to the 3.2 milestone Mar 15, 2014
@tractorcow tractorcow modified the milestones: 3.1.14, 3.2.0 Jun 16, 2015
@tractorcow
Copy link
Contributor

If this exists in 3.1 it needs to be fixed there also.

@sminnee sminnee removed this from the 3.1.14 milestone Jul 21, 2015
@dhensby
Copy link
Contributor

dhensby commented Aug 12, 2016

I don't get it, how's the problem replicated and how does it manifest itself?

@tractorcow
Copy link
Contributor

It sounds like the problem is limited to Member_Groupset. maybe just limit the suggested fix to that? (copy DataList::removeAll() to Member_Groupset).

@kinglozzer
Copy link
Member

Related to #3948?

@tractorcow
Copy link
Contributor

ok, I'm closing this in favour of #3948 which more precisely targets the issue (even though it's newer).

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

No branches or pull requests

6 participants