Andreas/markdownlint
1
0
Fork 0
mirror of https://github.com/DavidAnson/markdownlint.git synced 2025-12-19 15:30:13 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 12

Change helpers.getReferenceLinkImageData().shortcuts from Set to Map using the same schema as .references (fixes #685).

Browse source
This commit is contained in:
David Anson 2022-12-23 00:41:11 +00:00
parent f000a33481
commit 86a4a7de79
2 changed files with 24 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

21
demo/markdownlint-browser.js
View file

@ -792,7 +792,7 @@ module.exports.emphasisMarkersInContent = emphasisMarkersInContent;
function getReferenceLinkImageData(lineMetadata) { function getReferenceLinkImageData(lineMetadata) {
// Initialize return values // Initialize return values
const references = new Map(); const references = new Map();
const shortcuts = new Set(); const shortcuts = new Map();
const definitions = new Map(); const definitions = new Map();
const duplicateDefinitions = []; const duplicateDefinitions = [];
const definitionLineIndices = []; const definitionLineIndices = [];
@ -882,20 +882,23 @@ function getReferenceLinkImageData(lineMetadata) {
} }
const referenceIndex = referenceindex + const referenceIndex = referenceindex +
(topLevel ? -lineOffsets[lineIndex] : contentIndex); (topLevel ? -lineOffsets[lineIndex] : contentIndex);
const referenceDatum = [
lineIndex,
referenceIndex,
matchString.length,
matchText.length,
(matchLabel || "").length
];
if (shortcutLink) { if (shortcutLink) {
// Track separately due to ambiguity in "text [text] text" // Track separately due to ambiguity in "text [text] text"
shortcuts.add(label); const shortcutData = shortcuts.get(label) || [];
shortcutData.push(referenceDatum);
shortcuts.set(label, shortcutData);
} }
else { else {
// Track reference and location // Track reference and location
const referenceData = references.get(label) || []; const referenceData = references.get(label) || [];
referenceData.push([ referenceData.push(referenceDatum);
lineIndex,
referenceIndex,
matchString.length,
matchText.length,
matchLabel.length
]);
references.set(label, referenceData); references.set(label, referenceData);
} }
// Check for links embedded in brackets // Check for links embedded in brackets

21
helpers/helpers.js
View file

@ -824,7 +824,7 @@ module.exports.emphasisMarkersInContent = emphasisMarkersInContent;
function getReferenceLinkImageData(lineMetadata) { function getReferenceLinkImageData(lineMetadata) {
// Initialize return values // Initialize return values
const references = new Map(); const references = new Map();
const shortcuts = new Set(); const shortcuts = new Map();
const definitions = new Map(); const definitions = new Map();
const duplicateDefinitions = []; const duplicateDefinitions = [];
const definitionLineIndices = []; const definitionLineIndices = [];
@ -920,19 +920,22 @@ function getReferenceLinkImageData(lineMetadata) {
} }
const referenceIndex = referenceindex + const referenceIndex = referenceindex +
(topLevel ? -lineOffsets[lineIndex] : contentIndex); (topLevel ? -lineOffsets[lineIndex] : contentIndex);
const referenceDatum = [
lineIndex,
referenceIndex,
matchString.length,
matchText.length,
(matchLabel || "").length
];
if (shortcutLink) { if (shortcutLink) {
// Track separately due to ambiguity in "text [text] text" // Track separately due to ambiguity in "text [text] text"
shortcuts.add(label); const shortcutData = shortcuts.get(label) || [];
shortcutData.push(referenceDatum);
shortcuts.set(label, shortcutData);
} else { } else {
// Track reference and location // Track reference and location
const referenceData = references.get(label) || []; const referenceData = references.get(label) || [];
referenceData.push([ referenceData.push(referenceDatum);
lineIndex,
referenceIndex,
matchString.length,
matchText.length,
matchLabel.length
]);
references.set(label, referenceData); references.set(label, referenceData);
} }
// Check for links embedded in brackets // Check for links embedded in brackets