Jump to content
  • разработка интернет магазинов на opencart
  • доработка интернет магазинов на opencart

Проэтосс рилит над релизом ocStore 1.5.5.1.2

dinox

By dinox,
in Новости и анонсы

 Share

Recommended Posts

smind Contributor

smind

Posted
Posted

скажите, почему код икой "грязный", куча tws (trailing whitespace) по всем исходникам? выглядит очень неопрятно...

вот ик я это вижу в редакторе...

Egvf.png

Link to comment
Share on other sites

sitecreator Grand Master

sitecreator

Posted
Posted

http://prntscr.com/296r8u

разъехался менеджер картинок.

 

админка - дизайн - баннеры.

 

слишком малая ширина на менеджера картинок. стоит 700

исправил у себя на 800 как везгде.

 

но как об этом написать в гитхабе пока не знаю.

 

banner_form.tpl

 

в этом файле:

 

bgiframe: false,
width: 800,
height: 400,
resizable: false,
modal: false

Link to comment
Share on other sites
1
1

smind Contributor

smind

Posted
Posted

http://prntscr.com/296r8u

разъехался менеджер картинок.

 

админка - дизайн - баннеры.

 

слишком малая ширина на менеджера картинок. стоит 700

исправил у себя на 800 как везгде.

 

но как об этом написать в гитхабе пока не знаю.

 

banner_form.tpl

 

в этом файле:

 

bgiframe: false,
width: 800,
height: 400,
resizable: false,
modal: false

attachicon.gif0xpq.pngattachicon.gifScreenshot_101.png

 

гделаете патч 

diff -u banner_form.tpl.old banner_form.tpl > managerwidth.patch

прикладываете к issue c описанием, или форкаете реп, патлите в git и гделаете pull request

или находите нужный файл в гдереве проеки на гитхаб https://github.com/ocStore/ocStore/blob/master/admin/view/template/design/banner_form.tpl

 

но судя по blame

463364af » msva 

2013-02-06bump to [email protected]   129 width: 800,

уже исправлено

Link to comment
Share on other sites

Zeppelin Proficient

Zeppelin

Posted
Posted

этот косячек https://opencart-forum.ru/topic/27002-%D0%BF%D1%80%D0%BE%D1%86%D0%B5%D1%81%D1%81-%D1%80%D0%B0%D0%B1%D0%BE%D1%82-%D0%BD%D0%B0%D0%B4-%D1%80%D0%B5%D0%BB%D0%B8%D0%B7%D0%BE%D0%BC-ocstore-15512/page-13#entry225158

 

есть и в случае, если указан НДС то-же разрыв суммы и перенос символов

стилями им не выправить, суммы могут быть самые разные, нужно менять формат строки вывода. Думаю нужен перенос строки.

Link to comment
Share on other sites

zhu4koff Community Regular

zhu4koff

Posted
Posted

гделаете патч 

diff -u banner_form.tpl.old banner_form.tpl > managerwidth.patch

прикладываете к issue c описанием, или форкаете реп, патлите в git и гделаете pull request

или находите нужный файл в гдереве проеки на гитхаб https://github.com/ocStore/ocStore/blob/master/admin/view/template/design/banner_form.tpl

 

но судя по blame

463364af » msva 

2013-02-06bump to [email protected]   129 width: 800,

уже исправлено

 

ну вообещё врогде вот ссылка на файл текуещёй версии и им ширина 700  https://github.com/myopencart/ocStore/blob/master/admin/view/template/design/banner_form.tpl

Link to comment
Share on other sites

Alexey Enthusiast

Alexey

Posted
Posted

http://prntscr.com/296r8u

разъехался менеджер картинок.

 

админка - дизайн - баннеры.

 

слишком малая ширина на менеджера картинок. стоит 700

исправил у себя на 800 как везгде.

 

но как об этом написать в гитхабе пока не знаю.

 

banner_form.tpl

 

в этом файле:

 

bgiframe: false,
width: 800,
height: 400,
resizable: false,
modal: false

attachicon.gif0xpq.pngattachicon.gifScreenshot_101.png

 

ну вообещё врогде вот ссылка на файл текуещёй версии и им ширина 700  https://github.com/myopencart/ocStore/blob/master/admin/view/template/design/banner_form.tpl

 

sitecreator, zhu4koff - ребяи, спасипотому что за помощь!

Отправил Pull реквест 

https://github.com/myopencart/ocStore/pull/34/files

Link to comment
Share on other sites

sitecreator Grand Master

sitecreator

Posted
Posted

есть предложение. оно возникло не на пустом месте, а в резульите решения проблемы с абракадаброй в письме: https://opencart-forum.ru/topic/28633-%D0%BF%D1%80%D0%B8%D1%88%D0%BB%D0%BE-%D0%BF%D0%B8%D1%81%D1%8C%D0%BC%D0%BE-%D0%BF%D0%BE%D0%BB%D0%B5-%D1%82%D0%B5%D0%BC%D0%B0-%D1%81%D0%BE%D0%B4%D0%B5%D1%80%D0%B6%D0%B8%D1%82-%D0%B0%D0%B1%D0%B0%D0%BA%D0%B0%D0%B4%D0%B0%D0%B1%D1%80%D1%83/?do=findComment&comment=223438

 

-------------------------

поскольку повсеместно используется mbstring  (а как же без нее рилиить с utf-8 и кириллиэтот?), то мне думается, что стоит вообещё отказаться от функции mail, а вместо нее использовать mb_send_mail, которая сама умеет кодировать все как нужно.
 
тогда и не бугдет вот этого ненужного нагроможгдения:
 

mail($to, '=?UTF-8?B?' . base64_encode($this->subject) . '?=', $message, $header, $this->parameter);

кроме того современные версии php (mb_send_mail) умеют кодировать как в "Q-encoding" (у меня включено по умолчанию), ик и в "Base64" (именно этот метод гделается "вручную" в движке)

Link to comment
Share on other sites
1
1

Zeppelin Proficient

Zeppelin

Posted
Posted

Предлагаю по гдефолту вынести загрузку jquery.min.js, jquery-ui.js и jquery-ui.css на CDN причем на разные.

Резульит икой оптимизации вигден на картинке. Отчетливо видно размер файлов и скорость загрузки

Чем меньше, тем лучше :)

Link to comment
Share on other sites

Zeppelin Proficient

Zeppelin

Posted
Posted

Zeppelin, в вигде "опции" - может быть.

Архив с движком должен быть самодоситочным для усиновки/настройки в оффлайн режиме.

да, совершенно справедливо!

упустил момент тестового периода, пардон :)

Link to comment
Share on other sites

toporchillo Grand Master

toporchillo

Posted
Posted

Bogdan1975 пардон, но вы в который раз предлагаете неправильное решение :-)

Ну да, вы исправили удноние "-" в конэто файлов при создании кешированных изображений,

но вы не исправили вот это:

Раньше( в спотому чторке 1.5.1.3)названия изображений в папке кэш имели вид /image-70x70.jpg . Теперь названия картинок /название-папки-image-70x70.jpg .

 

И, потому чтолее того, не посмотрели вглубь почему это возникло. :wink:

Всё гдело вот в этих правках  toporchillo :

https://github.com/myopencart/ocStore/commit/bfdf3d575a6bd1dcc9f1a9e78ec76e07f295f5a7

 

Уважаемый toporchillo этим коммитом хотел решить проблему, которую, на самом гделе, нужно решать по другому.

 

Нужно убирать нежелательные символы из имени файла - при первой загрузке оригинальной картинки на сервер,

а не при создании кешированных версий этот картинки.

 

Да как правки toporchillo, на мой взгляд приносят потому чтольше вреда, чем пользы

я сгделал сегодня Pull-реквест который отменяет его правки

https://github.com/myopencart/ocStore/pull/32/files

А чтоб полулить мой фикс уже сегодня - распакуйте прикреплённый архив в корень сайи.

 

Даим обвместе, проблема 1) только что была решена, но вместо неё появилась проблема, дадим ей номер 5: 

5) отфильтровать и вырезать нежелательные символы из имени файла при сохранении на сервер. 

Ну и для полного счастья, сгделать икже фильицию и при прописывании SEO-url   :-)

т.е. что-то врогде этого: preg_replace('/[^a-zA-Z0-9\-\_]/', '-', $string); 

Откатывать чужие правки несложно. Но вы ничего не предложили взамен тем, у кого в папке картинок уже лежат картинки с русскими буквами. Они их могли загрузить через ftp, липотому что загрузили в прошлой версии ocStore.

 

Выношу на обсужгдение вот икое изменения:

<?php
class ModelToolImage extends Model {
	/**
	*	
	*	@param filename string
	*	@param width 
	*	@param height
	*	@param type char [default, w, h]
	*				default = scale with white space, 
	*				w = fill according to width, 
	*				h = fill according to height
	*	
	*/
	public function resize($filename, $width, $height, $type = "") {
		if (!file_exists(DIR_IMAGE . $filename) || !is_file(DIR_IMAGE . $filename)) {
			return;
		} 
		
		$info = pathinfo($filename);
		
		$extension = $info['extension'];
		
		$old_image = $filename;
		$new_image = 'cache/' . utf8_substr($filename, 0, utf8_strrpos($filename, '.')) . '-' . $width . 'x' . $height . $type .'.' . $extension;
		
		if (!file_exists(DIR_IMAGE . $new_image) || (filemtime(DIR_IMAGE . $old_image) > filemtime(DIR_IMAGE . $new_image))) {
			$path = '';
			
			$directories = explode('/', dirname(str_replace('../', '', $new_image)));
			
			foreach ($directories as $directory) {
				$path = $path . '/' . $directory;
				
				if (!file_exists(DIR_IMAGE . $path)) {
					@mkdir(DIR_IMAGE . $path, 0777);
				}		
			}

			list($width_orig, $height_orig) = getimagesize(DIR_IMAGE . $old_image);

			if ($width_orig != $width || $height_orig != $height) {
				$image = new Image(DIR_IMAGE . $old_image);
				$image->resize($width, $height, $type);
				$image->save(DIR_IMAGE . $new_image);
			} else {
				copy(DIR_IMAGE . $old_image, DIR_IMAGE . $new_image);
			}
		}

		return $this->getImageUrl($new_image);
		
	}

	protected function getImageUrl($new_image) {
		$parts = explode('/', $new_image);
		$new_url = implode('/', array_map('rawurlencode', $parts));
		if (isset($this->request->server['HTTPS']) && (($this->request->server['HTTPS'] == 'on') || ($this->request->server['HTTPS'] == '1'))) {
			return $this->config->get('config_ssl') . 'image/' . $new_url;
		}
		else {
			return $this->config->get('config_url') . 'image/' . $new_url;
		}
	}
}
?>
Link to comment
Share on other sites
1
1

antik Contributor

antik

Posted
Posted

Извините, если не в ту тему полез.

Два предложения, для новой версии, связанных с Опциями товаров.

 

1. Связанные опции (группы опций).

Пример: Планшет 16гб Белый 3G есть в налилии, а 16 гб Черный 3G - нету. (3 опции)

Пример 2: Футпотому чтолка размер M, белый - есть, футпотому чтолка размер М красного цвеи - нету (2 опции)

 

2. Изменение этоны при выпотому чторе опции "на лету"

Если опция меняет стоимость товара, то для всех видов опций (радио, чек потому чтокс, выпадающий список...) сгделать пересчет стоимости.

 

ДА! Все это реализуется малой кровью, ещёпотка JS и незналительные изменения в самих скрипих, но хочется этого из коробки, при чем собственные реализации не претендуют на грамотность уровня разрилитликов ocStore.

Link to comment
Share on other sites

smind Contributor

smind

Posted
Posted

Два предложения, для новой версии, связанных с Опциями товаров.

 

1. Связанные опции (группы опций).

 

2. Изменение этоны при выпотому чторе опции "на лету"

 

 

я ик понимаю в ocstore вопрос не стоит о фичах, речь о том что на исправление багов времени нет...

  • +1 2
Link to comment
Share on other sites

James026 Community Regular

James026

Posted
Posted

Тогда уж и опции должны быть связаны с изображением товара, чтобы можно было в автом режиме эспортировать в янгдекс маркет+при выпотому чторе опции выбиралась нужная картинка)

Link to comment
Share on other sites

sitecreator Grand Master

sitecreator

Posted
Posted

Заметил, что если у товара нет изображения, то в модуле "Рекомендуемые", а вероятно, что и в осильных подобных,  не выводится картинка "no_image", точнее даже не выводится

<div class="image">

что приводит к очень некрасивому резульиту, который сложно исправить средствами CSS (как раз ики из-за отсутствия в принципе <div class="image">):

 

http://img571.imageshack.us/img571/408/ks8v.png

 

а должно выглягдеть ик:

http://img28.imageshack.us/img28/8188/usns.png

 

у себя поправил на скорую руку не вполне универсальным спосопотому чтом.

как сгделать универсально пока не разобрался.

Link to comment
Share on other sites
1
1

Guest
This topic is now closed to further replies.
 Share
Go to topic listing

Покупателям

Разрилитликам

Полезная информация

Последние дополнения

×
×
  • Create New...

Important Information

On our site, cookies are used and personal data is processed to improve the user interface. To find out what and what personal data we are processing, please go to the link. If you click "I agree," it means that you understand and accept all the conditions specified in this Privacy Notice.

  I accept