Re: OpenCart 1.5.2 Bug Thread
Posted: Tue Apr 10, 2012 11:38 am
Has anyone tested the Priority order in the specials tab. It does not work in 1.5.2. for me?
OpenCart Community Forum - Discuss shopping cart and e-commerce solutions.
https://forum.opencart.com/
emmegi wrote:Hi,
I'm a newbie. A few days ago I opened a new store with the wonderful OpenCart.
I think the following code in catalog/model/product/account/return.php should be changed from
toCode: Select all
$query = $this->db->query("SELECT r.return_id, r.order_id, r.firstname, r.lastname, r.email, r.telephone, r.product, r.model, r.quantity, r.opened, rr.name as reason, ra.name as action, rs.name as status, r.comment, r.date_ordered, r.date_added, r.date_modified FROM `" . DB_PREFIX . "return` r LEFT JOIN " . DB_PREFIX . "return_reason rr ON (r.return_reason_id = rr.return_reason_id) LEFT JOIN " . DB_PREFIX . "return_action ra ON (r.return_action_id = ra.return_action_id) LEFT JOIN " . DB_PREFIX . "return_status rs ON (r.return_status_id = rs.return_status_id) WHERE return_id = '" . (int)$return_id . "' AND customer_id = '" . $this->customer->getId() . "'");
modified to correctly display the labels in languages other than EnglishCode: Select all
$query = $this->db->query("SELECT r.return_id, r.order_id, r.firstname, r.lastname, r.email, r.telephone, r.product, r.model, r.quantity, r.opened, rr.name as reason, ra.name as action, rs.name as status, r.comment, r.date_ordered, r.date_added, r.date_modified FROM `" . DB_PREFIX . "return` r LEFT JOIN " . DB_PREFIX . "return_reason rr ON (r.return_reason_id = rr.return_reason_id) LEFT JOIN " . DB_PREFIX . "return_action ra ON (r.return_action_id = ra.return_action_id) LEFT JOIN " . DB_PREFIX . "return_status rs ON (r.return_status_id = rs.return_status_id) WHERE return_id = '" . (int)$return_id . "' AND customer_id = '" . $this->customer->getId() . "' AND rr.language_id = '" . (int)$this->config->get('config_language_id') . "' AND ra.language_id = '" . (int)$this->config->get('config_language_id') . "' AND rs.language_id = '" . (int)$this->config->get('config_language_id') . "'");
Confirmed. Same issue with category copy as well.arames wrote:When we add a new language to our OpenCart (french, or german for example), then in the Admin / Customer area, ALL the fields are correctly being imported to the NEW language except for 1 field the Meta Tag Keywords, it showing BLANK (no data is being imported).
Code: Select all
meta_description= '" . $this->db->escape($product['meta_description']) . "',
Code: Select all
meta_description = '" . $this->db->escape($product['meta_description']) . "', meta_keyword = '" . $this->db->escape($product['meta_keyword']) . "',
Code: Select all
meta_description= '" . $this->db->escape($product['meta_description']) . "',
Code: Select all
meta_description = '" . $this->db->escape($product['meta_description']) . "', meta_keyword = '" . $this->db->escape($product['meta_keyword']) . "',
Code: Select all
meta_description= '" . $this->db->escape($category['meta_description']) . "',
Code: Select all
meta_description = '" . $this->db->escape($category['meta_description']) . "', meta_keyword = '" . $this->db->escape($category['meta_keyword']) . "',
Working fine here.allenshea wrote:Can anyone confirm if the currency is not updating it again?
My cart doesnt' update currency, don't know why.
Code: Select all
public function updateCurrencies($force = false) {
if (!extension_loaded('curl')) {
$this->log->write(__FUNCTION__ . ' :: CURL NOT LOADED');
return;
}
$data = array();
if ($force) {
$query = $this->db->query("SELECT * FROM " . DB_PREFIX . "currency WHERE code != '" . $this->db->escape($this->config->get('config_currency')) . "'");
} else {
$query = $this->db->query("SELECT * FROM " . DB_PREFIX . "currency WHERE code != '" . $this->db->escape($this->config->get('config_currency')) . "' AND date_modified < '" . $this->db->escape(date('Y-m-d H:i:s', strtotime('-1 day'))) . "'");
}
foreach ($query->rows as $result) {
$data[] = $this->config->get('config_currency') . $result['code'] . '=X';
}
$curl = curl_init();
curl_setopt($curl, CURLOPT_URL, 'http://download.finance.yahoo.com/d/quotes.csv?s=' . implode(',', $data) . '&f=sl1&e=.csv');
curl_setopt($curl, CURLOPT_RETURNTRANSFER, 1);
$content = curl_exec($curl);
if (curl_error($curl)) {
$this->log->write(__FUNCTION__ . ' :: ' . curl_error($curl));
}
curl_close($curl);
$lines = explode("\n", trim($content));
foreach ($lines as $line) {
$currency = utf8_substr($line, 4, 3);
$value = utf8_substr($line, 11, 6);
if ((float)$value) {
$this->db->query("UPDATE " . DB_PREFIX . "currency SET value = '" . (float)$value . "', date_modified = '" . $this->db->escape(date('Y-m-d H:i:s')) . "' WHERE code = '" . $this->db->escape($currency) . "'");
}
}
$this->db->query("UPDATE " . DB_PREFIX . "currency SET value = '1.00000', date_modified = '" . $this->db->escape(date('Y-m-d H:i:s')) . "' WHERE code = '" . $this->db->escape($this->config->get('config_currency')) . "'");
$this->cache->delete('currency');
}
already fixed in svn.Tcalp wrote:Pretty minor, but in /admin/model/localisation/tax_rate.php function addTaxRate() does not state date_modified in the insert statement, as such you end up with some pretty funky modified date being displayed in the list view.
Code: Select all
$this->config->get('config_layout_category');
$this->config->get('config_layout_information');
$this->config->get('config_layout_product');
ccM wrote:Can anyone else confirm problems with the Account and Affiliate layouts since r990? No modules showing up on fresh installs.
Perfect - Thanks Daniel!Daniel wrote:change the account route to account/%ccM wrote:Can anyone else confirm problems with the Account and Affiliate layouts since r990? No modules showing up on fresh installs.
R_D wrote:Don't know if this is really a bug but a customer can create multiple accounts by entering his/her email with capital or without capital letters or mix theme al around.
It is? seems like a horrible bug to me. I don't want customers complaining that they can't login because they used Johnsmith@site.com instead of johnsmith@site.com. That's a big bug.i2Paq wrote:R_D wrote:Don't know if this is really a bug but a customer can create multiple accounts by entering his/her email with capital or without capital letters or mix theme al around.
This is mentioned before but this is by design.......