Skip to content

Added ID-Based Duplicate Protection for Cloning #1327

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Memocana
Copy link

It's an old closed issue that's been never resolved but I made a solution for #708. I added a new option, allowDuplicates, that is true by default and it preserves the default behaviour of the code. If this tag is set to false, then it checks if there is a conflicting ID in the potential new parent. If there is a such ID, it basically behaves like put = false for that specific grouping.

@rsmike
Copy link

rsmike commented Jul 27, 2018

extremely useful!

@owen-m1
Copy link
Member

owen-m1 commented Jan 7, 2019

Please give a JSBin demonstration of what this feature does

@ikarthikng
Copy link

This is an extremely useful feature. Any idea when this feature will be available in the main CDN link?

@chimney-jim
Copy link

I could also really use this feature.

@waynevanson waynevanson added the missing reproduction Cannot help without an example label Apr 13, 2020
@waynevanson
Copy link
Contributor

waynevanson commented Apr 13, 2020

@Memocana and anyone else.

It looks like there is some demand for this feature. Would you be kind enough to produce an example for us? Thank you in advance.

You can create one using one of these templates.

@ledahu
Copy link

ledahu commented Feb 13, 2021

if it can help, using jquery

group: {
name:'myGRP',
put:function(to,from,item){
// console.log("FROM:"+$(from.el).attr('id'))
// console.log("ITEM:"+$(item).attr('id'))
// console.log("TO:"+$(to.el).attr('id'))
if($(to.el).find("#"+$(item).attr('id')).length>0) return false;
else return true;
},
},

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
missing reproduction Cannot help without an example
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants