php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Doc Bug #71797 GmagickDraw::setfillcolor() accepts GmagickPixel, not only string
Submitted: 2016-03-11 11:36 UTC Modified: 2016-07-10 13:12 UTC
Votes:2
Avg. Score:5.0 ± 0.0
Reproduced:2 of 2 (100.0%)
Same Version:2 (100.0%)
Same OS:0 (0.0%)
From: benjamin dot morel at gmail dot com Assigned: cmb (profile)
Status: Closed Package: Documentation problem
PHP Version: Irrelevant OS: N/A
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem:
20 + 22 = ?
Subscribe to this entry?

 
 [2016-03-11 11:36 UTC] benjamin dot morel at gmail dot com 
Description:
------------
setfillcolor() accepts a GmagickPixel, in addition to a string:
http://php.net/manual/en/gmagickdraw.setfillcolor.php

Source:
https://github.com/vitoc/gmagick/blob/master/gmagickdraw_methods.c#L637-L656

Proof:

$d = new GmagickDraw();
$d->setfillcolor(new GmagickPixel('#ff8800'));
echo $d->getfillcolor()->getcolor(); // rgb(65535,34952,0)

Expected result:
----------------
public GmagickDraw GmagickDraw::setfillcolor ( GmagickPixel|string $color )

Actual result:
--------------
public GmagickDraw GmagickDraw::setfillcolor ( string $color )

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2016-03-11 11:37 UTC] benjamin dot morel at gmail dot com 
Note that the $color parameter description should therefore be changed to:

GmagickPixel or string indicating color to use for filling.
 [2016-07-10 13:12 UTC] cmb@php.net 
Automatic comment from SVN on behalf of cmb
Revision: http://svn.php.net/viewvc/?view=revision&revision=339634
Log: Fix #71797: GmagickDraw::setfillcolor() accepts GmagickPixel, not only string
 [2016-07-10 13:12 UTC] cmb@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: cmb
 [2016-07-10 13:12 UTC] cmb@php.net 
This bug has been fixed in the documentation's XML sources. Since the
online and downloadable versions of the documentation need some time
to get updated, we would like to ask you to be a bit patient.

Thank you for the report, and for helping us make our documentation better.
 [2020-02-07 06:07 UTC] phpdocbot@php.net 
Automatic comment on behalf of cmb
Revision: http://git.php.net/?p=doc/en.git;a=commit;h=bff94a23450ac7d0c346ca74c9433dd55dbbc006
Log: Fix #71797: GmagickDraw::setfillcolor() accepts GmagickPixel, not only string
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Sun May 19 22:01:32 2024 UTC