Adjust currentAttr.name (fix inikulin#318)

calvinjuarez · Jun 24, 2020 · 62f7a59 · 62f7a59
1 parent 2e33574
commit 62f7a59
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/packages/parse5/lib/common/foreign-content.js b/packages/parse5/lib/common/foreign-content.js
@@ -17,7 +17,7 @@ const MIME_TYPES = {
 //Attributes
 const DEFINITION_URL_ATTR = 'definitionurl';
 const ADJUSTED_DEFINITION_URL_ATTR = 'definitionURL';
-const SVG_ATTRS_ADJUSTMENT_MAP = {
+const SVG_ATTRS_ADJUSTMENT_MAP = (exports.SVG_ATTRS_ADJUSTMENT_MAP = {
     attributename: 'attributeName',
     attributetype: 'attributeType',
     basefrequency: 'baseFrequency',
@@ -76,7 +76,7 @@ const SVG_ATTRS_ADJUSTMENT_MAP = {
     xchannelselector: 'xChannelSelector',
     ychannelselector: 'yChannelSelector',
     zoomandpan: 'zoomAndPan'
-};
+});
 
 const XML_ATTRS_ADJUSTMENT_MAP = {
     'xlink:actuate': { prefix: 'xlink', name: 'actuate', namespace: NS.XLINK },

diff --git a/packages/parse5/lib/extensions/location-info/tokenizer-mixin.js b/packages/parse5/lib/extensions/location-info/tokenizer-mixin.js
@@ -3,6 +3,7 @@
 const Mixin = require('../../utils/mixin');
 const Tokenizer = require('../../tokenizer');
 const PositionTrackingPreprocessorMixin = require('../position-tracking/preprocessor-mixin');
+const SVG_ATTRS_ADJUSTMENT_MAP = require('../../common/foreign-content').SVG_ATTRS_ADJUSTMENT_MAP;
 
 class LocationInfoTokenizerMixin extends Mixin {
     constructor(tokenizer) {
@@ -37,7 +38,9 @@ class LocationInfoTokenizerMixin extends Mixin {
             currentToken.location.attrs = Object.create(null);
         }
 
-        currentToken.location.attrs[currentAttr.name] = this.currentAttrLocation;
+        const name = SVG_ATTRS_ADJUSTMENT_MAP[currentAttr.name] || currentAttr.name;
+
+        currentToken.location.attrs[name] = this.currentAttrLocation;
     }
 
     _getOverriddenMethods(mxn, orig) {