Re: svn: /phpdoc/en/trunk/reference/mongo/ mongoclient/selectcollection.xml mongoclient/selectdb.xml mongodb/selectcollection.xml

On Mon, Jan 28, 2013 at 11:47 AM, Jeremy Mikola <jmikola@php.net> wrote: > jmikola Mon, 28 Jan 2013 19:47:52 +0000 > > Revision: > http://svn.php.net/viewvc?view=revision&revision=329315 > > Log: > Make Mongo db/collection select method docs consistent > > A vanilla Exception is thrown instead of InvalidArgumentException. > > Changed paths: > U phpdoc/en/trunk/reference/mongo/mongoclient/selectcollection.xml > U phpdoc/en/trunk/reference/mongo/mongoclient/selectdb.xml > U phpdoc/en/trunk/reference/mongo/mongodb/selectcollection.xml > > Modified: phpdoc/en/trunk/reference/mongo/mongoclient/selectcollection.xml > =================================================================== > --- phpdoc/en/trunk/reference/mongo/mongoclient/selectcollection.xml 2013-01-28 19:40:25 UTC > (rev 329314) > +++ phpdoc/en/trunk/reference/mongo/mongoclient/selectcollection.xml 2013-01-28 19:47:52 UTC > (rev 329315) > @@ -54,7 +54,7 @@ > <refsect1 role="errors"> > &reftitle.errors; > <para> > - Throws <classname>InvalidArgumentException</classname> if the database or > collection name is invalid. > + Throws <classname>Exception</classname> if the database or collection name is > invalid. > </para> > </refsect1> > > > Modified: phpdoc/en/trunk/reference/mongo/mongoclient/selectdb.xml > =================================================================== > --- phpdoc/en/trunk/reference/mongo/mongoclient/selectdb.xml 2013-01-28 19:40:25 UTC (rev > 329314) > +++ phpdoc/en/trunk/reference/mongo/mongoclient/selectdb.xml 2013-01-28 19:47:52 UTC (rev > 329315) > @@ -36,14 +36,14 @@ > <refsect1 role="returnvalues"> > &reftitle.returnvalues; > <para> > - Returns a new db object. > + Returns a new database object. "Returns new instance of <classname>MongoDB</classname>" would be much better then a "database object" imo. > <para> > - Returns the collection. > + Returns a new collection object. Please mention the classname -Hannes