Re: [drupal-devel] Forms API and readability
Konstantin Käfer wrote:
Angie, you said that you updated CODING_STANDARDS.html in contrib root. But obviously the file hasn't changed since september 11. Would you mind checking it in a second time?
Sorry, I should have been more clear... I was the one who updated it back in September to cover the // $Id$ commenting standard (as opposed to /* $Id$ */) but it has yet to appear on the actual coding standards page in the handbook. So I was curious to know what the actual procedure was in updating that page on the site. I've updated the CODING_STANDARDS.html page in CVS again now to include: * Added note about leaving off the ?> at the ends of code files (added full explanation as well to http://drupal.org/node/545) * Added section on how to format arrays, including consensus of this discussion * Fixed a small typo Could Dries or someone who has access please update http://drupal.org/node/318 to reflect these changes? -Angie
Could Dries or someone who has access please update http://drupal.org/node/318 to reflect these changes?
Thanks to marineam this is fixed now. There is an old checkout which file is php included here. I asked him to replace the file. I hope that soon cvs.drupal.org will move to Oregon and this file will be 'live' again. That's why I have not replaced the PHP command by the actual HTML. Regards NK
On 06 Nov 2005, at 03:03, Karoly Negyesi wrote:
Could Dries or someone who has access please update http:// drupal.org/node/318 to reflect these changes?
Thanks to marineam this is fixed now. There is an old checkout which file is php included here. I asked him to replace the file. I hope that soon cvs.drupal.org will move to Oregon and this file will be 'live' again. That's why I have not replaced the PHP command by the actual HTML.
The file is 'live', as we have a local checkout of the CVS repository. It has nothing to do with moving the repositories itself. 'marineam' did not fix it properly, causing CVS conflicts. I just fixed it properly. -- Dries Buytaert :: http://www.buytaert.net/
On Sun, 06 Nov 2005 10:32:05 +0100, Dries Buytaert <dries@buytaert.net> wrote:
On 06 Nov 2005, at 03:03, Karoly Negyesi wrote:
Could Dries or someone who has access please update http:// drupal.org/node/318 to reflect these changes?
Thanks to marineam this is fixed now. There is an old checkout which file is php included here. I asked him to replace the file. I hope that soon cvs.drupal.org will move to Oregon and this file will be 'live' again. That's why I have not replaced the PHP command by the actual HTML.
The file is 'live', as we have a local checkout of the CVS repository. It has nothing to do with moving the repositories itself. 'marineam' did not fix it properly, causing CVS conflicts. I just fixed it properly.
My bad. Wanted to offload something from you :( Regards NK
On 06 Nov 2005, at 02:37, Angie Byron wrote:
Could Dries or someone who has access please update http:// drupal.org/node/318 to reflect these changes?
Done. Thanks for investigating/reporting the problem. -- Dries Buytaert :: http://www.buytaert.net/
participants (3)
-
Angie Byron -
Dries Buytaert -
Karoly Negyesi