Opened 8 years ago

Closed 8 years ago

#13703 closed defect (fixed)

DnD code has unnecessary markupFactory() declarations

Reported by: bill Owned by: bill
Priority: low Milestone: 1.7
Component: DnD Version: 1.6.1
Keywords: Cc:
Blocked By: Blocking:

Description

markupFactory()'s third argument is the constructor, which means that (if implemented correctly) subclasses don't need to defined a markupFactory

Change History (1)

comment:1 Changed 8 years ago by bill

Resolution: → fixed
Status: new → closed

In [26261]:

Remove unneeded markupFactory definitions. Since the constructor is passed as third argument to markupFactory(), can just define a single factory in the base classes, and leave it out of subclasses. Fixes #13703 !strict.

Note: See TracTickets for help on using tickets.