highperfocused/amethyst
1
0
Fork 0
mirror of https://github.com/vitorpamplona/amethyst.git synced 2025-03-28 18:51:45 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fixes #1084. Opengraph parser fails when value-less properties are present.

Browse Source
This commit is contained in:
Vitor Pamplona 2024-09-16 11:23:41 -04:00
parent 428ed464af
commit 28da06ef01
2 changed files with 33 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
commons/src/androidTest/java/com/vitorpamplona/amethyst/commons/preview/MetaTagsParserTest.kt
View File

@ -78,4 +78,29 @@ class MetaTagsParserTest {
} }
} }
} }
@Test
fun testParseWithAdditionalTags() {
val input =
"""<html>
| <head>
| <meta data-preact-helmet property="og:title" content="Uma conversa com Antonio Vargas">
| </head>
|</html>
""".trimMargin()
val exp =
listOf(
listOf("property" to "og:title", "content" to "Uma conversa com Antonio Vargas"),
)
val metaTags = MetaTagsParser.parse(input).toList()
println(metaTags)
assertEquals(exp.size, metaTags.size)
metaTags.zip(exp).forEach { (meta, expAttrs) ->
expAttrs.forEach { (name, expValue) ->
assertEquals(expValue, meta.attr(name))
}
}
}
} }

9
commons/src/main/java/com/vitorpamplona/amethyst/commons/preview/MetaTagsParser.kt
View File

@ -248,7 +248,14 @@ object MetaTagsParser {
} }
c.isWhitespace() -> {} c.isWhitespace() -> {}
else -> return null
else -> {
// if it is expecting = but gets another name, starts another property
runCatching { attrs.add(Pair(input.slice(nameBegin..<nameEnd), "")) }
nameBegin = i
state = State.NAME
}
} }
} }