Page MenuHomePhabricator

VisualEditor: Remove transclusion node if we were just going to set it to empty content
Closed, ResolvedPublic



Steps to reproduce:

1.Open a page with VE
2.Go to Insert>Template
3.Click on Show Options
4.Click on the trash icon.
5.Click on "Add Content"

Observed Result:
The "Insert transclusion" button becomes enabled and an empty transclusion with empty content gets added upon clicking on it.

See the screenshot attached

Test Environment:
Browser: Chrome Version 26.0.1410.65
OS: MAC OS X 10. 8. 5

Version: unspecified
Severity: minor




Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 3:02 AM
bzimport set Reference to bz63341.
Ryasmeen created this task.Apr 1 2014, 12:32 AM

Created attachment 14990


Also, there is an error in the console:
Uncaught TypeError: Cannot read property 'length' of null

  • Bug 63340 has been marked as a duplicate of this bug. ***

See also bug 63340, which was duped to this one but is actually subtly different: it's about when the only part is a TemplatePlaceholderPage

Change 169638 had a related patch set uploaded by Alex Monk:
MWTransclusionModel: Exclude parts serialising to '' from the plain object

Change 169638 merged by jenkins-bot:
MWTransclusionModel: Exclude parts serialising to '' from the plain object

Verified the fix in test2

Verified the fix in production