Page MenuHomePhabricator

VisualEditor: Copy & paste should use ve.dm.Node#getClonedElement
Closed, ResolvedPublic

Description

Because we want to be able to make certain properties non-cloneable, e.g. in ve.dm.MWReferenceNode.prototype.getClonedElement we strip 'contentsUsed'.


Version: unspecified
Severity: normal

Details

Reference
bz49307

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 1:59 AM
bzimport set Reference to bz49307.
Esanders created this task.Jun 7 2013, 1:43 PM

Change 70913 had a related patch set uploaded by Catrope:
Invoke getClonedElement() on each element on copy

https://gerrit.wikimedia.org/r/70913

We should very that copy/pasting a named reference with contents, doesn't copy the contents.

"We should verify..."

Change 70913 merged by jenkins-bot:
Invoke getClonedElement() on each element on copy

https://gerrit.wikimedia.org/r/70913

Fixed and will go live in a few minutes.