Project

General

Profile

Coding standards and hints for Virtuemart » History » Version 10

Max Milbers, 08/29/2012 12:03 PM

1 1 Max Milbers
h1. Coding standards and hints for Virtuemart
2
3 3 Max Milbers
In general we avoid the use of globals. Globals are hard to maintain and they are unsure variables. 
4
5 1 Max Milbers
h2. Code Formatting:
6
7 4 Max Milbers
We follow the joomla standard in most cases. We use camelCase for methods. For faster copypasting and working with database fields, both variable and database fields are lowercase and seperated with "_". You may use for internal variables camelCase also.
8 1 Max Milbers
9
Example:
10
11
12
<pre>
13
function exampleMethod($data = 0) {
14
    if (!$product_id) $product_id = JRequest::getInt('product_id', 0);
15
    if ($product_id > 0) {
16
        ...;
17
    }
18
}
19
</pre>
20
21
h2. SQL Formatting:
22
23
The queries should follow this format:
24
25 10 Max Milbers
@$q  = 'SELECT `example_id` FROM  `#__vm_table` WHERE `user_id`= "' . (int)$user_id . '"';@
26 1 Max Milbers
27
or if the
28
29
$user_id is an array, use $user_id["myId"]
30
31
Placing of the ` improves speed, because the sql-engine can parse faster ( and do not have to reparse).
32
33
h2. Variable Formatting:
34
35
Due abstract logic many people name their variables or database fields just "id", "key", "value" and so on.
36
37
In our case we want to rewrite the structure of an old code, therefore it is very important to find the variable in the code rather fields in the database.
38
39
So we want a kind of this: <classname><tablename>variable like here
40
41
$vendor_id or $payment_method_id
42
43
Exception from this rule is the use of ordering and published. Ordering and published with 0 and 1 as values are useable with the joomla standard methods.
44
45
h2. Filename Formatting:
46
47
Filenames should be lowercase.
48
49
Models filenames (mymodel.php), view direcorynames (myview), all helper files and probably all other files loaded by the Joomla Framework, must be lowercase, no matter how you load the file:
50
51
 @$this->loadHelper('adminMenu'); // looks for the file helpers/adminmenu.php@
52
53
More complex filenames that are specific to your task and loaded by your own code (using require_once()), e.g. mySpecificClass.php, can be camelcase.
54
55
Read this article for more info http://docs.joomla.org/Talk:Developing_a_Model-View-Controller_Component_-_Part_1#Use_lowercase_file_and_folder_names_in_your_components.21 
56
57
h2. URL Requests:
58
59
As written in the section __Filename Formatting__ above, Joomla requires lowercase filenames. However, URL requests are not lowercased by Joomla. In order to find the proper views and controllers, make sure you use lowercase only in requests, e.g.:
60
61
  $this->setRedirect('index.php?option=com_virtuemart&view=updatesMigration', $msg); // Fails
62
$this->setRedirect('index.php?option=com_virtuemart&view=updatesmigration', $msg); // Works
63
64
65
h2. Character Encoding:
66
67
Like in joomla we use UTF-8. Please ensure the right character set in your IDE.
68 2 Max Milbers
69 6 Max Milbers
h2. Path constants
70 1 Max Milbers
71 6 Max Milbers
Always use:
72
73
JPATH_VM_SITE for the frontend (points to JPATH_SITE.DS.'components'.DS.'com_virtuemart')
74 1 Max Milbers
JPATH_VM_ADMINISTRATOR for the backend (points to JPATH_SITE.DS.'components'.DS.'com_virtuemart')
75
76
This is important for performance and that all pieces of code work in every application that means, plugins, modules and so on.
77
78 7 Max Milbers
h1. Database tables and fields
79
80
h2. Naming convention
81
82
In general we follow now the nooku conventions for naming tables and fields (http://nooku.assembla.com/spaces/nooku-framework/wiki/KDatabase). 
83
84
All tables must start with the component name, here virtuemart => #__virtuemart_ and be lowercase.
85
86
All non xref tables should use plural. Don't use underscore to make long words readable. Concat them like the germans do, for exampel:
87
vm_user_info => virtuemart_userinfos, vm_admin_menu => virtuemart_adminmenu, vm_payment_method => virtuemart_paymentmethods, ...
88
89
All xref tables are now added with underscore, the use of singular or plural gives a hint on the kind of the xref.
90
For exampel the xref table linking products to categories looks like #__virtuemart_product_categories. Or the hardcore exampel category_categories :-).
91
92
Next is to consider that all primary keys should be always the table name in singular + id on it. Look on this as exampel
93
94 8 Max Milbers
  CREATE TABLE IF NOT EXISTS `#__virtuemart_product_categories` (
95 7 Max Milbers
  `virtuemart_product_id` int(11) NOT NULL DEFAULT '0',
96
  `virtuemart_category_id` int(11) NOT NULL DEFAULT '0',
97
98
99 1 Max Milbers
h2. Standard publishing fields use:
100 9 Max Milbers
101 7 Max Milbers
Except the published field we use now the nooku conventions. So we use   
102 1 Max Milbers
103 7 Max Milbers
  `ordering` tinyint(2) NOT NULL,
104
  `shared` tinyint(1) NOT NULL,
105
  `published` tinyint(1) NOT NULL DEFAULT '1',
106
  `created_on` datetime NOT NULL default '0000-00-00 00:00:00',
107
  `created_by` int(11) NOT NULL DEFAULT 0,
108
  `modified_on` datetime NOT NULL DEFAULT '0000-00-00 00:00:00',
109
  `modified_by` int(11) NOT NULL DEFAULT 0,
110
  `locked_on` datetime NOT NULL DEFAULT '0000-00-00 00:00:00',
111
  `locked_by` int(11) NOT NULL DEFAULT 0,
112
113 2 Max Milbers
- publish_up and publish_down formatted in the db as datetime with standard 0000-00-00 00:00:00
114
115
The standard query is then 
116
117 5 Max Milbers
	@ $db = &JFactory::getDBO();
118
	$this->_nullDate = $db->getNullDate();
119
	$this->_now        = JFactory::getDate()->toMySQL();
120
121
 $query  = 'SELECT * FROM `#__sometable` WHERE yourexampel ="myexampelvalue" AND ';
122
 $query .= ' ( publish_up = '.$this->_db->Quote($this->_nullDate).' OR publish_up <= '.$this->_db->Quote($this->_now).' )' .
123
 ' AND ( publish_down = '.$this->_db->Quote($this->_nullDate).' OR publish_down >= '.$this->_db->Quote($this->_now).' ) '; @