Люди, что это за ошибка?

[Yesvik]

http://code.google.com/p/opencart/source/browse/trunk/upload/system/library/language.php?r=105 это было сделано в апреле 2010 года... потом удаляли, опять добавляли... у них на форуме был срач по этому поводу...

[freelancer]

http://code.google.c...guage.php?r=105 это было сделано в апреле 2010 года... потом удаляли, опять добавляли... у них на форуме был срач по этому поводу...

:ugeek:

а твоё мнение?

[Yesvik]

То как сделано сейчас http://code.google.com/p/opencart/source/browse/trunk/upload/system/library/language.php вполне разумно... зачем удаляли и чего спорили - не понимаю.

Данное решение не только защищает от полного отсутствия файла локализации, но и корректно отрабатывает когда в файле файле локализации не все фразы...

Например после обновления в дефолтном языке появились новые фразы, а в переводе этих фраз нет.... при этом никакой ругани не будет, а отсутствующие фразы будут отображены на дефолтном языке. (по этой причине и был оставлен английский язык в ocStore)

[freelancer]

То как сделано сейчас http://code.google.c...ry/language.php вполне разумно...

не видел этого кода, но постом выше сделал так же =)) только у меня короче в ущерб удобочитаемости и их default я заменил на alternative, потому как default - это язык заданный в настройках, т.е. текущий. путаница.

ну хоть в этом наши мысли сходятся.

[Yesvik]

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

default вполне адекватное имя переменной.. означает не тот язык который установлен по умолчанию в магазине, а дефолтный язык для проекта (файлы которого должны присутствовать всегда, даже если этот язык не используется в магазине)

[freelancer]

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

...

ты два линка дал. в первом да, во втором нет.

я ещё вот что думаю.. раз уж начали эту тему.

в контроллере приходится явно указывать фразу из файла языка, например

$this->data['text_telephone'] = $this->language->get('text_telephone');

а можно было бы автоматом загружать в $data контроллера всё, что в $_ language

[Yesvik]

Точно... как они достали... Мне больше нравится этот вариант http://code.google.com/p/opencart/source/browse/trunk/upload/system/library/language.php?r=124

[Yesvik]

в контроллере приходится явно указывать фразу из файла языка, например

$this->data['text_telephone'] = $this->language->get('text_telephone');

а можно было бы автоматом загружать в $data контроллера всё, что в $_ language

Эту тему уже обсуждали и было вполне разумное предложение https://opencartforum.com/topic/2707-%D1%80%D0%B0%D0%B7%D1%80%D0%B0%D0%B1%D0%BE%D1%82%D1%87%D0%B8%D0%BA%D0%B0%D0%BC-%D0%BD%D0%B0-%D0%B7%D0%B0%D0%BC%D0%B5%D1%82%D0%BA%D1%83-%D1%83%D0%BB%D1%83%D1%87%D1%88%D0%B5%D0%BD%D0%BD%D0%B0%D1%8F-%D1%80%D0%B0%D0%B1%D0%BE%D1%82%D0%B0-%D1%81-%D1%82/ которое все дружно проигнорировали...

[freelancer]

Эту тему уже обсуждали и было вполне разумное предложение http://opencartforum...-%D1%81-%D1%82/ которое все дружно проигнорировали...

всё уже придумали до меня..

да, там идея та же, но у меня реализация другая.

а что значит проигнорировали? ты же в OC Team. или кто решает что добавлять в сборку ocStore, а что нет?

[Sammy95]

Эту тему уже обсуждали и было вполне разумное предложение http://opencartforum...-%D1%81-%D1%82/ которое все дружно проигнорировали...

А какой смысл так перелопачивать ocStore, если в оригинальном опенкарте всё-равно останется по прежнему всё? Это только нарушит обратную совместимость. К тому же, здесь не переносят старые наработки при крупных обновлениях в апстриме. По крайней мере мои патчи из ветки 0.2 не попали в ocStore 1.x

[Sammy95]

а что значит проигнорировали? ты же в OC Team. или кто решает что добавлять в сборку ocStore, а что нет?

А тут каждый сам по себе. Если человек пропадёт, то и все его наработки в конечном счёте просто исчезнут сами собой. Поэтому и нет смысла вносить подобные отклонения от апстрима.

[freelancer]

Это только нарушит обратную совместимость...

о какой совместимости идет речь?

приведу пример. есть модуль CSV PRO для opencart, он не работает без напильника для ocStore

А тут каждый сам по себе. Если человек пропадёт, то и все его наработки в конечном счёте просто исчезнут сами собой. Поэтому и нет смысла вносить подобные отклонения от апстрима.

я не знаю ваших порядков, потому спрашиваю

[Sammy95]

о какой совместимости идет речь?

приведу пример. есть модуль CSV PRO для opencart, он не работает без напильника для ocStore

Я говорил о модулях и патчах, написанных для ocStore, в которых будет использоваться предложенная фича. Да, код уменьшится, разработка будет идти легче, но поставить такой модуль на оригинальный ОпенКарт уже 100% не получится без напильника. Т.е. гарантированно теряется обратная совместимость, которая и так уже не 100-процентная (см. собственный пример). А если человек, внёсший это изменение, вдруг исчезнет, то и в будущих версиях ocStore эти модули работать перестанут.

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

я не знаю ваших порядков, потому спрашиваю

Так я ответил на вопрос или нет?

[Oksana777]

Мне срочно для ocStore 0.2.0 необходим модуль который будет выводить категории и их рисунки на главной