Doc #73766 [Ver->Csd]: Reflection return method name as lowercases
| From: | danack@php.net | Date: | Fri, 11 Jun 2021 16:19:43 +0000 |
| Subject: | Doc #73766 [Ver->Csd]: Reflection return method name as lowercases | ||
| References: | 1 | Groups: | php.doc.bugs |
| Request: | Send a blank email to doc-bugs+get-18850@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=73766&edit=1
ID: 73766
Updated by: danack@php.net
Reported by: mozek007 at seznam dot cz
Summary: Reflection return method name as lowercases
-Status: Verified
+Status: Closed
Type: Documentation Problem
Package: imagick
Operating System: unix
PHP Version: 7.0.14
-Assigned To:
+Assigned To: danack
Block user comment: N
Private report: N
New Comment:
This is fixed in the next release of Imagick.
Previous Comments:
------------------------------------------------------------------------
[2016-12-20 14:20:19] danack@php.net
"but united coding style not."
The decision about case for method names was made a long time ago, by the previous maintainers of
Imagick. Imagick is a separate project from PHP, and so isn't required to have a
'united' code style with anything else.
Although I have been thinking about converting the case to be camelCase, it is really not that high
on my list of priorities. It would be higher if users of Imagick contributed more to any of:
* the documentation.
* code contributions to support new features of ImageMagick in Imagick.
* cash money (I need to acquire a Windows box to be able to investigate Windows issues, as well as
start producing 'official' binaries).
* anyone else helping to investigate issues either here or at https://github.com/mkoppanen/imagick/issues
Just to note, the earliest that changing the internal casing of the methods would possibly happen is
for the next major version of Imagick, for which there are other issues https://github.com/mkoppanen/imagick/issues/142
------------------------------------------------------------------------
[2016-12-17 00:52:36] cmb@php.net
Okay, reopening as doc bug.
------------------------------------------------------------------------
[2016-12-17 00:39:47] mozek007 at seznam dot cz
cmb: Thanks for explaining. I think in this case is wrong phpdoc
public function readImageBlob ($image, $filename = null) {}
and web documentation: http://php.net/manual/en/imagick.readimageblob.php.
method names are case-insensitive in PHP: Yes, but united coding style not.
------------------------------------------------------------------------
[2016-12-16 23:40:00] cmb@php.net
Thank you for taking the time to write to us, but this is not
a bug. Please double-check the documentation available at
http://www.php.net/manual/ and the instructions on how to
report
a bug at http://bugs.php.net/how-to-report.php
Note that readimageblob is the canonical name[1] and that method
names are case-insensitive in PHP, anyway.
[1] <https://github.com/mkoppanen/imagick/blob/3.4.2/imagick_class.c#L3766>
------------------------------------------------------------------------
[2016-12-16 23:36:14] mozek007 at seznam dot cz
same problem here: https://github.com/padraic/mockery/issues/162
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=73766
--
Edit this bug report at https://bugs.php.net/bug.php?id=73766&edit=1