Remove internal annotation

VincentLanglet · VincentLanglet · commit 7f2b7a3b02de · 2021-03-25T14:36:04.000+01:00
diff --git a/UPGRADE-5.3.md b/UPGRADE-5.3.md
@@ -60,6 +60,11 @@ Messenger
  * Deprecated the `prefetch_count` parameter in the AMQP bridge, it has no effect and will be removed in Symfony 6.0
  * Deprecated the use of TLS option for Redis Bridge, use `rediss://127.0.0.1` instead of `redis://127.0.0.1?tls=1`
 
+Mime
+----
+
+ * Remove internal annotation to the method `getHeaderBody`, `setHeaderBody`, `getHeaderParameter` and `setHeaderParameter`
+
 Notifier
 --------
 
diff --git a/src/Symfony/Component/Mime/Header/Headers.php b/src/Symfony/Component/Mime/Header/Headers.php
@@ -254,17 +254,11 @@ public function toArray(): array
         return $arr;
     }
 
-    /**
-     * @internal
-     */
     public function getHeaderBody($name)
     {
         return $this->has($name) ? $this->get($name)->getBody() : null;
     }
 
-    /**
-     * @internal
-     */
     public function setHeaderBody(string $type, string $name, $body): void
     {
         if ($this->has($name)) {
@@ -274,9 +268,6 @@ public function setHeaderBody(string $type, string $name, $body): void
         }
     }
 
-    /**
-     * @internal
-     */
     public function getHeaderParameter(string $name, string $parameter): ?string
     {
         if (!$this->has($name)) {
@@ -291,9 +282,6 @@ public function getHeaderParameter(string $name, string $parameter): ?string
         return $header->getParameter($parameter);
     }
 
-    /**
-     * @internal
-     */
     public function setHeaderParameter(string $name, string $parameter, $value): void
     {
         if (!$this->has($name)) {
diff --git a/src/Symfony/Component/Mime/Tests/Header/HeadersTest.php b/src/Symfony/Component/Mime/Tests/Header/HeadersTest.php
@@ -279,4 +279,39 @@ public function testToArray()
             "Foo: =?utf-8?Q?aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa?=\r\n =?utf-8?Q?aaaa?=",
         ], $headers->toArray());
     }
+
+    public function testHeaderBody()
+    {
+        $headers = new Headers();
+        $this->assertNull($headers->getHeaderBody('Content-Type'));
+        $headers->setHeaderBody('Text', 'Content-Type', 'type');
+        $this->assertSame('type', $headers->getHeaderBody('Content-Type'));
+    }
+
+    public function testHeaderParameter()
+    {
+        $headers = new Headers();
+        $this->assertNull($headers->getHeaderParameter('Content-Disposition', 'name'));
+
+        $headers->addParameterizedHeader('Content-Disposition', 'name');
+        $headers->setHeaderParameter('Content-Disposition', 'name', 'foo');
+        $this->assertSame('foo', $headers->getHeaderParameter('Content-Disposition', 'name'));
+    }
+
+    public function testHeaderParameterNotDefined()
+    {
+        $headers = new Headers();
+
+        $this->expectException(\LogicException::class);
+        $headers->setHeaderParameter('Content-Disposition', 'name', 'foo');
+    }
+
+    public function testSetHeaderParameterNotParameterized()
+    {
+        $headers = new Headers();
+        $headers->addTextHeader('Content-Disposition', 'name');
+
+        $this->expectException(\LogicException::class);
+        $headers->setHeaderParameter('Content-Disposition', 'name', 'foo');
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -254,17 +254,11 @@ public function toArray(): array`
`254`	`254`	`return $arr;`
`255`	`255`	`}`
`256`	`256`
`257`		`- /**`
`258`		`- * @internal`
`259`		`- */`
`260`	`257`	`public function getHeaderBody($name)`
`261`	`258`	`{`
`262`	`259`	`return $this->has($name) ? $this->get($name)->getBody() : null;`
`263`	`260`	`}`
`264`	`261`
`265`		`- /**`
`266`		`- * @internal`
`267`		`- */`
`268`	`262`	`public function setHeaderBody(string $type, string $name, $body): void`
`269`	`263`	`{`
`270`	`264`	`if ($this->has($name)) {`
`@@ -274,9 +268,6 @@ public function setHeaderBody(string $type, string $name, $body): void`
`274`	`268`	`}`
`275`	`269`	`}`
`276`	`270`
`277`		`- /**`
`278`		`- * @internal`
`279`		`- */`
`280`	`271`	`public function getHeaderParameter(string $name, string $parameter): ?string`
`281`	`272`	`{`
`282`	`273`	`if (!$this->has($name)) {`
`@@ -291,9 +282,6 @@ public function getHeaderParameter(string $name, string $parameter): ?string`
`291`	`282`	`return $header->getParameter($parameter);`
`292`	`283`	`}`
`293`	`284`
`294`		`- /**`
`295`		`- * @internal`
`296`		`- */`
`297`	`285`	`public function setHeaderParameter(string $name, string $parameter, $value): void`
`298`	`286`	`{`
`299`	`287`	`if (!$this->has($name)) {`