If you are an Opencart developer or you have gone through one of\u00a0Opencart’s\u00a0controller then there, you would find a repetitive code of line for fetching the language or performing some validations. Here, I’m providing some tips in order to minimise the code from several lines to 5-6 lines.<\/p>\n
Here, I’m covering a few examples to make your OPpencart\u00a0code optimised.<\/p>\n
Here’s the example of language in the product page and the appropriate solution to it. You would find a code like this on many pages.<\/p>\n
$this->load->language('catalog\/product');\r\n$data['text_enabled'] = $this->language->get('text_enabled');\r\n$data['text_disabled'] = $this->language->get('text_disabled');\r\n$data['text_none'] = $this->language->get('text_none');\r\n$data['text_yes'] = $this->language->get('text_yes');\r\n$data['text_no'] = $this->language->get('text_no');\r\n$data['text_plus'] = $this->language->get('text_plus');\r\n$data['text_minus'] = $this->language->get('text_minus');\r\n$data['text_default'] = $this->language->get('text_default');\r\n$data['text_option'] = $this->language->get('text_option');\r\n$data['text_option_value'] = $this->language->get('text_option_value');\r\n$data['text_select'] = $this->language->get('text_select');\r\n$data['text_percent'] = $this->language->get('text_percent');\r\n$data['text_amount'] = $this->language->get('text_amount');\r\n<\/pre>\nSo, this code is can be made very minimised by using just these line of codes.<\/p>\n
$language_array = $this->load->language('catalog\/product');\r\n\r\nforeach ($language_array as $language_key => $language_value) {\r\n\t$data[$language_key] = $language_value;\r\n}<\/pre>\nor, just writing,<\/p>\n
$data = array();\r\n$data = array_merge($data, $this->load->language('catalog\/product'));<\/pre>\nor, simply,<\/p>\n
$data = $this->load->language('catalog\/product');<\/pre>\nSimilarly, we have another code which delivers data to form whether for add or edit.<\/p>\n
if (isset($this->request->post['model'])) {\r\n\t$data['model'] = $this->request->post['model'];\r\n} elseif (!empty($product_info)) {\r\n\t$data['model'] = $product_info['model'];\r\n} else {\r\n\t$data['model'] = '';\r\n}\r\n\r\nif (isset($this->request->post['sku'])) {\r\n\t$data['sku'] = $this->request->post['sku'];\r\n} elseif (!empty($product_info)) {\r\n\t$data['sku'] = $product_info['sku'];\r\n} else {\r\n\t$data['sku'] = '';\r\n}\r\n\r\nif (isset($this->request->post['upc'])) {\r\n\t$data['upc'] = $this->request->post['upc'];\r\n} elseif (!empty($product_info)) {\r\n\t$data['upc'] = $product_info['upc'];\r\n} else {\r\n\t$data['upc'] = '';\r\n}\r\n\r\nif (isset($this->request->post['ean'])) {\r\n\t$data['ean'] = $this->request->post['ean'];\r\n} elseif (!empty($product_info)) {\r\n\t$data['ean'] = $product_info['ean'];\r\n} else {\r\n\t$data['ean'] = '';\r\n}<\/pre>\nThis code can be made optimised by putting all the indexes in one array and then iterating the code for those set of values like this.<\/p>\n
$form_keys = array('model','sku','upc','ean','location','jan','isbn','mpn');\/\/ more values can be added to this array\r\n\r\nforeach ($form_keys as $form_value) {\r\n\tif (isset($this->request->post[$form_value])) {\r\n\t\t$data[$form_value] = $this->request->post[$form_value];\r\n\t} elseif (!empty($product_info)) {\r\n\t\t$data[$form_value] = $product_info[$form_value];\r\n\t} else {\r\n\t\t$data[$form_value] = '';\r\n\t}\t\t\t\r\n}<\/pre>\nSimilar\u00a0will be the case with getting config values. For that too, you can use the above technique.<\/p>\n
These were a few optimisations. Other techniques will be posted in the upcoming blogs.<\/p>\n","protected":false},"excerpt":{"rendered":"