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 #75898 suggestion for imagecropauto() function
Submitted: 2018-01-31 22:25 UTC Modified: 2018-02-05 20:40 UTC
From: chris dot bjerken at columbian dot com Assigned: salathe (profile)
Status: Closed Package: GD related
PHP Version: 7.2.1 OS:
Private report: No CVE-ID: None
View Developer Edit
 [2018-01-31 22:25 UTC] chris dot bjerken at columbian dot com 
Description:
------------
---
From manual page: http://www.php.net/function.imagecropauto
---

I had a scenario today where I had to create an entire script as an alternative to imagick's trimimage method that in turn already existed natively in PHP with function imagecropauto().  You might imagine my frustration of this wasted time.

We can help anyone else avoid the same mistake in the future by building a dummy page named something like "imagetrim" to direct people to the imagecropauto page, much like http://php.net/manual/en/function.delete.php is a dummy page to direct people to the doc page for unlink.

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2018-02-01 22:28 UTC] cmb@php.net
-Package: Documentation problem +Package: GD related
 [2018-02-01 22:28 UTC] cmb@php.net 
I'd rather like to be redirected to the actually desired page, or
to see it listed in the search results, but as workaround a dummy
page might be in order – on the other hand, where would we stop?
Dummy pages for `chopimage` and `cropimage` might only be the next
step …
 [2018-02-01 22:38 UTC] chris dot bjerken at columbian dot com 
Good point. Let me ask you this - with that in mind, why was http://php.net/manual/en/function.delete.php or any other dummy pages made in the first place?
 [2018-02-02 03:21 UTC] philip@php.net 
How about instead a See Also link to this page:

  http://php.net/manual/en/function.imagecropauto.php

Is added to this page:

  http://php.net/manual/en/imagick.trimimage.php

Does this seem reasonable?

As for delete(), it was created long ago (in the 90's) by, well, probably gnomes.
 [2018-02-02 16:19 UTC] chris dot bjerken at columbian dot com 
I think that sounds like a good suggestion.
 [2018-02-05 20:38 UTC] salathe@php.net 
Automatic comment from SVN on behalf of salathe
Revision: http://svn.php.net/viewvc/?view=revision&revision=344186
Log: link to imagecropauto from Imagick::trimImage() (doc bug #75898)
 [2018-02-05 20:39 UTC] salathe@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: salathe
 [2018-02-05 20:39 UTC] salathe@php.net 
The fix for this bug has been committed.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.

 For Windows:

http://windows.php.net/snapshots/
 
Thank you for the report, and for helping us make PHP better.
 [2018-02-05 20:40 UTC] salathe@php.net 
Boo, that was the wrong canned message, which should have said:

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.

Thanks again!
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Sun Dec 22 06:01:30 2024 UTC