Changeset 59503

Timestamp:

12/10/2024 11:39:06 AM (3 months ago)

Author:

gziolo

Message:

HTML API: Remove nullable from get_breadcrumbs return type

Follow-up [58713]
Props jonsurrell, westonruter, gziolo.
Fixes #62674.

File:

• trunk/src/wp-includes/html-api/class-wp-html-processor.php (modified) (2 diffs)

trunk/src/wp-includes/html-api/class-wp-html-processor.php

@@ -1144 +1144 @@
      * They always include the entire path from the root HTML node to the matched element.
      *
-     * @todo It could be more efficient to expose a generator-based version of this function
-     *       to avoid creating the array copy on tag iteration. If this is done, it would likely
-     *       be more useful to walk up the stack when yielding instead of starting at the top.
-     *
-     * Example
+     * Example:
      *
      *     $processor = WP_HTML_Processor::create_fragment( '<p><strong><em><img></em></strong></p>' );
@@ -1156 +1152 @@
      * @since 6.4.0
      *
-     * @return string[]|null Array of tag names representing path to matched node, if matched, otherwise NULL.
-     */
-    public function get_breadcrumbs(): ?array {
+     * @return string[] Array of tag names representing path to matched node.
+     */
+    public function get_breadcrumbs(): array {
         return $this->breadcrumbs;
     }