[PHP-DOC] Contributions are ready for review

2011-07-11 Thread phpdoc
Hello PHP EN Documentation team,

There are contributions within the online editor queue for this language.
Please review, then commit or delete these patches.

Patches for review : 
---
* (update) On 2011-05-26 09:31:28 by uw : 
en/reference/mysqlnd_ms/concepts.xml
* (update) On 2011-06-27 21:34:04 by guilhermeblanco : 
en/language/predefined/errorexception/construct.xml
* (update) On 2011-07-01 07:28:07 by anonymous #4805 : 
en/reference/hash/functions/hash.xml
* (update) On 2011-07-08 15:17:18 by anonymous #5637 : 
en/reference/spl/appenditerator/construct.xml
* (update) On 2011-07-08 15:20:04 by anonymous #5637 : 
en/reference/spl/appenditerator.xml
* (update) On 2011-07-08 15:21:20 by anonymous #5637 : 
en/reference/spl/multipleiterator.xml
* (update) On 2011-07-08 15:30:07 by anonymous #5637 : 
en/reference/spl/multipleiterator/construct.xml
* (update) On 2011-07-11 01:20:04 by anonymous #5808 : 
en/language/oop5/patterns.xml



-- 
https://edit.php.net/
This email is send automatically by the Php Docbook Online Editor.


RE: [PHP-DOC] SQLSRV and PDO_SQLSRV Documentation Patch

2011-07-11 Thread Jonathan Guerin
Ok, we believe we have fixed the issues described, but are now running into a 
commit problem:

Commit failed (details follow):
Commit blocked by pre-commit hook (exit code 1) with output:
***
Access denied: Insufficient karma for jguerin to 
phpdoc/doc-base/trunk/entities/global.ent.
Contact phpdoc@lists.php.net for access.

How can we push updates to the Global Entities file, as we need to add some 
links in there?

Thanks!

Jonathan


-Original Message-
From: Philip Olson [mailto:phi...@roshambo.org] 
Sent: Tuesday, July 05, 2011 12:27 PM
To: Jonathan Guerin
Cc: phpdoc@lists.php.net
Subject: Re: [PHP-DOC] SQLSRV and PDO_SQLSRV Documentation Patch


On Jul 5, 2011, at 10:43 AM, Jonathan Guerin wrote:

 Update: Sounds like I sent this to the wrong mailing list. Resending to the 
 appropriate one!
 
 Hi everyone,
 
 Here is our patch for the initial addition of the PHP Manual for the 
 SQLSRV and PDO_SQLSRV extensions.

Overall it looks good and nicely detailed. However:

Please cleanup the whitespace, which should include using spaces instead of 
tabs. And the wide line widths makes it difficult to read, so should be closer 
to 80 than 300 in length.

And although we have a questionable and non-existent style guide (sorry about 
that, our bad):

- A few entities worth using (common for return value descriptions):

  - null; instead of NULL
  - true;  ''
  - false; ''

- Random:

  - refpurpose should not contain ending periods
  - Adding output to examples is often useful, and is done using screen 
within example
  -   where the example.outputs.similar; entity is useful in cases like this
  - Example within sqlsrv-num-fields.xml contains a parse error
  - You may want to link (and edit) the new install/cloud/azure.xml docs
  - Third-person active voice is preferred (don't worry about this now, too 
much work)

And feel free to ask further questions here or casually within #php.doc on 
EFNet IRC. The above are common problems for new docs, and I thought all of it 
was written down (???) but now I cannot find. Getting old I guess. But, thanks 
to your efforts, this will certainly be written down so people in the future 
will know from the beginning and perform less work. Again, sorry about that

 We would love to gather feedback, as well as understand the next 
 steps. Do we simply commit this patch once it is approved?

Yes, you'll commit it then random people (including you, at any time, no 
approval needed) will update it in the future.

Regards,
Philip




Re: [PHP-DOC] SQLSRV and PDO_SQLSRV Documentation Patch

2011-07-11 Thread Kalle Sommer Nielsen
Hi Jonathan

2011/7/12 Jonathan Guerin jgue...@microsoft.com:
 Ok, we believe we have fixed the issues described, but are now running into a 
 commit problem:

 Commit failed (details follow):
 Commit blocked by pre-commit hook (exit code 1) with output:
 ***
 Access denied: Insufficient karma for jguerin to 
 phpdoc/doc-base/trunk/entities/global.ent.
 Contact phpdoc@lists.php.net for access.

 How can we push updates to the Global Entities file, as we need to add some 
 links in there?

Philip will have to grant you karma, as currently your account does
not have access to any part of the svn tree.

-- 
regards,

Kalle Sommer Nielsen
ka...@php.net


Re: [PHP-DOC] SQLSRV and PDO_SQLSRV Documentation Patch

2011-07-11 Thread Philip Olson

On Jul 11, 2011, at 6:18 PM, Kalle Sommer Nielsen wrote:

 Hi Jonathan
 
 2011/7/12 Jonathan Guerin jgue...@microsoft.com:
 Ok, we believe we have fixed the issues described, but are now running into 
 a commit problem:
 
 Commit failed (details follow):
 Commit blocked by pre-commit hook (exit code 1) with output:
 ***
 Access denied: Insufficient karma for jguerin to 
 phpdoc/doc-base/trunk/entities/global.ent.
 Contact phpdoc@lists.php.net for access.
 
 How can we push updates to the Global Entities file, as we need to add some 
 links in there?
 
 Philip will have to grant you karma, as currently your account does
 not have access to any part of the svn tree.

Greetings Jonathan,

Your account lacked karma (and still does for pecl) but you now have phpdoc 
karma, welcome to the PHP documentation team! :)

Regards,
Philip