From 7911c9cdb155e56754e6afca27140dc5002dadb6 Mon Sep 17 00:00:00 2001
From: Jon Surrell <sirreal@users.noreply.github.com>
Date: Fri, 14 Aug 2026 09:02:34 +0400
Subject: [PATCH] Attach the deprecation version to the deprecating call

`export_uses()` wrote `deprecation_version` to `$out['functions'][0]`, the
first recorded function use, instead of the `_deprecated_*()` call that
carries the version. That only looked correct when the deprecating call
happened to be the first call in the scope, as it is in a deprecated file;
whenever another call precedes it, the version was attached to that
unrelated call and the deprecating call got none.

Build the use record first and attach the version to it before appending.

Found during the review of #262 and extracted here as a standalone change.
---
 lib/runner.php                                |  6 ++-
 .../phpunit/tests/export/uses/deprecated.inc  |  6 +++
 .../phpunit/tests/export/uses/deprecated.php  | 44 +++++++++++++++++++
 3 files changed, 54 insertions(+), 2 deletions(-)
 create mode 100644 tests/phpunit/tests/export/uses/deprecated.inc
 create mode 100644 tests/phpunit/tests/export/uses/deprecated.php

diff --git a/lib/runner.php b/lib/runner.php
index 70e77d92..cedebcb1 100644
--- a/lib/runner.php
+++ b/lib/runner.php
@@ -1292,7 +1292,7 @@ function export_uses( array $uses ) {
 
 				default:
 				case 'functions':
-					$out[ $type ][] = array(
+					$used = array(
 						'name'     => $name,
 						'line'     => $element->getLineNumber(),
 						'end_line' => $element->getNode()->getAttribute( 'endLine' ),
@@ -1310,9 +1310,11 @@ function export_uses( array $uses ) {
 							$version = (string) $arguments[1]->value->value;
 						}
 
-						$out[ $type ][0]['deprecation_version'] = $version;
+						$used['deprecation_version'] = $version;
 					}
 
+					$out[ $type ][] = $used;
+
 					break;
 			}
 		}
diff --git a/tests/phpunit/tests/export/uses/deprecated.inc b/tests/phpunit/tests/export/uses/deprecated.inc
new file mode 100644
index 00000000..44e6a819
--- /dev/null
+++ b/tests/phpunit/tests/export/uses/deprecated.inc
@@ -0,0 +1,6 @@
+<?php
+
+function old_thing() {
+	do_something_first();
+	_deprecated_function( __FUNCTION__, '6.1.0', 'new_thing' );
+}
diff --git a/tests/phpunit/tests/export/uses/deprecated.php b/tests/phpunit/tests/export/uses/deprecated.php
new file mode 100644
index 00000000..38411125
--- /dev/null
+++ b/tests/phpunit/tests/export/uses/deprecated.php
@@ -0,0 +1,44 @@
+<?php
+
+/**
+ * A test case for exporting the version of a deprecation.
+ */
+
+namespace WP_Parser\Tests;
+
+/**
+ * Test that the deprecation version is exported for the deprecating call itself.
+ */
+class Export_Deprecated_Function_Use extends Export_UnitTestCase {
+
+	/**
+	 * Test that the version is exported for the _deprecated_function() call.
+	 */
+	public function test_deprecating_call_has_version() {
+
+		$this->assertFunctionUsesFunction(
+			'old_thing'
+			, array(
+				'name'                => '_deprecated_function',
+				'line'                => 5,
+				'end_line'            => 5,
+				'deprecation_version' => '6.1.0',
+			)
+		);
+	}
+
+	/**
+	 * Test that the version isn't exported for a call preceding the deprecation.
+	 */
+	public function test_preceding_call_has_no_version() {
+
+		$this->assertFunctionUsesFunction(
+			'old_thing'
+			, array(
+				'name'     => 'do_something_first',
+				'line'     => 4,
+				'end_line' => 4,
+			)
+		);
+	}
+}