frontmatter: an unterminated quoted scalar is accepted silently and keeps its opening quote
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 82/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- javascript, node.js
Rechercherichtung
Lies zuerst site/lib/frontmatter.mjs und lib.test.mjs; untersuche, wie parseScalar zitierte Werte verarbeitet und wie Frontmatter-Fehler mit Zeilennummern gemeldet werden. Füge Fälle hinzu, die nicht abgeschlossene zitierte Skalare und Listenelemente zurückweisen, führe dann npm run validate aus und bestätige, dass das Fixture die Zeile des nicht geschlossenen Anführungszeichens meldet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Why it matters
name: "Foo (a missing closing quote, an easy slip when a name contains an apostrophe) parses as the string "Foo and flows into pages, data.json, the CSV and the feed with a stray quote. checkTitleMatchesName happens to catch it for name, but publisher, attribution_string, grid_dimensions, ue_versions and list items have no backstop.
Evidence (at 69387df)
site/lib/frontmatter.mjs:69-82 parseScalar only unquotes when the value both starts and ends with the same quote; otherwise it returns the raw text. Verified: parsing a frontmatter block containing name: "Foo yields meta.name === '"Foo' with errors empty.
Suggested approach
In parseScalar (or in parseFrontmatter where line numbers are known), when a trimmed value starts with " or ' and does not end with the matching quote, push line N: "key" opens a quote that is never closed. Same for list items.
Done when
lib.test.mjs has a rejecting case; npm run validate fails on a fixture with an unterminated quote and names the line.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 58
- Forks
- 15
- Ø Merge
- 16 Std. 53 Min.
- Gemergte PRs (30 T.)
- 14
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus TMHSDigital/Free-Game-Dev-Assets
-
community marketing priority: low
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
TMHSDigital/Free-Game-Dev-Assets#151 ·
Maintainer antworten meist innerhalb von 1 Tag
-
community marketing priority: medium
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
TMHSDigital/Free-Game-Dev-Assets#150 ·
Maintainer antworten meist innerhalb von 1 Tag
-
community enhancement priority: medium
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
TMHSDigital/Free-Game-Dev-Assets#149 ·
Maintainer antworten meist innerhalb von 1 Tag
-
community documentation priority: medium
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 88/100
TMHSDigital/Free-Game-Dev-Assets#148 ·
Maintainer antworten meist innerhalb von 1 Tag
-
site: the Unreal guide is missing from the homepage Guides list, and the hero calls tools "assets"Offenbug marketing priority: medium
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 85/100
TMHSDigital/Free-Game-Dev-Assets#147 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in TMHSDigital/Free-Game-Dev-Assets
Ähnliche Issues
-
bug Frontend P2-medium UI UX
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 76/100
Deepak3699/Ai_Mentor#427 ·
Maintainer antworten meist innerhalb von 1 Tag
-
needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
11ty/buildawesome#4407 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 77/100
keyxmakerx/Chronicle#1711 ·
Maintainer antworten meist innerhalb von 1 Tag
-
frontend
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 69/100
uoftblueprint/canada-basketball#79 ·
Maintainer antworten meist innerhalb von 1 Tag
-
New App Request
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 76/100
ThatGuySam/doesitarm#1126 · 1 Kommentar ·