Doc #73766 [Ver->Csd]: Reflection return method name as lowercases

From: 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

« previous php.doc.bugs (#18850) next »