brest001

2 часа назад, AlektroNik сказал:

@brest001 Нашел еще ошибочку.
При попытки редактировать цену доставки вылетает ошибка, но при этом цена доставки все же меняется
 

AH01071: Got error 'PHP message: PHP Fatal error:  Uncaught Error: Call to undefined method ompro\\omproapi::() in /workspace/sites/dev1c.site.ru/system/library/ompro/omproapi.php:2384
Stack trace:
#0 /workspace/sites/dev1c.site.ru/admin/controller/sale/ompro.php(6111): ompro\\omproapi->xEditCustomOrderShippingCost(Array)
#1 /workspace/sites/dev1c.site.ru_storage/modification/system/engine/action.php(79): ControllerSaleOMPro->xEditDbData()
#2 /workspace/sites/dev1c.site.ru/admin/controller/startup/router.php(26): Action->execute(Object(Registry), Array)
#3 /workspace/sites/dev1c.site.ru_storage/modification/system/engine/action.php(79): ControllerStartupRouter->index()
#4 /workspace/sites/dev1c.site.ru/system/engine/router.php(67): Action->execute(Object(Registry))
#5 /workspace/sites/dev1c.site.ru/system/engine/router.php(56): Router->execute(Object(Action))
#6 /workspace/sites/dev1c.site.ru/system/framework.php(168): Router->dispatch(Object(Action), Object(Action))
#7 /workspac...
', referer: https://dev1c.site.ru/admin/index.php?route=sale/ompro/orders&user_token=8ruog8YixYm627cyePsbjjcFBOGIr4Ww&pageid=13

 

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

Если вам уже сейчас нужно, то в system\library\ompro\omproapi.php

Строка 2384, замените if (isset($post['action'])) {

на: if (isset($post['action']) && trim($post['action']) !== '') {

Было бы неплохо проверить, т.к. у меня ошибка не воспроизводится. Спасибо за наводку )

В 04.06.2021 в 19:33, AlektroNik сказал:

<tbody> [[{totals}]]</tbody>

@brest001 Константин, ковырял, ковырял ... так и не смог понять откуда и как формируется переменная [[{totals}]].
Хотел посмотреть как название Купона вытащить. Думал эти итоги самому собрать ... а что-то не вышло и воспользовался пока [[{totals}]]

Минимальн ширини крутовата для столбика значений

min-width: 120px;

[[{totals}]] - переменные в шаблоне - это просто текст и их нельзя "разобрать" как вы написали в личке. Когда парсится шаблон для замены переменных на значения, то вместо этой переменной он берет все тоталы из заказа и выводит эти данные (подитог, налоги, доставка, купон и т.д.) по шаблону для итогов (например; название - значение). 

Если вы хотите вывести отдельно какое-то значение из тоталов, то можете взять за пример поле subtotal или shipping_cost и создать на их основе ваш запрос для купона, что-то вроде:

(SELECT COALESCE(SUM(ot.value), 0) FROM {DB_PREFIX}order_total AS ot WHERE ot.order_id = o.order_id AND code = 'coupon' LIMIT 1)

Цитата

Минимальн ширини крутовата для столбика значений

min-width: 120px;

Такие замечания даже не делайте, у всех все индивидуально, в зависимости от выводимых данных, где они выводятся, для чего, на каком экране и т.д. и т.п. И тут кому-то может понадобиться 121px.  другому 122px, и красненьким... третьему 1122px ... так что каждый сам под себя и настраивайте, для этого все есть, в код лезть не нужно.

29 минут назад, AlektroNik сказал:

Сразу скажу, это не претензия, а просто наблюдения.

 

Все же пришел к выводу, что Редактирование полей ЗАКАЗА напрямую с менеджера заказов это Зло.

А вот и рублика "Хотелка" подъехала
Вот если бы напрямую можно было бы редактировать непосредственно эти же значения у самого покупателя (таблица customer кажется), вот это было бы супер.

 

Дело в том, что когда редактируем эти поля через менеджер заказов, то редактирование происходит только внутри конкретного заказа.
Хотя изменение любого из этих значений должно сохраняться на века, что бы при следующем заказе не пришлось делать то же самое каждый раз.

А злом я его назвал потому, что менеджеры даже не будут пытаться в чем-то разбираться. Они запомнили, что уже редактировали покупателя и в следующий раз даже не будут обращать внимание на эти поля. Будут убеждены, что они их уже правили. Справедливости ради редактирование для менеджеров отключено по умолчанию, автор явно догадывался о возможных нюансах.

 

Тогда вам не менеджер заказов нужен, а редактор всего и вся из одной страницы. В общем, я не согласен и логика ломается - данные берутся из заказа, а получается редактировать нужно общие данные покупателя и отредактированные данные снова вставлять в заказ. Тем более, в магазине данные оплаты, доставки и основные данные покупателя - это разные данные. Следуя вашей логике, нужно брать основные данные и выводить в заказ, а если покупатель конкретно для этого заказа захочет изменить номер телефона (например он в поездке и у него другой номер) или почту, что тогда будете делать? - вся логика насмарку. Можно и еще разные ситуации спрогнозировать, когда такой подход будет только вносить путаницу.
В данном случае считаю кнопки для перехода в редактирование покупателя вполне достаточно.

47 минут назад, AlektroNik сказал:

@brest001 Константин, пытаюсь настроить кнопку WhatsApp с горем пополам получилось, но есть одно НО.
Когда я пытаюсь вставить тег email, ссылка ломается.
Получается ссылка на email внутри основной ссылки.
Можно как-то пометить тег, чтобы автоматом не переделывал email в ссылку?

Вот пример кода ссылки:

<a href="https://wa.me/[[{telephone_numeric}]]">Чат с пользователем WhatsApp [[{email}]]</a>

Вариант с добавлением поля в таблицу не подходит. Хотя функциональность классная

Попробовал изменить тип поля на текст, не помогло. Потом сверху почитал всплывающую подсказу и решил, что правильнее было бы отключить редактирование тега email. И да, это помогло.
Вопрос только один. Правильно ли я сделал или как-то можно было, и редактирование оставить и в ссылку вставить?

Правильно, а чтобы оставить редактирование здесь все написано http://brest001.ru/ompro_doc/ompro_faq.html#tab=tab_faq&item=item_002

Цитата

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

17 часов назад, AlektroNik сказал:

@brest001 Констанитин, возникла идея.
А может скролбар горизонтальный закрепить как-то внизу экрана как в PhpMyAdmin? Сейчас, если список длинный, срол бар находится внизу всего списка, а не внизу непосредственно окна.

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

11 часов назад, dejavydejavy сказал:

Подскажите у меня еще старая версия "Order Manager oc23x 1.3" я могу  безболезненно обновится.

Order Manager Pro - это отдельный модуль, устанавливайте и пользуйтесь, старой версии он не затронет.

На все предложения и "хотелки" отвечу позже по-порядку, по мере изучения. Я все вижу, смотрю что можно (или не можно) сделать.

5 часов назад, AlektroNik сказал:

Кеш чистил, не помогло ... да поидее и не должно. тут прям код отличается ... код уведомления то не менялся даже.

Впечатление все-равно, что у вас какие-то старые файлы работают. Я конечно не уверен, но мне эту ситуацию воспроизвести не удалось и по коду еще раз проверил, не должны теги сейчас удаляться из комментария в письме, у меня вот норм https://prnt.sc/13nfh4k

Кстати, мне помнится вы язык отдельно инсталлировали. В любом случае проверьте шаблоны нужного комментария для всех ли языков они с красным цветом, пересохраните настройки группы.

5 часов назад, AlektroNik сказал:

Мне кажется даже быстрее как-то менеджер заказов работать / грузиться стал со списками заказов.

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

5 часов назад, AlektroNik сказал:

Мне кажется пора внедрять кнопку удалить на странице "Выбор страниц" )))

Я думаю не так трудно зайти в список страниц и там удалить ) и так много всего в модуле, если добавлять что-то, то действительно нужное.

5 часов назад, AlektroNik сказал:

5. А вот "редактирования группы покупателя" это не понял / не нашел.
Наверное в списках заказа его пока нет как пример, но есть возможность отредактировать шаблон?
Может для 203 опять как-то шаблоны нужно подгрузить новые? Я пока именно обновлялся.

Да просто отключил "заморозку" настроек этого поля, в шаблоны не добавлял, там никакой премудрости нет - включили редактирование, добавили переменную в шаблон и всё.

5 часов назад, AlektroNik сказал:

Видимо только чеез создание нового фильтра.

Да, создавать, добавлять и пробовать. И так там тех фильтров немеряно, но все-равно у каждого будут свои нюансы в работе. Поэтому лучше всего пробовать создавать свои фильтры, под свои нужды.

5 часов назад, AlektroNik сказал:

@brest001В целом огромное спасибо за релиз. Производительность у Вас супер. Когда Вы только все успеваете ... Хотя ... я от Вас и в 03:00 письма получал ))) Жульничаете ... по 3 часа спите )))
С выходом из бетки полностью согласен 👍🍻

И вам спасибо за продуктивное сотрудничество.

1 час назад, AlektroNik сказал:

Если прям ошибка выскакивает, я думаю это решение лучше на постоянку добавить для всех.

Вообще-то, это не прям ошибка, а предупреждение, что в названии файла не должно быть кириллицы.

2 часа назад, kamrik сказал:

Нашел у себя одну небольшую странность. Если в названии загружаемого файла есть символ " № ", то выскакивает ошибка загрузки.

У всех так?) 

Если сильно напрягает, добавьте строчку в system\library\ompro\omproapi.php  https://prnt.sc/13mu496 Это функция транслитерации, там просто замена символа " № " не предусмотрена.

12 часов назад, Matvey сказал:

Добрый день.

Обновился с 2.0.1

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

<a class="btn btn-default btn-xs" data-toggle="tooltip" title="Обновить заказ" onclick="orderReload([[{order_id}]])"><i class="fa fa-refresh"></i></a>

На компе все ок.

Еще не могу зарегистрироваться на вашем сайте brest001, пишет такое мыло уже есть, но восстановление пароля не работает.

Разобрались. Небольшая коррекция все-таки понадобилась для нормальной работы.

В методе orderReload для обновления заказа кроме переданных аргументов берется id страницы из url, а при переходе из настроек в заказы открывается первая в списке страница, но pageid при этом в url не передается. Если у вас первой в списке стоит страница, где применяется обновление заказа, то после перехода в заказы нужно нажать еще раз в меню слева эту страницу, тогда в url появится &pageid=.. и все будет работать. Но чтобы нормально работало, замените этот файл admin\controller\sale\ompro_header.php (прикрепил). В архиве файл заменил.

ompro_header.php

6 часов назад, Matvey сказал:

<a class="btn btn-default btn-xs" data-toggle="tooltip" title="Обновить заказ" onclick="orderReload([[{order_id}]])"><i class="fa fa-refresh"></i></a>

Здравствуйте.

У вас нет второго аргумента для orderReload https://prnt.sc/13ljhnc

Если вы ничего в шаблонах не меняли, т.е. пользуетесь готовыми шаблонами, то включите новые страницы, добавленные в версии 2.0.2 https://prnt.sc/13ljmxt

А если вы что-то под себя настраивали, значит нужно проверять, что не так в ваших шаблонах.

Если не разберетесь, напишите в личку с доступами в админку и скажите что конкретно в каком шаблоне не работает.

Всем здравия!

Можно обновляться до 2.0.3.

Список изменений здесь http://brest001.ru/ompro_doc/ompro_history.html

8 минут назад, kamrik сказал:

Добрый вечер. Проверил сейчас свой личный кабинет. Он есть.  И  письма с ключами есть. Но вот покупок в нем нет(

Или их и не должно быть? Только лицензии?

Только лицензии. 

1 минуту назад, caspy сказал:

нет 

получилось спасибо

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

39 минут назад, caspy сказал:

Добрый день! 

у меня случидось несчастный слуси и переустановил ордер про . 

но у меня пишет опять активация 

что мне делать ? 

Кстати, вы ордер про с Order Manager Pro не перепутали, надеюсь?

39 минут назад, caspy сказал:

Добрый день! 

у меня случидось несчастный слуси и переустановил ордер про . 

но у меня пишет опять активация 

что мне делать ? 

Здравствуйте.

Отправляю ключ в личку, Зайдите в модуль и активируйте.

PS. Доступ в личный кабинет отправляется в письме при первом запросе лицензии.

В 21.05.2021 в 18:19, AlektroNik сказал:

Написал мод, проверил, теперь все работает.
Можно добавлять в сборку ))))

api_deleteApiSessionBySessonId_fix_v1.0.ocmod.zip 676 \u0411 · 0 downloads

Спасибо вам большое. В сборку добавлять не буду, так как для Order Manager Pro это не нужно, но здесь хорошо, что выложили, кому-нибудь точно пригодится.

4 часа назад, AlektroNik сказал:

Приступил к тестированию на ocStore 3.0.3.7

 

ОШИБКА появляется при установке основного модуля.

Ошибка кода(0): Error: Invalid default value for 'date_available'
Error No: 1067
ALTER TABLE `oc_product` ADD `cost` decimal(15,4) NOT NULL DEFAULT '0.0000' в /system/library/db/mysqli.php на строке 41

 

РЕШЕНИЕ

https://github.com/ATropichev/sql_mode_fix.ocmod
Может тоже добавить в FAQ или в Ваш фикс?

Спасибо! На ocStore 3.0.3.7 не тестировал, придется смотреть, может еще есть сюрпризы.

В фикс не получится, это нужно исправлять до установки модуля. Видимо включать надо в подготовку к установке, если ничего проще не придумаю.

7 минут назад, AlektroNik сказал:

А с этим что-то планируется делать? Може фикс какой дополнительный?

Не планирую, Новый модуль это не использует.

8 минут назад, AlektroNik сказал:

Как лучше быть сделать фикс для api.php или вылавливать модули, которые используют старую функцию и переписыать их?

Как по мне, самое простое и универсальное

Цитата

Т.е. в файле \admin\model\user\api.php (этот метод находится в конце), продублируйте его с названием без буквы i

	
	public function deleteApiSessionBySessonId($session_id) {
		$this->db->query("DELETE FROM `" . DB_PREFIX . "api_session` WHERE session_id = '" . $this->db->escape($session_id) . "'");
	}

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

5 часов назад, Tomas12312 сказал:

Здравствуйте, подскажите, при установке ompro_main_OC_30.ocmod.zip в админке, выдаёт ошибку "Директория admin/controller/sale не доступна для записи!"

Что делать?

Наверняка localcopy_OC_30.ocmod.zip пропустили. Не спешите, в инструкции ничего лишнего нет.

9 часов назад, AlektroNik сказал:

Жалко, что нет шаблонов. Но, лично я, этими функциями не пользуюсь так что ...

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

45 минут назад, kamrik сказал:

Да. Фикс стоит.

У меня этой ошибки нет, все нормально. Версия опенкарта у вас какая? Или лучше, чтобы я вас не мучил вопросами, если можете, дайте доступы в личку (админка, фтп). Надо посмотреть, что у вас там в этом файле.

1 час назад, kamrik сказал:

При удалении из модуля:

Notice: Undefined index: order_status_id in /home/h/________/test/public_html/system/library/ompro/ompro.php on line 1742{"success":"\u0423\u0441\u043f\u0435\u0448\u043d\u043e \u0443\u0434\u0430\u043b\u0435\u043d\u044b \u0437\u0430\u043a\u0430\u0437\u044b \u2116:16"}

Здравствуйте.

С этим понятно, разобрался, спасибо.

1 час назад, kamrik сказал:

Доброго дня! При попытке удаления заказа через стандартный редактор выдает ошибки:

2021-05-20 8:00:57 - PHP Notice:  Undefined variable: user_id in /home/h/______/test/storage/modification/catalog/model/checkout/order.php on line 435
2021-05-20 8:00:57 - PHP Notice:  Undefined variable: notify_sms in /home/h/_______/test/storage/modification/catalog/model/checkout/order.php on line 435
2021-05-20 8:00:57 - PHP Notice:  Undefined variable: notify_tlgrm in /home/h/_______/test/storage/modification/catalog/model/checkout/order.php on line 435
2021-05-20 8:00:57 - PHP Notice:  Undefined variable: file_name in /home/h/_______/test/storage/modification/catalog/model/checkout/order.php on line 435

Фикс устанавливали?