php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #81334 Rename imagettfbbox/text string/text parameter
Submitted: 2021-08-06 08:46 UTC Modified: 2021-08-06 08:55 UTC
Votes:1
Avg. Score:3.0 ± 0.0
Reproduced:0 of 0 (0.0%)
From: admin at kapsonfire dot de Assigned:
Status: Open Package: GD related
PHP Version: 8.0.9 OS: -
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: admin at kapsonfire dot de
New email:
PHP Version: OS:

 

 [2021-08-06 08:46 UTC] admin at kapsonfire dot de
Description:
------------
Since PHP supports spreading named parameters, i'd like to see 
imagettfbbox and imagettftext using the same paramater names.

imagettfbbox uses $string for the text
imagettfbbox uses $text   for the text



Test script:
---------------
<?php
//header("Content-Type: image/png");
$im = @imagecreate(200, 200)
or die("Cannot Initialize new GD image stream");
$background_color = imagecolorallocate($im, 50, 50, 200);
$text_color = imagecolorallocate($im, 255,255,255);

$fontConfig = [
    'size' => 75,
    'angle' => 0,
    'font_filename' => "Roboto-Regular.ttf",
    'string'    => 'PA',
];

$bb = imagettfbbox(...$fontConfig);
$w = $bb[2] - $bb[0];
$h = $bb[3] - $bb[1];
imagettftext(...([
        'image' => $im,
        'color' => $text_color,
        'x' => (100-$w/2),
        'y' => (100-$h/2),

             ]+$fontConfig));

imagepng($im);
imagedestroy($im);

Expected result:
----------------
No Error

Actual result:
--------------
Fatal error: Uncaught Error: Unknown named parameter $string in image.php:24

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2021-08-06 08:47 UTC] admin at kapsonfire dot de
sorry, misswrote
imagettfbbox uses $string for the text
imagettftext uses $text   for the text
 [2021-08-06 08:55 UTC] cmb@php.net
-Summary: - +Summary: Rename imagettfbbox/text string/text parameter
 [2021-08-06 08:55 UTC] cmb@php.net
Oh, that's indeed unfortunate.  I'm not sure whether we still
can change that in a minor version for BC reasons.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Thu Nov 21 19:01:29 2024 UTC