Opened 9 years ago

Closed 8 years ago

#3619 task closed fixed (fixed)

Fix the "import minidom as microdom" stuff in lore

Reported by: Jean-Paul Calderone Owned by:
Priority: normal Milestone:
Component: lore Keywords: easy
Cc: Branch: branches/lore-microdom-imports-3619
branch-diff, diff-cov, branch-cov, buildbot
Author: exarkun

Description

#3560 switched lore from microdom to minidom. To avoid lots and lots of extra diff, it left lore using the name "microdom" in most places, but made that actually refer to minidom.

Change all of the names used in Lore to minidom to make the code less confusing.

Change History (7)

comment:1 Changed 9 years ago by Jean-Paul Calderone

Keywords: easy added

comment:2 Changed 9 years ago by Jean-Paul Calderone

Author: exarkun
Branch: branches/lore-microdom-imports-3619

(In [26745]) Branching to 'lore-microdom-imports-3619'

comment:3 Changed 9 years ago by Jean-Paul Calderone

Keywords: review added
Owner: spiv deleted

I actually changed all the names to dom rather than minidom. Maybe this will afford us some more flexibility in changing to yet another DOM implementation in the future! Or maybe it doesn't matter.

Anyway I changed all the names and the tests all still pass.

comment:4 Changed 8 years ago by Glyph

Keywords: review removed
Owner: set to Jean-Paul Calderone

dom is still used as a parameter name a bunch of times in twisted.lore.lint, once in twisted.lore.slides, and as a variable in both of those modules as well as twisted.lore.latex.

But if the tests still pass once you've renamed those, go ahead and land it. Alternately, rename the import to something else, like 'minidom' as originally planned.

comment:5 Changed 8 years ago by Jean-Paul Calderone

(In [26928]) Rename this variable from dom to doc to avoid name collision with global

refs #3619

comment:6 Changed 8 years ago by Jean-Paul Calderone

Resolution: fixed
Status: newclosed

(In [26929]) Merge lore-microdom-imports-3619

Author: exarkun Reviewer: glyph Fixes: #3619

Change the minidom as microdom imports in Twisted Lore to be minidom as dom instead, and adjust all affected code.

comment:7 Changed 7 years ago by <automation>

Owner: Jean-Paul Calderone deleted
Note: See TracTickets for help on using tickets.