Hi Sherif, > Which methods are missing and from what classes?
The methods that I've found that are missing are: ImagickPixelIterator ImagickPixelIterator::getPixelIterator(Imagick source) Returns a new pixel iterator, static method ImagickPixelIterator ImagickPixelIterator::getPixelRegionIterator(Imagick source, int x, int y, int columns, int rows) Returns a new pixel region iterator, static method bool ImagickPixel::setColorValueQuantum(int color, float value) Sets the quantum color of the ImagickPixel. int ImagickPixel::getColorValueQuantum(int color) Gets the quantum color of the ImagickPixel string Imagick::__toString() Returns the current image as string The signatures and descriptions are from the Imagick source. cheers Dan On Sun, Oct 20, 2013 at 12:08 PM, Sherif Ramadan <theanomaly...@gmail.com> wrote: > > > > On Sun, Oct 20, 2013 at 7:31 AM, Dan Ackroyd <dan...@basereality.com> wrote: >> >> Hi, >> >> I've started out tackling the vast number of Imagick functions that have >> either no documentation at all, or lack examples. A couple of questions: >> >> 1) Should I submit a separate patch per change? Standard best practice is >> usually to do that, but seeing as it's SVN and may take time to merge each >> patch, would it make the people who are merging changes lives easier to have >> fewer patches? > > > Yes. Submit a separate patch for each change. > >> >> >> 2) Are there any guidelines on adding images to the doc? Some of the >> Imagick functions would seem to benefit from a pictorial example as they're >> much easier to explain visually than with text. If they're allowed, what >> size can be used and is there a recommended colour scheme? > > > Not really, but employ common sense where possible. i.e. we don't need > 1920x1024 pixel images as examples. > >> >> >> 3) Some methods are missing from the Imagick classes, and some are not >> marked as static when they should be. Is it possible for these to be edited >> online, or does that need to be generated from the source code? For example >> the page http://www.php.net/manual/en/class.imagickpixel.php seems to have >> the function list pulled in from somewhere else: >> >> <xi:include xpointer="xmlns(db=http://docbook.org/ns/docbook) >> xpointer(id('class.imagickpixel')/db:refentry/db:refsect1[@role='description']/descendant::db:methodsynopsis[1])" >> /> >> </classsynopsis> >> </section> >> </partintro> >> &reference.imagick.entities.imagickpixel; >> > > Which methods are missing and from what classes? To mark a class as > static/public/etc.. you simply apply a modifier tag to its methodsynopsis. > For example, in > /en/reference/imagick/imagickpixel/<method-you-want-to-modify>.xml there > will be a <methodsynopsis> entity towards the beginning of the file. For > example in getcolor.xml there is the following: > > <methodsynopsis> > <type>array</type><methodname>ImagickPixel::getColor</methodname> > ... > > Just add <modifier>static</modifier> if the method should be static, for > example. > >> >> 4) As much fun as it is being 'anonymous #33522' please could I have a >> login for the online editor? >> > > You can login to the online doc editor with a facebook/google login, I > believe, if you don't want to log in anonymously. It doesn't make much of a > difference. The OE is just a way for everyone to contribute to the docs with > minimal bar-to-entry. If you'd like to contribute to the documentation > regularly you'll want to familiarize yourself with our tool-chain. See > https://wiki.php.net/doc/howto and > https://wiki.php.net/doc/howto/gettingstarted > >> >> cheers >> Dan > >