Coding guidelines

In General

Make it easier to review

• Keep method cyclomatic complexity below 5.
• Keep methods short, below 20 lines

Naming

• Use meaningful method and variable names
• Keep referring to media types as mimetype, so that people woring on core understand what it means

Translations

Concatenated string cannot be processed by the l10n tools

Wrong

t('gallery',"This is my very long" +
" string, that I want to split");

Correct

t('gallery',"This is my very long string, that I want to split");

Tips

PHP uses a short-circuit evaluation technique when looking at conditions

if (conditionA && conditionB)

If conditionA is false, conditionB will never be tested.