best practices

A fix was recently committed to both Drupal 8 and Drupal 7 to use #attached for block_admin_display_form() instead of drupal_add_css(). It only came up in my "Your issues" block on my drupal.org dashboard because, it turned out, i was the one who had initially complained about it. And that's all i did- looking for core examples to tell readers of dgd7.org how to write good code, i noticed core was in fact doing it wrong.

Is this a generally accepted concept that links on a page should never go to that page itself?

Found this: "No link in a menu should simply reload the page itself"
http://www.webmasterworld.com/forum21/3705.htm

How to do it:

Menu System - Drupal API
http://api.drupal.org/api/group/menu/7

Introduction

This page is the seed of Public Display's collective knowledge base about security. Here you will find the main classes of vulnerability we need to be aware of, with a brief description.

• Use 2 space indents
• NO tabs - anywhere
• try not to leave trailing whitespace
• prefer parentheses around method arguments - ESPECIALLY when the argument list has more than one argument.
• Optimize your code for READABILITY first.
• always put whitespace between successive elements of a list (e.g. argument list, array)

Version Control with Git

• Commit locally very often - you can always rearrange it later.