Fallback to rendered title for posts/pages if raw is empty

nbradbury · claude · nbradbury · commit 9a6cb6abc241 · 2026-02-04T16:59:22.000-05:00
Some posts/pages may have an empty raw title but a valid rendered
title. This ensures dropdown items always show a title when available.

Co-Authored-By: Claude Opus 4.5 &lt;noreply@anthropic.com&gt;
diff --git a/WordPress/src/main/java/org/wordpress/android/ui/navmenus/data/NavMenuRestClient.kt b/WordPress/src/main/java/org/wordpress/android/ui/navmenus/data/NavMenuRestClient.kt
@@ -352,7 +352,10 @@ class NavMenuRestClient @Inject constructor(
             is WpRequestResult.Success -> {
                 appLogWrapper.d(AppLog.T.API, "Fetched ${response.response.data.size} $typeName")
                 val items = response.response.data.map {
-                    LinkableItemOption(it.id, it.title?.raw ?: "")
+                    val title = it.title?.raw?.takeIf { raw -> raw.isNotBlank() }
+                        ?: it.title?.rendered
+                        ?: ""
+                    LinkableItemOption(it.id, title)
                 }
                 LinkableItemsResult.Success(items)
             }

Original file line number	Diff line number	Diff line change
`@@ -352,7 +352,10 @@ class NavMenuRestClient @Inject constructor(`
`352`	`352`	`is WpRequestResult.Success -> {`
`353`	`353`	`appLogWrapper.d(AppLog.T.API, "Fetched ${response.response.data.size} $typeName")`
`354`	`354`	`val items = response.response.data.map {`
`355`		`- LinkableItemOption(it.id, it.title?.raw ?: "")`
	`355`	`+ val title = it.title?.raw?.takeIf { raw -> raw.isNotBlank() }`
	`356`	`+ ?: it.title?.rendered`
	`357`	`+ ?: ""`
	`358`	`+ LinkableItemOption(it.id, title)`
`356`	`359`	`}`
`357`	`360`	`LinkableItemsResult.Success(items)`
`358`	`361`	`}`