Project

Profile

Help

Bug #4126

closed

MetaTagAdjuster replicates namespace bindings

Added by Michael Kay about 5 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
2019-02-01
Due date:
% Done:

100%

Estimated time:
Legacy ID:
Applies to branch:
9.9, trunk
Fix Committed on Branch:
9.9, trunk
Fixed in Maintenance Release:
Platforms:

Description

The MetaTagAdjuster (used on the HTML serialization pipeline) contains the logic

for (NamespaceBinding ns : namespaceBindings) {
            if (inMetaTag) {
                namespaces.add(ns);
            } else {
                nextReceiver.namespace(namespaceBindings, properties);
            }
        }

which is clearly wrong when namespaceBindings contains more than one namespace; in line 5 namespaceBindings should be ns. We're only getting away with this because most paths send namespaces down the pipeline one at a time. Failure observed when we change that on the development branch.

Please register to edit this issue

Also available in: Atom PDF